[coreboot] [PATCH] v3: Replace magic numbers with symbolic constants

ron minnich rminnich at gmail.com
Thu Oct 9 17:06:27 CEST 2008


On Thu, Oct 9, 2008 at 2:25 AM, Carl-Daniel Hailfinger
<c-d.hailfinger.devel.2006 at gmx.net> wrote:

>>> -    if (!pci_conf1_find_device(0x1002,  0x4385, &dev)){
>>> +    if (!pci_conf1_find_device(PCI_VENDOR_ID_ATI,  0x4385, &dev)){
>>>
>>
>> Not sure about these at all. I think I prefer the number.
>>
>

if somebody someday put in 0x1003 for the vendor in that code, how
would you know it was wrong? Names are good :-)

ron




More information about the coreboot mailing list