[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Mon, 6 Jun 2016 22:59:03 +0300
From: Aleksei Mamlin <mamlinav@...il.com>
To: Boris Brezillon <boris.brezillon@...e-electrons.com>
Cc: Maxime Ripard <maxime.ripard@...e-electrons.com>,
Chen-Yu Tsai <wens@...e.org>,
Richard Weinberger <richard@....at>,
David Woodhouse <dwmw2@...radead.org>,
Brian Norris <computersforpeace@...il.com>,
linux-mtd@...ts.infradead.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
linux-sunxi@...glegroups.com
Subject: Re: [PATCH 5/7] mtd: nand: Add Hynix H27UBG8T2BTR-BC to nand_ids
table
On Mon, 6 Jun 2016 20:55:49 +0200
Boris Brezillon <boris.brezillon@...e-electrons.com> wrote:
> On Mon, 6 Jun 2016 13:24:22 +0300
> Aleksei Mamlin <mamlinav@...il.com> wrote:
>
> > Add the full description of the Hynix H27UBG8T2BTR-BC NAND chip in the
> > nand_ids table so that we can later use the NAND ECC infos and ONFI timings
> > mode in controller drivers.
>
> Still hoping to get this series [1] merged in 4.8, but if that's
> not the case, I'll apply your patch.
>
> BTW, that would be great if you could test it on your platforms.
>
It seems that Hynix-specific initialization code can't handle H27UBG8T2BTR-BC
chip:
[ 0.886153] nand: Could not find valid ONFI parameter page; aborting
[ 0.892665] nand: device found, Manufacturer ID: 0xad, Chip ID: 0xd7
[ 0.899025] nand: Hynix 1c03000.nand
[ 0.902596] nand: bus width 8 instead 16 bit
[ 0.906858] nand: No NAND device found
[ 0.910620] sunxi_nand 1c03000.nand: failed to init nand chips
[ 0.916528] sunxi_nand: probe of 1c03000.nand failed with error -22
>
> Regards,
>
> Boris
>
> [1]https://lkml.org/lkml/2016/5/27/264
>
> >
> > Signed-off-by: Aleksei Mamlin <mamlinav@...il.com>
> > ---
> > drivers/mtd/nand/nand_ids.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/drivers/mtd/nand/nand_ids.c b/drivers/mtd/nand/nand_ids.c
> > index ccc05f5..ccdc773 100644
> > --- a/drivers/mtd/nand/nand_ids.c
> > +++ b/drivers/mtd/nand/nand_ids.c
> > @@ -52,6 +52,10 @@ struct nand_flash_dev nand_flash_ids[] = {
> > { .id = {0xad, 0xde, 0x94, 0xda, 0x74, 0xc4} },
> > SZ_8K, SZ_8K, SZ_2M, NAND_NEED_SCRAMBLING, 6, 640,
> > NAND_ECC_INFO(40, SZ_1K), 4 },
> > + {"H27UBG8T2BTR-BC 32G 3.3V 8-bit",
> > + { .id = {0xad, 0xd7, 0x94, 0xda, 0x74, 0xc3} },
> > + SZ_8K, SZ_4K, SZ_2M, NAND_NEED_SCRAMBLING, 6, 640,
> > + NAND_ECC_INFO(40, SZ_1K), 0 },
> >
> > LEGACY_ID_NAND("NAND 4MiB 5V 8-bit", 0x6B, 4, SZ_8K, SP_OPTIONS),
> > LEGACY_ID_NAND("NAND 4MiB 3,3V 8-bit", 0xE3, 4, SZ_8K, SP_OPTIONS),
>
>
>
> --
> Boris Brezillon, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com
--
Thanks and regards,
Aleksei Mamlin
Powered by blists - more mailing lists