Rafaël Carré wrote:
On Sun, Oct 19, 2008, [EMAIL PROTECTED] wrote:
Date: 2008-10-19 16:11:01 +0200 (Sun, 19 Oct 2008)
New Revision: 18836

Log Message:
Add e200v2 and m200v2 targets.  Move the telechips lcd-ssd1815.c (currently 
used by Logik DAX and m200v1 ports) driver up in the target tree and share with 
the m200v2 - as2525 parts contributed by Rafael Carre.  Includes the start (but 
is still very incomplete) of an LCD driver for the e200v2.  m200v2 is not yet 
fully supported by mkamsboot - that will come soon.  Also some minor cleanups 
for the Clip.

Modified: trunk/firmware/export/config-clip.h
===================================================================
...
+#define MI4_FORMAT
+#define BOOTFILE_EXT    "sansa"
+#define BOOTFILE        "rockbox." BOOTFILE_EXT
+#define BOOTDIR "/.rockbox"

Isn't that needed only for PP targets ?

Thanks. MI4_FORMAT is wrong (that's only for the non-ipod PP targets), but the BOOTFILE definitions will be used - both in the bootloader and main Rockbox.

You can group the directions (GPIOA_DIR |= ((1<<7)|(1<<5)|(1<<4)|(1<<3)) ) it's
clearer to read

Sure, but as this is still a work-in-progress, I wanted to keep the code as close to the OF as possible, including all silliness - to make finishing (and double-checking) the work easier. It can be cleaned/optimised later, once it works.

+  GPIOD_DIR |= (1<<7);
+
+#if 0
+  /* TODO: This code is conditional on a variable in the OF init, we need to
+           work out what it means */
+
+  GPIOD_PIN(7) = (1<<7);
+  GPIOD_DIR |= (1<<7);
+#endif

This one is the button led, I think you can safely ignore it.

OK, thanks.

Copied: trunk/firmware/target/arm/lcd-ssd1815.c (from rev 18822, 
trunk/firmware/target/arm/tcc77x/lcd-ssd1815.c)
===================================================================
..
+#if CONFIG_CPU == AS3525
+#include "as3525.h"
+
+void lcd_write_command(int byte)
+{
+    DBOP_TIMPOL_23 = 0x6006E;

Isn't it better to match the model rather than the SoC ? (#ifdef SANSA_M200V2)

Maybe, but the telechips part of that file is common across both tcc77x devices that use the ssd1815 (m200v1 and Logik DAX), so the CPU seemed the appropriate #if there.

And since we only have one as3525 device using it, so I decided to use the CPU there as well.

But it's easily changed if a second as3525 device appears that uses the ssd1815, or if we want to merge the Clip's ssd1303 driver.


Dave.

Reply via email to