[flashrom] [RFC] JEDEC refactor w/ conversion notes and file eliminations

Sean Nelson audiohacked at gmail.com
Mon Jan 4 18:16:38 CET 2010


On 1/4/2010 6:05 AM, Carl-Daniel Hailfinger wrote:
> On 04.01.2010 07:39, Sean Nelson wrote:
>    
>> The patch converts jedec functions into mask-based generics which can
>> be used for many chip provided the only changes are the the addresses
>> are converted from 0x5555/0x2AAA to 0x555/0x2AA or similar.
>>      
>    
>> These functions won't work for the 0xAAA chips listed above, but most
>> of  listed chips should work with these changes. The other changes I
>> suggested will come in later patches, and the writing functions are
>> untouched since we'll move to a block writers system.
>>      
> Maybe snip the two sentences above from the changelog. (There are no
> "0xAAA chips listed above".)
>
>
>    
>> The patch mostly changes jedec.c, but a few other files are changed
>> because they use the jedec functions within their own functions.
>>
>> The patch also adds a copyright line to flashchips.c because of my
>> recent work in converting AMD and Atmel chips to use struct erase_block.
>>
>> Signed-off-by: Sean Nelson<audiohacked at gmail.com>
>>      
> Very nice.
> Acked-by: Carl-Daniel Hailfinger<c-d.hailfinger.devel.2006 at gmx.net>
>
> Regards,
> Carl-Daniel
>
>    
Thanks, committed in r828.

~Sean Nelson




More information about the flashrom mailing list