mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [RFC PATCH 0/3] Freescale/NXP i.MX mfgtools 3.0
@ 2020-03-24 14:53 Denis Osterland-Heim
  2020-03-24 14:53 ` [ptxdist] [RFC PATCH 2/3] host-libzip: new package Denis Osterland-Heim
                   ` (2 more replies)
  0 siblings, 3 replies; 16+ messages in thread
From: Denis Osterland-Heim @ 2020-03-24 14:53 UTC (permalink / raw)
  To: ptxdist

Hi,

this patch series adds support for the Freescale/NXP mfgtools 3.0.
You can use them with vanilla kernel and barebox.

There are two known issues:
 - the git downloads => no stable md5 sum
 - the linuxrc provided by imx-uuc uses array syntax

I have a version of the linuxrc script where I replaced the array
stuff with blank separation, but not sure if this should be put as
projectroot/linuxrc in ptxdist.

Here are some examples how to use it. I have tested it with a i.MX6ull EVK
barebox-2020.02.0 and linux-5.4.25.

The command `uuu barebox-image` is equivalent to `imx-usb-loader barebox-image`.

The example
https://www.barebox.org/doc/latest/user/usb.html#android-fastboot-support
can be done with a `uuu boot.uu` command.
boot.uu:
```
uuu_version 1.0.1
CFG: FB: -chip imx<variant> -pid 0x0104 -vid 0x1d6b
FB: flash kernel linuximage
FB: flash devicetree imx<your-board>.dtb
FB: flash initrd root.cpio.gz
FB: oem exec global linux.bootargs.fa=rdinit=/sbin/init
FB: oem exec bootm
FB: done
```

To run a shell command a kernel with USB gadgets/functionfs enabled
and imx-uuc install on target is needed.
This is an example of whole USB boot sequence and a command
executed in linux user space.
To see the output of the echo, the command is `uuu -v hello.uu`
hello.uu:
```
uuu_version 1.0.1
SDP: dcd -f barebox-image
SDP: write -f barebox-image -ivt 0
SDP: jump -f barebox-image
CFG: FB: -chip imx<variant> -pid 0x0104 -vid 0x1d6b
FB: flash kernel linuximage
FB: flash devicetree imx<your-board>.dtb
FB: flash initrd root.cpio.gz
FB: oem exec global linux.bootargs.fa=rdinit=/linuxrc
FB: oem exec bootm
FBK: ucmd echo "Hello World"
FBK: done
```

 rules/host-libzip.in     |  8 +++++++
 rules/host-libzip.make   | 11 +++++++++
 rules/host-mfgtools.in   | 13 ++++++++++
 rules/host-mfgtools.make | 43 +++++++++++++++++++++++++++++++++
 rules/imx-uuc.in         |  7 ++++++
 rules/imx-uuc.make       | 62 ++++++++++++++++++++++++++++++++++++++++++++++++
 6 files changed, 144 insertions(+)




Diehl Connectivity Solutions GmbH
Geschäftsführung: Horst Leonberger
Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
Nürnberg: HRB 32315
___________________________________________________________________________________________________

Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
- Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/

The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
- For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 16+ messages in thread

* [ptxdist] [RFC PATCH 2/3] host-libzip: new package
  2020-03-24 14:53 [ptxdist] [RFC PATCH 0/3] Freescale/NXP i.MX mfgtools 3.0 Denis Osterland-Heim
@ 2020-03-24 14:53 ` Denis Osterland-Heim
  2020-03-24 14:53 ` [ptxdist] [RFC PATCH 1/3] imx-uuc: " Denis Osterland-Heim
  2020-03-24 14:53 ` [ptxdist] [RFC PATCH 3/3] host-mfgtools: " Denis Osterland-Heim
  2 siblings, 0 replies; 16+ messages in thread
From: Denis Osterland-Heim @ 2020-03-24 14:53 UTC (permalink / raw)
  To: ptxdist

Signed-off-by: Denis Osterland-Heim <Denis.Osterland@diehl.com>
---
 rules/host-libzip.in   |  8 ++++++++
 rules/host-libzip.make | 11 +++++++++++
 2 files changed, 19 insertions(+)
 create mode 100644 rules/host-libzip.in
 create mode 100644 rules/host-libzip.make

diff --git a/rules/host-libzip.in b/rules/host-libzip.in
new file mode 100644
index 000000000..14689309f
--- /dev/null
+++ b/rules/host-libzip.in
@@ -0,0 +1,8 @@
+## SECTION=hosttools_noprompt
+
+config HOST_LIBZIP
+	tristate
+	select HOST_CMAKE
+	select HOST_ZLIB
+	select HOST_BZIP2
+	default y if ALLYES
diff --git a/rules/host-libzip.make b/rules/host-libzip.make
new file mode 100644
index 000000000..54b16cad0
--- /dev/null
+++ b/rules/host-libzip.make
@@ -0,0 +1,11 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2020 by Denis Osterland-Heim <Denis.Osterland@diehl.com>
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+HOST_PACKAGES-$(PTXCONF_HOST_LIBZIP) += host-libzip
+
+# vim: syntax=make
-- 
2.25.2



Diehl Connectivity Solutions GmbH
Geschäftsführung: Horst Leonberger
Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
Nürnberg: HRB 32315
___________________________________________________________________________________________________

Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
- Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/

The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
- For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 16+ messages in thread

* [ptxdist] [RFC PATCH 1/3] imx-uuc: new package
  2020-03-24 14:53 [ptxdist] [RFC PATCH 0/3] Freescale/NXP i.MX mfgtools 3.0 Denis Osterland-Heim
  2020-03-24 14:53 ` [ptxdist] [RFC PATCH 2/3] host-libzip: new package Denis Osterland-Heim
