[meta-fsl-arm][PATCH V2] nitrogen6x: Add support for i.MX6SDL CPU variants to SOC_FAMILY

[meta-fsl-arm][PATCH V2] nitrogen6x: Add support for i.MX6SDL CPU variants to SOC_FAMILY

[Eric Nelson]

The Nitrogen6x has options for i.MX6DL and i.MX6S, and
without mx6dl in the SOC_FAMILY, Dual-Lite/Solo features
of other components won't be built.

The problem triggering this patch is the firmware-imx package,
which will not install the Dual-Lite/Solo firmware for the
VPU (lib/firmware/vpu_fw_imx6d.bin).

Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com>
---
V2 updates the Subject line to include [meta-fsl-arm].

 conf/machine/nitrogen6x.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/conf/machine/nitrogen6x.conf b/conf/machine/nitrogen6x.conf
index 868d287..3a3707e 100644
--- a/conf/machine/nitrogen6x.conf
+++ b/conf/machine/nitrogen6x.conf
@@ -31,7 +31,7 @@
 include conf/machine/include/imx-base.inc
 include conf/machine/include/tune-cortexa9.inc
 
-SOC_FAMILY = "mx6:mx6q"
+SOC_FAMILY = "mx6::mx6dl:mx6q"
 
 KERNEL_DEVICETREE = "imx6q-sabrelite.dtb imx6q-nitrogen6x.dtb imx6dl-nitrogen6x.dtb"
 
-- 
1.9.1

Re: [meta-fsl-arm][PATCH V2] nitrogen6x: Add support for i.MX6SDL CPU variants to SOC_FAMILY

[Gary Thomas]

On 2014-05-04 07:52, Eric Nelson wrote:
> The Nitrogen6x has options for i.MX6DL and i.MX6S, and
> without mx6dl in the SOC_FAMILY, Dual-Lite/Solo features
> of other components won't be built.
>
> The problem triggering this patch is the firmware-imx package,
> which will not install the Dual-Lite/Solo firmware for the
> VPU (lib/firmware/vpu_fw_imx6d.bin).
>
> Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com>
> ---
> V2 updates the Subject line to include [meta-fsl-arm].

Wow, you do need more coffee.  [meta-fsl-arm-extra] perhaps?

>
>   conf/machine/nitrogen6x.conf | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/conf/machine/nitrogen6x.conf b/conf/machine/nitrogen6x.conf
> index 868d287..3a3707e 100644
> --- a/conf/machine/nitrogen6x.conf
> +++ b/conf/machine/nitrogen6x.conf
> @@ -31,7 +31,7 @@
>   include conf/machine/include/imx-base.inc
>   include conf/machine/include/tune-cortexa9.inc
>
> -SOC_FAMILY = "mx6:mx6q"
> +SOC_FAMILY = "mx6::mx6dl:mx6q"
>
>   KERNEL_DEVICETREE = "imx6q-sabrelite.dtb imx6q-nitrogen6x.dtb imx6dl-nitrogen6x.dtb"
>
>

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------

[meta-fsl-arm-extra][PATCH V2] nitrogen6x: Add support for i.MX6SDL CPU variants to SOC_FAMILY

[Eric Nelson]

The Nitrogen6x has options for i.MX6DL and i.MX6S, and
without mx6dl in the SOC_FAMILY, Dual-Lite/Solo features
of other components won't be built.

The problem triggering this patch is the firmware-imx package,
which will not install the Dual-Lite/Solo firmware for the
VPU (lib/firmware/vpu_fw_imx6d.bin).

Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com>
---
V2 updates the Subject line to include [meta-fsl-arm].
V3 updates the Subject line to include [meta-fsl-arm-extra]
   and proves that I should never submit patches before at
   least two cups of coffee and 15 minutes to allow absorption.

 conf/machine/nitrogen6x.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/conf/machine/nitrogen6x.conf b/conf/machine/nitrogen6x.conf
index 868d287..3a3707e 100644
--- a/conf/machine/nitrogen6x.conf
+++ b/conf/machine/nitrogen6x.conf
@@ -31,7 +31,7 @@
 include conf/machine/include/imx-base.inc
 include conf/machine/include/tune-cortexa9.inc
 
-SOC_FAMILY = "mx6:mx6q"
+SOC_FAMILY = "mx6::mx6dl:mx6q"
 
 KERNEL_DEVICETREE = "imx6q-sabrelite.dtb imx6q-nitrogen6x.dtb imx6dl-nitrogen6x.dtb"
 
-- 
1.9.1

Re: [meta-fsl-arm-extra][PATCH V2] nitrogen6x: Add support for i.MX6SDL CPU variants to SOC_FAMILY

[Gary Thomas]

On 2014-05-04 08:09, Eric Nelson wrote:
> The Nitrogen6x has options for i.MX6DL and i.MX6S, and
> without mx6dl in the SOC_FAMILY, Dual-Lite/Solo features
> of other components won't be built.
>
> The problem triggering this patch is the firmware-imx package,
> which will not install the Dual-Lite/Solo firmware for the
> VPU (lib/firmware/vpu_fw_imx6d.bin).
>
> Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com>
> ---
> V2 updates the Subject line to include [meta-fsl-arm].
> V3 updates the Subject line to include [meta-fsl-arm-extra]
>     and proves that I should never submit patches before at
>     least two cups of coffee and 15 minutes to allow absorption.
>
>   conf/machine/nitrogen6x.conf | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/conf/machine/nitrogen6x.conf b/conf/machine/nitrogen6x.conf
> index 868d287..3a3707e 100644
> --- a/conf/machine/nitrogen6x.conf
> +++ b/conf/machine/nitrogen6x.conf
> @@ -31,7 +31,7 @@
>   include conf/machine/include/imx-base.inc
>   include conf/machine/include/tune-cortexa9.inc
>
> -SOC_FAMILY = "mx6:mx6q"
> +SOC_FAMILY = "mx6::mx6dl:mx6q"

