> 
> This printk was added recently and results in ugly output on systems
> with no NAND:
> 
> NAND:  nand_get_flash_type: unknown NAND device: Manufacturer ID: 0x00,
> Chip ID: 0x00 0 MiB
> 
> instead of:
> 
> NAND:  0 MiB
> 
> Signed-off-by: Steve Sakoman <st...@sakoman.com>

Steve,

A proper patch would be very much appreciated instead of an updated patch 
within an e-mail chain.

Scott,

Is it ok if I add this to my tree?

Its part of a 11 patch series and a total of some 20 patches submitted by Steve 


> ---
>  drivers/mtd/nand/nand_base.c |    7 +++++--
>  1 files changed, 5 insertions(+), 2 deletions(-)
> 
> 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))
> +                     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);
>       }
> --
> 1.7.0.4


Regards,
Sandeep
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to