[Help] SST39VF6401B Support
Flávio Silveira
fggs at terra.com.br
Sun Jun 19 23:12:14 EDT 2011
Hi,
I'm attaching some other files to see if it helps finding what's wrong.
Thanks in advance!
----- Original Message -----
From: "Flávio Silveira" <fggs at terra.com.br>
To: "Fabio Giovagnini" <fabio.giovagnini at aurion-tech.com>;
<linux-mtd at lists.infradead.org>
Cc: "yidong zhang" <zhangyd6 at gmail.com>; <David.Woodhouse at intel.com>;
"Wolfram Sang" <w.sang at pengutronix.de>; <yegorslists at googlemail.com>;
"Guillaume LECERF" <glecerf at gmail.com>; <taliaferro62 at gmail.com>
Sent: Wednesday, June 15, 2011 9:55 PM
Subject: Re: [Help] SST39VF6401B Support
Hi guys,
I've tested this patch on kernel 2.6.23 (actually it's called 2.6.24.111)
and didn't work.
Below are my diffs and I can post original files as well, please help me
make it work!
Thanks in advance!
cfi_util.c
29a30,107
> int __xipram cfi_qry_present(struct map_info *map, __u32 base,
> struct cfi_private *cfi)
> {
> int osf = cfi->interleave * cfi->device_type; /* scale factor */
> map_word val[3];
> map_word qry[3];
>
> qry[0] = cfi_build_cmd('Q', map, cfi);
> qry[1] = cfi_build_cmd('R', map, cfi);
> qry[2] = cfi_build_cmd('Y', map, cfi);
>
> val[0] = map_read(map, base + osf*0x10);
> val[1] = map_read(map, base + osf*0x11);
> val[2] = map_read(map, base + osf*0x12);
>
> if (!map_word_equal(map, qry[0], val[0]))
> return 0;
>
> if (!map_word_equal(map, qry[1], val[1]))
> return 0;
>
> if (!map_word_equal(map, qry[2], val[2]))
> return 0;
>
> return 1; /* "QRY" found */
> }
> EXPORT_SYMBOL_GPL(cfi_qry_present);
>
> int __xipram cfi_qry_mode_on(uint32_t base, struct map_info *map,
> struct cfi_private *cfi)
> {
> cfi_send_gen_cmd(0xF0, 0, base, map, cfi, cfi->device_type, NULL);
> cfi_send_gen_cmd(0x98, 0x55, base, map, cfi, cfi->device_type, NULL);
> if (cfi_qry_present(map, base, cfi))
> return 1;
> /* QRY not found probably we deal with some odd CFI chips */
> /* Some revisions of some old Intel chips? */
> cfi_send_gen_cmd(0xF0, 0, base, map, cfi, cfi->device_type, NULL);
> cfi_send_gen_cmd(0xFF, 0, base, map, cfi, cfi->device_type, NULL);
> cfi_send_gen_cmd(0x98, 0x55, base, map, cfi, cfi->device_type, NULL);
> if (cfi_qry_present(map, base, cfi))
> return 1;
> /* ST M29DW chips */
> cfi_send_gen_cmd(0xF0, 0, base, map, cfi, cfi->device_type, NULL);
> cfi_send_gen_cmd(0x98, 0x555, base, map, cfi, cfi->device_type, NULL);
> if (cfi_qry_present(map, base, cfi))
> return 1;
> /* some old SST chips, e.g. 39VF160x/39VF320x */
> cfi_send_gen_cmd(0xF0, 0, base, map, cfi, cfi->device_type, NULL);
> cfi_send_gen_cmd(0xAA, 0x5555, base, map, cfi, cfi->device_type, NULL);
> cfi_send_gen_cmd(0x55, 0x2AAA, base, map, cfi, cfi->device_type, NULL);
> cfi_send_gen_cmd(0x98, 0x5555, base, map, cfi, cfi->device_type, NULL);
> if (cfi_qry_present(map, base, cfi))
> return 1;
> /* SST 39VF640xB */
> cfi_send_gen_cmd(0xF0, 0, base, map, cfi, cfi->device_type, NULL);
> cfi_send_gen_cmd(0xAA, 0x555, base, map, cfi, cfi->device_type, NULL);
> cfi_send_gen_cmd(0x55, 0x2AA, base, map, cfi, cfi->device_type, NULL);
> cfi_send_gen_cmd(0x98, 0x555, base, map, cfi, cfi->device_type, NULL);
> if (cfi_qry_present(map, base, cfi))
> return 1;
> /* QRY not found */
> return 0;
> }
> EXPORT_SYMBOL_GPL(cfi_qry_mode_on);
>
> void __xipram cfi_qry_mode_off(uint32_t base, struct map_info *map,
> struct cfi_private *cfi)
> {
> cfi_send_gen_cmd(0xF0, 0, base, map, cfi, cfi->device_type, NULL);
> cfi_send_gen_cmd(0xFF, 0, base, map, cfi, cfi->device_type, NULL);
> /* M29W128G flashes require an additional reset command
> when exit qry mode */
> if ((cfi->mfr == CFI_MFR_ST) && (cfi->id == 0x227E || cfi->id == 0x7E))
> cfi_send_gen_cmd(0xF0, 0, base, map, cfi, cfi->device_type, NULL);
> }
> EXPORT_SYMBOL_GPL(cfi_qry_mode_off);
>
cfi_cmdset_0002.c
219a220,253
> static void fixup_old_sst_eraseregion(struct mtd_info *mtd)
> {
> struct map_info *map = mtd->priv;
> struct cfi_private *cfi = map->fldrv_priv;
>
> /*
> * These flashes report two seperate eraseblock regions based on the
> * sector_erase-size and block_erase-size, although they both operate
> on the
> * same memory. This is not allowed according to CFI, so we just pick
> the
> * sector_erase-size.
> */
> cfi->cfiq->NumEraseRegions = 1;
> }
>
> static void fixup_sst39vf_rev_b(struct mtd_info *mtd, void *param)
> {
> struct map_info *map = mtd->priv;
> struct cfi_private *cfi = map->fldrv_priv;
>
> fixup_old_sst_eraseregion(mtd);
>
> cfi->addr_unlock1 = 0x555;
> cfi->addr_unlock2 = 0x2AA;
> }
>
> /* Used to fix CFI-Tables of chips without Extended Query Tables */
> static struct cfi_fixup cfi_nopri_fixup_table[] = {
> { CFI_MFR_SST, 0x235C, fixup_sst39vf_rev_b, NULL, }, // SST39VF3202B
> { CFI_MFR_SST, 0x235D, fixup_sst39vf_rev_b, NULL, }, // SST39VF3201B
> { CFI_MFR_SST, 0x236C, fixup_sst39vf_rev_b, NULL, }, // SST39VF6402B
> { CFI_MFR_SST, 0x236D, fixup_sst39vf_rev_b, NULL, }, // SST39VF6401B
> { 0, 0, NULL, NULL }
> };
>
----- Original Message -----
From: "Fabio Giovagnini" <fabio.giovagnini at aurion-tech.com>
To: <linux-mtd at lists.infradead.org>
Cc: "yidong zhang" <zhangyd6 at gmail.com>; <David.Woodhouse at intel.com>;
"Wolfram Sang" <w.sang at pengutronix.de>; <yegorslists at googlemail.com>;
"Guillaume LECERF" <glecerf at gmail.com>; <taliaferro62 at gmail.com>
Sent: Friday, October 22, 2010 2:50 PM
Subject: Re: [Help] SST39VF6401B Support
Hi Guys,
As Gullelm remeber for sure, I used the jedec probe for such a flash in a
very
old 2.6 kernel.
Now I'm migrating to the newets one kernel.
Do you tell me how the kernel will be updated with the last mtd stuff
developed; or in place of it how to patch a standard kernel?
Thanks a lot and sorry for my question if it is improper
In data venerdì 22 ottobre 2010 17:42:39, yidong zhang ha scritto:
: > On Fri, Oct 22, 2010 at 11:36 PM, Guillaume LECERF <glecerf at gmail.com>
wrote:
> > 2010/10/22 yidong zhang <zhangyd6 at gmail.com>:
> >>> Could you try setting cif->cfiq->EraseRegionInfo[0] to 0x02003FF in
> >>> fixup_sst39vf_rev_b() ?
> >>
> >> Hi
> >> how about this attached patch.
> >
> > If it works for you, I'll send a patch series for SST 39VF640xB and
> > 38VF640x, because both chips needs 0x50 as erase command.
>
> Sorry, i cannot test it right now. I will sent the result as soon as i
> can.
>
> > --
> > Guillaume LECERF
> > GeeXboX developer - www.geexbox.org
>
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/
--
Ing. Fabio Giovagnini
Aurion s.r.l.
P.I e C.F.
00885711200
skype: aurion.giovagnini
Tel. +39.051.594.78.24
Cell. +39.335.83.50.919
www.aurion-tech.com
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cfi_cmdset_0002.c
Type: application/octet-stream
Size: 52791 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-mtd/attachments/20110620/15fdaa22/attachment-0003.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cfi_util.c
Type: application/octet-stream
Size: 6988 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-mtd/attachments/20110620/15fdaa22/attachment-0004.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gen_probe.c
Type: application/octet-stream
Size: 7011 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-mtd/attachments/20110620/15fdaa22/attachment-0005.obj>
More information about the linux-mtd
mailing list