[U-Boot] [PATCH v2] mtd: nand: new base driver for memory mapped nand devices
Mike Frysinger
vapier at gentoo.org
Wed May 6 21:14:53 CEST 2009
On Wednesday 06 May 2009 14:19:07 Scott Wood wrote:
> Mike Frysinger wrote:
> >>> +/* #define CONFIG_NAND_PLAT */
> >>
> >> Why is this commented out?
> >
> > because it's a driver for an optional add-on card that people usually
> > dont have, let alone plugged in
>
> OK, was hoping there would be at least one config that selects it so it
> gets compilation exposure.
i have another board that does rely on it that i'll be merging in the next
cycle
> >>> +#define NAND_PLAT_DEV_READY(chip) ((*pPORTFIO & BFIN_NAND_READY)
> >>> ? 1 : 0)
> >>
> >> Why not just (*pPORTFIO & BFIN_NAND_READY)?
> >
> > i thought the nand/mtd layers expect 1/0 only ? if the higher layers
> > dont care, then there's no reason.
>
> A quick grep doesn't show any non-boolean uses.
sounds good, i'll make the change then
-mike
More information about the U-Boot
mailing list