[meta-freescale] [PATCH 1/3] cm-fx6:u-boot: add u-boot configuration for cm-fx6 machines

[Daiane Angolini]

On Thu, Apr 9, 2015 at 4:05 AM, Valentin Raevsky
<valentin at compulab.co.il> wrote:
> Add u-boot configuration for cm-fx6 machines.

I would prefer if you could elaborate it. For example, add here where
is the initial support for the cm-fx6, if it has some additional
support more than the very basic needed to bootup like splash screen,
USB, whatever...

>
> Signed-off-by: Valentin Raevsky <valentin at compulab.co.il>
> ---
>  recipes-bsp/u-boot/u-boot-cm-fx6_2014.10.bb |   17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
>  create mode 100644 recipes-bsp/u-boot/u-boot-cm-fx6_2014.10.bb
>
> diff --git a/recipes-bsp/u-boot/u-boot-cm-fx6_2014.10.bb b/recipes-bsp/u-boot/u-boot-cm-fx6_2014.10.bb
> new file mode 100644
> index 0000000..463da67
> --- /dev/null
> +++ b/recipes-bsp/u-boot/u-boot-cm-fx6_2014.10.bb
> @@ -0,0 +1,17 @@
> +require recipes-bsp/u-boot/u-boot.inc
> +
> +DESCRIPTION = "u-boot which includes support for CompuLab boards such as cm-fx6."

"such as" makes me think that there are other CompuLab boards added
and cm-fx6 is only one example.

I would prefer if you could drop the "such as" and the machine name.

In fact, the ideal is to have the board supported in u-boot mainline.......


Daiane

> +LICENSE = "GPLv2+"
> +LIC_FILES_CHKSUM = "file://configs/cm_fx6_defconfig;md5=549452418e9e099e5fe19ef1ac65e892"
> +
> +PROVIDES += "u-boot"
> +
> +SRCBRANCH = "utilite/master"
> +SRCREV = "6803a75f3d40bee3b53f56e7b49c70465819b7ca"
> +SRC_URI = "git://github.com/utilite-computer/u-boot.git;branch=${SRCBRANCH}"
> +SRC_URI[md5sum] = "fd8234c5b3a460430689848c1f16acef"
> +
> +S = "${WORKDIR}/git"
> +
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> +COMPATIBLE_MACHINE = "cm-fx6"
> --
> 1.7.9.5
>
> --
> _______________________________________________
> meta-freescale mailing list
> meta-freescale at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale