[flashrom] [PATCH] Use extract_param everywhere

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Tue Jul 6 11:58:48 CEST 2010


On 06.07.2010 11:52, Carl-Daniel Hailfinger wrote:
> On 05.07.2010 22:32, Michael Karcher wrote:
>   
>> Please change the pcidev stuff to use common code in pcidev_init and
>> recheck the ordering in the buspirate file. With these changes:
>>
>> Acked-by: Michael Karcher <flashrom at mkarcher.dialup.fu-berlin.de>
>>     
>
> Done, thanks for the review.
>
> New version, addresses all comments except the it87spi stuff which will
> be handled in an updated it8705 support patch.
>
> Various places in the flashrom source feature custom parameter
> extraction from programmer_param. This led to wildly differing syntax
> for programmer parameters, and it also voids pretty much every
> assumption you could make about programmer_param. The latter is a
> problem for libflashrom.
>
>
> Use extract_param everywhere, clean up related code and make it more
> foolproof.
> Add two instances of exit(1) where we have no option to return an error.
> Remove six instances of exit(1) where returning an error was possible.
>
> WARNING: This changes programmer parameter syntax for a few programmers!
>
> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>   

I reused your ack and committed in r1070.

Regards,
Carl-Daniel

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





More information about the flashrom mailing list