On Thursday 10 December 2009, Michael S. Tsirkin wrote:
> The recent e1000 bug made the important of using
> symbolic macros for pci config access clear for me.
> So I started going over drivers and converting
> to symbolic constants instead of hard-coded ones.
> I did a large part until I run out of steam.
> Maybe some brave soul will take up converting
> the rest of them, or maybe I will: note that
> when converting bridges one should be careful
> to use bridge macros where appropriate.

Seeing as you're introducing a huge amount of churn, wouldn't it be better to 
come up with a sane abstraction for initializing the the PCI config data (c.f. 
pci_config_set_vendor_id)?

Paul


Reply via email to