[U-Boot] [U-Boot, v3] nand/denali: Adding Denali NAND driver support

Chin Liang See clsee at altera.com
Wed Mar 5 18:35:42 CET 2014


On Mon, 2014-03-03 at 20:24 -0600, Scott Wood wrote:
> On Thu, Feb 27, 2014 at 11:05:06AM -0600, Chin Liang See wrote:
> > To add the Denali NAND driver support into U-Boot. It required
> > information such as register base address from configuration
> > header file  within include/configs folder.
> > 
> > Signed-off-by: Chin Liang See <clsee at altera.com>
> > Cc: Artem Bityutskiy <artem.bityutskiy at linux.intel.com>
> > Cc: David Woodhouse <David.Woodhouse at intel.com>
> > Cc: Brian Norris <computersforpeace at gmail.com>
> > Cc: Scott Wood <scottwood at freescale.com>
> > 
> > ---
> > Changes for v3
> > - Fixed coding style
> > Changes for v2
> > - Enable this driver support for SOCFPGA
> 
> Sorry, didn't see this when I replied to v2.  Some of those comments
> still apply, though.


Yup, fixed


> 
> > +/*
> > + * This macro divides two integers and rounds fractional values up
> > + * to the nearest integer value.
> > + */
> > +#define CEIL_DIV(X, Y) (((X)%(Y)) ? ((X)/(Y)+1) : ((X)/(Y)))
> 
> How is this different from DIV_ROUND_UP(), other than being more
> complicated?


All change to use DIV_ROUND_UP as no need to duplicate

Thanks

Chin Liang

> 
> -Scott
> 




More information about the U-Boot mailing list