<br><br><div class="gmail_quote">On Tue, Oct 28, 2008 at 2:15 PM, Marc Jones <span dir="ltr"><<a href="mailto:Marc.Jones@amd.com">Marc.Jones@amd.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;">
Myles Watson 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 class="Ih2E3d">
This patch documents the unreadable function in northbridge/amd/k8/pci.c and<br>
cleans up the NULL pointer protection.<br>
<br></div>
Signed-off-by: Myles Watson <<a href="mailto:mylesgw@gmail.com" target="_blank">mylesgw@gmail.com</a> <mailto:<a href="mailto:mylesgw@gmail.com" target="_blank">mylesgw@gmail.com</a>>><br>
</blockquote>
<br>
Untested but this looks correct to me. See if Ron has some comment.<br>
<br>
Acked-by: Marc Jones <<a href="mailto:marc.jones@amd.com" target="_blank">marc.jones@amd.com</a>><div class="Ih2E3d"></div></blockquote><div><br>Thanks.<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 class="Ih2E3d"><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
It also begs the question why cpu_bus_scan and cpu_bus_init aren't needed in v3.  They aren't called.  Is that an oversight, or should they be removed.<br>
<br>
</blockquote>
<br></div>
I think it is because the HT init (cpu setup) is all done pre-memory and done by the time dts is used. This could change if the dts is present in stage1.</blockquote><div><br>Do you know what the corresponding functions are in v3?<br>
<br>Thanks,<br>Myles <br></div></div><br>