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, 17 insertions, 19 deletions

diff --git a/drivers/video/via/vt1636.c b/drivers/video/via/vt1636.c
index a6b37494e79a..4589c6e73c5d 100644
--- a/drivers/video/via/vt1636.c
+++ b/drivers/video/via/vt1636.c
@@ -27,9 +27,8 @@ u8 viafb_gpio_i2c_read_lvds(struct lvds_setting_information
 {
         u8 data;
 
-        viaparinfo->shared->i2c_stuff.i2c_port = plvds_chip_info->i2c_port;
-        viafb_i2c_readbyte(plvds_chip_info->lvds_chip_slave_addr, index, &data);
-
+        viafb_i2c_readbyte(plvds_chip_info->i2c_port,
+                           plvds_chip_info->lvds_chip_slave_addr, index, &data);
         return data;
 }
 
@@ -39,14 +38,13 @@ void viafb_gpio_i2c_write_mask_lvds(struct lvds_setting_information
 {
         int index, data;
 
-        viaparinfo->shared->i2c_stuff.i2c_port = plvds_chip_info->i2c_port;
-
         index = io_data.Index;
         data = viafb_gpio_i2c_read_lvds(plvds_setting_info, plvds_chip_info,
                 index);
         data = (data & (~io_data.Mask)) | io_data.Data;
 
-        viafb_i2c_writebyte(plvds_chip_info->lvds_chip_slave_addr, index, data);
+        viafb_i2c_writebyte(plvds_chip_info->i2c_port,
+                            plvds_chip_info->lvds_chip_slave_addr, index, data);
 }
 
 void viafb_init_lvds_vt1636(struct lvds_setting_information
@@ -159,7 +157,7 @@ void viafb_disable_lvds_vt1636(struct lvds_setting_information
         }
 }
 
-bool viafb_lvds_identify_vt1636(void)
+bool viafb_lvds_identify_vt1636(u8 i2c_adapter)
 {
         u8 Buffer[2];
 
@@ -170,23 +168,23 @@ bool viafb_lvds_identify_vt1636(void)
         VT1636_LVDS_I2C_ADDR;
 
         /* Check vendor ID first: */
-        viafb_i2c_readbyte((u8) viaparinfo->chip_info->lvds_chip_info.
-        lvds_chip_slave_addr,
-                    0x00, &Buffer[0]);
-        viafb_i2c_readbyte((u8) viaparinfo->chip_info->lvds_chip_info.
-                lvds_chip_slave_addr,
-                    0x01, &Buffer[1]);
+        viafb_i2c_readbyte(i2c_adapter,
+                           (u8) viaparinfo->chip_info->lvds_chip_info.lvds_chip_slave_addr,
+                           0x00, &Buffer[0]);
+        viafb_i2c_readbyte(i2c_adapter,
+                           (u8) viaparinfo->chip_info->lvds_chip_info.lvds_chip_slave_addr,
+                           0x01, &Buffer[1]);
 
         if (!((Buffer[0] == 0x06) && (Buffer[1] == 0x11)))
                 return false;
 
         /* Check Chip ID: */
-        viafb_i2c_readbyte((u8) viaparinfo->chip_info->lvds_chip_info.
-        lvds_chip_slave_addr,
-                    0x02, &Buffer[0]);
-        viafb_i2c_readbyte((u8) viaparinfo->chip_info->lvds_chip_info.
-                lvds_chip_slave_addr,
-                    0x03, &Buffer[1]);
+        viafb_i2c_readbyte(i2c_adapter,
+                           (u8) viaparinfo->chip_info->lvds_chip_info.lvds_chip_slave_addr,
+                           0x02, &Buffer[0]);
+        viafb_i2c_readbyte(i2c_adapter,
+                           (u8) viaparinfo->chip_info->lvds_chip_info.lvds_chip_slave_addr,
+                           0x03, &Buffer[1]);
         if ((Buffer[0] == 0x45) && (Buffer[1] == 0x33)) {
                 viaparinfo->chip_info->lvds_chip_info.lvds_chip_name =
                         VT1636_LVDS;