Re: [flashrom] [PATCH] Add support for Spansion S25FL127S.

[Stefan Tauner]

Hello Jernej,

thanks for your report!

On Wed, 14 Jan 2015 00:02:22 +0100 (CET)
Jernej Škrabec  wrote:
> Now I manage to test again and as you can see from attachments, everything is OK for 64 KiB version.

Can you please test probing without the -c parameter? That's much more
important than the rest.
> However, I found out that chip definitions are duplicated:
>
> S25FL127S-64KB, line 10503
> S25FL127S-256kB, line 10536
> S25FL127S-64kB, line 10569
> S25FL127S-256kB, line 10613

Those must be local modifications on your side. On r1868:
$ grep -n S25FL127S flashchips.c
10505:		.name		= "S25FL127S-64kB", /* hybrid: 32 (top or bottom) 4 kB sub-sectors + 64 kB sectors */
10549:		.name		= "S25FL127S-256kB", /* uniform 256kB sectors */