Messages in this thread | | | Date | Thu, 10 Feb 2011 22:41:31 +0100 | From | Wolfram Sang <> | Subject | Re: [PATCH] mtd: cfi_cmdset_0002: fix sector erase command for AM29LV040B |
| |
On Fri, Feb 11, 2011 at 12:04:37AM +0300, Antony Pavlov wrote: > The AM29LV040B chips use 0x50 as sector erase command instead of 0x30 [1]. > > [1] http://www.datasheetcatalog.com/datasheets_pdf/A/M/2/9/AM29LV040B.shtml
I am confused. The datasheet says on page 15 it uses 0x30 while your description above says it uses 0x50...
> Signed-off-by: Antony Pavlov <antony@niisi.msk.ru> > --- > drivers/mtd/chips/cfi_cmdset_0002.c | 9 +++++++++ > 1 files changed, 9 insertions(+), 0 deletions(-) > > diff --git a/drivers/mtd/chips/cfi_cmdset_0002.c b/drivers/mtd/chips/cfi_cmdset_0002.c > index f072fcf..a401dbe 100644 > --- a/drivers/mtd/chips/cfi_cmdset_0002.c > +++ b/drivers/mtd/chips/cfi_cmdset_0002.c > @@ -332,6 +332,14 @@ static void fixup_s29gl032n_sectors(struct mtd_info *mtd) > } > } > > +static void fixup_use_sector_erase_cmd_30(struct mtd_info *mtd) > +{ > + struct map_info *map = mtd->priv; > + struct cfi_private *cfi = map->fldrv_priv; > + > + cfi->sector_erase_cmd = CMD(0x30);
... but here you change it to 0x30 (which is the default) instead of 0x50?
Please clarify,
Wolfram
-- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | [unhandled content-type:application/pgp-signature] | |