From: Scott Wood <scottwood@freescale.com>
Date: Mon, 3 Mar 2014 20:24:42 -0600
Subject: [U-Boot] [U-Boot,
	v3] nand/denali: Adding Denali NAND driver support
In-Reply-To: <1393520706-10312-1-git-send-email-clsee@altera.com>
References: <1393520706-10312-1-git-send-email-clsee@altera.com>
Message-ID: <20140304022441.GA28533@home.buserror.net>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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@altera.com>
> Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
> Cc: David Woodhouse <David.Woodhouse@intel.com>
> Cc: Brian Norris <computersforpeace@gmail.com>
> Cc: Scott Wood <scottwood@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.

> +/*
> + * 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?

-Scott