[flashrom] [PATCH] Fix AAI code

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Thu Jul 29 18:32:51 CEST 2010


On 29.07.2010 18:18, Uwe Hermann wrote:
> On Thu, Jun 24, 2010 at 02:41:38AM +0200, Carl-Daniel Hailfinger wrote:
>   
>> The AAI code rewrite in r1052 introduced a bug: The writelen of AAI
>> continuation is 3 bytes, but the code incorrectly had 6 bytes there.
>> This causes all AAI writes (except the first two bytes of a chip) to fail.
>> Thanks to den_m for reporting the bug and for testing the fix.
>>
>> Functional change:
>> Fix AAI continuation command write length (was 6, should be 3).
>>
>> Cosmetic change:
>> Change the whitespace of a few #defines in spi.h to have aligned values
>> to improve readability (maybe someone would have spotted the AAI bug if
>> the values had been aligned in the first place).
>>
>> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>>     
>
> Acked-by: Uwe Hermann <uwe at hermann-uwe.de>
>   

Thanks, committed in r1121.

Regards,
Carl-Daniel

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





More information about the flashrom mailing list