Re: [meta-freescale] [PATCH v2] u-boot-fslc: Upgrade to 2017.07-based release

2017-07-10 Thread Otavio Salvador 
On Mon, Jul 10, 2017 at 4:29 PM, Daiane Angolini  wrote:
> On Mon, Jul 10, 2017 at 4:05 PM, Otavio Salvador
>  wrote:
>> This release provides a huge improvement regarding the support for SPL
>> to more boards.
>>
>
>> diff --git a/conf/machine/imx6qdlsabresd.conf 
>> b/conf/machine/imx6qdlsabresd.conf
>> new file mode 100644
>> index ..d221da15
>> --- /dev/null
>> +++ b/conf/machine/imx6qdlsabresd.conf
>> @@ -0,0 +1,32 @@
>> +#@TYPE: Machine
>> +#@NAME: NXP i.MX6Q/DL SABRE Smart Device
>> +#@SOC: i.MX6Q
>> +#@DESCRIPTION: Machine configuration for NXP i.MX6Q/DL SABRE Smart Device
>
> It also includes imx6QP
>
> I think we must have a README explaining in a more verbose way this change.
>
> I suggest a comment:
>
>
> // This machine works for:
> // * imx6qpsabresd
> // * imx6qsabresd
> // * imx6dlsabresd
> // This means one SDCard flashed with one image targeting this machine
> is able to boot all those boards.
>
> I think it's important to include the machine name as it was before
> the change, and make sure to say why it has changed "one binary for
> all machines".

I sent a v3; please take a look if it solves your concerns.


-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
meta-freescale mailing list
meta-freescale@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-freescale

Re: [meta-freescale] [PATCH v2] u-boot-fslc: Upgrade to 2017.07-based release

2017-07-10 Thread Daiane Angolini 
On Mon, Jul 10, 2017 at 4:05 PM, Otavio Salvador
 wrote:
> This release provides a huge improvement regarding the support for SPL
> to more boards.
>

> diff --git a/conf/machine/imx6qdlsabresd.conf 
> b/conf/machine/imx6qdlsabresd.conf
> new file mode 100644
> index ..d221da15
> --- /dev/null
> +++ b/conf/machine/imx6qdlsabresd.conf
> @@ -0,0 +1,32 @@
> +#@TYPE: Machine
> +#@NAME: NXP i.MX6Q/DL SABRE Smart Device
> +#@SOC: i.MX6Q
> +#@DESCRIPTION: Machine configuration for NXP i.MX6Q/DL SABRE Smart Device

It also includes imx6QP

I think we must have a README explaining in a more verbose way this change.

I suggest a comment:


// This machine works for:
// * imx6qpsabresd
// * imx6qsabresd
// * imx6dlsabresd
// This means one SDCard flashed with one image targeting this machine
is able to boot all those boards.

I think it's important to include the machine name as it was before
the change, and make sure to say why it has changed "one binary for
all machines".

Daiane