@ 2020-03-24 14:53 ` Denis Osterland-Heim
  2020-03-25 18:43   ` Roland Hieber
  2020-03-24 14:53 ` [ptxdist] [RFC PATCH 3/3] host-mfgtools: " Denis Osterland-Heim
  2 siblings, 1 reply; 16+ messages in thread
From: Denis Osterland-Heim @ 2020-03-24 14:53 UTC (permalink / raw)
  To: ptxdist

NXP user space util used by mfgtool.

Signed-off-by: Denis Osterland-Heim <Denis.Osterland@diehl.com>
---
 rules/imx-uuc.in   |  7 ++++++
 rules/imx-uuc.make | 62 ++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 69 insertions(+)
 create mode 100644 rules/imx-uuc.in
 create mode 100644 rules/imx-uuc.make

diff --git a/rules/imx-uuc.in b/rules/imx-uuc.in
new file mode 100644
index 000000000..ec0cbcd3a
--- /dev/null
+++ b/rules/imx-uuc.in
@@ -0,0 +1,7 @@
+## SECTION=communication
+
+config IMX_UUC
+	tristate
+	prompt "imx-uuc"
+	help
+	  A Daemon wait for Freescale/NXP mfgtools host's command.
diff --git a/rules/imx-uuc.make b/rules/imx-uuc.make
new file mode 100644
index 000000000..d275ec8af
--- /dev/null
+++ b/rules/imx-uuc.make
@@ -0,0 +1,62 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2016-2020 by Denis Osterland <Denis.Osterland@diehl.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.
+#
+
+PACKAGES-$(PTXCONF_IMX_UUC) += imx-uuc
+
+#
+# Paths and names
+#
+IMX_UUC_VERSION    := d6afb27e55d73d7ad08cd2dd51c784d8ec9694dc
+IMX_UUC_MD5        := 1a807ab59464385309b92794b43b5caa
+IMX_UUC            := imx-uuc-$(IMX_UUC_VERSION)
+IMX_UUC_SUFFIX     := tar.gz
+IMX_UUC_URL        := https://github.com/NXPmicro/imx-uuc.git;tag=$(IMX_UUC_VERSION)
+IMX_UUC_SOURCE     := $(SRCDIR)/$(IMX_UUC).$(IMX_UUC_SUFFIX)
+IMX_UUC_DIR        := $(BUILDDIR)/$(IMX_UUC)
+IMX_UUC_LICENSE    := GPL-2.0-or-later
+IMX_UUC_LICENSE_FILES	:= \
+	file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263
+
+# ----------------------------------------------------------------------------
+# Prepare
+# ----------------------------------------------------------------------------
+
+IMX_UUC_CONF_TOOL	:= NO
+
+# ----------------------------------------------------------------------------
+# Compile
+# ----------------------------------------------------------------------------
+
+IMX_UUC_MAKE_ENV       := $(CROSS_ENV)
+
+# ----------------------------------------------------------------------------
+# Target-Install
+# ----------------------------------------------------------------------------
+
+$(STATEDIR)/imx-uuc.targetinstall:
+	@$(call targetinfo)
+
+	@$(call install_init, imx-uuc)
+	@$(call install_fixup, imx-uuc,PRIORITY,optional)
+	@$(call install_fixup, imx-uuc,SECTION,base)
+	@$(call install_fixup, imx-uuc,AUTHOR,"Denis Osterland <Denis.Osterland@diehl.com>")
+	@$(call install_fixup, imx-uuc,DESCRIPTION,missing)
+
+	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/uuc)
+	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/ufb)
+	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/sdimage)
+	@$(call install_alternative, imx-uuc, 0, 0, 0755, /linuxrc)
+	@$(call install_alternative, imx-uuc, 0, 0, 0644, /fat)
+
+	@$(call install_finish, imx-uuc)
+
+	@$(call touch)
+
+# vim: syntax=make
-- 
2.25.2



Diehl Connectivity Solutions GmbH
Geschäftsführung: Horst Leonberger
Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
Nürnberg: HRB 32315
___________________________________________________________________________________________________

Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
- Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/

The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
- For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 16+ messages in thread

* [ptxdist] [RFC PATCH 3/3] host-mfgtools: new package
  2020-03-24 14:53 [ptxdist] [RFC PATCH 0/3] Freescale/NXP i.MX mfgtools 3.0 Denis Osterland-Heim
  2020-03-24 14:53 ` [ptxdist] [RFC PATCH 2/3] host-libzip: new package Denis Osterland-Heim
  2020-03-24 14:53 ` [ptxdist] [RFC PATCH 1/3] imx-uuc: " Denis Osterland-Heim
@ 2020-03-24 14:53 ` Denis Osterland-Heim
  2020-03-25 18:33   ` Roland Hieber
  2 siblings, 1 reply; 16+ messages in thread
From: Denis Osterland-Heim @ 2020-03-24 14:53 UTC (permalink / raw)
  To: ptxdist

Tool to USB boot i.MX systems from NXP via Android fastboot.

Signed-off-by: Denis Osterland-Heim <Denis.Osterland@diehl.com>
---
 rules/host-mfgtools.in   | 13 ++++++++++++
 rules/host-mfgtools.make | 43 ++++++++++++++++++++++++++++++++++++++++
 2 files changed, 56 insertions(+)
 create mode 100644 rules/host-mfgtools.in
 create mode 100644 rules/host-mfgtools.make

diff --git a/rules/host-mfgtools.in b/rules/host-mfgtools.in
new file mode 100644
index 000000000..c9046f10d
--- /dev/null
+++ b/rules/host-mfgtools.in
@@ -0,0 +1,13 @@
+## SECTION=hosttools
+
+config HOST_MFGTOOLS
+	tristate
+	default y if ALLYES
+	prompt "uuu (Universal Update Utility), mfgtools 3.0"
+	select HOST_CMAKE
+	select HOST_LIBUSB
+	select HOST_ZLIB
+	select HOST_BZIP2
+	select HOST_LIBZIP
+	help
+	  Freescale/NXP I.MX Chip image deploy tools.
diff --git a/rules/host-mfgtools.make b/rules/host-mfgtools.make
new file mode 100644
index 000000000..fb4b2cf05
--- /dev/null
+++ b/rules/host-mfgtools.make
@@ -0,0 +1,43 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2020 by Denis Osterland-Heim <Denis.Osterland@diehl.com>
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+HOST_PACKAGES-$(PTXCONF_HOST_MFGTOOLS) += host-mfgtools
+
+#
+# Paths and names
+#
+HOST_MFGTOOLS_VERSION	:= 1.3.136-9-g47ddef5
+HOST_MFGTOOLS_MD5	:= 1555253943e1f39746694365c7f89eb1
+HOST_MFGTOOLS		:= mfgtools-$(HOST_MFGTOOLS_VERSION)
+HOST_MFGTOOLS_SUFFIX	:= tar.gz
+HOST_MFGTOOLS_URL	:= https://github.com/NXPmicro/mfgtools.git;tag=47ddef51d6c24b1f917f348d0f62b4af38852994
+HOST_MFGTOOLS_SOURCE	:= $(SRCDIR)/$(HOST_MFGTOOLS).$(HOST_MFGTOOLS_SUFFIX)
+HOST_MFGTOOLS_DIR	:= $(HOST_BUILDDIR)/$(HOST_MFGTOOLS)
+HOST_MFGTOOLS_LICENSE	:= BSD-3-Clause
+HOST_MFGTOOLS_LICENSE_FILES	:= \
+	file://LICENSE;md5=38ec0c18112e9a92cffc4951661e85a5
+
+# ----------------------------------------------------------------------------
+# Extract
+# ----------------------------------------------------------------------------
+
+$(STATEDIR)/host-mfgtools.extract:
+	@$(call targetinfo)
+	@$(call clean, $(HOST_MFGTOOLS_DIR))
+	@$(call extract, HOST_MFGTOOLS)
+	@echo "uuu_$(HOST_MFGTOOLS_VERSION)" > $(HOST_MFGTOOLS_DIR)/.tarball-version
+	@$(call patchin, HOST_MFGTOOLS)
+	@$(call touch)
+
+# ----------------------------------------------------------------------------
+# Prepare
+# ----------------------------------------------------------------------------
+
+HOST_MFGTOOLS_CONF_TOOL	:= cmake
+
+# vim: syntax=make
-- 
2.25.2



Diehl Connectivity Solutions GmbH
Geschäftsführung: Horst Leonberger
Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
Nürnberg: HRB 32315
___________________________________________________________________________________________________

Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
- Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/

The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
- For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 3/3] host-mfgtools: new package
  2020-03-24 14:53 ` [ptxdist] [RFC PATCH 3/3] host-mfgtools: " Denis Osterland-Heim
@ 2020-03-25 18:33   ` Roland Hieber
  2020-03-26  8:28     ` Denis Osterland-Heim
  0 siblings, 1 reply; 16+ messages in thread
From: Roland Hieber @ 2020-03-25 18:33 UTC (permalink / raw)
  To: Denis Osterland-Heim; +Cc: ptxdist

On Tue, Mar 24, 2020 at 02:53:23PM +0000, Denis Osterland-Heim wrote:
> Tool to USB boot i.MX systems from NXP via Android fastboot.
> 
> Signed-off-by: Denis Osterland-Heim <Denis.Osterland@diehl.com>
> ---
>  rules/host-mfgtools.in   | 13 ++++++++++++
>  rules/host-mfgtools.make | 43 ++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 56 insertions(+)
>  create mode 100644 rules/host-mfgtools.in
>  create mode 100644 rules/host-mfgtools.make
> 
> diff --git a/rules/host-mfgtools.in b/rules/host-mfgtools.in
> new file mode 100644
> index 000000000..c9046f10d
> --- /dev/null
> +++ b/rules/host-mfgtools.in
> @@ -0,0 +1,13 @@
> +## SECTION=hosttools
> +
> +config HOST_MFGTOOLS
> +	tristate
> +	default y if ALLYES
> +	prompt "uuu (Universal Update Utility), mfgtools 3.0"
> +	select HOST_CMAKE
> +	select HOST_LIBUSB
> +	select HOST_ZLIB
> +	select HOST_BZIP2
> +	select HOST_LIBZIP
> +	help
> +	  Freescale/NXP I.MX Chip image deploy tools.
> diff --git a/rules/host-mfgtools.make b/rules/host-mfgtools.make
> new file mode 100644
> index 000000000..fb4b2cf05
> --- /dev/null
> +++ b/rules/host-mfgtools.make
> @@ -0,0 +1,43 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2020 by Denis Osterland-Heim <Denis.Osterland@diehl.com>
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +HOST_PACKAGES-$(PTXCONF_HOST_MFGTOOLS) += host-mfgtools
> +
> +#
> +# Paths and names
> +#
> +HOST_MFGTOOLS_VERSION	:= 1.3.136-9-g47ddef5

Apparently this commit was released a few days ago as version 1.3.154:
<https://github.com/NXPmicro/mfgtools/releases>

> +HOST_MFGTOOLS_MD5	:= 1555253943e1f39746694365c7f89eb1
> +HOST_MFGTOOLS		:= mfgtools-$(HOST_MFGTOOLS_VERSION)
> +HOST_MFGTOOLS_SUFFIX	:= tar.gz
> +HOST_MFGTOOLS_URL	:= https://github.com/NXPmicro/mfgtools.git;tag=47ddef51d6c24b1f917f348d0f62b4af38852994

So you can use the default release tarball here, and use the default
extract stage below.

