[U-Boot] [PATCH 5/7] board: ti: am57xx-idk: Auto detect LCD Panel

[Roger Quadros]

On 26/01/17 19:33, menon.nishanth at 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?

cheeers,
-roger