[flashrom] [PATCH] do something useful with the dediprog LEDs
reinauer at google.com
Thu Jan 27 22:02:48 CET 2011
On Mon, Jan 24, 2011 at 2:14 AM, Mathias Krause
<mathias.krause at secunet.com> wrote:
> On 21.01.2011 20:09, Stefan Reinauer wrote:
>> This patch makes use of the dediprog LEDs.
>> It would be nice to have some "more global" infrastructure for device
>> dependent status reporting. Right now some scenarios can not be
>> reported as an error, though the user would think of them as one. For
>> example, if the chip is IDed successfully but not supported by
>> flashrom, thus it can not be flashed. At best, the dediprog driver
>> sees a successful RDID command, but has no knowledge about the failure
>> to read/write the chip. Therefore it can not report it.
>> However, the attached patch improves my work flow here quite a bit, so
>> I thought it should be shared.
> Thanks for the patch. Albeit it would be good to have a "noleds" option
> to disable the functionality because we don't know if it breaks older
> firmware versions.
All changes added, except this one. Instead the updated version
explicitly support older firmware versions.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 7170 bytes
Desc: not available
More information about the flashrom