<br><br><div class="gmail_quote">On Fri, Nov 14, 2008 at 9:43 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;">
OK, to use this I still see a problem.<br>
<br>
In order to make the pnp devices not be dynamic, we have lost the<br>
ability to have default superio settings that don't have to be done in<br>
the mainboard.</blockquote><div><br>It could be done in superio.c where the mask is being set. <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;">

In other words, in the current system, we have this *in the superio*:<br>
<br>
-       /* Floppy */<br>
<div class="Wj3C7c">-       floppydev = "0x0";<br>
<br>
</div>now, for each and every *mainboard* that uses that superio, we have to<br>
set these defaults in the mainboard.<br>
<br>
That's not good.</blockquote><div><br>That was one of my questions.  Aren't these settings mainboard specific?  In that case we want to set them in each mainboard.  Otherwise it'll bite lots of people.<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;">
The problem is easily solved: create files such as<br>
superio/common/com1.dts, and use them instead of the pnp.dts which has<br>
non-optimal default settings. Or, just create<br>
superio/winbond/whatever/comport.dts, keyboard.dts, etc. and use them.</blockquote><div><br>If we're going that way, I'd rather just use what we have and change the PNP code to match. (Let it create dynamic devices but pass more information in the info structure so that the enables get set, etc.)  The hierarchy buys us nothing if we just make lots of little files and types.<br>
<br>BTW: Does anyone see PNP devices besides the SuperIOs?  I.E., will the PNP code ever get used for anything else? <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;">
I think this is closer but maybe still not right. This could be the<br>
template for all other dts setups if we get it right.<font color="#888888"><br>
</font></blockquote></div><br>Yes.  Let's get it right.<br><br>Thanks,<br>Myles<br>