[flashrom] [PATCH] print.c fix - remove empty pcidevs header

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Mon Dec 14 01:08:43 CET 2009


Hi Adam,

On 24.11.2009 16:52, Carl-Daniel Hailfinger wrote:
> On 24.11.2009 16:46, Adam Jurkowski wrote:
>   
>> Don't print out supported PCI devices header if all following lines
>> are excluded from build.
>>     
>
> Your patch unfortunately conflicts with
> http://patchwork.coreboot.org/patch/576/ but I'll merge it once that
> other patch is committed.
>   

The conflicting patch has now been committed.
Would you mind regenerating your patch and resending it with the
original changelog? I have roughly 50 patches I need to manage and this
would reduce my load. Feel free to include my Acked-by statement in your
changelog to get a speedier commit.

One minor comment, though. Could you use parentheses like this?
#if (NIC3COM_SUPPORT == 1) || (GFXNVIDIA_SUPPORT == 1) ||
(DRKAISER_SUPPORT == 1) || (SATASII_SUPPORT == 1)

IMHO that makes the code more readable.


> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>   

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