> +#@MAINTAINER: Otavio Salvador 
> +
> +MACHINEOVERRIDES =. "mx6:mx6q:mx6dl:"
> +
> +require conf/machine/include/imx-base.inc
> +require conf/machine/include/tune-cortexa9.inc
> +
> +KERNEL_DEVICETREE = " \
> +imx6qp-sabresd.dtb imx6qp-sabresd-btwifi.dtb imx6qp-sabresd-hdcp.dtb \
> +imx6qp-sabresd-ldo.dtb imx6qp-sabresd-ldo-pcie-cert.dtb \
> +\
> +imx6q-sabresd.dtb imx6q-sabresd-ldo.dtb imx6q-sabresd-hdcp.dtb \
> +imx6q-sabresd-enetirq.dtb imx6q-sabresd-btwifi.dtb \
> +\
> +imx6dl-sabresd.dtb imx6dl-sabresd-ldo.dtb imx6dl-sabresd-hdcp.dtb \
> +imx6dl-sabresd-enetirq.dtb imx6dl-sabresd-btwifi.dtb \
> +"
> +
> +UBOOT_MACHINE ?= "mx6sabresd_defconfig"
> +UBOOT_MAKE_TARGET = "all"
> +UBOOT_SUFFIX = "img"
> +SPL_BINARY = "SPL"
> +
> +SERIAL_CONSOLE = "115200 ttymxc0"
> +
> +MACHINE_FIRMWARE_append_mx6 = " linux-firmware-ath6k"
> +
> +MACHINE_FEATURES += " pci wifi bluetooth"
> diff --git a/conf/machine/imx6qpsabreauto.conf 
> b/conf/machine/imx6qpsabreauto.conf
> deleted file mode 100644
> index 505d341c..
> --- a/conf/machine/imx6qpsabreauto.conf
> +++ /dev/null
> @@ -1,24 +0,0 @@
> -#@TYPE: Machine
> -#@NAME: NXP i.MX6Q Plus SABRE Automotive
> -#@SOC: i.MX6QP
> -#@DESCRIPTION: Machine configuration for NXP i.MX6QP SABRE Automotive
> -#@MAINTAINER: Lauren Post 
> -
> -MACHINEOVERRIDES =. "mx6:mx6q:"
> -
> -require conf/machine/include/imx6sabreauto-common.inc
> -
> -KERNEL_DEVICETREE = "imx6qp-sabreauto.dtb imx6qp-sabreauto-ecspi.dtb 
> imx6qp-sabreauto-flexcan1.dtb \
> - imx6qp-sabreauto-gpmi-weim.dtb"
> -
> -PREFERRED_PROVIDER_u-boot = "u-boot-imx"
> -PREFERRED_PROVIDER_virtual/bootloader = "u-boot-imx"
> -PREFERRED_PROVIDER_virtual/kernel = "linux-imx"
> -
> -UBOOT_CONFIG ??= "sd"
> -UBOOT_CONFIG[sd] = "mx6qpsabreauto_config,sdcard"
> -UBOOT_CONFIG[eimnor] = "mx6qpsabreauto_eimnor_config"
> -UBOOT_CONFIG[nand] = "mx6qpsabreauto_nand_config,ubifs"
> -UBOOT_CONFIG[spinor] = "mx6qpsabreauto_spinor_config"
> -UBOOT_CONFIG[sata] = "mx6qpsabreauto_sata_config"
> -UBOOT_CONFIG[mfgtool] = "mx6qpsabreauto_config"
> diff --git a/conf/machine/imx6qpsabresd.conf b/conf/machine/imx6qpsabresd.conf
> deleted file mode 100644
> index 369d5b7d..
> --- a/conf/machine/imx6qpsabresd.conf
> +++ /dev/null
> @@ -1,20 +0,0 @@
> -#@TYPE: Machine
> -#@NAME: NXP i.MX6Q Plus SABRE Smart Device
> -#@SOC: i.MX6QP
> -#@DESCRIPTION: Machine configuration for NXP i.MX6QP SABRE Smart Device
> -#@MAINTAINER: Lauren Post 
> -
> -MACHINEOVERRIDES =. "mx6:mx6q:"
> -
> -require conf/machine/include/imx6sabresd-common.inc
> -
> -KERNEL_DEVICETREE = "imx6qp-sabresd.dtb imx6qp-sabresd-btwifi.dtb 
> imx6qp-sabresd-hdcp.dtb \
> - imx6qp-sabresd-ldo.dtb imx6qp-sabresd-ldo-pcie-cert.dtb"
> -
> -PREFERRED_PROVIDER_u-boot = "u-boot-imx"
> -PREFERRED_PROVIDER_virtual/bootloader = "u-boot-imx"
> -
> -UBOOT_CONFIG ??= "sd"
> -UBOOT_CONFIG[sd] = "mx6qpsabresd_config,sdcard"
> -UBOOT_CONFIG[sata] = "mx6qpsabresd_sata_config"
> -UBOOT_CONFIG[mfgtool] = "mx6qpsabresd_config"
> diff --git a/conf/machine/imx6qsabreauto.conf 
> b/conf/machine/imx6qsabreauto.conf
> deleted file mode 100644
> index e419c5d4..
> --- a/conf/machine/imx6qsabreauto.conf
> +++ /dev/null
> @@ -1,20 +0,0 @@
> -#@TYPE: Machine
> -#@NAME: NXP i.MX6Q SABRE Automotive
> -#@SOC: i.MX6Q
> -#@DESCRIPTION: Machine configuration for NXP i.MX6Q SABRE Automotive
> -#@MAINTAINER: Daiane Angolini 
> -
> -MACHINEOVERRIDES =. "mx6:mx6q:"
> -
> -require conf/machine/include/imx6sabreauto-common.inc
> -
> -KERNEL_DEVICETREE = "imx6q-sabreauto.dtb imx6q-sabreauto-gpmi-weim.dtb 
> imx6q-sabreauto-ecspi.dtb \
> - imx6q-sabreauto-flexcan1.dtb 
> imx6q-sabreauto-enetirq.dtb"
> -
> -UBOOT_CONFIG ??= "sd"
> -UBOOT_CONFIG[sd] = "mx6qsabreauto_config,sdcard"
> -UBOOT_CONFIG[eimnor] =