> +HOST_MFGTOOLS_SOURCE	:= $(SRCDIR)/$(HOST_MFGTOOLS).$(HOST_MFGTOOLS_SUFFIX)
> +HOST_MFGTOOLS_DIR	:= $(HOST_BUILDDIR)/$(HOST_MFGTOOLS)
> +HOST_MFGTOOLS_LICENSE	:= BSD-3-Clause
> +HOST_MFGTOOLS_LICENSE_FILES	:= \
> +	file://LICENSE;md5=38ec0c18112e9a92cffc4951661e85a5
> +
> +# ----------------------------------------------------------------------------
> +# Extract
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/host-mfgtools.extract:
> +	@$(call targetinfo)
> +	@$(call clean, $(HOST_MFGTOOLS_DIR))
> +	@$(call extract, HOST_MFGTOOLS)
> +	@echo "uuu_$(HOST_MFGTOOLS_VERSION)" > $(HOST_MFGTOOLS_DIR)/.tarball-version

NB: The released tarballs also don't have this file, is it really needed?

 - Roland

> +	@$(call patchin, HOST_MFGTOOLS)
> +	@$(call touch)
> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +HOST_MFGTOOLS_CONF_TOOL	:= cmake
> +
> +# vim: syntax=make
> -- 
> 2.25.2
> 
> 
> 
> Diehl Connectivity Solutions GmbH
> Geschäftsführung: Horst Leonberger
> Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> Nürnberg: HRB 32315
> ___________________________________________________________________________________________________
> 
> Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
> Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
> - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> 
> The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
> mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
> - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

-- 
Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
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] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 1/3] imx-uuc: new package
  2020-03-24 14:53 ` [ptxdist] [RFC PATCH 1/3] imx-uuc: " Denis Osterland-Heim
@ 2020-03-25 18:43   ` Roland Hieber
  2020-03-25 18:52     ` Roland Hieber
  2020-03-26  9:59     ` Guillermo Rodriguez Garcia
  0 siblings, 2 replies; 16+ messages in thread
From: Roland Hieber @ 2020-03-25 18:43 UTC (permalink / raw)
  To: ptxdist

On Tue, Mar 24, 2020 at 02:53:23PM +0000, Denis Osterland-Heim wrote:
> NXP user space util used by mfgtool.
> 
> Signed-off-by: Denis Osterland-Heim <Denis.Osterland@diehl.com>
> ---
>  rules/imx-uuc.in   |  7 ++++++
>  rules/imx-uuc.make | 62 ++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 69 insertions(+)
>  create mode 100644 rules/imx-uuc.in
>  create mode 100644 rules/imx-uuc.make
> 
> diff --git a/rules/imx-uuc.in b/rules/imx-uuc.in
> new file mode 100644
> index 000000000..ec0cbcd3a
> --- /dev/null
> +++ b/rules/imx-uuc.in
> @@ -0,0 +1,7 @@
> +## SECTION=communication
> +
> +config IMX_UUC
> +	tristate
> +	prompt "imx-uuc"
> +	help
> +	  A Daemon wait for Freescale/NXP mfgtools host's command.
> diff --git a/rules/imx-uuc.make b/rules/imx-uuc.make
> new file mode 100644
> index 000000000..d275ec8af
> --- /dev/null
> +++ b/rules/imx-uuc.make
> @@ -0,0 +1,62 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2016-2020 by Denis Osterland <Denis.Osterland@diehl.com>
> +#
> +# See CREDITS for details about who has contributed to this project.

This line keeps coming back to haunt us… :)
https://git.pengutronix.de/cgit/ptxdist/commit?h=cbd6bda680c6

> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +PACKAGES-$(PTXCONF_IMX_UUC) += imx-uuc
> +
> +#
> +# Paths and names
> +#
> +IMX_UUC_VERSION    := d6afb27e55d73d7ad08cd2dd51c784d8ec9694dc

Nitpick: I don't know how picky opkg-based systems are with having
monotonically increasing version numbers, but in case someone uses them
to update software, you could make them happy by providing a fake
'git describe' tag as a version:

IMX_UUC_VERSION_HASH    := d6afb27e55d73d7ad08cd2dd51c784d8ec9694dc
IMX_UUC_VERSION         := 

monotonically increasing version number instead of a randomly changing
hash. If you put something like '0.0-1-gd6afb27e55d73d7ad08cd2d', git
will still be able to extract the correct commit hash from it.

> +IMX_UUC_MD5        := 1a807ab59464385309b92794b43b5caa
> +IMX_UUC            := imx-uuc-$(IMX_UUC_VERSION)
> +IMX_UUC_SUFFIX     := tar.gz
> +IMX_UUC_URL        := https://github.com/NXPmicro/imx-uuc.git;tag=$(IMX_UUC_VERSION)
> +IMX_UUC_SOURCE     := $(SRCDIR)/$(IMX_UUC).$(IMX_UUC_SUFFIX)
> +IMX_UUC_DIR        := $(BUILDDIR)/$(IMX_UUC)
> +IMX_UUC_LICENSE    := GPL-2.0-or-later
> +IMX_UUC_LICENSE_FILES	:= \
> +	file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263
> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +IMX_UUC_CONF_TOOL	:= NO
> +
> +# ----------------------------------------------------------------------------
> +# Compile
> +# ----------------------------------------------------------------------------
> +
> +IMX_UUC_MAKE_ENV       := $(CROSS_ENV)
> +
> +# ----------------------------------------------------------------------------
> +# Target-Install
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/imx-uuc.targetinstall:
> +	@$(call targetinfo)
> +
> +	@$(call install_init, imx-uuc)
> +	@$(call install_fixup, imx-uuc,PRIORITY,optional)
> +	@$(call install_fixup, imx-uuc,SECTION,base)
> +	@$(call install_fixup, imx-uuc,AUTHOR,"Denis Osterland <Denis.Osterland@diehl.com>")
> +	@$(call install_fixup, imx-uuc,DESCRIPTION,missing)
> +
> +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/uuc)
> +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/ufb)
> +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/sdimage)
> +	@$(call install_alternative, imx-uuc, 0, 0, 0755, /linuxrc)

Upstream has a bold choice of config file locations...

 - Roland

> +	@$(call install_alternative, imx-uuc, 0, 0, 0644, /fat)
> +
> +	@$(call install_finish, imx-uuc)
> +
> +	@$(call touch)
> +
> +# vim: syntax=make
> -- 
> 2.25.2
> 
> 
> 
> Diehl Connectivity Solutions GmbH
> Geschäftsführung: Horst Leonberger
> Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> Nürnberg: HRB 32315
> ___________________________________________________________________________________________________
> 
> Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
> Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
> - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> 
> The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
> mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
> - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

-- 
Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
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] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 1/3] imx-uuc: new package
  2020-03-25 18:43   ` Roland Hieber
@ 2020-03-25 18:52     ` Roland Hieber
  2020-03-25 20:40       ` Roland Hieber
  2020-03-26  9:59     ` Guillermo Rodriguez Garcia
  1 sibling, 1 reply; 16+ messages in thread
From: Roland Hieber @ 2020-03-25 18:52 UTC (permalink / raw)
  To: Denis Osterland-Heim; +Cc: ptxdist

On Wed, Mar 25, 2020 at 07:43:46PM +0100, Roland Hieber wrote:
> On Tue, Mar 24, 2020 at 02:53:23PM +0000, Denis Osterland-Heim wrote:
> > NXP user space util used by mfgtool.
> > 
> > Signed-off-by: Denis Osterland-Heim <Denis.Osterland@diehl.com>
> > ---
> >  rules/imx-uuc.in   |  7 ++++++
> >  rules/imx-uuc.make | 62 ++++++++++++++++++++++++++++++++++++++++++++++
> >  2 files changed, 69 insertions(+)
> >  create mode 100644 rules/imx-uuc.in
> >  create mode 100644 rules/imx-uuc.make
> > 
> > diff --git a/rules/imx-uuc.in b/rules/imx-uuc.in
> > new file mode 100644
> > index 000000000..ec0cbcd3a
> > --- /dev/null
> > +++ b/rules/imx-uuc.in
> > @@ -0,0 +1,7 @@
> > +## SECTION=communication
> > +
> > +config IMX_UUC
> > +	tristate
> > +	prompt "imx-uuc"
> > +	help
> > +	  A Daemon wait for Freescale/NXP mfgtools host's command.
> > diff --git a/rules/imx-uuc.make b/rules/imx-uuc.make
> > new file mode 100644
> > index 000000000..d275ec8af
> > --- /dev/null
> > +++ b/rules/imx-uuc.make
> > @@ -0,0 +1,62 @@
> > +# -*-makefile-*-
> > +#
> > +# Copyright (C) 2016-2020 by Denis Osterland <Denis.Osterland@diehl.com>
> > +#
> > +# See CREDITS for details about who has contributed to this project.
> 
> This line keeps coming back to haunt us… :)
> https://git.pengutronix.de/cgit/ptxdist/commit?h=cbd6bda680c6
> 
> > +#
> > +# For further information about the PTXdist project and license conditions
> > +# see the README file.
> > +#
> > +
> > +PACKAGES-$(PTXCONF_IMX_UUC) += imx-uuc
> > +
> > +#
> > +# Paths and names
> > +#
> > +IMX_UUC_VERSION    := d6afb27e55d73d7ad08cd2dd51c784d8ec9694dc
> > +IMX_UUC_MD5        := 1a807ab59464385309b92794b43b5caa

