>Number:         33393
>Category:       port-amd64
>Synopsis:       viaide driver does not recognise VT8237R variant, so only PIO available.
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    port-amd64-maintainer
>State:          open
>Class:          change-request
>Submitter-Id:   net
>Arrival-Date:   Sun Apr 30 14:15:00 +0000 2006
>Originator:     root admin
>Release:        NetBSD 3.0.0_STABLE
>Organization:
  AXDF
>Environment:
	
	
System: NetBSD blants 3.0.0_STABLE NetBSD 3.0.0_STABLE (NITROTOMINE) #5: Sun Apr 30 21:01:43 EST 2006 root@blants:/usr/src/sys/arch/amd64/compile/NITROTOMINE amd64
Architecture: x86_64
Machine: amd64
>Description:
  viaide driver does not recognise and hence does not support VT8237R SATA interface.
  Works with pciide, but only in PIO mode.
>How-To-Repeat:
	N/A
>Fix:
  Patches attached.

--- viaide.c	2006-04-30 20:17:01.000000000 +1000
+++ viaide.c	2006-04-30 21:01:13.000000000 +1000
@@ -160,7 +160,12 @@
 	  "VIA Technologies VT8237 SATA Controller",
 	  via_sata_chip_map,
 	},
-	{ 0,
+	{ PCI_PRODUCT_VIATECH_VT8237R_SATA,
+    0,
+    "VIA Technologies VT8237R SATA Controller",
+    via_sata_chip_map,
+  },
+  { 0,
 	  0,
 	  NULL,
 	  NULL

--- pcidevs	2006-04-30 23:54:54.000000000 +1000
+++ pcidevs	2006-04-30 20:47:36.000000000 +1000
@@ -2922,6 +2922,7 @@
 product VIATECH VT8623		0x3123	VT8623 (Apollo CLE266) CPU-PCI Bridge
 product VIATECH VT8233A		0x3147	VT8233A PCI-ISA Bridge
 product VIATECH VT8237_SATA	0x3149	VT8237 Integrated SATA Controller
+product VIATECH VT8237R_SATA 0x3349	VT8237R Integrated SATA Controller
 product VIATECH VT6421_RAID	0x3249	VT6421 Serial RAID Controller
 product VIATECH KT880_3         0x3269  KT880 CPU to PCI bridge
 product VIATECH VT8235		0x3177	VT8235 (Apollo KT400) PCI-ISA Bridge

--- pcidevs.h	2006-04-30 23:56:42.000000000 +1000
+++ pcidevs.h	2006-04-30 23:57:06.000000000 +1000
@@ -2929,6 +2929,7 @@
 #define	PCI_PRODUCT_VIATECH_VT8623	0x3123		/* VT8623 (Apollo CLE266) CPU-PCI Bridge */
 #define	PCI_PRODUCT_VIATECH_VT8233A	0x3147		/* VT8233A PCI-ISA Bridge */
 #define	PCI_PRODUCT_VIATECH_VT8237_SATA	0x3149		/* VT8237 Integrated SATA Controller */
+#define	PCI_PRODUCT_VIATECH_VT8237R_SATA	0x3349		/* VT8237R Integrated SATA Controller */
 #define	PCI_PRODUCT_VIATECH_VT6421_RAID	0x3249		/* VT6421 Serial RAID Controller */
 #define	PCI_PRODUCT_VIATECH_KT880_3	0x3269		/* KT880 CPU to PCI bridge */
 #define	PCI_PRODUCT_VIATECH_VT8235	0x3177		/* VT8235 (Apollo KT400) PCI-ISA Bridge */

--- pcidevs_data.h	2006-04-30 23:56:42.000000000 +1000
+++ pcidevs_data.h	2006-04-30 23:57:06.000000000 +1000
@@ -9768,6 +9768,10 @@
 	    "VT8237 Integrated SATA Controller",
 	},
 	{
+	    PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VT8237R_SATA,
+	    "VT8237R Integrated SATA Controller",
+	},
+	{
 	    PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VT6421_RAID,
 	    "VT6421 Serial RAID Controller",
 	},
@@ -10320,4 +10324,4 @@
 	    "Video Controller",
 	},
 };
-const int pci_nproducts = 2001;
+const int pci_nproducts = 2002;

>Unformatted: