]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
ARM: OMAP: lcd_lph8923: fix register read / write
authorImre Deak <imre.deak@nokia.com>
Mon, 20 Feb 2006 16:51:18 +0000 (18:51 +0200)
committerJuha Yrjola <juha.yrjola@nokia.com>
Mon, 20 Feb 2006 16:51:18 +0000 (18:51 +0200)
A local variable hiding the one with the same name at an upper scope,
resulted in storing the register address instead of the desired value in
LCD registers.

In register writes there is no extra cycle after the command, so we use
only 9-bit word size.

Fix from Peter Ujfalusi <peter.ujfalusi@nokia.com>.

Signed-off-by: Imre Deak <imre.deak@nokia.com>
Signed-off-by: Juha Yrjölä <juha.yrjola@nokia.com>
drivers/video/omap/lcd_lph8923.c

index 64e1c54b9b3d1fc5a1e15f668a97c9230a57ff0e..69f5f1843354299bb728937824a7561bf4831651 100644 (file)
@@ -163,13 +163,7 @@ static void lph8923_write(int cmd, const u8 *buf, int len)
 
        spi_message_init(&m);
        m.spi = lph8923.spi;
-
-       if (len > 1) {
-               cmd <<= 1;
-               set_spi_data_width(10);
-       } else {
-               set_spi_data_width(9);
-       }
+       set_spi_data_width(9);
 
        t.cs_change     = 0;
        w               = cmd;
@@ -183,12 +177,8 @@ static void lph8923_write(int cmd, const u8 *buf, int len)
        if (!len)
                return;
 
-       set_spi_data_width(9);
-
        t.tx_buf = &w;
        for (i = 0; i < len; i++) {
-               u16 w;
-
                spi_message_init(&m);
                m.spi = lph8923.spi;
                spi_message_add_tail(&t, &m);