On Mon, Aug 13, 2018 at 4:25 PM, Chen-Yu Tsai <[email protected]> wrote:
> 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

Removed 'default n' and

Applied to u-boot-sunxi/master
_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot

Reply via email to