<div dir="ltr">On Fri, Oct 17, 2008 at 5:12 PM, Alex Mauer <span dir="ltr"><<a href="mailto:hawke@hawkesnest.net">hawke@hawkesnest.net</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Corey Osgood wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
2008/10/17 Uwe Hermann <<a href="mailto:uwe@hermann-uwe.de" target="_blank">uwe@hermann-uwe.de</a> <mailto:<a href="mailto:uwe@hermann-uwe.de" target="_blank">uwe@hermann-uwe.de</a>>><div><div></div><div class="Wj3C7c">
<br>
<br>
    See patch.<br>
<br>
<br>
    Drop unneeded enable_mainboard_devices() function, the lines<br>
<br>
          register "fn_ctrl_lo" = "0x80"<br>
          register "fn_ctrl_hi" = "0x1d"<br>
<br>
    in the Config.lb file should do exactly the same (but please test that)!<br>
<br>
    Also, drop some unused variables and (IMHO) unneeded debug messages.<br>
<br>
<br>
NACK. If the devices aren't enabled/disabled (and in the process hidden/revealed) before PCI bringup, coreboot will get very confused when they disappear and appear out of nowhere.<br>
</div></div></blockquote>
<br>
would it be feasible to have enable_mainboard_devices() read the structure setup by Config.lb so that those values can come from just one place?<br>
</blockquote></div><br>No, unfortunately Config.lb is only available in post-raminit code. This is one of the things that I hope to solve in V3. fn_ctrl_lo in v3 can be replaced with actual function names in the dts (usb_ehci, , and with Carl-Daniel's dts->flatfile work (whenever it's ready ;)), it can be accessed from stage1/initram. This is just one of the reasons I've been concentrating on getting the cn700 running on v3 more then worrying about CAR on v2.<br>
<br>-Corey<br></div>