Skip to contentSkip to conversation

Add VX800/VX820 support to i2c-viapro

PATCH In review Rudolf Marek 2 participants 2 messages Aug 10, 2008 – Aug 12, 2008 1 message not in the archive

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Add VX800/VX820 support to i2c-viapro

Hello,

Thanks to new datasheets published on http://linux.via.com.tw we can now add support for VX800/VX820 chipsets.

If anyone has the hardware, please test the patch in attachment. Some simple test with i2cdump should be enough the hardware controller is same as on other chipsets.

Thank you,

Thanks,
Rudolf

Documentation/i2c/busses/i2c-viapro +6 -2
Index: linux-2.6.27-rc2/Documentation/i2c/busses/i2c-viapro
===================================================================
@@ -16,6 +16,9 @@
   * VIA Technologies, Inc. CX700
     Datasheet: available on request and under NDA from VIA
 
+  * VIA Technologies, Inc. VX800/VX820
+    Datasheet: available on http://linux.via.com.tw
+
 Authors:
 	Kyösti Mälkki <[email protected]>,
 	Mark D. Studebaker <mdsxyz123-/[email protected]>,
@@ -49,6 +52,7 @@
  device 1106:3372   (VT8237S)
  device 1106:3287   (VT8251)
  device 1106:8324   (CX700)
+ device 1106:8353   (VX800/VX820)
 
 If none of these show up, you should look in the BIOS for settings like
 enable ACPI / SMBus or even USB.
@@ -57,5 +61,5 @@
 VT8231), this driver supports I2C block transactions. Such transactions
 are mainly useful to read from and write to EEPROMs.
 
-The CX700 additionally appears to support SMBus PEC, although this driver
-doesn't implement it yet.
+The CX700/VX800/VX820 additionally appears to support SMBus PEC, although
+this driver doesn't implement it yet.
drivers/i2c/busses/Kconfig +3 -1
Index: linux-2.6.27-rc2/drivers/i2c/busses/Kconfig
===================================================================
@@ -209,7 +209,7 @@
 	  will be called i2c-via.
 
 config I2C_VIAPRO
-	tristate "VIA VT82C596/82C686/82xx and CX700"
+	tristate "VIA VT82C596/82C686/82xx and CX700/VX800/VX820"
 	depends on PCI
 	help
 	  If you say yes to this option, support will be included for the VIA
@@ -223,6 +223,8 @@
 	    VT8237R/A/S
 	    VT8251
 	    CX700
+	    VX800
+	    VX820
 
 	  This driver can also be built as a module.  If so, the module
 	  will be called i2c-viapro.
drivers/i2c/busses/i2c-viapro.c +4 -0
Index: linux-2.6.27-rc2/drivers/i2c/busses/i2c-viapro.c
===================================================================
@@ -36,6 +36,7 @@
    VT8237S            0x3372             yes
    VT8251             0x3287             yes
    CX700              0x8324             yes
+   VX800/VX820        0x8353             yes
 
    Note: we assume there can only be one device, with one SMBus interface.
 */
@@ -396,6 +397,7 @@
 
 	switch (pdev->device) {
 	case PCI_DEVICE_ID_VIA_CX700:
+	case PCI_DEVICE_ID_VIA_VX800:
 	case PCI_DEVICE_ID_VIA_8251:
 	case PCI_DEVICE_ID_VIA_8237:
 	case PCI_DEVICE_ID_VIA_8237A:
@@ -459,6 +461,8 @@
 	  .driver_data = SMBBA3 },
 	{ PCI_DEVICE(PCI_VENDOR_ID_VIA, PCI_DEVICE_ID_VIA_CX700),
 	  .driver_data = SMBBA3 },
+	{ PCI_DEVICE(PCI_VENDOR_ID_VIA, PCI_DEVICE_ID_VIA_VX800),
+	  .driver_data = SMBBA3 },
 	{ 0, }
 };
 

_______________________________________________
i2c mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/i2c

A message that is not in the archive (sent off-list or not yet received): 489F3E6D.1010509-/[email protected]

#2 lore raw

Hi Rudolf,

Quoted 10 lines · Rudolf Marek

Hello,

Thanks to new datasheets published on http://linux.via.com.tw we can now add support for VX800/VX820 chipsets.

If anyone has the hardware, please test the patch in attachment. Some simple test with i2cdump should be enough the hardware controller is same as on other chipsets.

Thanks. I've queued this patch for 2.6.28, so it's in linux-next now. I hope we can get some testing feedback in the next weeks.

Signature · 6 lines
Jean Delvare

_______________________________________________
i2c mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/i2c