<br><br><div class="gmail_quote">On Tue, Jul 14, 2009 at 5:50 AM, Werner Almesberger <span dir="ltr"><<a href="mailto:werner@openmoko.org">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;">
I finally got to giving the LCM a more thorough review. Sorry for<br>
the delay.<br>
<br>
I made the following small changes:<br>
<br>
- Shifted parallel traces to better show their grouping.<br>
- Swapped values of C6001 and C6007 to match GTA02 schematics.<br>
- added docs/ecn/ecn0001/lcm-wiring.txt with a map between LCM wiring for<br>
GTA02 and gta02-core.</blockquote><div><br>Thanks! <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
<br>
Unresolved issues:<br>
<br>
- LED+ and LED- are unconnected</blockquote><div> </div><div>Fixed. I also had to add these power symbols and updated the PMU schematics to use these as well. <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
- LEND seems to be unused (and is easily confused with LDEN) ?</blockquote><div><br>That's odd. It was used as the GPS reset signal in GTA01 and NC in GTA02. However, the 2442 datasheet says that it is need for TFT screens. Maybe the LCD driver chipset already takes care of this? <br>
</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
- the names LCD_xCS and LCD_xRES are a bit confusing. The LCD_xxx nets<br>
don't always follow LCD or CPU naming anyway, so we can change them.<br>
How about LCD_nCS or nLCD_CS ?</blockquote><div><br>Agreed. I changed them to LCD_nCS because it looked better to me. <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<br>
Thanks,<br>
<font color="#888888">- Werner<br>
</font></blockquote></div><br>