[coreboot] Patch for ST M50FLW080A support (now with Signed-off-by - Line)

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Mon Apr 28 21:40:51 CEST 2008


On 28.04.2008 19:56, Stefan Reinauer wrote:
> Claus Gindhart wrote:
>> The generic jedec.c does not work for the ST M50FLW flash
>> devices, because they need an unlock command first.
>> For this reason, ST M50FLW support is moved to a new HW support
>> module, because any change in jedec.c
>> would bear the risk to cause problems with the already
>> supported devices.
>> It's already tested with ST M50FLW080A; the other
>> chips of this family i dont have available, so i couldnt
>> test it.
>>
>> Signed-off-by: Claus Gindhart <claus.gindhart at kontron.com>
>>   
> Due to your commit message and the many TODOs in jedec.c that file was
> ommitted. The rest is commited in r3274.
>
> Thank you very much for your contribution.

This was a bit too fast. The probe function looks like an exact copy of
probe_jedec and can be dropped. The flashrom restructuring I have in
mind benefits from a reduction of probe functions and allows
faster+generic probing.

Regards,
Carl-Daniel




More information about the coreboot mailing list