Could you write the funciton where I have to act? I agree the issue about a bug into the cfi_cmdset_0002.c
Ciao Alle 10:47, domenica 7 marzo 2010, Fabio Giovagnini ha scritto: > Ok I'll try. > > Alle 02:17, domenica 7 marzo 2010, Erwin Authried ha scritto: > > Am Samstag, den 06.03.2010, 16:59 -0500 schrieb Mike Frysinger: > > > On Saturday 06 March 2010 12:17:12 Fabio Giovagnini wrote: > > > > According to you, it is easier to add jedec entry or it is better to > > > > modify cfi_cmdset_0002.c? > > > > I'm newbe, so I aks: what could be the trade off between the two > > > > solutions? > > > > > > one can be sent upstream (jedec) and one cannot (hacking cmdset in non- > > > standard ways) > > > -mike > > > > I took a look at AMD's CFI specification. Both of the extended query > > tables are optional. Thus, it looks like relying that at least one of > > those tables exist is a bug in cfi_cmdset_0002. I believe this can be > > fixed this way: > > > > ... > > __u16 adr = primary?cfi->cfiq->P_ADR:cfi->cfiq->A_ADR; > > if (cfi->cfi_mode==CFI_MODE_CFI && adr){ > > unsigned char bootloc; > > /* > > * It's a real CFI chip, not one for which the probe > > * routine faked a CFI structure. So we read the feature > > * table from it. > > */ > > struct cfi_pri_amdstd *extp; > > ... > > > > Fabio, would you like to try if this works? I don't have a flash without > > extended query table, thus I can't test this. > > > > -Erwin > > > > _______________________________________________ > > uClinux-dev mailing list > > uClinux-dev@uclinux.org > > http://mailman.uclinux.org/mailman/listinfo/uclinux-dev > > This message was resent by uclinux-dev@uclinux.org > > To unsubscribe see: > > http://mailman.uclinux.org/mailman/options/uclinux-dev -- Fabio Giovagnini Aurion s.r.l. CF e P.IVA 00885711200 Tel. +39.051.594.78.24 Cell. +39.335.8350919 www.aurion-tech.com account telefono VoIP skype (www.skype.com): aurion.giovagnini _______________________________________________ uClinux-dev mailing list uClinux-dev@uclinux.org http://mailman.uclinux.org/mailman/listinfo/uclinux-dev This message was resent by uclinux-dev@uclinux.org To unsubscribe see: http://mailman.uclinux.org/mailman/options/uclinux-dev