I also think that :: is [benignly] incorrect since this is just a list
of colon separated machine/SOC variants.  It would also be incorrect in
at least this one other file:
   meta-fsl-arm-extra/conf/machine/cubox-i.conf:SOC_FAMILY = "mx6::mx6dl:mx6q"

>
>   KERNEL_DEVICETREE = "imx6q-sabrelite.dtb imx6q-nitrogen6x.dtb imx6dl-nitrogen6x.dtb"
>
>

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------

Re: [meta-fsl-arm-extra][PATCH V2] nitrogen6x: Add support for i.MX6SDL CPU variants to SOC_FAMILY

[Eric Bénard]

Le Sun, 04 May 2014 08:15:21 -0600,
Gary Thomas <gary@mlbassoc.com> a écrit :

> 
> On 2014-05-04 08:09, Eric Nelson wrote:
> > The Nitrogen6x has options for i.MX6DL and i.MX6S, and
> > without mx6dl in the SOC_FAMILY, Dual-Lite/Solo features
> > of other components won't be built.
> >
> > The problem triggering this patch is the firmware-imx package,
> > which will not install the Dual-Lite/Solo firmware for the
> > VPU (lib/firmware/vpu_fw_imx6d.bin).
> >
> > Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com>
> > ---
> > V2 updates the Subject line to include [meta-fsl-arm].
> > V3 updates the Subject line to include [meta-fsl-arm-extra]
> >     and proves that I should never submit patches before at
> >     least two cups of coffee and 15 minutes to allow absorption.
> >
> >   conf/machine/nitrogen6x.conf | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/conf/machine/nitrogen6x.conf b/conf/machine/nitrogen6x.conf
> > index 868d287..3a3707e 100644
> > --- a/conf/machine/nitrogen6x.conf
> > +++ b/conf/machine/nitrogen6x.conf
> > @@ -31,7 +31,7 @@
> >   include conf/machine/include/imx-base.inc
> >   include conf/machine/include/tune-cortexa9.inc
> >
> > -SOC_FAMILY = "mx6:mx6q"
> > +SOC_FAMILY = "mx6::mx6dl:mx6q"
> 
> I also think that :: is [benignly] incorrect since this is just a list
> of colon separated machine/SOC variants.  It would also be incorrect in
> at least this one other file:
>    meta-fsl-arm-extra/conf/machine/cubox-i.conf:SOC_FAMILY = "mx6::mx6dl:mx6q"
> 
this seems to be a sed effect.

Eric

[meta-fsl-arm-extra][PATCH V4] nitrogen6x: Add support for i.MX6SDL CPU variants to SOC_FAMILY

[Eric Nelson]

The Nitrogen6x has options for i.MX6DL and i.MX6S, and
without mx6dl in the SOC_FAMILY, Dual-Lite/Solo features
of other components won't be built.

The problem triggering this patch is the firmware-imx package,
which will not install the Dual-Lite/Solo firmware for the
VPU (lib/firmware/vpu_fw_imx6d.bin).

Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com>
---
V2 updates the Subject line to include [meta-fsl-arm].
V3 updates the Subject line to include [meta-fsl-arm-extra]
   and proves that I should never submit patches before at
   least two cups of coffee and 15 minutes to allow absorption.
V4 updates removes the empty field in the colon-separated list

 conf/machine/nitrogen6x.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/conf/machine/nitrogen6x.conf b/conf/machine/nitrogen6x.conf
index 868d287..49a33a1 100644
--- a/conf/machine/nitrogen6x.conf
+++ b/conf/machine/nitrogen6x.conf
@@ -31,7 +31,7 @@
 include conf/machine/include/imx-base.inc
 include conf/machine/include/tune-cortexa9.inc
 
-SOC_FAMILY = "mx6:mx6q"
+SOC_FAMILY = "mx6:mx6dl:mx6q"
 
 KERNEL_DEVICETREE = "imx6q-sabrelite.dtb imx6q-nitrogen6x.dtb imx6dl-nitrogen6x.dtb"
 
-- 
1.9.1

[meta-fsl-arm-extra][PATCH] cubox-i: remove extraneous colons from SOC_FAMILY

[Eric Nelson]

The SOC_FAMILY variable is a colon-separated list and
cubox-i had an extra colon.

Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com>
---
 conf/machine/cubox-i.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/conf/machine/cubox-i.conf b/conf/machine/cubox-i.conf
index 897c0ea..99416c1 100644
--- a/conf/machine/cubox-i.conf
+++ b/conf/machine/cubox-i.conf
@@ -12,7 +12,7 @@
 require conf/machine/include/imx-base.inc
 require conf/machine/include/tune-cortexa9.inc
 
-SOC_FAMILY = "mx6::mx6dl:mx6q"
+SOC_FAMILY = "mx6:mx6dl:mx6q"
 
 PREFERRED_PROVIDER_u-boot = "u-boot-cubox-i"
 PREFERRED_PROVIDER_virtual/kernel = "linux-cubox-i"
-- 
1.9.1