<br><br><div class="gmail_quote">On Wed, Nov 12, 2008 at 10:37 AM, ron minnich <span dir="ltr"><<a href="mailto:rminnich@gmail.com">rminnich@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">On Wed, Nov 12, 2008 at 8:25 AM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
<br>
> I forgot the rule.  I'm down the path of creating static devices for each<br>
> PNP device. :(<br>
<br>
</div>if you make a dts per PNP device, the dtc will do that for you.<br>
<br>
don't go too far without a sanity test with me or you will go insane :-)<br>
<div class="Ih2E3d"><br>
<br>
> Which has absolutely no knowledge of the dts.  We could, however, create an<br>
> initialization function that copies all of the information from the dts into<br>
> the specific device.  It just has to be done before resource allocation,<br>
> which means that pnp_enable_devices needs to be split and renamed.<br>
<br>
</div>as marc pointed out, the phase3_chip_setup_dev does have dts knowledge.<br>
<br>
I'm seeing the problem more now. As you have  pointed out, the pnp<br>
stuff has no good connection to anything else, and this needs to be<br>
fixed.<br>
<div class="Ih2E3d"><br>
<br>
> I have a generic pnp dts which has members io,irq,io2,irq2,drq<br>
<br>
</div>can I see it?</blockquote><div><br>Shield your eyes from the ugliness :)<br><br>This patch creates a superio/winbond/*/pnp.dts which is generic enough that it probably should be extended if needed and moved to device/pnp.dts or superio/pnp.dts or something like that.<br>
<br>I wish that I could put pnp@W83267HF_KBC and have the define come out in the statictree.c, but it doesn't like that, so I have to duplicate the information.<br><br>Right now it does pnp@5 as a static child of ioport@2e.  I have to fix that up in the code by assigning the pnp device's port from its parents.  That could move.<br>
<br>In the pnp code, I changed it so that it doesn't allocate a pnp device if it doesn't find it, because that would mean that all the values were wrong for the ports, irqs, etc.<br><br>Signed-off-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>><br>
<br>I signed it off in case I'm hit by a truck, for Uwe, but it shouldn't be committed.<br><br>Thanks,<br>Myles <br></div></div><br>