[U-Boot] [PATCH v4 09/11] S3C24XX: Add NAND Flash driver

Scott Wood scottwood at freescale.com
Wed Sep 19 18:19:01 CEST 2012


On 09/19/2012 06:25:25 AM, José Miguel Gonçalves wrote:
> NAND Flash driver with HW ECC for the S3C24XX SoCs.
> Currently it only supports SLC NAND chips.
> 
> Signed-off-by: José Miguel Gonçalves <jose.goncalves at inov.pt>
> ---
> Changes for v2:
>    - Coding style cleanup
>    - Use of clrsetbits_le32()
>    - Use of register bit macros instead of magic numbers
> 
> Changes for v3:
>    - Removed magic numbers
>    - Removed a macro to declare a void printf()
>    - Replaced one printf() with a puts()
> 
> Changes for v4:
>    - Coding style cleanup
>    - Use of a struct to store chip private data
>    - Replaced u_long by u32
>    - Replaced u_char by uint8_t
>    - Added error message in s3c_nand_select_chip()
>    - Optimization of s3c_nand_hwcontrol()
> ---
>  drivers/mtd/nand/Makefile       |    1 +
>  drivers/mtd/nand/s3c24xx_nand.c |  255  
> +++++++++++++++++++++++++++++++++++++++
>  2 files changed, 256 insertions(+)
>  create mode 100644 drivers/mtd/nand/s3c24xx_nand.c

Acked-by: Scott Wood <scottwood at freescale.com>

> +	nand->priv = (void *)&chip_info[chip_n++];

Hmm, I was going to say that the cast could be dropped, but I guess  
it's because of the const.  It doesn't need to hold up acceptance of  
the patch, but it'd probably be better to drop the const instead (more  
flexible too if additional private data needs to be declared).

-Scott


More information about the U-Boot mailing list