[meta-xilinx] [PATCH 2/2] zc702-zynq7 : Define boot files so that wic can make SD card images that boot.

[Nathan Rossi]

> -----Original Message-----
> From: Philip Balister [mailto:philip at balister.org]
> Sent: Saturday, November 15, 2014 3:49 AM
> To: Nathan Rossi
> Cc: meta-xilinx at yoctoproject.org
> Subject: Re: [meta-xilinx] [PATCH 2/2] zc702-zynq7 : Define boot files so
> that wic can make SD card images that boot.
> 
> On 11/11/2014 10:23 AM, Philip Balister wrote:
> > On 11/04/2014 10:42 PM, Nathan Rossi wrote:
> >>> -----Original Message-----
> >>> From: meta-xilinx-bounces at yoctoproject.org [mailto:meta-xilinx-
> >>> bounces at yoctoproject.org] On Behalf Of Philip Balister
> >>> Sent: Saturday, November 01, 2014 12:26 AM
> >>> To: meta-xilinx at yoctoproject.org
> >>> Subject: [meta-xilinx] [PATCH 2/2] zc702-zynq7 : Define boot files so
> that
> >>> wic can make SD card images that boot.
> >>>
> >>> Signed-off-by: Philip Balister <philip at balister.org>
> >>> ---
> >>>  conf/machine/zc702-zynq7.conf | 2 ++
> >>>  1 file changed, 2 insertions(+)
> >>>
> >>> diff --git a/conf/machine/zc702-zynq7.conf b/conf/machine/zc702-
> zynq7.conf
> >>> index 25bd338..915c78b 100644
> >>> --- a/conf/machine/zc702-zynq7.conf
> >>> +++ b/conf/machine/zc702-zynq7.conf
> >>> @@ -24,3 +24,5 @@ MACHINE_DEVICETREE := " \
> >>>  		"
> >>>
> >>>  MACHINE_KCONFIG := "common/linux/rtc.cfg"
> >>> +
> >>> +IMAGE_BOOT_FILES = "boot.bin u-boot.img uEnv.txt uImage-zc702-
> zynq7.dtb
> >>> uImage"
> >>
> >> Should be a ?= in case local.conf has a custom value.
> >>
> >> This looks like a good candidate to be split up a-bit and put in
> machine-xilinx-board.inc?
> >>
> >> machine-xilinx-board.inc:
> >> IMAGE_BOOT_FILES ?= "${KERNEL_IMAGETYPE} u-boot.${UBOOT_SUFFIX}"
> >
> > UBOOT_SUFFIX is defined in a u-boot include, not a machine one. So this
> > variable is not available when wic is running. Hardcoding to img for
> > testing.

I made a patch to fix up this, moving UBOOT_SUFFIX to being defined in machine-xilinx-default.inc.

> 
> Any thoughts on how to clear this up? Should we include boot.bin and the
> devicetree file here? Hmm, maybe not devicetree file.

For now I think leaving the device tree and boot.bin appends in the zc702-zynq7.conf is fine. In the future it can be refactored, when it makes sense.

Regards,
Nathan

> 
> Philip
> 
> >
> > Philip
> >
> >>
> >> zc702-zynq7.conf:
> >> IMAGE_BOOT_FILES += "boot.bin uEnv.txt ${KERNEL_IMAGETYPE}-
> ${MACHINE}.dtb"
> >>
> >> So that at some point the "boot.bin" could move into the machine-
> xilinx-board.inc definition.
> >>
> >> Regards,
> >> Nathan
> >>
> >>> --
> >>> 1.8.3.1
> >>>
> >>> --
> >>> _______________________________________________
> >>> meta-xilinx mailing list
> >>> meta-xilinx at yoctoproject.org
> >>> https://lists.yoctoproject.org/listinfo/meta-xilinx
> >>
> >>