[flashrom] [PATCH] Move serial handling from serprog.c to serial.c
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Mon Nov 23 20:21:53 CET 2009
On 23.11.2009 16:33, Uwe Hermann wrote:
> On Sat, Nov 21, 2009 at 01:29:11PM +0100, Carl-Daniel Hailfinger wrote:
>
>> Move serial handling from serprog.c to serial.c.
>> This is the first step in enabling platform independent serprog and it
>> also allows other drivers to use serial port functionality without
>> requiring serprog.
>>
>> Pure code move, no code changed.
>>
>> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>>
>
> Acked-by: Uwe Hermann <uwe at hermann-uwe.de>
>
> Build-tested, worked ok.
>
Thanks, r771.
>> +void __attribute__((noreturn)) sp_die(char *msg)
>> +{
>> + perror(msg);
>> + exit(1);
>>
>
> EXIT_FAILURE maybe, though I'm not sure we have consistent error codes
> everywhere, yet.
>
We need a generic die() and a variant die_perror(). Both of them should
call programmer_shutdown() or use proper atexit stuff.
Regards,
Carl-Daniel
--
Developer quote of the month:
"We are juggling too many chainsaws and flaming arrows and tigers."
More information about the flashrom
mailing list