viafb: rework the I2C support in the VIA framebuffer driver

This patch changes the way how the various I2C busses are used internally
inside the viafb driver:  Previosuly, only a single i2c_adapter was created,
even though two different hardware I2C busses are accessed: A structure member
in a global variable was modified to indicate the bus to be used.

Now, all existing hardware busses are registered with the i2c core, and the
viafb_i2c_{read,write}byte[s]() function take the adapter number as function
call parameter, rather than referring to the global structure member.

[jc: even more painful merge with mainline changes ->2.6.34]
[jc: painful merge with OLPC changes]

Cc: Florian Tobias Schandinat <FlorianSchandinat@gmx.de>
Cc: ScottFang@viatech.com.cn
Cc: JosephChan@via.com.tw
Signed-off-by: Harald Welte <HaraldWelte@viatech.com>
Signed-off-by: Jonathan Corbet <corbet@lwn.net>

1 files changed, 15 insertions, 20 deletions

diff --git a/drivers/video/via/dvi.c b/drivers/video/via/dvi.c
index abe59b8c7a05..be513701e4e6 100644
--- a/drivers/video/via/dvi.c
+++ b/drivers/video/via/dvi.c
@@ -96,7 +96,7 @@ int viafb_tmds_trasmitter_identify(void)
         viaparinfo->chip_info->tmds_chip_info.tmds_chip_name = VT1632_TMDS;
         viaparinfo->chip_info->
                 tmds_chip_info.tmds_chip_slave_addr = VT1632_TMDS_I2C_ADDR;
-        viaparinfo->chip_info->tmds_chip_info.i2c_port = I2CPORTINDEX;
+        viaparinfo->chip_info->tmds_chip_info.i2c_port = VIA_I2C_ADAP_31;
         if (check_tmds_chip(VT1632_DEVICE_ID_REG, VT1632_DEVICE_ID) != FAIL) {
                 /*
                  * Currently only support 12bits,dual edge,add 24bits mode later
@@ -110,7 +110,7 @@ int viafb_tmds_trasmitter_identify(void)
                           viaparinfo->chip_info->tmds_chip_info.i2c_port);
                 return OK;
         } else {
-                viaparinfo->chip_info->tmds_chip_info.i2c_port = GPIOPORTINDEX;
+                viaparinfo->chip_info->tmds_chip_info.i2c_port = VIA_I2C_ADAP_2C;
                 if (check_tmds_chip(VT1632_DEVICE_ID_REG, VT1632_DEVICE_ID)
                     != FAIL) {
                         tmds_register_write(0x08, 0x3b);
@@ -160,32 +160,26 @@ int viafb_tmds_trasmitter_identify(void)
 
 static void tmds_register_write(int index, u8 data)
 {
-        viaparinfo->shared->i2c_stuff.i2c_port =
-                viaparinfo->chip_info->tmds_chip_info.i2c_port;
-
-        viafb_i2c_writebyte(viaparinfo->chip_info->tmds_chip_info.
-                tmds_chip_slave_addr, index,
-                     data);
+        viafb_i2c_writebyte(viaparinfo->chip_info->tmds_chip_info.i2c_port,
+                            viaparinfo->chip_info->tmds_chip_info.tmds_chip_slave_addr,
+                            index, data);
 }
 
 static int tmds_register_read(int index)
 {
         u8 data;
 
-        viaparinfo->shared->i2c_stuff.i2c_port =
-                viaparinfo->chip_info->tmds_chip_info.i2c_port;
-        viafb_i2c_readbyte((u8) viaparinfo->chip_info->
-            tmds_chip_info.tmds_chip_slave_addr,
-                        (u8) index, &data);
+        viafb_i2c_readbyte(viaparinfo->chip_info->tmds_chip_info.i2c_port,
+                           (u8) viaparinfo->chip_info->tmds_chip_info.tmds_chip_slave_addr,
+                           (u8) index, &data);
         return data;
 }
 
 static int tmds_register_read_bytes(int index, u8 *buff, int buff_len)
 {
-        viaparinfo->shared->i2c_stuff.i2c_port =
-                viaparinfo->chip_info->tmds_chip_info.i2c_port;
-        viafb_i2c_readbytes((u8) viaparinfo->chip_info->tmds_chip_info.
-                         tmds_chip_slave_addr, (u8) index, buff, buff_len);
+        viafb_i2c_readbytes(viaparinfo->chip_info->tmds_chip_info.i2c_port,
+                            (u8) viaparinfo->chip_info->tmds_chip_info.tmds_chip_slave_addr,
+                            (u8) index, buff, buff_len);
         return 0;
 }
 
@@ -541,9 +535,10 @@ void viafb_dvi_enable(void)
                                 else
                                         data = 0x37;
                                 viafb_i2c_writebyte(viaparinfo->chip_info->
-                                             tmds_chip_info.
-                                             tmds_chip_slave_addr,
-                                             0x08, data);
+                                                       tmds_chip_info.i2c_port,
+                                                    viaparinfo->chip_info->
+                                                       tmds_chip_info.tmds_chip_slave_addr,
+                                                    0x08, data);
                         }
                 }
         }