Sorry, I slipped while writing the mail. What I wanted to write in the
first place:

Nitpick: I don't know how picky opkg-based systems are with having
monotonically increasing version numbers, but in case someone uses them
to update software in the field, you could make them happy by providing
a fake 'git describe' tag as a version:

IMX_UUC_VERSION    := 0.0-1-gd6afb27e55d73d7ad08cd2dd51c784d8ec9694dc

and then increase the -1 every time for a version bump. This way git
can extract the commit ID from the version info and opkg versions are
increasing monotonically. The MD5 will be different though.

 - Roland

> > +IMX_UUC            := imx-uuc-$(IMX_UUC_VERSION)
> > +IMX_UUC_SUFFIX     := tar.gz
> > +IMX_UUC_URL        := https://github.com/NXPmicro/imx-uuc.git;tag=$(IMX_UUC_VERSION)
> > +IMX_UUC_SOURCE     := $(SRCDIR)/$(IMX_UUC).$(IMX_UUC_SUFFIX)
> > +IMX_UUC_DIR        := $(BUILDDIR)/$(IMX_UUC)
> > +IMX_UUC_LICENSE    := GPL-2.0-or-later
> > +IMX_UUC_LICENSE_FILES	:= \
> > +	file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263
> > +
> > +# ----------------------------------------------------------------------------
> > +# Prepare
> > +# ----------------------------------------------------------------------------
> > +
> > +IMX_UUC_CONF_TOOL	:= NO
> > +
> > +# ----------------------------------------------------------------------------
> > +# Compile
> > +# ----------------------------------------------------------------------------
> > +
> > +IMX_UUC_MAKE_ENV       := $(CROSS_ENV)
> > +
> > +# ----------------------------------------------------------------------------
> > +# Target-Install
> > +# ----------------------------------------------------------------------------
> > +
> > +$(STATEDIR)/imx-uuc.targetinstall:
> > +	@$(call targetinfo)
> > +
> > +	@$(call install_init, imx-uuc)
> > +	@$(call install_fixup, imx-uuc,PRIORITY,optional)
> > +	@$(call install_fixup, imx-uuc,SECTION,base)
> > +	@$(call install_fixup, imx-uuc,AUTHOR,"Denis Osterland <Denis.Osterland@diehl.com>")
> > +	@$(call install_fixup, imx-uuc,DESCRIPTION,missing)
> > +
> > +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/uuc)
> > +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/ufb)
> > +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/sdimage)
> > +	@$(call install_alternative, imx-uuc, 0, 0, 0755, /linuxrc)
> 
> Upstream has a bold choice of config file locations...
> 
>  - Roland
> 
> > +	@$(call install_alternative, imx-uuc, 0, 0, 0644, /fat)
> > +
> > +	@$(call install_finish, imx-uuc)
> > +
> > +	@$(call touch)
> > +
> > +# vim: syntax=make
> > -- 
> > 2.25.2
> > 
> > 
> > 
> > Diehl Connectivity Solutions GmbH
> > Geschäftsführung: Horst Leonberger
> > Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> > Nürnberg: HRB 32315
> > ___________________________________________________________________________________________________
> > 
> > Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> > Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
> > Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
> > - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> > 
> > The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
> > mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
> > - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> > 
> > _______________________________________________
> > ptxdist mailing list
> > ptxdist@pengutronix.de
> > 
> 
> -- 
> Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
> Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
> 31137 Hildesheim, Germany                | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686         | Fax:   +49-5121-206917-5555 |
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
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] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 1/3] imx-uuc: new package
  2020-03-25 18:52     ` Roland Hieber
@ 2020-03-25 20:40       ` Roland Hieber
  2020-03-26  9:03         ` Denis Osterland-Heim
  0 siblings, 1 reply; 16+ messages in thread
From: Roland Hieber @ 2020-03-25 20:40 UTC (permalink / raw)
  To: Denis Osterland-Heim; +Cc: ptxdist

On Wed, Mar 25, 2020 at 07:52:54PM +0100, Roland Hieber wrote:
> On Wed, Mar 25, 2020 at 07:43:46PM +0100, Roland Hieber wrote:
> > On Tue, Mar 24, 2020 at 02:53:23PM +0000, Denis Osterland-Heim wrote:
> > > NXP user space util used by mfgtool.
> > > 
> > > Signed-off-by: Denis Osterland-Heim <Denis.Osterland@diehl.com>
> > > ---
> > >  rules/imx-uuc.in   |  7 ++++++
> > >  rules/imx-uuc.make | 62 ++++++++++++++++++++++++++++++++++++++++++++++
> > >  2 files changed, 69 insertions(+)
> > >  create mode 100644 rules/imx-uuc.in
> > >  create mode 100644 rules/imx-uuc.make
> > > 
> > > diff --git a/rules/imx-uuc.in b/rules/imx-uuc.in
> > > new file mode 100644
> > > index 000000000..ec0cbcd3a
> > > --- /dev/null
> > > +++ b/rules/imx-uuc.in
> > > @@ -0,0 +1,7 @@
> > > +## SECTION=communication
> > > +
> > > +config IMX_UUC
> > > +	tristate
> > > +	prompt "imx-uuc"
> > > +	help
> > > +	  A Daemon wait for Freescale/NXP mfgtools host's command.

Oh, and also keep the tabs as indentation here, not spaces (except for
the two spaces before the help text).

> > > diff --git a/rules/imx-uuc.make b/rules/imx-uuc.make
> > > new file mode 100644
> > > index 000000000..d275ec8af
> > > --- /dev/null
> > > +++ b/rules/imx-uuc.make
> > > @@ -0,0 +1,62 @@
> > > +# -*-makefile-*-
> > > +#
> > > +# Copyright (C) 2016-2020 by Denis Osterland <Denis.Osterland@diehl.com>
> > > +#
> > > +# See CREDITS for details about who has contributed to this project.
> > 
> > This line keeps coming back to haunt us… :)
> > https://git.pengutronix.de/cgit/ptxdist/commit?h=cbd6bda680c6
> > 
> > > +#
> > > +# For further information about the PTXdist project and license conditions
> > > +# see the README file.
> > > +#
> > > +
> > > +PACKAGES-$(PTXCONF_IMX_UUC) += imx-uuc
> > > +
> > > +#
> > > +# Paths and names
> > > +#
> > > +IMX_UUC_VERSION    := d6afb27e55d73d7ad08cd2dd51c784d8ec9694dc
> > > +IMX_UUC_MD5        := 1a807ab59464385309b92794b43b5caa

Same here: tabs for alignment before the :=.

> Sorry, I slipped while writing the mail. What I wanted to write in the
> first place:
> 
> Nitpick: I don't know how picky opkg-based systems are with having
> monotonically increasing version numbers, but in case someone uses them
> to update software in the field, you could make them happy by providing
> a fake 'git describe' tag as a version:
> 
> IMX_UUC_VERSION    := 0.0-1-gd6afb27e55d73d7ad08cd2dd51c784d8ec9694dc
> 
> and then increase the -1 every time for a version bump. This way git
> can extract the commit ID from the version info and opkg versions are
> increasing monotonically. The MD5 will be different though.
> 
>  - Roland
> 
> > > +IMX_UUC            := imx-uuc-$(IMX_UUC_VERSION)
> > > +IMX_UUC_SUFFIX     := tar.gz
> > > +IMX_UUC_URL        := https://github.com/NXPmicro/imx-uuc.git;tag=$(IMX_UUC_VERSION)
> > > +IMX_UUC_SOURCE     := $(SRCDIR)/$(IMX_UUC).$(IMX_UUC_SUFFIX)
> > > +IMX_UUC_DIR        := $(BUILDDIR)/$(IMX_UUC)
> > > +IMX_UUC_LICENSE    := GPL-2.0-or-later
> > > +IMX_UUC_LICENSE_FILES	:= \
> > > +	file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263
> > > +
> > > +# ----------------------------------------------------------------------------
> > > +# Prepare
> > > +# ----------------------------------------------------------------------------
> > > +
> > > +IMX_UUC_CONF_TOOL	:= NO
> > > +
> > > +# ----------------------------------------------------------------------------
> > > +# Compile
> > > +# ----------------------------------------------------------------------------
> > > +
> > > +IMX_UUC_MAKE_ENV       := $(CROSS_ENV)
> > > +
> > > +# ----------------------------------------------------------------------------
> > > +# Target-Install
> > > +# ----------------------------------------------------------------------------
> > > +
> > > +$(STATEDIR)/imx-uuc.targetinstall:
> > > +	@$(call targetinfo)

And here too, otherwise make will complain for recipes.

 - Roland

> > > +
> > > +	@$(call install_init, imx-uuc)
> > > +	@$(call install_fixup, imx-uuc,PRIORITY,optional)
> > > +	@$(call install_fixup, imx-uuc,SECTION,base)
> > > +	@$(call install_fixup, imx-uuc,AUTHOR,"Denis Osterland <Denis.Osterland@diehl.com>")
> > > +	@$(call install_fixup, imx-uuc,DESCRIPTION,missing)
> > > +
> > > +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/uuc)
> > > +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/ufb)
> > > +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/sdimage)
> > > +	@$(call install_alternative, imx-uuc, 0, 0, 0755, /linuxrc)
> > 
> > Upstream has a bold choice of config file locations...
> > 
> >  - Roland
> > 
> > > +	@$(call install_alternative, imx-uuc, 0, 0, 0644, /fat)
> > > +
> > > +	@$(call install_finish, imx-uuc)
> > > +
> > > +	@$(call touch)
> > > +
> > > +# vim: syntax=make
> > > -- 
> > > 2.25.2
> > > 
> > > 
> > > 
> > > Diehl Connectivity Solutions GmbH
> > > Geschäftsführung: Horst Leonberger
> > > Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> > > Nürnberg: HRB 32315
> > > ___________________________________________________________________________________________________
> > > 
> > > Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> > > Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
> > > Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
> > > - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> > > 
> > > The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
> > > mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
> > > - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> > > 
> > > _______________________________________________
> > > ptxdist mailing list
> > > ptxdist@pengutronix.de
> > > 
> > 
> > -- 
> > Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
> > Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
> > 31137 Hildesheim, Germany                | Phone: +49-5121-206917-0    |
> > Amtsgericht Hildesheim, HRA 2686         | Fax:   +49-5121-206917-5555 |
> > 
> > _______________________________________________
> > ptxdist mailing list
> > ptxdist@pengutronix.de
> 
> -- 
> Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
> Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
> 31137 Hildesheim, Germany                | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686         | Fax:   +49-5121-206917-5555 |
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
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] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 3/3] host-mfgtools: new package
  2020-03-25 18:33   ` Roland Hieber
