<div dir="ltr">On Tue, Oct 14, 2008 at 6:46 AM, Carl-Daniel Hailfinger <span dir="ltr"><<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.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;"><div class="Ih2E3d">On 14.10.2008 05:33, Corey Osgood wrote:<br>
> See attached. Stage2 has to wait until CAR can be disabled on C7.<br>
><br>
</div>> Add stage1 support for vt8237[RS] to v3.<br>
><br>
> Signed-off-by: Corey Osgood <<a href="mailto:corey.osgood@gmail.com">corey.osgood@gmail.com</a>><br>
><br>
<br>
Looks good. I didn't understand one part of the code and would<br>
appreciate clarifications, though.<br>
Acked-by: Carl-Daniel Hailfinger <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>></blockquote><div><br>With the fixes you mentioned, committed in r928<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
> +void enable_smbus(u16 smbus_io_base)<br>
> +{<br>
> +     u32 dev;<br>
> +<br>
> +     /* Power management controller */<br>
> +     pci_conf1_find_device(PCI_VENDOR_ID_VIA, PCI_DEVICE_ID_VIA_VT8237R_LPC,<br>
> +                             &dev);<br>
><br>
<br>
OK, here you look for PCI_DEVICE_ID_VIA_VT8237R_LPC<br>
<br>
> +<br>
> +     if (pci_conf1_read_config16(dev, PCI_DEVICE_ID) !=<br>
> +                             PCI_DEVICE_ID_VIA_VT8237R_LPC) {<br>
><br>
<br>
And here you check what? If pci_conf1_find_device is not broken and<br>
returns a nonzero value, the expression above should be always false. I<br>
believe this is a romcc relic.</blockquote></div><br>A relic of my own laziness, actually. I meant to look up pci_conf1_fird_device() to see what it returned, threw that in as a stopgap, and forgot about it.<br><br>Thanks,<br>
Corey<br></div>