* [ptxdist] [PATCH] stm32flash: New package @ 2018-10-05 9:25 Guillermo Rodriguez 2018-10-05 10:30 ` Roland Hieber 2018-10-06 17:14 ` [ptxdist] [PATCH v2] " Guillermo Rodriguez 0 siblings, 2 replies; 7+ messages in thread From: Guillermo Rodriguez @ 2018-10-05 9:25 UTC (permalink / raw) To: ptxdist; +Cc: Guillermo Rodriguez Open source cross platform flash program for the STM32 ARM microcontrollers using the built-in ST serial bootloader over UART or I2C. Signed-off-by: Guillermo Rodriguez <guille.rodriguez@gmail.com> --- rules/stm32flash.in | 9 ++++++++ rules/stm32flash.make | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 66 insertions(+) create mode 100644 rules/stm32flash.in create mode 100644 rules/stm32flash.make diff --git a/rules/stm32flash.in b/rules/stm32flash.in new file mode 100644 index 0000000..877eed3 --- /dev/null +++ b/rules/stm32flash.in @@ -0,0 +1,9 @@ +## SECTION=shell_and_console + +config STM32FLASH + tristate + prompt "stm32flash" + help + Open source cross platform flash program for the STM32 ARM + microcontrollers using the built-in ST serial bootloader over UART + or I2C. diff --git a/rules/stm32flash.make b/rules/stm32flash.make new file mode 100644 index 0000000..975d02e --- /dev/null +++ b/rules/stm32flash.make @@ -0,0 +1,57 @@ +# -*-makefile-*- +# +# Copyright (C) 2018 by Guillermo Rodriguez <guille.rodriguez@gmail.com> +# +# See CREDITS for details about who has contributed to this project. +# +# For further information about the PTXdist project and license conditions +# see the README file. +# + +# +# We provide this package +# +PACKAGES-$(PTXCONF_STM32FLASH) += stm32flash + +# +# Paths and names +# +STM32FLASH_VERSION := 0.5 +STM32FLASH_SUFFIX := tar.gz +STM32FLASH_MD5 := 40f673502949f3bb655d2bcc539d7b6a +STM32FLASH := stm32flash-$(STM32FLASH_VERSION) +STM32FLASH_URL := https://sourceforge.net/projects/stm32flash/files/$(STM32FLASH).$(STM32FLASH_SUFFIX)/download +STM32FLASH_DIR := $(BUILDDIR)/$(STM32FLASH) +STM32FLASH_SOURCE := $(SRCDIR)/$(STM32FLASH).$(STM32FLASH_SUFFIX) +STM32FLASH_LICENSE := GPLv2 + + +# ---------------------------------------------------------------------------- +# Prepare +# ---------------------------------------------------------------------------- + +STM32FLASH_CONF_TOOL := NO +STM32FLASH_MAKE_ENV := $(CROSS_ENV) +STM32FLASH_INSTALL_OPT := PREFIX=/usr install + +# ---------------------------------------------------------------------------- +# Target-Install +# ---------------------------------------------------------------------------- + +$(STATEDIR)/stm32flash.targetinstall: + @$(call targetinfo) + + @$(call install_init, stm32flash) + @$(call install_fixup, stm32flash, PRIORITY,optional) + @$(call install_fixup, stm32flash, SECTION,base) + @$(call install_fixup, stm32flash, AUTHOR,"Guillermo Rodriguez <guille.rodriguez@gmail.com>") + @$(call install_fixup, stm32flash, DESCRIPTION,missing) + + @$(call install_copy, stm32flash, 0, 0, 0755, -, /usr/bin/stm32flash) + + @$(call install_finish, stm32flash) + + @$(call touch) + + +# vim: syntax=make -- 2.5.4 (Apple Git-61) _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ptxdist] [PATCH] stm32flash: New package 2018-10-05 9:25 [ptxdist] [PATCH] stm32flash: New package Guillermo Rodriguez @ 2018-10-05 10:30 ` Roland Hieber 2018-10-05 10:43 ` Guillermo Rodriguez Garcia 2018-10-06 17:14 ` [ptxdist] [PATCH v2] " Guillermo Rodriguez 1 sibling, 1 reply; 7+ messages in thread From: Roland Hieber @ 2018-10-05 10:30 UTC (permalink / raw) To: ptxdist; +Cc: Guillermo Rodriguez Hi, On Fri, Oct 05, 2018 at 11:25:45AM +0200, Guillermo Rodriguez wrote: > Open source cross platform flash program for the STM32 ARM > microcontrollers using the built-in ST serial bootloader over UART > or I2C. > > Signed-off-by: Guillermo Rodriguez <guille.rodriguez@gmail.com> > --- > rules/stm32flash.in | 9 ++++++++ > rules/stm32flash.make | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 66 insertions(+) > create mode 100644 rules/stm32flash.in > create mode 100644 rules/stm32flash.make > > diff --git a/rules/stm32flash.in b/rules/stm32flash.in > new file mode 100644 > index 0000000..877eed3 > --- /dev/null > +++ b/rules/stm32flash.in > @@ -0,0 +1,9 @@ > +## SECTION=shell_and_console > + > +config STM32FLASH > + tristate > + prompt "stm32flash" > + help > + Open source cross platform flash program for the STM32 ARM > + microcontrollers using the built-in ST serial bootloader over UART > + or I2C. > diff --git a/rules/stm32flash.make b/rules/stm32flash.make > new file mode 100644 > index 0000000..975d02e > --- /dev/null > +++ b/rules/stm32flash.make > @@ -0,0 +1,57 @@ > +# -*-makefile-*- > +# > +# Copyright (C) 2018 by Guillermo Rodriguez <guille.rodriguez@gmail.com> > +# > +# See CREDITS for details about who has contributed to this project. > +# > +# For further information about the PTXdist project and license conditions > +# see the README file. > +# > + > +# > +# We provide this package > +# > +PACKAGES-$(PTXCONF_STM32FLASH) += stm32flash > + > +# > +# Paths and names > +# > +STM32FLASH_VERSION := 0.5 > +STM32FLASH_SUFFIX := tar.gz > +STM32FLASH_MD5 := 40f673502949f3bb655d2bcc539d7b6a > +STM32FLASH := stm32flash-$(STM32FLASH_VERSION) > +STM32FLASH_URL := https://sourceforge.net/projects/stm32flash/files/$(STM32FLASH).$(STM32FLASH_SUFFIX)/download Better use $(call ptx/mirror, SF, ...) here to get a list of SourceForge mirrors. Have a look at other packages for how to use it. > +STM32FLASH_DIR := $(BUILDDIR)/$(STM32FLASH) > +STM32FLASH_SOURCE := $(SRCDIR)/$(STM32FLASH).$(STM32FLASH_SUFFIX) > +STM32FLASH_LICENSE := GPLv2 > + > + > +# ---------------------------------------------------------------------------- > +# Prepare > +# ---------------------------------------------------------------------------- > + > +STM32FLASH_CONF_TOOL := NO > +STM32FLASH_MAKE_ENV := $(CROSS_ENV) > +STM32FLASH_INSTALL_OPT := PREFIX=/usr install For readability, keep these options on a single line each: STM32FLASH_INSTALL_OPT := \ PREFIX=/usr \ install Also most packages keep *_INSTALL_* variables under the Install section header, and *_MAKE_* variables under the "Compile" section header. I would suggest to do it here too to get a unified look over package rules. > + > +# ---------------------------------------------------------------------------- > +# Target-Install > +# ---------------------------------------------------------------------------- > + > +$(STATEDIR)/stm32flash.targetinstall: > + @$(call targetinfo) > + > + @$(call install_init, stm32flash) > + @$(call install_fixup, stm32flash, PRIORITY,optional) > + @$(call install_fixup, stm32flash, SECTION,base) > + @$(call install_fixup, stm32flash, AUTHOR,"Guillermo Rodriguez <guille.rodriguez@gmail.com>") > + @$(call install_fixup, stm32flash, DESCRIPTION,missing) > + > + @$(call install_copy, stm32flash, 0, 0, 0755, -, /usr/bin/stm32flash) > + > + @$(call install_finish, stm32flash) > + > + @$(call touch) > + While you're at it, no need for a double blank line here :) - Roland > + > +# vim: syntax=make > -- > 2.5.4 (Apple Git-61) > > > _______________________________________________ > ptxdist mailing list > ptxdist@pengutronix.de -- Roland Hieber | r.hieber@pengutronix.de | Pengutronix e.K. | https://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim | Phone: +49-5121-206917-5086 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ptxdist] [PATCH] stm32flash: New package 2018-10-05 10:30 ` Roland Hieber @ 2018-10-05 10:43 ` Guillermo Rodriguez Garcia 2018-10-05 12:07 ` Roland Hieber 2018-10-05 14:04 ` Michael Olbrich 0 siblings, 2 replies; 7+ messages in thread From: Guillermo Rodriguez Garcia @ 2018-10-05 10:43 UTC (permalink / raw) To: r.hieber; +Cc: ptxdist Hi, El vie., 5 oct. 2018 a las 12:30, Roland Hieber (<r.hieber@pengutronix.de>) escribió: > > Hi, > > On Fri, Oct 05, 2018 at 11:25:45AM +0200, Guillermo Rodriguez wrote: > > Open source cross platform flash program for the STM32 ARM > > microcontrollers using the built-in ST serial bootloader over UART > > or I2C. > > > > Signed-off-by: Guillermo Rodriguez <guille.rodriguez@gmail.com> > > --- > > rules/stm32flash.in | 9 ++++++++ > > rules/stm32flash.make | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++ > > 2 files changed, 66 insertions(+) > > create mode 100644 rules/stm32flash.in > > create mode 100644 rules/stm32flash.make > > > > diff --git a/rules/stm32flash.in b/rules/stm32flash.in > > new file mode 100644 > > index 0000000..877eed3 > > --- /dev/null > > +++ b/rules/stm32flash.in > > @@ -0,0 +1,9 @@ > > +## SECTION=shell_and_console > > + > > +config STM32FLASH > > + tristate > > + prompt "stm32flash" > > + help > > + Open source cross platform flash program for the STM32 ARM > > + microcontrollers using the built-in ST serial bootloader over UART > > + or I2C. > > diff --git a/rules/stm32flash.make b/rules/stm32flash.make > > new file mode 100644 > > index 0000000..975d02e > > --- /dev/null > > +++ b/rules/stm32flash.make > > @@ -0,0 +1,57 @@ > > +# -*-makefile-*- > > +# > > +# Copyright (C) 2018 by Guillermo Rodriguez <guille.rodriguez@gmail.com> > > +# > > +# See CREDITS for details about who has contributed to this project. > > +# > > +# For further information about the PTXdist project and license conditions > > +# see the README file. > > +# > > + > > +# > > +# We provide this package > > +# > > +PACKAGES-$(PTXCONF_STM32FLASH) += stm32flash > > + > > +# > > +# Paths and names > > +# > > +STM32FLASH_VERSION := 0.5 > > +STM32FLASH_SUFFIX := tar.gz > > +STM32FLASH_MD5 := 40f673502949f3bb655d2bcc539d7b6a > > +STM32FLASH := stm32flash-$(STM32FLASH_VERSION) > > +STM32FLASH_URL := https://sourceforge.net/projects/stm32flash/files/$(STM32FLASH).$(STM32FLASH_SUFFIX)/download > > Better use $(call ptx/mirror, SF, ...) here to get a list of SourceForge > mirrors. Have a look at other packages for how to use it. Will do. > > > +STM32FLASH_DIR := $(BUILDDIR)/$(STM32FLASH) > > +STM32FLASH_SOURCE := $(SRCDIR)/$(STM32FLASH).$(STM32FLASH_SUFFIX) > > +STM32FLASH_LICENSE := GPLv2 > > + > > + > > +# ---------------------------------------------------------------------------- > > +# Prepare > > +# ---------------------------------------------------------------------------- > > + > > +STM32FLASH_CONF_TOOL := NO > > +STM32FLASH_MAKE_ENV := $(CROSS_ENV) > > +STM32FLASH_INSTALL_OPT := PREFIX=/usr install > > For readability, keep these options on a single line each: > > STM32FLASH_INSTALL_OPT := \ > PREFIX=/usr \ > install > > Also most packages keep *_INSTALL_* variables under the Install section > header, and *_MAKE_* variables under the "Compile" section header. I > would suggest to do it here too to get a unified look over package > rules. Uhm, most of the existing packages I have checked define MAKE_ and INSTALL_ vars in the prepare stage. See for example bzip2 (this is the one I used as a reference), busybox, i2c-tools, openssl, zip ... > > > + > > +# ---------------------------------------------------------------------------- > > +# Target-Install > > +# ---------------------------------------------------------------------------- > > + > > +$(STATEDIR)/stm32flash.targetinstall: > > + @$(call targetinfo) > > + > > + @$(call install_init, stm32flash) > > + @$(call install_fixup, stm32flash, PRIORITY,optional) > > + @$(call install_fixup, stm32flash, SECTION,base) > > + @$(call install_fixup, stm32flash, AUTHOR,"Guillermo Rodriguez <guille.rodriguez@gmail.com>") > > + @$(call install_fixup, stm32flash, DESCRIPTION,missing) > > + > > + @$(call install_copy, stm32flash, 0, 0, 0755, -, /usr/bin/stm32flash) > > + > > + @$(call install_finish, stm32flash) > > + > > + @$(call touch) > > + > > While you're at it, no need for a double blank line here :) OK :) Guillermo > > - Roland > > > + > > +# vim: syntax=make > > -- > > 2.5.4 (Apple Git-61) > > > > > > _______________________________________________ > > ptxdist mailing list > > ptxdist@pengutronix.de > > -- > Roland Hieber | r.hieber@pengutronix.de | > Pengutronix e.K. | https://www.pengutronix.de/ | > Peiner Str. 6-8, 31137 Hildesheim | Phone: +49-5121-206917-5086 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- Guillermo Rodriguez Garcia guille.rodriguez@gmail.com _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ptxdist] [PATCH] stm32flash: New package 2018-10-05 10:43 ` Guillermo Rodriguez Garcia @ 2018-10-05 12:07 ` Roland Hieber 2018-10-05 14:04 ` Michael Olbrich 1 sibling, 0 replies; 7+ messages in thread From: Roland Hieber @ 2018-10-05 12:07 UTC (permalink / raw) To: Guillermo Rodriguez Garcia; +Cc: ptxdist > > Also most packages keep *_INSTALL_* variables under the Install section > > header, and *_MAKE_* variables under the "Compile" section header. I > > would suggest to do it here too to get a unified look over package > > rules. > > Uhm, most of the existing packages I have checked define MAKE_ and > INSTALL_ vars in the prepare stage. See for example bzip2 (this is the > one I used as a reference), busybox, i2c-tools, openssl, zip ... Hm. The style seems to be very mixed between packages. Sorry, my bad. - Roland -- Roland Hieber | r.hieber@pengutronix.de | Pengutronix e.K. | https://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim | Phone: +49-5121-206917-5086 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ptxdist] [PATCH] stm32flash: New package 2018-10-05 10:43 ` Guillermo Rodriguez Garcia 2018-10-05 12:07 ` Roland Hieber @ 2018-10-05 14:04 ` Michael Olbrich 2018-10-06 17:05 ` Guillermo Rodriguez Garcia 1 sibling, 1 reply; 7+ messages in thread From: Michael Olbrich @ 2018-10-05 14:04 UTC (permalink / raw) To: ptxdist On Fri, Oct 05, 2018 at 12:43:01PM +0200, Guillermo Rodriguez Garcia wrote: > > > +# ---------------------------------------------------------------------------- > > > +# Prepare > > > +# ---------------------------------------------------------------------------- > > > + > > > +STM32FLASH_CONF_TOOL := NO > > > +STM32FLASH_MAKE_ENV := $(CROSS_ENV) > > > +STM32FLASH_INSTALL_OPT := PREFIX=/usr install > > > > For readability, keep these options on a single line each: > > > > STM32FLASH_INSTALL_OPT := \ > > PREFIX=/usr \ > > install > > > > Also most packages keep *_INSTALL_* variables under the Install section > > header, and *_MAKE_* variables under the "Compile" section header. I > > would suggest to do it here too to get a unified look over package > > rules. > > Uhm, most of the existing packages I have checked define MAKE_ and > INSTALL_ vars in the prepare stage. See for example bzip2 (this is the > one I used as a reference), busybox, i2c-tools, openssl, zip ... I'd put it all in the prepare section if _only_ these variables are defined and the other sections are skipped entirely. If the actual compile / install target is defined as well, then the variable should be in the corresponding section. Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ptxdist] [PATCH] stm32flash: New package 2018-10-05 14:04 ` Michael Olbrich @ 2018-10-06 17:05 ` Guillermo Rodriguez Garcia 0 siblings, 0 replies; 7+ messages in thread From: Guillermo Rodriguez Garcia @ 2018-10-06 17:05 UTC (permalink / raw) To: ptxdist El vie., 5 oct. 2018 a las 16:04, Michael Olbrich (<m.olbrich@pengutronix.de>) escribió: > > On Fri, Oct 05, 2018 at 12:43:01PM +0200, Guillermo Rodriguez Garcia wrote: > > > > +# ---------------------------------------------------------------------------- > > > > +# Prepare > > > > +# ---------------------------------------------------------------------------- > > > > + > > > > +STM32FLASH_CONF_TOOL := NO > > > > +STM32FLASH_MAKE_ENV := $(CROSS_ENV) > > > > +STM32FLASH_INSTALL_OPT := PREFIX=/usr install > > > > > > For readability, keep these options on a single line each: > > > > > > STM32FLASH_INSTALL_OPT := \ > > > PREFIX=/usr \ > > > install > > > > > > Also most packages keep *_INSTALL_* variables under the Install section > > > header, and *_MAKE_* variables under the "Compile" section header. I > > > would suggest to do it here too to get a unified look over package > > > rules. > > > > Uhm, most of the existing packages I have checked define MAKE_ and > > INSTALL_ vars in the prepare stage. See for example bzip2 (this is the > > one I used as a reference), busybox, i2c-tools, openssl, zip ... > > I'd put it all in the prepare section if _only_ these variables are > defined and the other sections are skipped entirely. If the actual compile > / install target is defined as well, then the variable should be in the > corresponding section. Allright. That's exactly the case here, only these variables are defined and there is nothing else in the compile / install sections. I'll send v2 of the patch then. BR, Guillermo Rodriguez Garcia guille.rodriguez@gmail.com _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* [ptxdist] [PATCH v2] stm32flash: New package 2018-10-05 9:25 [ptxdist] [PATCH] stm32flash: New package Guillermo Rodriguez 2018-10-05 10:30 ` Roland Hieber @ 2018-10-06 17:14 ` Guillermo Rodriguez 1 sibling, 0 replies; 7+ messages in thread From: Guillermo Rodriguez @ 2018-10-06 17:14 UTC (permalink / raw) To: ptxdist; +Cc: Guillermo Rodriguez Open source cross platform flash program for the STM32 ARM microcontrollers using the built-in ST serial bootloader over UART or I2C. Signed-off-by: Guillermo Rodriguez <guille.rodriguez@gmail.com> --- rules/stm32flash.in | 9 +++++++++ rules/stm32flash.make | 56 +++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 65 insertions(+) create mode 100644 rules/stm32flash.in create mode 100644 rules/stm32flash.make diff --git a/rules/stm32flash.in b/rules/stm32flash.in new file mode 100644 index 0000000..877eed3 --- /dev/null +++ b/rules/stm32flash.in @@ -0,0 +1,9 @@ +## SECTION=shell_and_console + +config STM32FLASH + tristate + prompt "stm32flash" + help + Open source cross platform flash program for the STM32 ARM + microcontrollers using the built-in ST serial bootloader over UART + or I2C. diff --git a/rules/stm32flash.make b/rules/stm32flash.make new file mode 100644 index 0000000..a767a5f --- /dev/null +++ b/rules/stm32flash.make @@ -0,0 +1,56 @@ +# -*-makefile-*- +# +# Copyright (C) 2018 by Guillermo Rodriguez <guille.rodriguez@gmail.com> +# +# See CREDITS for details about who has contributed to this project. +# +# For further information about the PTXdist project and license conditions +# see the README file. +# + +# +# We provide this package +# +PACKAGES-$(PTXCONF_STM32FLASH) += stm32flash + +# +# Paths and names +# +STM32FLASH_VERSION := 0.5 +STM32FLASH_SUFFIX := tar.gz +STM32FLASH_MD5 := 40f673502949f3bb655d2bcc539d7b6a +STM32FLASH := stm32flash-$(STM32FLASH_VERSION) +STM32FLASH_URL := $(call ptx/mirror, SF, stm32flash/$(STM32FLASH).$(STM32FLASH_SUFFIX)) +STM32FLASH_DIR := $(BUILDDIR)/$(STM32FLASH) +STM32FLASH_SOURCE := $(SRCDIR)/$(STM32FLASH).$(STM32FLASH_SUFFIX) +STM32FLASH_LICENSE := GPLv2 + + +# ---------------------------------------------------------------------------- +# Prepare +# ---------------------------------------------------------------------------- + +STM32FLASH_CONF_TOOL := NO +STM32FLASH_MAKE_ENV := $(CROSS_ENV) +STM32FLASH_INSTALL_OPT := PREFIX=/usr install + +# ---------------------------------------------------------------------------- +# Target-Install +# ---------------------------------------------------------------------------- + +$(STATEDIR)/stm32flash.targetinstall: + @$(call targetinfo) + + @$(call install_init, stm32flash) + @$(call install_fixup, stm32flash, PRIORITY,optional) + @$(call install_fixup, stm32flash, SECTION,base) + @$(call install_fixup, stm32flash, AUTHOR,"Guillermo Rodriguez <guille.rodriguez@gmail.com>") + @$(call install_fixup, stm32flash, DESCRIPTION,missing) + + @$(call install_copy, stm32flash, 0, 0, 0755, -, /usr/bin/stm32flash) + + @$(call install_finish, stm32flash) + + @$(call touch) + +# vim: syntax=make -- 2.5.4 (Apple Git-61) _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-10-06 17:14 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-10-05 9:25 [ptxdist] [PATCH] stm32flash: New package Guillermo Rodriguez 2018-10-05 10:30 ` Roland Hieber 2018-10-05 10:43 ` Guillermo Rodriguez Garcia 2018-10-05 12:07 ` Roland Hieber 2018-10-05 14:04 ` Michael Olbrich 2018-10-06 17:05 ` Guillermo Rodriguez Garcia 2018-10-06 17:14 ` [ptxdist] [PATCH v2] " Guillermo Rodriguez
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox