[coreboot] patch: resource map proposal.

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Mon Aug 4 20:11:39 CEST 2008


On 04.08.2008 19:56, Peter Stuge wrote:
> On Mon, Aug 04, 2008 at 07:50:32PM +0200, Carl-Daniel Hailfinger wrote:
>   
>>> a patch to apply on top of Ron's commit.
>>>       
>> If I'm successful, I'll be undoing 50% of Ron's patch.
>>     
>
> That doesn't matter. This is early work in progress.
> Key word: progress
>   

If something has to be rolled back, it's the opposite of progress.


>> -	pci_write_config32(a, b, c);
>> +	pci_cf8_conf1.write32(NULL, 0, a, b, c);
>>
>> Can't we just use a macro for that stuff?
>> #define PCI_WRITE_CONFIG32_EARLY(a,b,c) pci_cf8_conf1.write32(NULL, 0, a, b, c);
>>     
>
> Sorry, I don't see the point. I expect pci_cf8_* to change at least
> once anyway. I think time is better spent elsewhere.
>   

That's why I don't want it committed in the first place. The least
intrusive change would be the following #define at the beginning of each
affected C file:

> #define pci_write_config32(a,b,c) pci_cf8_conf1.write32(NULL, 0, a, b, c);
>   


No other code changes needed. It reduces code churn and keeps the
familiar interface. With that change, I see no reason to hold the commit
back.

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the coreboot mailing list