[meta-xilinx] [PATCH 1/2] u-boot-xlnx.inc : Add support for building working SPL.

[Nathan Rossi]

> -----Original Message-----
> From: meta-xilinx-bounces at yoctoproject.org [mailto:meta-xilinx-
> bounces at yoctoproject.org] On Behalf Of Philip Balister
> Sent: Friday, October 10, 2014 4:10 AM
> To: meta-xilinx at yoctoproject.org
> Subject: [meta-xilinx] [PATCH 1/2] u-boot-xlnx.inc : Add support for
> building working SPL.

Missing patch 2/2?

> 
> You need to add the files ps7_init.[ch] via a board specific
> SRC_URI append. The ps7 files should be GPL licensed to avoid
> license headaches.
> 
> Signed-off-by: Philip Balister <philip at balister.org>
> ---
>  recipes-bsp/u-boot/u-boot-xlnx.inc | 22 +++++++++++++++++++++-
>  1 file changed, 21 insertions(+), 1 deletion(-)
> 
> diff --git a/recipes-bsp/u-boot/u-boot-xlnx.inc b/recipes-bsp/u-boot/u-
> boot-xlnx.inc
> index ee6b32c..6f5e5c6 100644
> --- a/recipes-bsp/u-boot/u-boot-xlnx.inc
> +++ b/recipes-bsp/u-boot/u-boot-xlnx.inc
> @@ -4,7 +4,27 @@ LICENSE = "GPLv2+"
> 
>  XILINX_EXTENSION = "-xilinx"
> 
> -FILESEXTRAPATHS += "${THISDIR}/u-boot-xlnx:"
> +FILESEXTRAPATHS += "${THISDIR}/u-boot-xlnx:${THISDIR}/${MACHINE}:"
>  SRC_URI = "git://github.com/Xilinx/u-boot-xlnx.git;protocol=https"
> 
> +SPL_BINARY = "boot.bin"
> +UBOOT_SUFFIX = "img"
> +
> +do_configure_prepend () {
> +	if test -e ${WORKDIR}/ps7_init.h; then
> +		cp ${WORKDIR}/ps7_init.h ${S}/board/xilinx/zynq/
> +		cp ${WORKDIR}/ps7_init.c ${S}/board/xilinx/zynq/
> +	fi

Worth checking for ps7_init_gpl.* as well, probably before the ps7_init.* as the _gpl part implies that it is more likely to be the properly licensed version.

I am interested in making ps7_init.c/h a package/dependency in its own right, that would provide the ps7_init.c/h files in the machines sysroot. That way a bsp could have a recipe that provides the files (which might exist in a git repo, in the layer or in a tarball, etc). Any ideas/opinions on handling these files in a formalized way?

Also MicroBlaze has a similar need for this sort of thing but for the xparameters.h file.

> +}
> +
> +do_deploy_append () {
> +	if test -e ${WORKDIR}/uEnv.txt; then
> +		cp ${WORKDIR}/uEnv.txt ${DEPLOYDIR}
> +	fi

There is already some recipe infrastructure for 'environment' files, I haven't tested it but it looks like it might work for what you have here.
https://github.com/openembedded/oe-core/blob/master/meta/recipes-bsp/u-boot/u-boot.inc#L41

Looks like all that needs to be set is:
UBOOT_ENV = "uEnv"

> +	if test -e ${WORKDIR}fpga.bin; then
> +		cp ${WORKDIR}/fpga.bin ${DEPLOYDIR}
> +	fi

Not sure if this belongs in the u-boot recipe, any specific reason for it here? Or just a convenience thing (so that you can put all the files in the 'u-boot/${MACHINE}/' directory)?

> +}
> +
>  include u-boot-extra.inc
> +
> --
> 1.8.3.1
> 
> --
> _______________________________________________
> meta-xilinx mailing list
> meta-xilinx at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-xilinx