Re: [coreboot] Patch for ST M50FLW080A support

[Paul Menzel]

Dear Claus,

thank you for your great work.

Am Freitag, den 25.04.2008, 14:52 +0200 schrieb Claus Gindhart:
> 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 couldn
> test it.

Although I cannot test or verify your patch, I think you have to add a
Signed-Off-By line [1]. Otherwise your patch cannot be acked and then
commited.

Thanks,

Paul

[1] http://www.coreboot.org/Development_Guidelines#Sign-off_Procedure