[coreboot] patch: stage 2 ide device for mcp55

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Tue Aug 12 03:36:41 CEST 2008


On 11.08.2008 18:03, ron minnich wrote:
> On Mon, Aug 11, 2008 at 3:20 AM, Carl-Daniel Hailfinger
>   
>> Maybe move the hunk above into an extra function so we can use the
>> generic subsystem ID infrastructure?
>>     
>
> It's gone. There is not one on v3 and I see no need to add it. The
>   

We have a generic subsystem ID infrastructure in v3. Look at
pci_dev_enable_resources().

> propagation of subsystem stuff can be done by the dtc, AFAICS. We do
> this:
> 1. add subsystem info to dts and device struct
>   

We only have per-mainboard subsystem info in the dts. We need to add
per-device subsystem info to the dts.

> 2. allow users to specify subsystem to any node in the dts. All nodes
> under that
>    node will inherit that id unless they similarly set it.
>   

Not yet done.

> 3. at the root, we set subsystem to (I assume) mainboard VID and DID
>   

We set a special variable mainboard_subsystem_vendor and ..._id as part
of the mainboard struct.

> 4. at flatten tree time, we walk the tree and propagate subsystem VID
> and DID to
> children. This ensures that all devices have it at compile time
>   

Not yet.

> 5. In a final pass at runtime (probably phase6) we walk the tree and,
> for devices that set 'support subsystem VID/DID', we set it into the
> hardware.
>   

We do that in pci_dev_enable_resources().

> Done. Run time mess becomes compile time mess. Unless I'm missing something.
>
> If I am, let me know.
>   

See above.

A few seconds ago, I posted a patch to move the mcp55 subsystem ID code
to the existing v3 infrastructure.

Regards,
Carl-Daniel

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





More information about the coreboot mailing list