<br><br><div class="gmail_quote">On Thu, Nov 13, 2008 at 12:14 PM, 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;">
{<br>
        device_operations = "w83627thg_ops";<br>
        /* To override any of these, put the over-ride in mainboard dts. */<br>
<br>
        /* COM1 */<br>
        pnp@2{<br>
<div class="Ih2E3d">                 com1dev = "2";<br>
                 com1enable = "0";<br>
                 com1io = "0x3f8";<br>
                 com1irq = "4";<br>
</div>        };<br>
<br>
<br>
<br>
Questions I have no answer to:<br>
Before I put the pnp2 in , we got this:<br>
<br>
<br>
struct superio_winbond_w83627thg_dts_config domain_0_ioport_2e = {<br>
        .com1dev = 0x2,<br>
        .com1enable = 0x0,<br>
        .com1io = 0x3f8,<br>
<br>
etc.<br>
<br>
What should we get now?</blockquote><div>struct superio_common_pnp domain_0_ioport_2e_pnp_2 = {<br>
        .enable = 0x0,<br>
        .io = 3f8,<br>
<br>struct superio_common_pnp domain_0_ioport_2e_pnp_5 = {<br>
        .enable = 0x0,<br>
        .io = 3f8,<br>
etc.<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Should pnp2 be a device? </blockquote><div>That's the way it is now.  I'm just making them static instead of dynamic. <br>
</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Child or sibling of w83627thg? </blockquote><div>Child?  That's how I did it. <br>
</div><div> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">What<br>
device_operations should it have?</blockquote><div>Enable and set.<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> One possibiltiy: if no<br>

device_operations property in the pnp@2 node,<br>
inherit from parent.</blockquote><div><br>That's how I did it the first time, but then I saw that you were already passing it into the PNP code in the info structure, so I took it back out.<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
What do you want to see?<font color="#888888"><br>
</font></blockquote></div><br>I'm not picky if it works.  I could go back and implement it with it all inside the device.  I would just have to change the PNP code a _lot_ more than I did.  There would have to be more passes so that you could get the devices set up, then go back and initialize them with resources.  I was trying to minimize changes.<br>
<br>Thanks,<br>Myles<br>