@ 2020-03-26  8:28     ` Denis Osterland-Heim
  2020-03-26 16:16       ` Michael Olbrich
  0 siblings, 1 reply; 16+ messages in thread
From: Denis Osterland-Heim @ 2020-03-26  8:28 UTC (permalink / raw)
  To: rhi; +Cc: ptxdist

Hi Roland,

sorry for the confusion yesterday, seems it was to late for me...

Am Mittwoch, den 25.03.2020, 19:33 +0100 schrieb Roland Hieber:
> On Tue, Mar 24, 2020 at 02:53:23PM +0000, Denis Osterland-Heim wrote:
> > Tool to USB boot i.MX systems from NXP via Android fastboot.
> > 
> > Signed-off-by: Denis Osterland-Heim <Denis.Osterland@diehl.com>
> > ---
> >  rules/host-mfgtools.in   | 13 ++++++++++++
> >  rules/host-mfgtools.make | 43 ++++++++++++++++++++++++++++++++++++++++
> >  2 files changed, 56 insertions(+)
> >  create mode 100644 rules/host-mfgtools.in
> >  create mode 100644 rules/host-mfgtools.make
> > 
> > diff --git a/rules/host-mfgtools.in b/rules/host-mfgtools.in
> > new file mode 100644
> > index 000000000..c9046f10d
> > --- /dev/null
> > +++ b/rules/host-mfgtools.in
> > @@ -0,0 +1,13 @@
> > +## SECTION=hosttools
> > +
> > +config HOST_MFGTOOLS
> > +	tristate
> > +	default y if ALLYES
> > +	prompt "uuu (Universal Update Utility), mfgtools 3.0"
> > +	select HOST_CMAKE
> > +	select HOST_LIBUSB
> > +	select HOST_ZLIB
> > +	select HOST_BZIP2
> > +	select HOST_LIBZIP
> > +	help
> > +	  Freescale/NXP I.MX Chip image deploy tools.
> > diff --git a/rules/host-mfgtools.make b/rules/host-mfgtools.make
> > new file mode 100644
> > index 000000000..fb4b2cf05
> > --- /dev/null
> > +++ b/rules/host-mfgtools.make
> > @@ -0,0 +1,43 @@
> > +# -*-makefile-*-
> > +#
> > +# Copyright (C) 2020 by Denis Osterland-Heim <Denis.Osterland@diehl.com>
> > +#
> > +# For further information about the PTXdist project and license conditions
> > +# see the README file.
> > +#
> > +
> > +HOST_PACKAGES-$(PTXCONF_HOST_MFGTOOLS) += host-mfgtools
> > +
> > +#
> > +# Paths and names
> > +#
> > +HOST_MFGTOOLS_VERSION	:= 1.3.136-9-g47ddef5
> 
> Apparently this commit was released a few days ago as version 1.3.154:
> <https://github.com/NXPmicro/mfgtools/releases>
I have already changed it locally, v2 will come ;-)

> 
> > +HOST_MFGTOOLS_MD5	:= 1555253943e1f39746694365c7f89eb1
> > +HOST_MFGTOOLS		:= mfgtools-$(HOST_MFGTOOLS_VERSION)
> > +HOST_MFGTOOLS_SUFFIX	:= tar.gz
> > +HOST_MFGTOOLS_URL	:= https://github.com/NXPmicro/mfgtools.git;tag=47ddef51d6c24b1f917f348d0f62b4af38852994
> 
> So you can use the default release tarball here, and use the default
> extract stage below.
Not yet, but hopefully with next version.

> 
> > +HOST_MFGTOOLS_SOURCE	:= $(SRCDIR)/$(HOST_MFGTOOLS).$(HOST_MFGTOOLS_SUFFIX)
> > +HOST_MFGTOOLS_DIR	:= $(HOST_BUILDDIR)/$(HOST_MFGTOOLS)
> > +HOST_MFGTOOLS_LICENSE	:= BSD-3-Clause
> > +HOST_MFGTOOLS_LICENSE_FILES	:= \
> > +	file://LICENSE;md5=38ec0c18112e9a92cffc4951661e85a5
> > +
> > +# ----------------------------------------------------------------------------
> > +# Extract
> > +# ----------------------------------------------------------------------------
> > +
> > +$(STATEDIR)/host-mfgtools.extract:
> > +	@$(call targetinfo)
> > +	@$(call clean, $(HOST_MFGTOOLS_DIR))
> > +	@$(call extract, HOST_MFGTOOLS)
> > +	@echo "uuu_$(HOST_MFGTOOLS_VERSION)" > $(HOST_MFGTOOLS_DIR)/.tarball-version
> 
> NB: The released tarballs also don't have this file, is it really needed?
Unfortunately it is still need, until upstream generates a tarball with the file.
https://github.com/NXPmicro/mfgtools/issues/173

Regards Denis

> 
>  - Roland
> 
> > +	@$(call patchin, HOST_MFGTOOLS)
> > +	@$(call touch)
> > +
> > +# ----------------------------------------------------------------------------
> > +# Prepare
> > +# ----------------------------------------------------------------------------
> > +
> > +HOST_MFGTOOLS_CONF_TOOL	:= cmake
> > +
> > +# vim: syntax=make
> > -- 
> > 2.25.2
> > 
> > 
> > 
> > Diehl Connectivity Solutions GmbH
> > Geschäftsführung: Horst Leonberger
> > Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> > Nürnberg: HRB 32315
> > ___________________________________________________________________________________________________
> > 
> > Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> > Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
> > Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
> > - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> > 
> > The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
> > mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
> > - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> > 
> > _______________________________________________
> > ptxdist mailing list
> > ptxdist@pengutronix.de
> > 
> 
> 


Diehl Connectivity Solutions GmbH
Geschäftsführung: Horst Leonberger
Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
Nürnberg: HRB 32315
___________________________________________________________________________________________________

Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
- Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/

The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
- For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 1/3] imx-uuc: new package
  2020-03-25 20:40       ` Roland Hieber
@ 2020-03-26  9:03         ` Denis Osterland-Heim
  2020-03-26  9:47           ` rhi
  2020-03-26 16:23           ` Michael Olbrich
  0 siblings, 2 replies; 16+ messages in thread
From: Denis Osterland-Heim @ 2020-03-26  9:03 UTC (permalink / raw)
  To: rhi; +Cc: ptxdist

Hi Roland,

