Haven't yet heard back about SVN commit permissions so here is another patch.<br><br><div class="gmail_quote"><div class="im">On Sun, Jun 21, 2009 at 5:30 PM, Werner Almesberger <span dir="ltr"><<a href="mailto:werner@openmoko.org" target="_blank">werner@openmoko.org</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><br></div>
Bug (just one):<br>
<br>
- the RPxxxx have no value</blockquote></div><div><br>Fixed. <br></div><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;"><br>
<br>
Compatibility:<br>
<br>
- Openmoko have RP2201 upside-down, which changes the pin numbers.<br>
I think this deserves an ECN, small as it may be ...</blockquote></div><div><br>I just flipped the symbol to match the OM schematic so that an ECN isn't needed.<br> <br></div><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;">
<br>
<br>
Style (all of the issues are actually general):<br>
<br>
- for the bead, its characteristics should be more interesting than<br>
the part number (similar to what I said about varistors).</blockquote></div><div><br>Fixed. I listed current rating and impedance for the bead.<br> <br></div><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;">
<br>
<br>
- most of the caps should have placement constraints. We could either<br>
indicate them visually by placing them next to the respective pins,<br>
or add a comment that says where they should go. That will help<br>
later, when doing the layout.</blockquote></div><div><br>Fixed. I placed the caps next to the pins because it made it more clear for me. <br></div><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;">
<br>
<br>
- also, please don't forget to X the unconnected pins</blockquote></div><div><br>Done.<br><br><br>Let me know if you see anything else!<br><br>Luke.<br></div></div>