[flashrom] [PATCH] Annotate evil twins

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Thu Jul 22 16:05:37 CEST 2010


On 22.07.2010 15:43, Daniel Lenski wrote:
> On Thu, 2010-07-22 at 14:54 +0200, Carl-Daniel Hailfinger wrote:
>   
>> Evil twins are chips with identical IDs and different/incompatible
>> characteristics (e.g. eraseblock layout, write strategy or size).
>> Annotate them so flashrom can tell the user that it's not a matter of
>> simply picking one of the chips in a list of multiple chips found.
>>
>> This patch only touches the AMIC A25L40PT and A25L40PU.
>> I'm fairly sure there are more evil twins, but those two were touched
>> recently.
>>
>> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>>     
>
> I should point out that the A25L40PT and A25L40PU got erase-tested by
> Rudolf Marek while we were trying to figure out if they were really evil
> twins.  My patch added the erase-tested bit, but I think you took this
> out.
>   

Yes, sorry for not mentioning this in the commit mail. Right now we only
check if at least one erase function worked. There is no way (except
editing the source code) to test all erase functions for a given chip.
This will change once
[PATCH] Move implicit erase out of chip drivers, clean up
is committed and we have partial write.

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the flashrom mailing list