On Mon, Aug 13, 2018 at 6:29 PM, Jagan Teki <[email protected]> wrote:
> On Wed, Jul 25, 2018 at 5:32 AM, Andre Przywara <[email protected]> 
> wrote:
>> "default" lines in Kconfig are processed in order, the first hit will
>> stop considering subsequent lines. In the case of the DRAM_ODT_EN symbol
>> that means that everything following the first two lines will never be
>> checked:
>> ------------
>> config DRAM_ODT_EN
>>         bool "sunxi dram odt enable"
>>         default n if !MACH_SUN8I_A23
>>         default y if MACH_SUN8I_A23
>>         default y if MACH_SUN8I_R40
>>         default y if MACH_SUN50I
>> ------------
>>
>> Assuming that the "default y" for the A64 and the R40 were a deliberate
>> choice, fix the Kconfig stanza to take this into account.
>> Also remove the now redundant lines from the respective defconfigs.
>>
>> Signed-off-by: Andre Przywara <[email protected]>
>> ---
>
> Tested-by: Jagan Teki <[email protected]> # A64, R40
>
> Can someone confirm A23?

Tested-by: Chen-Yu Tsai <[email protected]> # A23
_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot

Reply via email to