[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
Thu May 14 20:41:21 CEST 2015
On Thu, 2015-05-14 at 18:55 +0200, Hans de Goede wrote:
> >> +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?
>
> Because it is used directly as an int in the code, otherwise I
> need to add #ifdef-ery. I guess I could make it a bool and use
> IS_ENABLED(), but that does lead to slightly less readable code
> IMHO.
I'm afraid I think IS_ENABLED is the way to go though, it's the lesser
of two evils compared with using an int for a boolean option.
Ian.
More information about the U-Boot
mailing list