Am Mittwoch, den 25.03.2020, 21:40 +0100 schrieb Roland Hieber:
> On Wed, Mar 25, 2020 at 07:52:54PM +0100, Roland Hieber wrote:
> > On Wed, Mar 25, 2020 at 07:43:46PM +0100, Roland Hieber wrote:
> > > On Tue, Mar 24, 2020 at 02:53:23PM +0000, Denis Osterland-Heim wrote:
> > > > NXP user space util used by mfgtool.
> > > > 
> > > > Signed-off-by: Denis Osterland-Heim <Denis.Osterland@diehl.com>
> > > > ---
> > > >  rules/imx-uuc.in   |  7 ++++++
> > > >  rules/imx-uuc.make | 62 ++++++++++++++++++++++++++++++++++++++++++++++
> > > >  2 files changed, 69 insertions(+)
> > > >  create mode 100644 rules/imx-uuc.in
> > > >  create mode 100644 rules/imx-uuc.make
> > > > 
> > > > diff --git a/rules/imx-uuc.in b/rules/imx-uuc.in
> > > > new file mode 100644
> > > > index 000000000..ec0cbcd3a
> > > > --- /dev/null
> > > > +++ b/rules/imx-uuc.in
> > > > @@ -0,0 +1,7 @@
> > > > +## SECTION=communication
> > > > +
> > > > +config IMX_UUC
> > > > +	tristate
> > > > +	prompt "imx-uuc"
> > > > +	help
> > > > +	  A Daemon wait for Freescale/NXP mfgtools host's command.
> 
> Oh, and also keep the tabs as indentation here, not spaces (except for
> the two spaces before the help text).
Yes, of course. It are tabs. When I select it, it is bigger than one space.

> 
> > > > diff --git a/rules/imx-uuc.make b/rules/imx-uuc.make
> > > > new file mode 100644
> > > > index 000000000..d275ec8af
> > > > --- /dev/null
> > > > +++ b/rules/imx-uuc.make
> > > > @@ -0,0 +1,62 @@
> > > > +# -*-makefile-*-
> > > > +#
> > > > +# Copyright (C) 2016-2020 by Denis Osterland <Denis.Osterland@diehl.com>
> > > > +#
> > > > +# See CREDITS for details about who has contributed to this project.
> > > 
> > > This line keeps coming back to haunt us… :)
> > > https://git.pengutronix.de/cgit/ptxdist/commit?h=cbd6bda680c6
I see. I will remove it. It is lying around since 2016.

> > > 
> > > > +#
> > > > +# For further information about the PTXdist project and license conditions
> > > > +# see the README file.
> > > > +#
> > > > +
> > > > +PACKAGES-$(PTXCONF_IMX_UUC) += imx-uuc
> > > > +
> > > > +#
> > > > +# Paths and names
> > > > +#
> > > > +IMX_UUC_VERSION    := d6afb27e55d73d7ad08cd2dd51c784d8ec9694dc
> > > > +IMX_UUC_MD5        := 1a807ab59464385309b92794b43b5caa
> 
> Same here: tabs for alignment before the :=.
I will change it.

> 
> > Sorry, I slipped while writing the mail. What I wanted to write in the
> > first place:
> > 
> > Nitpick: I don't know how picky opkg-based systems are with having
> > monotonically increasing version numbers, but in case someone uses them
> > to update software in the field, you could make them happy by providing
> > a fake 'git describe' tag as a version:
> > 
> > IMX_UUC_VERSION    := 0.0-1-gd6afb27e55d73d7ad08cd2dd51c784d8ec9694dc
> > 
> > and then increase the -1 every time for a version bump. This way git
> > can extract the commit ID from the version info and opkg versions are
> > increasing monotonically. The MD5 will be different though.
Cool. I will apply.

I hope for a release, but looks not promising.
https://github.com/NXPmicro/imx-uuc/issues/12

> > 
> >  - Roland
> > 
> > > > +IMX_UUC            := imx-uuc-$(IMX_UUC_VERSION)
> > > > +IMX_UUC_SUFFIX     := tar.gz
> > > > +IMX_UUC_URL        := https://github.com/NXPmicro/imx-uuc.git;tag=$(IMX_UUC_VERSION)
> > > > +IMX_UUC_SOURCE     := $(SRCDIR)/$(IMX_UUC).$(IMX_UUC_SUFFIX)
> > > > +IMX_UUC_DIR        := $(BUILDDIR)/$(IMX_UUC)
> > > > +IMX_UUC_LICENSE    := GPL-2.0-or-later
> > > > +IMX_UUC_LICENSE_FILES	:= \
> > > > +	file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263
> > > > +
> > > > +# ----------------------------------------------------------------------------
> > > > +# Prepare
> > > > +# ----------------------------------------------------------------------------
> > > > +
> > > > +IMX_UUC_CONF_TOOL	:= NO
> > > > +
> > > > +# ----------------------------------------------------------------------------
> > > > +# Compile
> > > > +# ----------------------------------------------------------------------------
> > > > +
> > > > +IMX_UUC_MAKE_ENV       := $(CROSS_ENV)
> > > > +
> > > > +# ----------------------------------------------------------------------------
> > > > +# Target-Install
> > > > +# ----------------------------------------------------------------------------
> > > > +
> > > > +$(STATEDIR)/imx-uuc.targetinstall:
> > > > +	@$(call targetinfo)
> 
> And here too, otherwise make will complain for recipes.
definitely a tab.
Do you have problems to apply it?

> 
>  - Roland
> 
> > > > +
> > > > +	@$(call install_init, imx-uuc)
> > > > +	@$(call install_fixup, imx-uuc,PRIORITY,optional)
> > > > +	@$(call install_fixup, imx-uuc,SECTION,base)
> > > > +	@$(call install_fixup, imx-uuc,AUTHOR,"Denis Osterland <Denis.Osterland@diehl.com>")
> > > > +	@$(call install_fixup, imx-uuc,DESCRIPTION,missing)
> > > > +
> > > > +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/uuc)
> > > > +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/ufb)
> > > > +	@$(call install_copy, imx-uuc, 0, 0, 0755, -, /usr/bin/sdimage)
> > > > +	@$(call install_alternative, imx-uuc, 0, 0, 0755, /linuxrc)
> > > 
> > > Upstream has a bold choice of config file locations...
I am not sure what you want to say here.
https://github.com/NXPmicro/imx-uuc/blob/master/linuxrc
sets up the USB function fs stuff for mfgtool.
Unfortunately it uses array syntax, which is not supported by busybox.
You may enable bash (have not tested and shebang says sh) or change the script.
That´s what I did.
Maybe I find the time to fix the script upstream, to use only sh features.

Regards Denis

> > > 
> > >  - Roland
> > > 
> > > > +	@$(call install_alternative, imx-uuc, 0, 0, 0644, /fat)
> > > > +
> > > > +	@$(call install_finish, imx-uuc)
> > > > +
> > > > +	@$(call touch)
> > > > +
> > > > +# vim: syntax=make
> > > > -- 
> > > > 2.25.2
> > > > 
> > > > 
> > > > 
> > > > Diehl Connectivity Solutions GmbH
> > > > Geschäftsführung: Horst Leonberger
> > > > Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> > > > Nürnberg: HRB 32315
> > > > ___________________________________________________________________________________________________
> > > > 
> > > > Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> > > > Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
> > > > Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
> > > > - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> > > > 
> > > > The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
> > > > mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
> > > > - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> > > > 
> > > > _______________________________________________
> > > > ptxdist mailing list
> > > > ptxdist@pengutronix.de
> > > > 
> > > 
> > > -- 
> > > Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
> > > Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
> > > 31137 Hildesheim, Germany                | Phone: +49-5121-206917-0    |
> > > Amtsgericht Hildesheim, HRA 2686         | Fax:   +49-5121-206917-5555 |
> > > 
> > > _______________________________________________
> > > ptxdist mailing list
> > > ptxdist@pengutronix.de
> > 
> > -- 
> > Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
> > Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
> > 31137 Hildesheim, Germany                | Phone: +49-5121-206917-0    |
> > Amtsgericht Hildesheim, HRA 2686         | Fax:   +49-5121-206917-5555 |
> > 
> > _______________________________________________
> > ptxdist mailing list
> > ptxdist@pengutronix.de
> 
> 


Diehl Connectivity Solutions GmbH
Geschäftsführung: Horst Leonberger
Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
Nürnberg: HRB 32315
___________________________________________________________________________________________________

Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
- Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/

The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
- For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 1/3] imx-uuc: new package
  2020-03-26  9:03         ` Denis Osterland-Heim
@ 2020-03-26  9:47           ` rhi
  2020-03-26 16:23           ` Michael Olbrich
  1 sibling, 0 replies; 16+ messages in thread
From: rhi @ 2020-03-26  9:47 UTC (permalink / raw)
  To: Denis Osterland-Heim; +Cc: ptxdist

On Thu, Mar 26, 2020 at 09:03:58AM +0000, Denis Osterland-Heim wrote:
> Hi Roland,
> 
> Am Mittwoch, den 25.03.2020, 21:40 +0100 schrieb Roland Hieber:
> > On Wed, Mar 25, 2020 at 07:52:54PM +0100, Roland Hieber wrote:
> > > On Wed, Mar 25, 2020 at 07:43:46PM +0100, Roland Hieber wrote:
> > > > On Tue, Mar 24, 2020 at 02:53:23PM +0000, Denis Osterland-Heim wrote:
> > > > > +# ----------------------------------------------------------------------------
> > > > > +# Target-Install
> > > > > +# ----------------------------------------------------------------------------
> > > > > +
> > > > > +$(STATEDIR)/imx-uuc.targetinstall:
> > > > > +	@$(call targetinfo)
> > 
> > And here too, otherwise make will complain for recipes.
> definitely a tab.
> Do you have problems to apply it?

No, because it is a new file. Strange, now I see it as a tab too.
Sometimes my screen setup is somehow broken and makes tabs smaller than
usual... So nothing to worry about then, sorry.

 - Roland

-- 
Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
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] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 1/3] imx-uuc: new package
  2020-03-25 18:43   ` Roland Hieber
  2020-03-25 18:52     ` Roland Hieber
@ 2020-03-26  9:59     ` Guillermo Rodriguez Garcia
  1 sibling, 0 replies; 16+ messages in thread
From: Guillermo Rodriguez Garcia @ 2020-03-26  9:59 UTC (permalink / raw)
  To: ptxdist

El mié., 25 mar. 2020 a las 19:44, Roland Hieber
(<rhi@pengutronix.de>) escribió:
> > +IMX_UUC_VERSION    := d6afb27e55d73d7ad08cd2dd51c784d8ec9694dc
>
> Nitpick: I don't know how picky opkg-based systems are with having
> monotonically increasing version numbers, but in case someone uses them
> to update software, you could make them happy by providing a fake
> 'git describe' tag as a version:

It is picky, and will refuse to upgrade if the version number seems to
be lower than whatever is currently installed.
So yes, the package version should be monotonically increasing and not
simply a commit hash.

When I face this case in my own packages I normally define two
different varaibles, e.g.:

IMX_UUC_VERSION := 0.0.1
IMX_UUC_GIT_TAG := d6afb27e55d73d7ad08cd2dd51c784d8ec9694dc

and then use IMX_UUC_VERSION everywhere except in the URL:

IMX_UUC_URL        :=
https://github.com/NXPmicro/imx-uuc.git;tag=$(IMX_UUC_GIT_TAG)

Guillermo

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 3/3] host-mfgtools: new package
  2020-03-26  8:28     ` Denis Osterland-Heim
@ 2020-03-26 16:16       ` Michael Olbrich
  2020-03-26 16:25         ` Denis Osterland-Heim
  0 siblings, 1 reply; 16+ messages in thread
From: Michael Olbrich @ 2020-03-26 16:16 UTC (permalink / raw)
  To: ptxdist

On Thu, Mar 26, 2020 at 08:28:26AM +0000, Denis Osterland-Heim wrote:
> Hi Roland,
> 
> sorry for the confusion yesterday, seems it was to late for me...
> 
> Am Mittwoch, den 25.03.2020, 19:33 +0100 schrieb Roland Hieber:
> > On Tue, Mar 24, 2020 at 02:53:23PM +0000, Denis Osterland-Heim wrote:
> > > Tool to USB boot i.MX systems from NXP via Android fastboot.
> > > 
> > > Signed-off-by: Denis Osterland-Heim <Denis.Osterland@diehl.com>
> > > ---
> > >  rules/host-mfgtools.in   | 13 ++++++++++++
> > >  rules/host-mfgtools.make | 43 ++++++++++++++++++++++++++++++++++++++++
> > >  2 files changed, 56 insertions(+)
> > >  create mode 100644 rules/host-mfgtools.in
> > >  create mode 100644 rules/host-mfgtools.make
> > > 
> > > diff --git a/rules/host-mfgtools.in b/rules/host-mfgtools.in
> > > new file mode 100644
> > > index 000000000..c9046f10d
> > > --- /dev/null
> > > +++ b/rules/host-mfgtools.in
> > > @@ -0,0 +1,13 @@
> > > +## SECTION=hosttools
> > > +
> > > +config HOST_MFGTOOLS
> > > +	tristate
> > > +	default y if ALLYES
> > > +	prompt "uuu (Universal Update Utility), mfgtools 3.0"
> > > +	select HOST_CMAKE
> > > +	select HOST_LIBUSB
> > > +	select HOST_ZLIB
> > > +	select HOST_BZIP2
> > > +	select HOST_LIBZIP
> > > +	help
> > > +	  Freescale/NXP I.MX Chip image deploy tools.
> > > diff --git a/rules/host-mfgtools.make b/rules/host-mfgtools.make
> > > new file mode 100644
> > > index 000000000..fb4b2cf05
> > > --- /dev/null
> > > +++ b/rules/host-mfgtools.make
> > > @@ -0,0 +1,43 @@
> > > +# -*-makefile-*-
> > > +#
> > > +# Copyright (C) 2020 by Denis Osterland-Heim <Denis.Osterland@diehl.com>
> > > +#
> > > +# For further information about the PTXdist project and license conditions
> > > +# see the README file.
> > > +#
> > > +
> > > +HOST_PACKAGES-$(PTXCONF_HOST_MFGTOOLS) += host-mfgtools
> > > +
> > > +#
> > > +# Paths and names
> > > +#
> > > +HOST_MFGTOOLS_VERSION	:= 1.3.136-9-g47ddef5
> > 
> > Apparently this commit was released a few days ago as version 1.3.154:
> > <https://github.com/NXPmicro/mfgtools/releases>
> I have already changed it locally, v2 will come ;-)
> 
> > 
> > > +HOST_MFGTOOLS_MD5	:= 1555253943e1f39746694365c7f89eb1
> > > +HOST_MFGTOOLS		:= mfgtools-$(HOST_MFGTOOLS_VERSION)
> > > +HOST_MFGTOOLS_SUFFIX	:= tar.gz
> > > +HOST_MFGTOOLS_URL	:= https://github.com/NXPmicro/mfgtools.git;tag=47ddef51d6c24b1f917f348d0f62b4af38852994
> > 
> > So you can use the default release tarball here, and use the default
> > extract stage below.
> Not yet, but hopefully with next version.
 
> > 
> > > +HOST_MFGTOOLS_SOURCE	:= $(SRCDIR)/$(HOST_MFGTOOLS).$(HOST_MFGTOOLS_SUFFIX)
> > > +HOST_MFGTOOLS_DIR	:= $(HOST_BUILDDIR)/$(HOST_MFGTOOLS)
> > > +HOST_MFGTOOLS_LICENSE	:= BSD-3-Clause
> > > +HOST_MFGTOOLS_LICENSE_FILES	:= \
> > > +	file://LICENSE;md5=38ec0c18112e9a92cffc4951661e85a5
> > > +
> > > +# ----------------------------------------------------------------------------
> > > +# Extract
> > > +# ----------------------------------------------------------------------------
> > > +
> > > +$(STATEDIR)/host-mfgtools.extract:
> > > +	@$(call targetinfo)
> > > +	@$(call clean, $(HOST_MFGTOOLS_DIR))
> > > +	@$(call extract, HOST_MFGTOOLS)
> > > +	@echo "uuu_$(HOST_MFGTOOLS_VERSION)" > $(HOST_MFGTOOLS_DIR)/.tarball-version
> > 
> > NB: The released tarballs also don't have this file, is it really needed?
> Unfortunately it is still need, until upstream generates a tarball with the file.
> https://github.com/NXPmicro/mfgtools/issues/173

Do this in extract.post not extract.

Michael

> Regards Denis
> 
> > 
> >  - Roland
> > 
> > > +	@$(call patchin, HOST_MFGTOOLS)
> > > +	@$(call touch)
> > > +
> > > +# ----------------------------------------------------------------------------
> > > +# Prepare
> > > +# ----------------------------------------------------------------------------
> > > +
> > > +HOST_MFGTOOLS_CONF_TOOL	:= cmake
> > > +
> > > +# vim: syntax=make
> > > -- 
> > > 2.25.2
> > > 
> > > 
> > > 
> > > Diehl Connectivity Solutions GmbH
> > > Geschäftsführung: Horst Leonberger
> > > Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> > > Nürnberg: HRB 32315
> > > ___________________________________________________________________________________________________
> > > 
> > > Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> > > Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
> > > Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
> > > - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> > > 
> > > The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
> > > mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
> > > - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> > > 
> > > _______________________________________________
> > > ptxdist mailing list
> > > ptxdist@pengutronix.de
> > > 
> > 
> > 
> 
> 
> Diehl Connectivity Solutions GmbH
> Geschäftsführung: Horst Leonberger
> Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> Nürnberg: HRB 32315
> ___________________________________________________________________________________________________
> 
> Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
> Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
> - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> 
> The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
> mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
> - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
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] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 1/3] imx-uuc: new package
  2020-03-26  9:03         ` Denis Osterland-Heim
  2020-03-26  9:47           ` rhi
@ 2020-03-26 16:23           ` Michael Olbrich
  2020-03-26 16:32             ` Denis Osterland-Heim
  1 sibling, 1 reply; 16+ messages in thread
From: Michael Olbrich @ 2020-03-26 16:23 UTC (permalink / raw)
  To: ptxdist

On Thu, Mar 26, 2020 at 09:03:58AM +0000, Denis Osterland-Heim wrote:
> Am Mittwoch, den 25.03.2020, 21:40 +0100 schrieb Roland Hieber:
> > On Wed, Mar 25, 2020 at 07:52:54PM +0100, Roland Hieber wrote:
> > > Nitpick: I don't know how picky opkg-based systems are with having
> > > monotonically increasing version numbers, but in case someone uses them
> > > to update software in the field, you could make them happy by providing
> > > a fake 'git describe' tag as a version:
> > > 
> > > IMX_UUC_VERSION    := 0.0-1-gd6afb27e55d73d7ad08cd2dd51c784d8ec9694dc
> > > 
> > > and then increase the -1 every time for a version bump. This way git
> > > can extract the commit ID from the version info and opkg versions are
> > > increasing monotonically. The MD5 will be different though.
> Cool. I will apply.

Please use the Date of the Last commit. Also: github can produce stable
tarballs for any commit. Take a look at how this is handled in glslang.make
for example.

Michael

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
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] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 3/3] host-mfgtools: new package
  2020-03-26 16:16       ` Michael Olbrich
