<br><br><div class="gmail_quote">On Thu, Dec 18, 2008 at 3:21 PM, Marc Jones <span dir="ltr"><<a href="mailto:marcj303@gmail.com">marcj303@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><div></div><div class="Wj3C7c">On Thu, Dec 18, 2008 at 12:58 PM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
><br>
><br>
> On Thu, Dec 18, 2008 at 12:21 PM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
>><br>
>><br>
>> On Thu, Dec 18, 2008 at 11:51 AM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
>>><br>
>>><br>
>>> On Thu, Dec 18, 2008 at 11:41 AM, Marc Jones <<a href="mailto:marcj303@gmail.com">marcj303@gmail.com</a>> wrote:<br>
>>>><br>
>>>> On Wed, Dec 17, 2008 at 1:42 PM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
>>>> > serengeti.diff and dtc-links.diff make serengeti find all of its<br>
>>>> > devices<br>
>>>> > correctly.<br>
>>>> ><br>
>>>> > serengeti.diff<br>
>>>> > - takes out the multiple links for the Opteron (they weren't working<br>
>>>> > correctly)<br>
>>>> > - adds the 8132 devices and makes them pci_a so they don't conflict<br>
>>>> > with the<br>
>>>> > 8111<br>
>>>> > - corrects the disabled flags (I think we should have a build error<br>
>>>> > somewhere when there are incorrect flags)<br>
>>>> > - moves the lpc device to the lpc bus<br>
>>>> > - adds back in the second Opteron<br>
>>>> ><br>
>>>> > Signed-off-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>><br>
>>>> ><br>
>>>><br>
>>>> Acked-by: Marc Jones <<a href="mailto:marcj303@gmail.com">marcj303@gmail.com</a>><br>
>>>> But what about amd8151?<br>
><br>
> I was thinking the 8151 was on a different link of the same Opteron.  It's<br>
> not.  Since it's on a different Opteron, it's easy.  Sorry for the<br>
> confusion.<br>
><br>
> This patch adds the 8151 and makes it compile.  Tested on the free version,<br>
> so the 8151 just gets disabled.<br>
><br>
> Signed-off-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>><br>
><br>
> BTW: it still needs the dtc patch.  If you don't have it, this adds yet<br>
> another case where two links are created instead of one.<br>
<br>
</div></div>It would be unusual for two IO bridges  to be on the same CPU. I can't<br>
think of every seeing that.<br>
I have no way to test this now but it looks good so, let's get your<br>
work in. This ack is for the dts </blockquote><div><br>Rev 1080. <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;">and the dtc changes.<br>

<div><div class="Wj3C7c"></div></div></blockquote><div><br>Rev 1079. <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;"><div><div class="Wj3C7c">
<br>
Acked-by: Marc Jones <<a href="mailto:marcj303@gmail.com">marcj303@gmail.com</a>><br>
</div></div></blockquote></div><br>Thanks,<br>Myles<br>