From nobody@FreeBSD.org  Thu Sep 11 06:45:45 2008
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 5385C1065670
	for <freebsd-gnats-submit@FreeBSD.org>; Thu, 11 Sep 2008 06:45:45 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (www.freebsd.org [IPv6:2001:4f8:fff6::21])
	by mx1.freebsd.org (Postfix) with ESMTP id 60F558FC12
	for <freebsd-gnats-submit@FreeBSD.org>; Thu, 11 Sep 2008 06:45:44 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (localhost [127.0.0.1])
	by www.freebsd.org (8.14.2/8.14.2) with ESMTP id m8B6jhEK064686
	for <freebsd-gnats-submit@FreeBSD.org>; Thu, 11 Sep 2008 06:45:43 GMT
	(envelope-from nobody@www.freebsd.org)
Received: (from nobody@localhost)
	by www.freebsd.org (8.14.2/8.14.1/Submit) id m8B6jhLB064685;
	Thu, 11 Sep 2008 06:45:43 GMT
	(envelope-from nobody)
Message-Id: <200809110645.m8B6jhLB064685@www.freebsd.org>
Date: Thu, 11 Sep 2008 06:45:43 GMT
From: Raul <raul@b2n.org>
To: freebsd-gnats-submit@FreeBSD.org
Subject: [ata] [patch] new pci ids for SiI3112 present on adaptec aar-1225sa rev a2
X-Send-Pr-Version: www-3.1
X-GNATS-Notify:

>Number:         127289
>Category:       kern
>Synopsis:       [ata] [patch] new pci ids for SiI3132 present on adaptec aar-1225sa rev a2
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-bugs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Thu Sep 11 06:50:01 UTC 2008
>Closed-Date:    Thu Apr 28 19:01:20 UTC 2011
>Last-Modified:  Thu Apr 28 19:01:20 UTC 2011
>Originator:     Raul
>Release:        FreeBSD 7.1-PRERELEASE amd64
>Organization:
not applicable
>Environment:
FreeBSD xxx 7.1-PRERELEASE FreeBSD 7.1-PRERELEASE #0: Sun Sep  7 21:10:45 CEST 2008     raul@xxx:/usr/obj/usr/src/sys/GENERIC  amd64
>Description:
Commented on:

http://lists.freebsd.org/pipermail/freebsd-drivers/2008-September/000770.html

after the patch, 'pciconf -l -v' relevant output look like this:

atapci0@pci0:7:0:0:     class=0x010401 card=0x02449005 chip=0x02441095 rev=0x01 hdr=0x00
    vendor     = 'Silicon Image Inc (Was: CMD Technology Inc)'
    class      = mass storage
    subclass   = RAID

>How-To-Repeat:
Not applicable; simply not supported at the moment.
>Fix:
Apply the patch supplied by Volker.

Patch attached with submission follows:

--- sys/dev/ata/ata-pci.h.orig	2008-09-06 22:09:23.000000000 +0200
+++ sys/dev/ata/ata-pci.h	2008-09-06 22:12:10.000000000 +0200
@@ -304,6 +304,7 @@
 #define ATA_SII3112_1           0x02401095
 #define ATA_SII3124		0x31241095
 #define ATA_SII3132		0x31321095
+#define ATA_SII3132_1           0x02441095
 #define ATA_SII0680             0x06801095
 #define ATA_CMD646              0x06461095
 #define ATA_CMD648              0x06481095


>Release-Note:
>Audit-Trail:

From: Raul <raul@b2n.org>
To: bug-followup@FreeBSD.org, raul@b2n.org
Cc:  
Subject: Re: kern/127289: [ata] [patch] new pci ids for SiI3112 present on
	adaptec aar-1225sa rev a2
Date: Thu, 11 Sep 2008 09:09:27 +0200

 --=-P95J5HycTuvyEgx6rdQy
 Content-Type: text/plain
 Content-Transfer-Encoding: 7bit
 
 The other half of the patch :/
 
 --=-P95J5HycTuvyEgx6rdQy
 Content-Disposition: attachment; filename=ata-chipset.diff
 Content-Type: text/x-patch; name=ata-chipset.diff; charset=utf-8
 Content-Transfer-Encoding: 7bit
 
 --- sys/dev/ata/ata-chipset.c.orig	2008-09-06 22:12:23.000000000 +0200
 +++ sys/dev/ata/ata-chipset.c	2008-09-06 22:13:07.000000000 +0200
 @@ -4458,6 +4458,7 @@
       { ATA_SII3112_1, 0x00, SIIMEMIO, SIIBUG,    ATA_SA150, "SiI 3112" },
       { ATA_SII3124,   0x00, SIIPRBIO, SII4CH,    ATA_SA300, "SiI 3124" },
       { ATA_SII3132,   0x00, SIIPRBIO, 0,         ATA_SA300, "SiI 3132" },
 +     { ATA_SII3132_1, 0x00, SIIPRBIO, 0,         ATA_SA300, "SiI 3132" },
       { ATA_SII0680,   0x00, SIIMEMIO, SIISETCLK, ATA_UDMA6, "SiI 0680" },
       { ATA_CMD649,    0x00, 0,        SIIINTR,   ATA_UDMA5, "CMD 649" },
       { ATA_CMD648,    0x00, 0,        SIIINTR,   ATA_UDMA4, "CMD 648" },
 
 --=-P95J5HycTuvyEgx6rdQy--
 

