kern/128950: [patch] Marvell 88SE6121 PATA controller not supported

[Thomas Nyström]

>Number:         128950
>Category:       kern
>Synopsis:       [patch] Marvell 88SE6121 PATA controller not supported
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Tue Nov 18 00:30:01 UTC 2008
>Closed-Date:
>Last-Modified:
>Originator:     Thomas Nyström
>Release:        FreeBSD 7.1-PRERELEASE
>Organization:
Svensk Aktuell Elektronik AB
>Environment:
FreeBSD oscar.thn.saeab.se 7.1-PRERELEASE FreeBSD 7.1-PRERELEASE #0: Mon Nov 17 23:45:13 CET 2008    root at semla.thn.saeab.se:/usr/obj/usr/src/sys/TURE  i386

>Description:
Asus P5Q motherboard uses a Marvell 88SE6121 for its IDE channel.
This controller is not recognized and handled correctly by FreeBSD.

pciconf -lv for this chip:
atapci0 at pci0:3:0:0:     class=0x01018f card=0x82e01043 chip=0x612111ab rev=0xb2 hdr=0x00
    vendor     = 'Marvell Semiconductor (Was: Galileo Technology Ltd)'
    device     = '6121 SATA2 Controller'
    class      = mass storage
    subclass   = ATA

>How-To-Repeat:
N/A
>Fix:


Patch attached with submission follows:

--- ata-pci.h.orig      2008-11-17 01:47:20.000000000 +0100
+++ ata-pci.h   2008-11-17 23:38:40.000000000 +0100
@@ -211,6 +211,7 @@
 #define ATA_M88SX6041           0x604111ab
 #define ATA_M88SX6081           0x608111ab
 #define ATA_M88SX6101           0x610111ab
+#define ATA_M88SX6121           0x612111ab
 #define ATA_M88SX6145           0x614511ab
 
 #define ATA_MICRON_ID           0x1042
--- ata-chipset.c.orig  2008-11-17 01:47:20.000000000 +0100
+++ ata-chipset.c       2008-11-17 23:38:53.000000000 +0100
@@ -2555,6 +2555,7 @@
      { ATA_M88SX6041, 0, 4, MV60XX, ATA_SA300, "88SX6041" },
      { ATA_M88SX6081, 0, 8, MV60XX, ATA_SA300, "88SX6081" },
      { ATA_M88SX6101, 0, 1, MV61XX, ATA_UDMA6, "88SX6101" },
+     { ATA_M88SX6121, 0, 1, MV61XX, ATA_UDMA6, "88SX6121" },
      { ATA_M88SX6145, 0, 2, MV61XX, ATA_UDMA6, "88SX6145" },
      { 0, 0, 0, 0, 0, 0}};


>Release-Note:
>Audit-Trail:
>Unformatted: