Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels
@ 2013-05-03  8:27 Henk Fijnvandraat
  2013-05-03  8:27 ` [Buildroot] [PATCH 2/2] firmware-imx-version: bump to version 1.1.0 Henk Fijnvandraat
  2013-05-03 12:01 ` [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels H.Fijnvandraat at inter.nl.net
  0 siblings, 2 replies; 6+ messages in thread
From: Henk Fijnvandraat @ 2013-05-03  8:27 UTC (permalink / raw)
  To: buildroot

imx-lib: bump to version 1.1.0

Signed-off-by: Henk Fijnvandraat <h.fijnvandraat@inter.nl.net>
---
 package/imx-lib/imx-lib.mk |   14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/package/imx-lib/imx-lib.mk b/package/imx-lib/imx-lib.mk
index c168c80..c398568 100644
--- a/package/imx-lib/imx-lib.mk
+++ b/package/imx-lib/imx-lib.mk
@@ -1,10 +1,22 @@
 #############################################################
 #
+# freescale's imx-version-level
+#
+#############################################################
+
+# Current version level for a number of freescale software packages
+#
+# firmware-imx, imx-lib and gpu-viv-bin-mx6q
+
+IMX_VERSION_LEVEL = 1.1.0
+
+#############################################################
+#
 # imx-lib
 #
 #############################################################
 
-IMX_LIB_VERSION = 12.09.01
+IMX_LIB_VERSION = $(IMX_VERSION_LEVEL)
 # No official download site from freescale, just this mirror
 IMX_LIB_SITE    = http://download.ossystems.com.br/bsp/freescale/source
 IMX_LIB_LICENSE = LGPLv2.1+
-- 
1.7.10.4

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

* [Buildroot] [PATCH 2/2] firmware-imx-version: bump to version 1.1.0
  2013-05-03  8:27 [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels Henk Fijnvandraat
@ 2013-05-03  8:27 ` Henk Fijnvandraat
  2013-05-03 12:01 ` [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels H.Fijnvandraat at inter.nl.net
  1 sibling, 0 replies; 6+ messages in thread
From: Henk Fijnvandraat @ 2013-05-03  8:27 UTC (permalink / raw)
  To: buildroot

(indirect version increase through refactoring)

Signed-off-by: Henk Fijnvandraat <h.fijnvandraat@inter.nl.net>
---
 package/firmware-imx/firmware-imx.mk |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/firmware-imx/firmware-imx.mk b/package/firmware-imx/firmware-imx.mk
index 3b43bef..167eb35 100644
--- a/package/firmware-imx/firmware-imx.mk
+++ b/package/firmware-imx/firmware-imx.mk
@@ -4,7 +4,7 @@
 #
 #############################################################
 
-FIRMWARE_IMX_VERSION = 12.09.01
+FIRMWARE_IMX_VERSION = $(IMX_VERSION_LEVEL)
 # No official download site from freescale, just this mirror
 FIRMWARE_IMX_SITE = http://download.ossystems.com.br/bsp/freescale/source
 FIRMWARE_IMX_SOURCE = firmware-imx-$(FIRMWARE_IMX_VERSION).bin
-- 
1.7.10.4

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

* [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels
  2013-05-03  8:27 [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels Henk Fijnvandraat
  2013-05-03  8:27 ` [Buildroot] [PATCH 2/2] firmware-imx-version: bump to version 1.1.0 Henk Fijnvandraat
@ 2013-05-03 12:01 ` H.Fijnvandraat at inter.nl.net
  2013-05-03 17:13   ` Arnout Vandecappelle
  2013-05-03 17:29   ` Thomas Petazzoni
  1 sibling, 2 replies; 6+ messages in thread
From: H.Fijnvandraat at inter.nl.net @ 2013-05-03 12:01 UTC (permalink / raw)
  To: buildroot

Sorry,

But I have to withdraw these patches.

Due to the sequence in which the .mk files are included IMX_VERSION_LEVEL
is defined when imx-lib.mk is processed, while it is already needed for
firmware-imx (and  gpu-viv-bin-mx6q if it would be introduced)

Placing it in firmware-imx would do for now, but then no dependencies can
be placed alphabetically before it later.

It appears we need some other mechanism to rule them all.

Henk


Op Vr, 3 mei, 2013 10:27 am schreef Henk Fijnvandraat:
> imx-lib: bump to version 1.1.0
>
>
> Signed-off-by: Henk Fijnvandraat <h.fijnvandraat@inter.nl.net>
> ---
> package/imx-lib/imx-lib.mk |   14 +++++++++++++- 1 file changed, 13
> insertions(+), 1 deletion(-)
>
> diff --git a/package/imx-lib/imx-lib.mk b/package/imx-lib/imx-lib.mk index
> c168c80..c398568 100644 --- a/package/imx-lib/imx-lib.mk
> +++ b/package/imx-lib/imx-lib.mk
> @@ -1,10 +1,22 @@
> #############################################################
> #
> +# freescale's imx-version-level
> +#
> +#############################################################
> +
> +# Current version level for a number of freescale software packages
> +#
> +# firmware-imx, imx-lib and gpu-viv-bin-mx6q
> +
> +IMX_VERSION_LEVEL = 1.1.0
> +
> +#############################################################
> +#
> # imx-lib
> #
> #############################################################
>
>
> -IMX_LIB_VERSION = 12.09.01
> +IMX_LIB_VERSION = $(IMX_VERSION_LEVEL)
> # No official download site from freescale, just this mirror
> IMX_LIB_SITE    = http://download.ossystems.com.br/bsp/freescale/source
> IMX_LIB_LICENSE = LGPLv2.1+
> --
> 1.7.10.4
>
>
> _______________________________________________
> buildroot mailing list buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
>
>

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

* [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels
  2013-05-03 12:01 ` [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels H.Fijnvandraat at inter.nl.net
@ 2013-05-03 17:13   ` Arnout Vandecappelle
  2013-05-05 19:41     ` Peter Korsgaard
  2013-05-03 17:29   ` Thomas Petazzoni
  1 sibling, 1 reply; 6+ messages in thread
From: Arnout Vandecappelle @ 2013-05-03 17:13 UTC (permalink / raw)
  To: buildroot

On 03/05/13 14:01, H.Fijnvandraat at inter.nl.net wrote:
> Sorry,
>
> But I have to withdraw these patches.
>
> Due to the sequence in which the .mk files are included IMX_VERSION_LEVEL
> is defined when imx-lib.mk is processed, while it is already needed for
> firmware-imx (and  gpu-viv-bin-mx6q if it would be introduced)
>
> Placing it in firmware-imx would do for now, but then no dependencies can
> be placed alphabetically before it later.
>
> It appears we need some other mechanism to rule them all.

  Ah, yes stupid of me. Normally the order of assignments doesn't matter, 
but the generic-target contains an 'ifndef $(2)_VERSION'.

  So the only solution is to collect the imx packages in an imx 
subdirectory, like qt5. I've told Henk earlier that we try to avoid 
creating subdirectories in the package directory, but perhaps this is a 
good reason to do it. What do the others think?

  Regards,
  Arnout

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

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

* [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels
  2013-05-03 12:01 ` [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels H.Fijnvandraat at inter.nl.net
  2013-05-03 17:13   ` Arnout Vandecappelle
@ 2013-05-03 17:29   ` Thomas Petazzoni
  1 sibling, 0 replies; 6+ messages in thread
From: Thomas Petazzoni @ 2013-05-03 17:29 UTC (permalink / raw)
  To: buildroot

Dear H.Fijnvandraat at inter.nl.net,

On Fri, 3 May 2013 14:01:04 +0200 (CEST), H.Fijnvandraat at inter.nl.net
wrote:

> But I have to withdraw these patches.
> 
> Due to the sequence in which the .mk files are included
> IMX_VERSION_LEVEL is defined when imx-lib.mk is processed, while it
> is already needed for firmware-imx (and  gpu-viv-bin-mx6q if it would
> be introduced)
> 
> Placing it in firmware-imx would do for now, but then no dependencies
> can be placed alphabetically before it later.
> 
> It appears we need some other mechanism to rule them all.

This shouldn't matter because we're using '=' everywhere, so they are
recursively expanded, which means that the values are expanded only
when the value is actually needed. So the inclusion order shouldn't
matter.

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

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

* [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels
  2013-05-03 17:13   ` Arnout Vandecappelle
@ 2013-05-05 19:41     ` Peter Korsgaard
  0 siblings, 0 replies; 6+ messages in thread
From: Peter Korsgaard @ 2013-05-05 19:41 UTC (permalink / raw)
  To: buildroot

>>>>> "Arnout" == Arnout Vandecappelle <arnout@mind.be> writes:

Hi,

 Arnout>  Ah, yes stupid of me. Normally the order of assignments doesn't
 Arnout> matter, but the generic-target contains an 'ifndef $(2)_VERSION'.

 Arnout>  So the only solution is to collect the imx packages in an imx
 Arnout> subdirectory, like qt5. I've told Henk earlier that we try to
 Arnout> avoid creating subdirectories in the package directory, but
 Arnout> perhaps this is a good reason to do it. What do the others
 Arnout> think?

Yes, I guess this is the best option.

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2013-05-05 19:41 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-05-03  8:27 [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels Henk Fijnvandraat
2013-05-03  8:27 ` [Buildroot] [PATCH 2/2] firmware-imx-version: bump to version 1.1.0 Henk Fijnvandraat
2013-05-03 12:01 ` [Buildroot] [PATCH 1/2] Refactoring Freescale's iMX software revision levels H.Fijnvandraat at inter.nl.net
2013-05-03 17:13   ` Arnout Vandecappelle
2013-05-05 19:41     ` Peter Korsgaard
2013-05-03 17:29   ` Thomas Petazzoni

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