[flashrom] [PATCH] Intel Poulsbo chipset support

Adam Jurkowski adam.jurkowski at kontron.pl
Wed Dec 2 18:14:14 CET 2009


Hi Carl-Daniel,

On 2009-12-02 16:51, Carl-Daniel Hailfinger wrote:
> Is there any reason you can't call enable_flash_ich(dev, name, 0xd8)?
> That would eliminate some of the code.
>   
It's a matter of taste actually.

Using enable_flash_ich would eliminate only printf_debug lines and half 
of one PCI register value modification.

If I do all inside the function then I use word-access once instead of 
using byte-access twice.
I'm also able to print out the state of prefetch_flag right after 
lock_enable and write_enable but before the whole 16-bit register value.

Best regards,
Adam




More information about the flashrom mailing list