+ ioport@2e { + /config/("superio/winbond/w83627hf/dts"); + pnp@2 { + /config/("superio/winbond/w83627hf/pnp.dts"); + enabled; + io = "0x3f8"; + irq = "4"; + }; + pnp@5 { + /config/("superio/winbond/w83627hf/pnp.dts"); + enabled; + io = "0x60"; + io2 = "0x62"; + irq = "1"; + irq2 = "12"; + }; + pnp@a { + /config/("superio/winbond/w83627hf/pnp.dts"); + enabled; + io = "0x290"; + irq = "5"; + }; + }; };
A few comments here. First, per the discussion in the other thread, it's not clear that people want to see all this dts stuff in the mainboard for each and every individual device. Second, every device has an io and io2 now --even those that don't support it? Third, it's not clear that having a "generic" pnp is going to capture all the possible combinations; we tried this on v2 and it did not work out that well. Fourth, I find this less readable than the old pnp:
- /* Floppy */ - floppydev = "0x0"; - floppyenable = "0"; - floppyio = "0x3f0"; - floppyirq = "0x60"; - floppydrq = "0x02"; - - /* Parallel port */ - ppdev = "2"; - ppenable = "0"; - ppio = "0x378"; - ppirq = "7"; - - /* COM1 */ - com1dev = "2"; - com1enable = "0"; - com1io = "0x3f8"; - com1irq = "4"; - - /* COM2 */ - com2dev = "3"; - com2enable = "0"; - com2io = "0x2f8"; - com2irq = "3"; - - /* Keyboard */ - kbdev = "5"; - kbenable = "0"; - kbio = "0x60"; - kbio2 = "0x62"; - kbirq = "1"; - kbirq2 = "12"; - - /* Consumer IR */ - cirdev = "6"; - cirenable = "0"; - - /* Game port */ - gamedev = "7"; - gameenable = "0"; - gameio = "0x220"; - gameio2 = "0x400"; - gameirq = "9"; - - /* GPIO2 */ - gpio2dev = "8"; - gpio2enable = "0"; - - /* GPIO3 */ - gpio3dev = "9"; - gpio3enable = "0"; - - /* ACPI */ - acpidev = "0xa"; - acpienable = "0"; - - /* Hardware Monitor */ - hwmdev = "0xb"; - hwmenable = "0"; - hwmio = "0x290"; - hwmirq = "5";
The naming is a lot more meaningful for me than the generic one-size-fits-all pnp stuff.
Also, note from the other thread on the 82801gx south, it almost seems people would prefer this one-dts-has-it-all form over the other form.
Lotsa confusion out here with the dts right now.
ron