[U-Boot] [PATCH V2 08/11] mtd: nand: supress 'unknown NAND' warning if no nand is found
Scott Wood
scottwood at freescale.com
Tue Sep 7 19:37:50 CEST 2010
On Sun, 5 Sep 2010 14:37:13 -0700
Steve Sakoman <sakoman at gmail.com> wrote:
> On Sun, Sep 5, 2010 at 3:59 AM, Sergei Shtylyov <sshtylyov at mvista.com> wrote:
> > On 02-09-2010 19:33, Steve Sakoman wrote:
> >
> >> diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
> >> index ed1c9c9..b2400dd 100644
> >> --- a/drivers/mtd/nand/nand_base.c
> >> +++ b/drivers/mtd/nand/nand_base.c
> >> @@ -2653,8 +2653,11 @@ static struct nand_flash_dev *nand_get_flash_type(struct mtd_info *mtd,
> >> }
> >>
> >> if (!type) {
> >> - printk(KERN_INFO "%s: unknown NAND device: Manufacturer ID:"
> >> - " 0x%02x, Chip ID: 0x%02x\n", __func__,
> >> + /* supress warning if there is no nand */
> >> + if ((*maf_id != 0x00) && (*maf_id != 0xff) &&
> >> + (dev_id != 0x00) && (dev_id != 0xff))
> >
> > Parens around != are not necessary.
>
> Understood -- I just think it is easier to read/understand with
> parens. If the general consensus is that I should remove them I am
> happy to do so.
My vote is no parens.
-Scott
More information about the U-Boot
mailing list