Thanks Sam and Denys. I just submitted the v2 patch.

Regards,
Hongmei

> -----Original Message-----
> From: Nelson, Sam
> Sent: Wednesday, February 08, 2017 3:25 PM
> To: Dmytriyenko, Denys
> Cc: Gou, Hongmei; [email protected]
> Subject: RE: [meta-arago] [morty/krogoth][patch] ti-tisdk-makefile: add
> am572x-idk-pru-excl-uio.dtb to linux-dtbs
> 
> Ok. I misunderstood.
> Yes. It makes sense make this addition in makefile conditional with
> ENABLE_TI_UIO_DEVICES.
> 
> Hongmei,
> See reference in
> meta-ti/recipes-kernel/linux/ti-uio.inc
> 
> KERNEL_DEVICETREE_append_am57xx-evm =
> "${@base_conditional("ENABLE_TI_UIO_DEVICES", "1", " am572x-idk-pru-
> excl-uio.dtb", "", d)}"
> 
> 
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Wednesday, February 08, 2017 3:05 PM
> > To: Nelson, Sam
> > Cc: Gou, Hongmei; [email protected]
> > Subject: Re: [meta-arago] [morty/krogoth][patch] ti-tisdk-makefile:
> > add am572x-idk-pru-excl-uio.dtb to linux-dtbs
> >
> > Right, but my point was that the corresponding dtb will not even be
> > installed when uio is not enabled with ENABLE_TI_UIO_DEVICES
> >
> >
> > On Wed, Feb 08, 2017 at 03:00:09PM -0500, Nelson, Sam wrote:
> > > My understanding is just the device tree source compilation should
> > > NOT
> > depend on the UIO enabled in kernel.
> > > So I don't expect it to fail.
> > >
> > > > -----Original Message-----
> > > > From: Gou, Hongmei
> > > > Sent: Monday, February 06, 2017 12:10 PM
> > > > To: Dmytriyenko, Denys; Nelson, Sam
> > > > Cc: [email protected]
> > > > Subject: RE: [meta-arago] [morty/krogoth][patch]
> > > > ti-tisdk-makefile: add am572x-idk-pru-excl-uio.dtb to linux-dtbs
> > > >
> > > > Looping in Sam for this question...
> > > >
> > > > > -----Original Message-----
> > > > > From: Dmytriyenko, Denys
> > > > > Sent: Monday, February 06, 2017 11:52 AM
> > > > > To: Gou, Hongmei
> > > > > Cc: [email protected]
> > > > > Subject: Re: [meta-arago] [morty/krogoth][patch] ti-tisdk-makefile:
> > > > > add am572x-idk-pru-excl-uio.dtb to linux-dtbs
> > > > >
> > > > > Will this fail if UIO is not enabled?
> > > > >
> > > > >
> > > > > On Mon, Feb 06, 2017 at 11:43:57AM -0500, Hongmei Gou wrote:
> > > > > > Signed-off-by: Hongmei Gou <[email protected]>
> > > > > > ---
> > > > > >  .../recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb       
> > > > > >    | 4
> ++-
> > -
> > > > > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > > > >
> > > > > > diff --git
> > > > > > a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-m
> > > > > > akefil
> > > > > > e_
> > > > > > 1.0.bb
> > > > > > b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-m
> > > > > > akefil
> > > > > > e_
> > > > > > 1.0.bb
> > > > > > index 8cc0758..c27bba6 100644
> > > > > > ---
> > > > > > a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-m
> > > > > > akefil
> > > > > > e_
> > > > > > 1.0.bb
> > > > > > +++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tis
> > > > > > +++ dk-mak
> > > > > > +++ ef
> > > > > > +++ ile_1.0.bb
> > > > > > @@ -50,7 +50,7 @@ SRC_URI = "\
> > > > > >      file://Makefile_barcode-roi \  "
> > > > > >
> > > > > > -PR = "r77"
> > > > > > +PR = "r78"
> > > > > >
> > > > > >  MAKEFILES_MATRIX_GUI = "matrix-gui-browser \
> > > > > >                          refresh-screen \ @@ -188,7 +188,7 @@
> > > > > > KERNEL_DEVICETREE_ti43x = "am43x-epos-evm.dtb am437x-gp-
> > evm.dtb
> > > > > > am437x-gp-evm-hd  KERNEL_DEVICETREE_beaglebone = "am335x-
> > > > > bone.dtb am335x-boneblack.dtb am335x-bonegreen.dtb"
> > > > > >  KERNEL_DEVICETREE_omap5-evm = "omap5-uevm.dtb"
> > > > > >  KERNEL_DEVICETREE_dra7xx-evm = "dra7-evm.dtb dra7-evm-lcd-
> > lg.dtb
> > > > > dra7-evm-lcd-osd.dtb dra72-evm.dtb dra72-evm-revc.dtb dra72-evm-
> > lcd-
> > > > > lg.dtb dra72-evm-lcd-osd.dtb"
> > > > > > -KERNEL_DEVICETREE_am57xx-evm = "am57xx-beagle-x15.dtb
> > am57xx-
> > > > > beagle-x15-revb1.dtb am57xx-evm.dtb am57xx-evm-reva3.dtb
> am571x-
> > > > > idk.dtb am572x-idk.dtb am571x-idk-lcd-osd.dtb am572x-idk-lcd-
> > osd.dtb"
> > > > > > +KERNEL_DEVICETREE_am57xx-evm = "am57xx-beagle-x15.dtb
> > am57xx-
> > > > > beagle-x15-revb1.dtb am57xx-evm.dtb am57xx-evm-reva3.dtb
> am571x-
> > > > > idk.dtb am572x-idk.dtb am571x-idk-lcd-osd.dtb
> > > > > am572x-idk-lcd-osd.dtb am572x-idk-pru-excl-uio.dtb"
> > > > > >  KERNEL_DEVICETREE_k2hk-evm = "keystone-k2hk-evm.dtb"
> > > > > >  KERNEL_DEVICETREE_k2e = "keystone-k2e-evm.dtb"
> > > > > >  KERNEL_DEVICETREE_k2g = "keystone-k2g-evm.dtb"
> > > > > > --
> > > > > > 1.9.1
> > > > > >
> > > > > > _______________________________________________
> > > > > > meta-arago mailing list
> > > > > > [email protected]
> > > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Reply via email to