Mostly cosmetic, partial fixes for STM3240G-EVAL LCD

git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@4705 42af7a65-404d-4744-a932-0658087f49c3

ChangeLog

@@ -2708,6 +2708,6 @@
 	  position, not an offset.
 	* graphics/nxtk/nxtk_drawframe.c:  Framed windows are now drawn in three
 	  colors (instead of just two).
-	* drivers/input/stmpe11_*:  Beginning of a driver to support STMicro STMPE11
+	* drivers/input/stmpe11_*:  Added a driver to support the STMicro STMPE11
 	  IO Expander and touchscreen driver.
 

configs/stm3240g-eval/README.txt

@@ -1081,6 +1081,21 @@ Where <subdir> is one of the following:
       CONFIG_STM32_CODESOURCERYW=y  : CodeSourcery under Windows
       CONFIG_LCD_LANDSCAPE=y        : 320x240 landscape
 
+  nxwm
+  ----
+    This is a special configuration setup for the NxWM window manager
+    UnitTest.  The NxWM window manager can be found here:
+
+      trunk/NxWidgets/nxwm
+
+    The NxWM unit test can be found at:
+
+      trunk/NxWidgets/UnitTests/nxwm
+
+    Documentation for installing the NxWM unit test can be found here:
+
+      trunk/NxWidgets/UnitTests/READEM.txt
+
   ostest:
   ------
     This configuration directory, performs a simple OS test using

configs/stm3240g-eval/src/up_lcd.c

@@ -101,7 +101,7 @@
 #    error "Cannot define both rportrait and any other orientations"
 #  endif
 #elif defined(CONFIG_LCD_LANDSCAPE)
-#  ifdef defined(CONFIG_LCD_RLANDSCAPE)
+#  ifdef CONFIG_LCD_RLANDSCAPE
 #    error "Cannot define both landscape and any other orientations"
 #  endif
 #elif !defined(CONFIG_LCD_RLANDSCAPE)
@@ -141,7 +141,7 @@
 /* STM3240G-EVAL LCD Hardware Definitions *********************************************/
 /* LCD /CS is CE4,  Bank 3 of NOR/SRAM Bank 1~4 */
 
-#define STM3240G_LCDBASE      ((uint32_t)(0x60000000 | 0x08000000))
+#define STM3240G_LCDBASE      ((uintptr_t)(0x60000000 | 0x08000000))
 #define LCD                   ((struct lcd_regs_s *)STM3240G_LCDBASE)
 
 #define LCD_REG_0             0x00
@@ -488,12 +488,15 @@ static inline void stm3240g_writegram(uint16_t rgbval)
  *   this LCD type.  When reading 16-bit gram data, there may be some shifts in the
  *   returned data:
  *
- *   - ILI932x: Unknown -- assuming no shift in the return data
+ *   - ILI932x: Discard first dummy read; no shift in the return data
  *
  **************************************************************************************/
  
 static void stm3240g_readnosetup(FAR uint16_t *accum)
 {
+  /* Read-ahead one pixel */
+
+  *accum  = LCD->value;
 }
 
 /**************************************************************************************
@@ -507,14 +510,12 @@ static void stm3240g_readnosetup(FAR uint16_t *accum)
  *
  **************************************************************************************/
 
-#if !defined(CONFIG_STM32_ILI9320_DISABLE) || !defined(CONFIG_STM32_ILI9325_DISABLE)
 static uint16_t stm3240g_readnoshift(FAR uint16_t *accum)
 {
   /* Read the value (GRAM register already selected) */
 
   return LCD->value;
 }
-#endif
 
 /**************************************************************************************
  * Name:  stm3240g_setcursor
@@ -593,11 +594,7 @@ static int stm3240g_putrun(fb_coord_t row, fb_coord_t col, FAR const uint8_t *bu
    * the STM3240G-EVAL is used as the top.
    */
 
-  /* Set the cursor position */
-
-  stm3240g_setcursor(col, row);
-
-  /* Then write the GRAM data, manually incrementing X */
+  /* Write the GRAM data, manually incrementing X */
 
   for (i = 0; i < npixels; i++)
     {

drivers/input/stmpe11_temp.c

@@ -132,12 +132,12 @@ uint16_t stmpe11_tempread(STMPE11_HANDLE handle)
 
   stmpe11_putreg8(priv, STMPE11_TEMP_CTRL, (TEMP_CTRL_ACQ|TEMP_CTRL_ENABLE));
   
-  /* Read the tempreature */
+  /* Read the temperature */
 
   temp1 = stmpe11_getreg8(priv, STMPE11_SYS_CTRL2);
   temp2 = stmpe11_getreg8(priv, STMPE11_SYS_CTRL2+1);
 
-  /* Scale the tempreature */
+  /* Scale the temperature (where Vio is assumed to be .33) */
 
   temp = ((uint32_t)(temp1 & 3) << 8) | temp2;
   temp = (uint32_t)((33 * temp * 100) / 751);