[flashrom] [PATCH] Support for S25FL127S

Jernej Škrabec jernej.skrabec at planet.si
Mon Dec 1 23:22:41 CET 2014


Hello Stefan,

sorry for late response, I've been busy these days. I hope that this new patch is acceptable for now.

Best regards,
Jernej Škrabec 

----- Izvorno sporočilo -----
Od: "Stefan Tauner" <stefan.tauner at alumni.tuwien.ac.at>
Za: "Jernej Škrabec" <jernej.skrabec at planet.si>
Kp: flashrom at flashrom.org
Poslano: sobota, 29. november 2014 21:03:26
Zadeva: Re: [flashrom] [PATCH] Support for S25FL127S

On Mon, 24 Nov 2014 23:59:15 +0100 (CET)
Jernej Škrabec <jernej.skrabec at planet.si> wrote:

> Hi!
> 
> I'm attaching a patch which adds support for Spansion S25FL127S flash chip in 64KiB and 256KiB mode. I also tested 64KiB mode with FT4232H Mini Module which works fine (please see attached logs).

Hello Jernej,

thanks for your patch! Please note that before we can include your
patch you need to sign the Developer's Certificate of Origin by
stating so in the patch/email. We can not include your patch as is!
Please read
http://flashrom.org/Development_Guidelines#Sign-off_Procedure
for details, thanks.

Apart from that the naming of the chip definitions does not make much
sense. The model number of the S25FL127S does not denote the sector
layout in any way (unlike other Spansion chips). It would be best to
write a custom probing function that checks for the flags in the
configuration register and alters struct flashchip accordingly (similar
to what we do for at45db chips).
But for now I would accept a simple suffix to the "S25FL127S" name and
propose S25FL127S-64kB and S25FL127S-256kB.

Also, the comment in the patch is not correct AFAICS. Quote from the
datasheet "A P4E command applied to a sector that is larger than
4 kbytes will not be executed and will not set the E_ERR status."
This contradicts the comment that states that a bit is changed.
There are some minor other things I'd like to change but I can make
these changes myself. Thank you very much for your effort. Please reply
with your sign-off so that I can integrate the patch soon.
-- 
Kind regards/Mit freundlichen Grüßen, Stefan Tauner
-------------- next part --------------
A non-text attachment was scrubbed...
Name: S25FL127S_support.patch
Type: text/x-patch
Size: 2571 bytes
Desc: not available
URL: <http://www.flashrom.org/pipermail/flashrom/attachments/20141201/abdcdaa7/attachment.patch>


More information about the flashrom mailing list