[meta-freescale] [PATCH v2] u-boot-fslc: Upgrade to 2017.07-based release

2017-07-10 Thread Otavio Salvador 
This release provides a huge improvement regarding the support for SPL
to more boards.

We now uses this and migrated following boards:

 - imx6qpsabresd -> imx6qdlsabresd
 - imx6qsabresd -> imx6qdlsabresd
 - imx6dlsabresd -> imx6qdlsabresd
 - imx6solosabresd -> imx6qdlsabresd
 - imx6qpsabreauto -> imx6qdlsabreauto
 - imx6qsabreauto -> imx6qdlsabreauto
 - imx6dlsabreauto -> imx6qdlsabreauto
 - imx6solosabreauto -> imx6qdlsabreauto

So now, the 8 boards are covered by 2 machine files. This drastically
reduces the build time and allow for a better user experience as we
can use the same image to test different boards.

On top of the official 2017.07 release, following patches are
included:

f2f4106d7e embestmx6boards: Use PARTUUID to specify the rootfs location
ee34bd7e30 mx6cuboxi: Use PARTUUID to specify the rootfs location
a58be2196b wandboard: Use PARTUUID to specify the rootfs location
5be2f9ed4f wandboard: Set fdt based on board_rev and board_name
d7d18cac0e warp: Change default bootenv configuration
3348be11be mx6sabresd: Enable video interfaces in bootargs
28b300063a iMX6: Change mmcroot to use fixed mmc block index for USDHC
0eeb1e9a98 mx6sabresd: Use LDO dtb file until LDO bypass support is added
83818d88d0 mx6slevk: Use LDO dtb file until LDO bypass support is added
029f4b321a mx6sabreauto: Update to SPL only mode
873656c162 mx6qsabreauto: Add SPL support
183c4ba268 mx6: soc: Move mxs_dma_init() into the mxs nand driver

Signed-off-by: Otavio Salvador 
---

