[coreboot] patch: two bugs in the cs5536 ide code

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Wed May 7 13:43:09 CEST 2008


On 07.05.2008 12:29, Stefan Reinauer wrote:
> ron minnich wrote:
>   
>> I know some people will get upset by the find_pci_device, but:
>> This patch fixes a long-standing problem in the cs5536 driver, that is
>> probably also in v2.
>>
>> The ide_init is called with the sb device but needs the IDE device,
>> which is different.
>>   
>>     
> Then that is the error that needs to be fixed. ide_init needs to be
> called with the IDE device.
>
>   
>> Signed-off-by: Ronald G. Minnich <rminnich at gmail.com>
>>   
>>     
>
> I veto. The above is against the concept of constructors we have in v3
> or drivers we have in v2. If we start searching for device X in code
> attached to device Y, we could as well just make a long linear piece of
> code searching for all devices and stop walking over the device tree.
>   

See my counter-proposal patch in this thread from a few hours ago. It
should address your worries.

Regards,
Carl-Daniel




More information about the coreboot mailing list