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

[menon.nishanth at gmail.com]

On Fri, Jan 27, 2017 at 2:54 AM, Roger Quadros <rogerq at ti.com> wrote:
> 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?

That'd make sense.

Regards,
Nishanth Menon