Changes in v2:
- include fslc specific patches
- fix typo on commit log (Fabio Estevam)
- improve log to mention specific patches

 conf/machine/imx6dlsabreauto.conf  | 19 -
 conf/machine/imx6dlsabresd.conf| 19 -
 conf/machine/imx6qdlsabreauto.conf | 32 ++
 conf/machine/imx6qdlsabresd.conf   | 32 ++
 conf/machine/imx6qpsabreauto.conf  | 24 
 conf/machine/imx6qpsabresd.conf| 20 --
 conf/machine/imx6qsabreauto.conf   | 20 --
 conf/machine/imx6qsabresd.conf | 17 
 conf/machine/imx6solosabreauto.conf| 21 --
 conf/machine/imx6solosabresd.conf  | 19 -
 conf/machine/imx6sxsabreauto.conf  |  9 +-
 conf/machine/imx6sxsabresd.conf|  9 +-
 conf/machine/include/imx6sabreauto-common.inc  | 10 ---
 conf/machine/include/imx6sabresd-common.inc| 10 ---
 ...t_2016.09.bb => u-boot-fslc-mxsboot_2017.07.bb} |  6 ++--
 recipes-bsp/u-boot/u-boot-fslc_2017.03.bb  |  6 
 recipes-bsp/u-boot/u-boot-fslc_2017.07.bb  |  6 
 17 files changed, 89 insertions(+), 190 deletions(-)
 delete mode 100644 conf/machine/imx6dlsabreauto.conf
 delete mode 100644 conf/machine/imx6dlsabresd.conf
 create mode 100644 conf/machine/imx6qdlsabreauto.conf
 create mode 100644 conf/machine/imx6qdlsabresd.conf
 delete mode 100644 conf/machine/imx6qpsabreauto.conf
 delete mode 100644 conf/machine/imx6qpsabresd.conf
 delete mode 100644 conf/machine/imx6qsabreauto.conf
 delete mode 100644 conf/machine/imx6qsabresd.conf
 delete mode 100644 conf/machine/imx6solosabreauto.conf
 delete mode 100644 conf/machine/imx6solosabresd.conf
 delete mode 100644 conf/machine/include/imx6sabreauto-common.inc
 delete mode 100644 conf/machine/include/imx6sabresd-common.inc
 rename recipes-bsp/u-boot/{u-boot-fslc-mxsboot_2016.09.bb => 
u-boot-fslc-mxsboot_2017.07.bb} (87%)
 delete mode 100644 recipes-bsp/u-boot/u-boot-fslc_2017.03.bb
 create mode 100644 recipes-bsp/u-boot/u-boot-fslc_2017.07.bb

diff --git a/conf/machine/imx6dlsabreauto.conf 
b/conf/machine/imx6dlsabreauto.conf
deleted file mode 100644
index 51fb7028..
--- a/conf/machine/imx6dlsabreauto.conf
+++ /dev/null
@@ -1,19 +0,0 @@
-#@TYPE: Machine
-#@NAME: NXP i.MX6DL SABRE Automotive
-#@SOC: i.MX6DL
-#@DESCRIPTION: Machine configuration for NXP i.MX6DL SABRE Automotive
-#@MAINTAINER: Daiane Angolini 
-
-MACHINEOVERRIDES =. "mx6:mx6dl:"
-
-require conf/machine/include/imx6sabreauto-common.inc
-
-KERNEL_DEVICETREE = "imx6dl-sabreauto.dtb imx6dl-sabreauto-gpmi-weim.dtb 
imx6dl-sabreauto-ecspi.dtb \
- imx6dl-sabreauto-flexcan1.dtb 
imx6dl-sabreauto-enetirq.dtb"
-
-UBOOT_CONFIG ??= "sd"
-UBOOT_CONFIG[sd] = "mx6dlsabreauto_config,sdcard"
-UBOOT_CONFIG[eimnor] = "mx6dlsabreauto_eimnor_config"
-UBOOT_CONFIG[nand] = "mx6dlsabreauto_nand_config,ubifs"
-UBOOT_CONFIG[spinor] = "mx6dlsabreauto_spinor_config"
-UBOOT_CONFIG[mfgtool] = "mx6dlsabreauto_config"
diff --git a/conf/machine/imx6dlsabresd.conf b/conf/machine/imx6dlsabresd.conf
deleted file mode 100644
index 27b8391c..
--- a/conf/machine/imx6dlsabresd.conf
+++ /dev/null
@@ -1,19 +0,0 @@
-#@TYPE: Machine
-#@NAME: NXP i.MX6DL SABRE Smart