@ 2020-03-26 16:25         ` Denis Osterland-Heim
  0 siblings, 0 replies; 16+ messages in thread
From: Denis Osterland-Heim @ 2020-03-26 16:25 UTC (permalink / raw)
  To: ptxdist

Hi Michael,

Am Donnerstag, den 26.03.2020, 17:16 +0100 schrieb Michael Olbrich:
> On Thu, Mar 26, 2020 at 08:28:26AM +0000, Denis Osterland-Heim wrote:
> > Hi Roland,
> > 
> > sorry for the confusion yesterday, seems it was to late for me...
> > 
> > Am Mittwoch, den 25.03.2020, 19:33 +0100 schrieb Roland Hieber:
> > > On Tue, Mar 24, 2020 at 02:53:23PM +0000, Denis Osterland-Heim wrote:
> > > > Tool to USB boot i.MX systems from NXP via Android fastboot.
> > > > 
...
> > > > +
> > > > +# ----------------------------------------------------------------------------
> > > > +# Extract
> > > > +# ----------------------------------------------------------------------------
> > > > +
> > > > +$(STATEDIR)/host-mfgtools.extract:
> > > > +	@$(call targetinfo)
> > > > +	@$(call clean, $(HOST_MFGTOOLS_DIR))
> > > > +	@$(call extract, HOST_MFGTOOLS)
> > > > +	@echo "uuu_$(HOST_MFGTOOLS_VERSION)" > $(HOST_MFGTOOLS_DIR)/.tarball-version
> > > 
> > > NB: The released tarballs also don't have this file, is it really needed?
> > 
> > Unfortunately it is still need, until upstream generates a tarball with the file.
> > https://github.com/NXPmicro/mfgtools/issues/173
> 
> Do this in extract.post not extract.
Okay.

Regards Denis

> 
> Michael
> 
> > Regards Denis
> > 
> > > 
> > >  - Roland
> > > 
> > > > +	@$(call patchin, HOST_MFGTOOLS)
> > > > +	@$(call touch)
> > > > +
> > > > +# ----------------------------------------------------------------------------
> > > > +# Prepare
> > > > +# ----------------------------------------------------------------------------
> > > > +
> > > > +HOST_MFGTOOLS_CONF_TOOL	:= cmake
> > > > +
> > > > +# vim: syntax=make
> > > > -- 
> > > > 2.25.2
> > > > 
> > > > 
> > > > 
> > > > Diehl Connectivity Solutions GmbH
> > > > Geschäftsführung: Horst Leonberger
> > > > Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> > > > Nürnberg: HRB 32315
> > > > ___________________________________________________________________________________________________
> > > > 
> > > > Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> > > > Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
> > > > Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
> > > > - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> > > > 
> > > > The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
> > > > mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
> > > > - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> > > > 
> > > > _______________________________________________
> > > > ptxdist mailing list
> > > > ptxdist@pengutronix.de
> > > > 
> > > 
> > > 
> > 
> > 
> > Diehl Connectivity Solutions GmbH
> > Geschäftsführung: Horst Leonberger
> > Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> > Nürnberg: HRB 32315
> > ___________________________________________________________________________________________________
> > 
> > Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> > Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
> > Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
> > - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> > 
> > The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
> > mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
> > - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> > _______________________________________________
> > ptxdist mailing list
> > ptxdist@pengutronix.de
> 
> 


Diehl Connectivity Solutions GmbH
Geschäftsführung: Horst Leonberger
Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
Nürnberg: HRB 32315
___________________________________________________________________________________________________

Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
- Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/

The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
- For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [ptxdist] [RFC PATCH 1/3] imx-uuc: new package
  2020-03-26 16:23           ` Michael Olbrich
@ 2020-03-26 16:32             ` Denis Osterland-Heim
  0 siblings, 0 replies; 16+ messages in thread
From: Denis Osterland-Heim @ 2020-03-26 16:32 UTC (permalink / raw)
  To: ptxdist

Hi Michael,

Am Donnerstag, den 26.03.2020, 17:23 +0100 schrieb Michael Olbrich:
> On Thu, Mar 26, 2020 at 09:03:58AM +0000, Denis Osterland-Heim wrote:
> > Am Mittwoch, den 25.03.2020, 21:40 +0100 schrieb Roland Hieber:
> > > On Wed, Mar 25, 2020 at 07:52:54PM +0100, Roland Hieber wrote:
> > > > Nitpick: I don't know how picky opkg-based systems are with having
> > > > monotonically increasing version numbers, but in case someone uses them
> > > > to update software in the field, you could make them happy by providing
> > > > a fake 'git describe' tag as a version:
> > > > 
> > > > IMX_UUC_VERSION    := 0.0-1-gd6afb27e55d73d7ad08cd2dd51c784d8ec9694dc
> > > > 
> > > > and then increase the -1 every time for a version bump. This way git
> > > > can extract the commit ID from the version info and opkg versions are
> > > > increasing monotonically. The MD5 will be different though.
> > 
> > Cool. I will apply.
> 
> Please use the Date of the Last commit. Also: github can produce stable
> tarballs for any commit. Take a look at how this is handled in glslang.make
> for example.
Looks perfect!
Thanks, then I will switch to that.

Regards Denis

> 
> Michael
> 


Diehl Connectivity Solutions GmbH
Geschäftsführung: Horst Leonberger
Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
Nürnberg: HRB 32315
___________________________________________________________________________________________________

Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht.
Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
- Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/

The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by
mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. 
- For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2020-03-26 16:32 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-24 14:53 [ptxdist] [RFC PATCH 0/3] Freescale/NXP i.MX mfgtools 3.0 Denis Osterland-Heim
2020-03-24 14:53 ` [ptxdist] [RFC PATCH 2/3] host-libzip: new package Denis Osterland-Heim
2020-03-24 14:53 ` [ptxdist] [RFC PATCH 1/3] imx-uuc: " Denis Osterland-Heim
2020-03-25 18:43   ` Roland Hieber
2020-03-25 18:52     ` Roland Hieber
2020-03-25 20:40       ` Roland Hieber
2020-03-26  9:03         ` Denis Osterland-Heim
2020-03-26  9:47           ` rhi
2020-03-26 16:23           ` Michael Olbrich
2020-03-26 16:32             ` Denis Osterland-Heim
2020-03-26  9:59     ` Guillermo Rodriguez Garcia
2020-03-24 14:53 ` [ptxdist] [RFC PATCH 3/3] host-mfgtools: " Denis Osterland-Heim
2020-03-25 18:33   ` Roland Hieber
2020-03-26  8:28     ` Denis Osterland-Heim
2020-03-26 16:16       ` Michael Olbrich
2020-03-26 16:25         ` Denis Osterland-Heim

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox