[U-Boot] [PATCH 2/2] sunxi: Make dram odt-en configurable through Kconfig for A33 based boards
Ian Campbell
ijc+uboot at hellion.org.uk
Wed May 13 21:26:20 CEST 2015
On Wed, 2015-05-13 at 17:09 +0200, Hans de Goede wrote:
> diff --git a/board/sunxi/Kconfig b/board/sunxi/Kconfig
> index 940b6c7..d4ae6c7 100644
> --- a/board/sunxi/Kconfig
> +++ b/board/sunxi/Kconfig
> @@ -95,6 +95,14 @@ config DRAM_ZQ
> ---help---
> Set the dram zq value.
>
> +if MACH_SUN8I_A33
Shouldn't this be a "depends on MACH_SUN8I_A33" in the entry itself? I
see we use if a lot in this file, is there a reason for that or just how
it has been done?
In any case using if here is at least consistent so no strong objection
on that grounds.
> +config DRAM_ODT_EN
> + int "sunxi dram odt enable"
> + default 0
> + ---help---
> + Set this to 1 to enable dram odt (on die termination)
Why is this an int rather than a bool?
More information about the U-Boot
mailing list