Subject: Re: kern/36759 (patch: AMD PCI product id updates)
To: None <jnemeth@NetBSD.org, gnats-admin@netbsd.org,>
From: John Nemeth <jnemeth@cornerstoneservice.ca>
List: netbsd-bugs
Date: 08/14/2007 16:20:02
The following reply was made to PR kern/36759; it has been noted by GNATS.

From: jnemeth@cornerstoneservice.ca (John Nemeth)
To: "Christoph Egger" <Christoph_Egger@gmx.de>,
	gnats-bugs@NetBSD.org, netbsd-bugs@NetBSD.org
Cc: 
Subject: Re: kern/36759 (patch: AMD PCI product id updates)
Date: Tue, 14 Aug 2007 09:16:20 -0700

 On Aug 14,  5:03pm, "Christoph Egger" wrote:
 } > The following reply was made to PR kern/36759; it has been noted by GNATS.
 } > 
 } > From: "Christoph Egger" <Christoph_Egger@gmx.de>
 } > To: gnats-bugs@NetBSD.org, jnemeth@netbsd.org,
 } > 	gnats-admin@netbsd.org, netbsd-bugs@netbsd.org,
 } > 	kern-bug-people@netbsd.org
 } > Cc: 
 } > Subject: Re: kern/36759 (patch: AMD PCI product id updates)
 } > Date: Sat, 11 Aug 2007 10:47:07 +0200
 } > 
 } >  > Synopsis: patch: AMD PCI product id updates
 } >  > 
 } >  > State-Changed-From-To: open->feedback
 } >  > State-Changed-By: jnemeth@netbsd.org
 } >  > State-Changed-When: Sat, 11 Aug 2007 02:07:11 +0000
 } >  > State-Changed-Why:
 } >  > I committed the patch, but since it didn't patch cleanly,
 } >  > I had to do it by hand.  Please check it.
 } >  
 } >  Will do once it is in anonymous cvs. It is not yet as of this writing.
 } 
 } The applied patch is fine with me. You can close this
 } PR - or apply the patch below and then close it. :-)
 } 
 } Index: pcidevs
 } ===================================================================
 } RCS file: /cvsroot/src/sys/dev/pci/pcidevs,v
 } retrieving revision 1.889
 } diff -u -p -r1.889 pcidevs
 } --- pcidevs     11 Aug 2007 02:05:11 -0000      1.889
 } +++ pcidevs     13 Aug 2007 09:21:33 -0000
 } @@ -1862,6 +1862,10 @@ product MARVELL 88SX5080 0x5080  88SX5080
 }  product MARVELL 88SX5081       0x5081  88SX5081 SATA
 }  product MARVELL 88SX6041       0x6041  88SX6041 SATA
 }  product MARVELL 88SX6081       0x6081  88SX6041 SATA
 } +product MARVELL 88SE6101       0x6101  88SE6101 single-port PATA133 Controller
 } +product MARVELL 88SE6121       0x6121  88SE6102 SATA II Controller
 } +product MARVELL 88SE614X       0x6141  88SE614X SATA II PCI-E Controller
 } +product MARVELL 88SE6145       0x6145  88SE6145 SATA II PCI-E Controller
 }  product MARVELL        GT64130         0x6320  GT-64130 System Controller
 }  product MARVELL        GT64260         0x6430  GT-64260 System Controller
 }  product MARVELL        GT64360         0x6460  MV6436x System Controller
 
      I applied the patch.  I assume the second line should have said:
 
 +product MARVELL 88SE6121       0x6121  88SE6121 SATA II Controller
 
 }-- End of excerpt from "Christoph Egger"