[OE-core] [for dylan][PATCH 3/3] linux-dtb.inc: Fix dtb generation for kernels newer than 3.8

[Cooper Jr., Franklin]

> -----Original Message-----
> From: Paul Eggleton [mailto:paul.eggleton at linux.intel.com]
> Sent: Tuesday, August 20, 2013 1:24 PM
> To: Otavio Salvador
> Cc: Cooper Jr., Franklin; Patches and discussions about the oe-core layer
> Subject: Re: [for dylan][PATCH 3/3] linux-dtb.inc: Fix dtb generation for kernels
> newer than 3.8
> 
> Hi Otavio,
> 
> On Tuesday 20 August 2013 15:17:16 Otavio Salvador wrote:
> > On Tue, Aug 20, 2013 at 1:51 PM, Franklin S. Cooper Jr
> > <fcooper at ti.com>
> wrote:
> > > From: Otavio Salvador <otavio at ossystems.com.br>
> > >
> > > The 3.8 kernel has change the default directory where the dtb file
> > > is stored. The change has been done at:
> > >
> > > ,----[ Quote of 3.8 kernel change ]
> > >
> > > | commit 499cd8298628eeabf0eb5eb6525d4faa0eec80d8
> > > | Author: Grant Likely <grant.likely at secretlab.ca>
> > > | Date:   Tue Nov 27 16:29:11 2012 -0700
> > > |
> > > |     ARM: dt: change .dtb build rules to build in dts directory
> > > |
> > > |     The current rules have the .dtb files build in a different directory
> > > |     from the .dts files. The only reason for this is that it was what
> > > |     PowerPC has done historically. This patch changes ARM to use the
> > > |     generic
> > > |     dtb rule which builds .dtb files in the same directory as the source
> > > |     .dts.
> > > |
> > > |     Cc: Russell King <linux at arm.linux.org.uk>
> > > |     Cc: Arnd Bergmann <arnd at arndb.de>
> > > |     Acked-by: Olof Johansson <olof at lixom.net>
> > > |     Cc: linux-arm-kernel at lists.infradead.org
> > > |     Signed-off-by: Grant Likely <grant.likely at secretlab.ca>
> > > |     [swarren: added rm command for old stale .dtb files]
> > > |     Signed-off-by: Stephen Warren <swarren at nvidia.com>
> > > |     Signed-off-by: Rob Herring <rob.herring at calxeda.com>
> > >
> > > `----
> > >
> > > This change adds support for both places to backward and forward
> > > compatibility.
> > >
> > > Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
> > > Acked-by: Bruce Ashfield <bruce.ashfield at windriver.com>
> >
> > Franklin, I agree this is the way to go but I think it is a bit risky
> > to change it in dylan.
> >
> > I think it could be included in meta-ti or other layers (in case it is
> > need for dylan) but it has breakage risk as noticed today in some
> > behaviour change as found in meta-xilinx.
> >
> > Paul, please be conservative about this and my previous linux-dtb
> > patch. In case it is going to be applied in dylan we need to let it
> > sleep some more time in master to find any other side effect due the
> > behaviour change.
> 
> Agreed, I'd very much prefer to hold on this until it has had time to soak in
> master.
[Franklin] 
No problem. I will put this on the back burner for some time.
> 
> Cheers,
> Paul
> 
> --
> 
> Paul Eggleton
> Intel Open Source Technology Centre