<br><br><div class="gmail_quote">On Mon, Jul 20, 2009 at 8:14 AM, Rudolf Marek <span dir="ltr"><<a href="mailto:r.marek@assembler.cz">r.marek@assembler.cz</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;">
Hi Myles,<br>
<div class="im"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
That's the way the rest of the boards did it.  I'm sorry I missed this<br>
southbridge.  I thought I got them all.<br>
</blockquote>
<br></div>
Ok. Please prepare the patch.</blockquote><div>Attached.<br><br>Signed-off-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> <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="im"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I noticed in your patch that you're setting the limit = base+size-1.  The<br>
limit is not the same as the end of the resource.  For most I/O, the limit<br>
should be 0xffff.<br>
</blockquote>
<br></div>
aha true. This was cut and paste from k8t890 where it was used to setup APIC resource. For mem resource it should be how? I don't understand the gran too.</blockquote><div>The limit should always be the upper bound that a type of resource could be set to.  For I/O 0xffff, for MEM, 0xffffffff...<br>
<br>gran sets the granularity of an allocation.  For example, on K8 you have to allocate I/O to links at least 4K at a time, so the granularity is 12.  gran and align should never matter for fixed resources. <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;">
For this SB (the VT8237S) we will definitely need to dynamically allocate a mmio BAR for SPI controller. I will prepare a patch, but cannot test.<br>
<br>
I would like to ask if you care about the decoding range for flashes - if I need some other resource for that and also about 0xfee... for MSI.<font color="#888888"></font></blockquote><div><br>Anything above 0xfec... doesn't matter, since you have an APIC resource there.<br>
<br>Thanks,<br>Myles <br></div></div><br>