> -----Original Message-----
> From: Nathan Rossi [mailto:[email protected]]
> Sent: Tuesday, October 10, 2017 11:51 PM
> To: Manjukumar Harthikote Matha <[email protected]>
> Cc: Alistair Francis <[email protected]>; [email protected]
> Subject: Re: [meta-xilinx] [PATCH 5/7] Update recipes for Xilinx v2017.3 
> release
> 
> On 11 October 2017 at 09:05, Manjukumar Harthikote Matha
> <[email protected]> wrote:
> > Hi Nathan,
> >
> >> -----Original Message-----
> >> From: [email protected] [mailto:meta-xilinx-
> >> [email protected]] On Behalf Of Alistair Francis
> >> Sent: Tuesday, October 10, 2017 3:13 PM
> >> To: Nathan Rossi <[email protected]>
> >> Cc: [email protected]
> >> Subject: Re: [meta-xilinx] [PATCH 5/7] Update recipes for Xilinx
> >> v2017.3 release
> >>
> >> On Sat, Oct 7, 2017 at 2:57 AM, Nathan Rossi <[email protected]> 
> >> wrote:
> >> > Update the arm-trusted-firmware, pmu-firmware, u-boot-xlnx,
> >> > linux-xlnx, qemu-xilinx and qemu-devicetrees recipes for to the 
> >> > 'xilinx-v2017.3'
> >> > release tags.
> >> >
> >> > Drop/update existing patches where applicable.
> >> >
> >> > Signed-off-by: Nathan Rossi <[email protected]>
> >>
> >> Reviewed-by: Alistair Francis <[email protected]>
> >>
> >> Thanks,
> >> Alistair
> >>
> >> > ---
> >
> > <.....>
> >
> >> > diff --git
> >> > a/recipes-bsp/u-boot/u-boot-xlnx/v2017.1/arm-zynqmp-xilinx_zynqmp.h
> >> > -Au
> >> > to-boot-in-JTAG-if-imag.patch
> >> > b/recipes-bsp/u-boot/u-boot-xlnx/v2017.3/arm-zynqmp-xilinx_zynqmp.h
> >> > -Au
> >> > to-boot-in-JTAG-if-imag.patch
> >> > similarity index 100%
> >> > rename from
> >> > recipes-bsp/u-boot/u-boot-xlnx/v2017.1/arm-zynqmp-xilinx_zynqmp.h-A
> >> > uto -boot-in-JTAG-if-imag.patch rename to
> >> > recipes-bsp/u-boot/u-boot-xlnx/v2017.3/arm-zynqmp-xilinx_zynqmp.h-A
> >> > uto -boot-in-JTAG-if-imag.patch diff --git
> >
> > meta-xilinx should not be a holding ground for non upstreamable patches.
> > Implementation has to be from kernel-fitimage.bbclass inclusion.
> > Please don't add this in meta-xilinx
> >
> 
> This is not adding it, just moving the existing patch to apply to the newer 
> u-boot
> recipe.
> 

Agreed, we are porting a non-upstreamable patch regardless. 
> It does however need to go though. 

I completely disagree, it's not an option to proceed. I cannot accept this path.

But I have not had a chance to sort that out yet,
> the biggest problem with using fit is that it would need to embed the rootfs 
> (which is
> passed at runqemu time). Which  means that using a disk interface is a better 
> way to
> go, but since qemu boots u-boot instead of a kernel directly it is not so 
> straight
> forward to pass the root= param to the kernel.
> 

We could also look into kernel-fitimage.bbclass to introduce the fit to 
resolve. 

Thanks,
Manju
-- 
_______________________________________________
meta-xilinx mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/meta-xilinx

Reply via email to