On Fri, Jan 27, 2017 at 2:54 AM, Roger Quadros <rog...@ti.com> wrote: > On 26/01/17 19:33, menon.nisha...@gmail.com wrote: >> [...] >> >> >>> diff --git a/include/configs/ti_omap5_common.h >>> b/include/configs/ti_omap5_common.h >>> index 6de19c2..1fa2f82 100644 >>> --- a/include/configs/ti_omap5_common.h >>> +++ b/include/configs/ti_omap5_common.h >>> @@ -91,16 +91,22 @@ >>> "setenv fdtfile am57xx-beagle-x15.dtb; fi;" \ >>> "if test $board_name = beagle_x15_revb1; then " \ >>> "setenv fdtfile am57xx-beagle-x15-revb1.dtb; fi;" \ >>> - "if test $board_name = am572x_idk; then " \ >>> - "setenv fdtfile am572x-idk.dtb; fi;" \ >>> "if test $board_name = am57xx_evm; then " \ >>> "setenv fdtfile am57xx-beagle-x15.dtb; fi;" \ >>> "if test $board_name = am57xx_evm_reva3; then " \ >>> "setenv fdtfile am57xx-beagle-x15.dtb; fi;" \ >>> + "if test $board_name = am572x_idk && test $idk_lcd = no; >>> then " \ >>> + "setenv fdtfile am572x-idk.dtb; fi;" \ >>> + "if test $board_name = am572x_idk && test $idk_lcd = >>> osd101t2045; then " \ >>> + "setenv fdtfile am572x-idk-lcd-osd.dtb; fi;" \ >>> + "if test $board_name = am572x_idk && test $idk_lcd = >>> osd101t2587; then " \ >>> + "setenv fdtfile am572x-idk-lcd-osd101t2587.dtb; >>> fi;" \ >>> "if test $board_name = am571x_idk && test $idk_lcd = no; >>> then " \ >>> "setenv fdtfile am571x-idk.dtb; fi;" \ >>> "if test $board_name = am571x_idk && test $idk_lcd = >>> osd101t2045; then " \ >>> "setenv fdtfile am571x-idk-lcd-osd.dtb; fi;" \ >>> + "if test $board_name = am571x_idk && test $idk_lcd = >>> osd101t2587; then " \ >>> + "setenv fdtfile am571x-idk-lcd-osd101t2587.dtb; >>> fi;" \ >> >> >> please drop the fdt selection from your patches. upstream will be >> using dt overlay for these variations. rest of the detection code is >> ok to go. >> > > OK. > Just to clarify, dt overlay code will use the idk_lcd variable to patch the > DT right?
That'd make sense. Regards, Nishanth Menon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot