<br><br><div class="gmail_quote">On Tue, Nov 11, 2008 at 3:08 PM, Myles Watson <span dir="ltr"><<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">> > > resources.<br>
> > > For example:<br>
> > > >     PNP 2e.0: size 8 align 3 gran 3 limit 7ff flags 100 index 60<br>
> > > which is the floppy device has this resource definition<br>
> > >     { &w83627hf_ops, W83627HF_FDC,  PNP_IO0 | PNP_IRQ0 | PNP_DRQ0, {<br>
> > > 0x07f8, 0}, },<br>
> > > and this dts entry<br>
> > >     /* Floppy */<br>
> > >     floppydev = "0x0";<br>
> > >     floppyenable = "0";<br>
> > >     floppyio = "0x3f0";<br>
> > >     floppyirq = "0x60";<br>
> > >     floppydrq = "0x02";<br>
> ><br>
> > So you're saying it should be a fixed resource size 8 base 0x3f0?<br>
><br>
> Yes, if it were enabled. I don't know what the device code does if it is<br>
> disabled.<br>
<br>
</div>Good point.  I looked, and this doesn't ever get checked.  All these devices<br>
show up as dynamic devices and lose this information.  I guess some phase3<br>
scan function needs to be implemented that takes this into account.<br>
<br>
Thanks,<br>
<font color="#888888">Myles</font></blockquote><div> </div></div> static void phase3_chip_setup_dev(struct device *dev)<br> {<br>-       pnp_enable_devices(dev, &w83627hf_ops, ARRAY_SIZE(pnp_dev_info), pnp_dev_info);<br>
+       struct superio_winbond_w83627hf_dts_config *conf;<br>+       conf = dev->device_configuration;<br>+<br>+       if (conf->com1enable)<br>+               pnp_enable_devices(dev, &w83627hf_ops, 1, &pnp_dev_info[2]);<br>
+       if (conf->com2enable)<br>+               pnp_enable_devices(dev, &w83627hf_ops, 1, &pnp_dev_info[3]);<br>+       if (conf->kbenable)<br>+               pnp_enable_devices(dev, &w83627hf_ops, 1, &pnp_dev_info[4]);<br>
+       if (conf->hwmenable)<br>+               pnp_enable_devices(dev, &w83627hf_ops, 1, &pnp_dev_info[9]);<br> }<br><br><br>Are we thinking something like this?  Something different?  I don't understand how we were hoping those dts values would make it into devices.<br>
<br>Thanks,<br>Myles<br>