From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Date: Thu, 24 Jun 2010 11:56:33 +0200 Subject: [U-Boot] [PATCH v2] NAND: show manufacturer and device ID for unknown chips In-Reply-To: <20100623215301.GA9212@schlenkerla.am.freescale.net> References: <20100623215301.GA9212@schlenkerla.am.freescale.net> Message-ID: <201006241156.33244.florian@openwrt.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hey Scott, On Wednesday 23 June 2010 23:53:01 Scott Wood wrote: > From: Florian Fainelli > > When the NAND part is not supported, it is useful to show the manufacturer > and device ID to help debugging and reporting. > > Signed-off-by: Florian Fainelli > Signed-off-by: Scott Wood > --- > Whitespace amended to be consistent with rest of file. Thanks, I was just about to respin it. > > Applied to u-boot-nand-flash next. > > drivers/mtd/nand/nand_base.c | 6 +++++- > 1 files changed, 5 insertions(+), 1 deletions(-) > > diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c > index 7171bdd..ed1c9c9 100644 > --- a/drivers/mtd/nand/nand_base.c > +++ b/drivers/mtd/nand/nand_base.c > @@ -2652,8 +2652,12 @@ static struct nand_flash_dev > *nand_get_flash_type(struct mtd_info *mtd, } > } > > - if (!type) > + if (!type) { > + printk(KERN_INFO "%s: unknown NAND device: Manufacturer ID:" > + " 0x%02x, Chip ID: 0x%02x\n", __func__, > + *maf_id, dev_id); > return ERR_PTR(-ENODEV); > + } > > if (!mtd->name) > mtd->name = type->name;