From: Raul <raul@b2n.org>
To: bug-followup@FreeBSD.org, raul@b2n.org
Cc:  
Subject: Re: kern/127289: [ata] [patch] new pci ids for SiI3112 present on
	adaptec aar-1225sa rev a2
Date: Thu, 11 Sep 2008 09:29:41 +0200

 Please, forget the synopsis, the chip is SiI3132.
 
 Sorry for the noise.
 
 Raul
 

From: Raul <raul@b2n.org>
To: bug-followup@FreeBSD.org, raul@b2n.org
Cc:  
Subject: kern/127289: [ata] [patch] new pci ids for SiI3132 present on adaptec
 aar-1225sa rev a2
Date: Thu, 26 Mar 2009 09:41:49 +0100

 After several months of flawless duty, any chance to get it on STABLE 
 before code freeze?
 
 Thanks a lot in advance.
State-Changed-From-To: open->patched 
State-Changed-By: mav 
State-Changed-When: Thu May 7 19:04:17 UTC 2009 
State-Changed-Why:  
Patch committed to 8-CURRENT. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=127289 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/127289: commit references a PR
Date: Thu,  7 May 2009 19:17:16 +0000 (UTC)

 Author: mav
 Date: Thu May  7 19:17:06 2009
 New Revision: 191897
 URL: http://svn.freebsd.org/changeset/base/191897
 
 Log:
   Add ID of one more SII3132 revision found on adaptec aar-1225sa rev a2.
   
   PR:		kern/127289
 
 Modified:
   head/sys/dev/ata/ata-pci.h
   head/sys/dev/ata/chipsets/ata-siliconimage.c
 
 Modified: head/sys/dev/ata/ata-pci.h
 ==============================================================================
 --- head/sys/dev/ata/ata-pci.h	Thu May  7 18:36:47 2009	(r191896)
 +++ head/sys/dev/ata/ata-pci.h	Thu May  7 19:17:06 2009	(r191897)
 @@ -345,6 +345,7 @@ struct ata_pci_controller {
  #define ATA_SII3124		0x31241095
  #define ATA_SII3132		0x31321095
  #define ATA_SII3132_1		0x02421095
 +#define ATA_SII3132_2		0x02441095
  #define ATA_SII0680             0x06801095
  #define ATA_CMD646              0x06461095
  #define ATA_CMD648              0x06481095
 
 Modified: head/sys/dev/ata/chipsets/ata-siliconimage.c
 ==============================================================================
 --- head/sys/dev/ata/chipsets/ata-siliconimage.c	Thu May  7 18:36:47 2009	(r191896)
 +++ head/sys/dev/ata/chipsets/ata-siliconimage.c	Thu May  7 19:17:06 2009	(r191897)
 @@ -99,6 +99,7 @@ ata_sii_probe(device_t dev)
       { ATA_SII3124,   0x00, SII_PRBIO, SII_4CH,    ATA_SA300, "3124" },
       { ATA_SII3132,   0x00, SII_PRBIO, 0,          ATA_SA300, "3132" },
       { ATA_SII3132_1, 0x00, SII_PRBIO, 0,          ATA_SA300, "3132" },
 +     { ATA_SII3132_2, 0x00, SII_PRBIO, 0,          ATA_SA300, "3132" },
       { ATA_SII0680,   0x00, SII_MEMIO, SII_SETCLK, ATA_UDMA6, "680" },
       { ATA_CMD649,    0x00, 0,         SII_INTR,   ATA_UDMA5, "(CMD) 649" },
       { ATA_CMD648,    0x00, 0,         SII_INTR,   ATA_UDMA4, "(CMD) 648" },
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: patched->closed 
State-Changed-By: jhb 
State-Changed-When: Thu Apr 28 18:57:33 UTC 2011 
State-Changed-Why:  
Fix was merged to 8 earlier and just now to 7. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=127289 
>Unformatted:
