From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Thu, 20 Jun 2013 15:57:04 -0500
Subject: [U-Boot] [PATCH v2 5/5] ARM: at91: atmel_nand: add code to
 check the ONFI parameter ECC requirement
In-Reply-To: <1371466281-31922-6-git-send-email-josh.wu@atmel.com> (from
	josh.wu@atmel.com on Mon Jun 17 05:51:21 2013)
Message-ID: <1371761824.11064.15@snotra>
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 06/17/2013 05:51:21 AM, Josh Wu wrote:
> +static int pmecc_choose_ecc(struct atmel_nand_host *host,
> +		struct nand_chip *chip,
> +		int *cap, int *sector_size)
> +{
> +	/* Get ECC requirement from ONFI parameters */
> +	*cap = *sector_size = 0;
> +	if (chip->onfi_version) {
> +		if (!get_onfi_ecc_param(chip, cap, sector_size))
> +			pr_debug("ONFI params, minimum required ECC: %d  
> bits in %d bytes\n",
> +				*cap, *sector_size);
> +		else
> +			dev_info(NULL, "NAND chip ECC reqirement is in  
> Extended ONFI parameter, we don't support yet.\n");
> +	} else {
> +		dev_info(NULL, "NAND chip is not ONFI compliant, assume  
> ecc_bits is 2 in 512 bytes");
> +	}

Don't pass NULL to dev_info().  Either pass host->dev as the Linux code  
does (and use dev_dbg rather than pr_debug), or just use printf() (and  
debug()) if there's no intent to sync this change back to Linux.  It  
doesn't matter if host->dev doesn't exist in U-Boot, as the macro  
doesn't use that parameter, but the only reason to use Linux-style  
prints is to keep the code common with Linux.

-Scott