kern/55926: [PATCH] VIA VT8237 support

[Hideyuki KURASHINA]

>Number:         55926
>Category:       kern
>Synopsis:       [PATCH] VIA VT8237 support
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          update
>Submitter-Id:   current-users
>Arrival-Date:   Sun Aug 24 09:00:21 PDT 2003
>Closed-Date:
>Last-Modified:
>Originator:     Hideyuki KURASHINA
>Release:        
>Organization:
>Environment:

>Description:

	I received a mail at FreeBSD mailing list in Japan, which is
	related to VIA VT8237.  SHIOTA-san, who is original reporter,
	bought ASUS A7V600 that has KT600 Chipset.  Unfortunately,
	FreeBSD 5.1-CURRENT recognizes that chip as follows;

	  ad0: 114473MB <ST3120026A> [232581/16/63] at ata0-master PIO4
	  acd0: DVD-ROM <ASUS DVD-ROM E612> at ata1-master PIO4

	He made a simple patch that attached at end of this mail, applying to
	his system results like following;

	  atapci0: <VIA 8237 UDMA133 controller> port 0xa400-0xa4ff,0xa800-0xa80f,0xb000-0xb003, 0xb400-0xb407,0xb800-0xb803,0xd000-0xd007 irq 3 at device 15.0 on pci0
	  ata2: at 0xd000 on atapci0
	  ata3: at 0xb400 on atapci0
	  atapci1: <VIA 8237 UDMA133 controller> port 0xa000-0xa00f  at device 15.1 on pci0
	  ata0: at 0x1f0 irq 14 on atapci1
	  ata1: at 0x170 irq 15 on atapci1
	  ad0: 114473MB <ST3120026A> [232581/16/63] at ata0-master UDMA100
	  ata1-master: DMA limited to UDMA33, non-ATA66 cable or device
	  acd0: DVD-ROM <ASUS DVD-ROM E612> at ata1-master UDMA33

	According to SHIOTA-san's testing, this change yields 12 times faster
	than before.

	I've checked VIA's web site,

	  IDE Support, Linux Application Notes
	    http://www.viaarena.com/?PageID=297

	and then downloaded

	  VT8237 South Bridge SATA RAID controller
	    http://downloads.viaarena.com/LinuxApplicationNotes/Aug03/VIA%20RH7.2%20SATA%20Device%20ver0.8.gz

	then extracted it.  The file `pcitable' in distfile shows

	  0x1106  0x3149  "viasraid"      "VIATECH|VIA VT6420 RAID Controller"

	It seems that 0x3149 is sufficient for VT8237, I think.

	Could you please commit this?

>How-To-Repeat:
	
>Fix:

	Apply following patch;

Index: ata-chipset.c
===================================================================
RCS file: /home/ncvs/src/sys/dev/ata/ata-chipset.c,v
retrieving revision 1.34
diff -u -r1.34 ata-chipset.c
--- ata-chipset.c	24 Aug 2003 09:22:26 -0000	1.34
+++ ata-chipset.c	24 Aug 2003 15:33:41 -0000
@@ -2052,6 +2052,7 @@
      { ATA_VIA8233C,  0x00, VIA100, 0x00,   ATA_UDMA5, "VIA 8233C" },
      { ATA_VIA8233A,  0x00, VIA133, 0x00,   ATA_UDMA6, "VIA 8233A" },
      { ATA_VIA8235,   0x00, VIA133, 0x00,   ATA_UDMA6, "VIA 8235" },
+     { ATA_VIA8237,   0x00, VIA133, 0x00,   ATA_UDMA6, "VIA 8237" },
      { 0, 0, 0, 0, 0, 0 }};
     char buffer[64];
 
Index: ata-pci.h
===================================================================
RCS file: /home/ncvs/src/sys/dev/ata/ata-pci.h,v
retrieving revision 1.13
diff -u -r1.13 ata-pci.h
--- ata-pci.h	24 Aug 2003 09:22:26 -0000	1.13
+++ ata-pci.h	24 Aug 2003 15:34:38 -0000
@@ -215,6 +215,7 @@
 #define ATA_VIA8233A		0x31471106
 #define ATA_VIA8233C		0x31091106
 #define ATA_VIA8235		0x31771106
+#define ATA_VIA8237		0x31491106
 #define ATA_VIA8361		0x31121106
 #define ATA_VIA8363		0x03051106
 #define ATA_VIA8371		0x03911106
>Release-Note:
>Audit-Trail:
>Unformatted: