> Here is a proposed change to dts. I've forgotten all I ever knew about
> the 8111 so I am pretty sure this is incomplete, but it's something
> like what we want.
> 
> 
> Index: mainboard/amd/serengeti/dts
> ===================================================================
> --- mainboard/amd/serengeti/dts       (revision 904)
> +++ mainboard/amd/serengeti/dts       (working copy)
> @@ -28,18 +28,17 @@
>               /config/("northbridge/amd/k8/domain");
>               [EMAIL PROTECTED],0{
>               };
> -             /* guesses; we need a real lspci */
>               [EMAIL PROTECTED],0 {
>                       /config/("northbridge/amd/k8/pci");
> -                     [EMAIL PROTECTED],0 {
> +                     [EMAIL PROTECTED],0 {
>
/config/("southbridge/amd/amd8111/amd8111.dts");
> +                             [EMAIL PROTECTED],0 {
> +
/config/("southbridge/amd/amd8111/nic.dts");
> +                             };
>                       };
>                       [EMAIL PROTECTED],0 {
>                               /config/("southbridge/amd/amd8111/ide.dts");
>                       };
> -                     [EMAIL PROTECTED],0 {
> -                             /config/("southbridge/amd/amd8111/nic.dts");
> -                     };
>               };
>               [EMAIL PROTECTED],0 {
>                       /config/("northbridge/amd/k8/pci");
> 
> It still doesn't seem quite right. Segher?

I've played with it a little more, no luck yet.  Could you help me
understand the naming convention?

[EMAIL PROTECTED],0 = pci at device 5 function 0 ?

I don't see how we say there's a pci bridge at device 0 function 0 on this
bus, then specify the devices on the pci bus from there.

Should we be using the amd8111/pci.dts here too?  Maybe that's why you had
the phase3_scan = 0, since the pci bus should take care of its scan?

Thanks,
Myles


--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to