[meta-freescale] [meta-fsl-arm-extra][PATCH 1/3] gateworks ventana: add machine conf

[Otavio Salvador]

Hello Pushpal,

I would like to thank you to prepare the patches for the integration
of Gateworks board. There are some possible changes which I want to
propose, please look at the comments inline ...

On Fri, Apr 17, 2015 at 9:04 PM, Pushpal Sidhu <psidhu at gateworks.com> wrote:
> This commit adds a machine conf for the Gateworks Ventana product
> family. For more information, please see the product site at
> http://www.gateworks.com/product#ventana
>
> Signed-off-by: Pushpal Sidhu <psidhu at gateworks.com>

This commit should be the 'last one' as it depends on the u-boot and
kernel availability to proper work. So this should be the 3/3 patch.

...
> --- /dev/null
> +++ b/conf/machine/gwventana.conf
> @@ -0,0 +1,60 @@
> +#@TYPE: Machine
> +#@NAME: i.MX6Q/DL Ventana Platform
> +#@SOC: i.MX6Q/DL
> +#@DESCRIPTION: Machine configuration for Gateworks Ventana boards
...

Daiane, please see if those works for the Documentation use or might
needs adjustments.

...
> +# Bootloader configs:
> +UBOOT_ENTRYPOINT := "10008000"

This is the same as the default. Please remove it as it is a duplication.

> +UBOOT_CONFIG ??= "std"
> +UBOOT_CONFIG[std] = "gwventana_config"
> +
> +# Put bootscript on board
> +do_rootfs[depends] += "u-boot-script-gateworks:do_install"

The right task to depends is do_deploy otherwise you can have problems
in a build from an empty tmp directory with hot sstate-cache.

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750