Subject: Re: kern/36759 (patch: AMD PCI product id updates)
To: Christoph Egger <Christoph_Egger@gmx.de>
From: John Nemeth <jnemeth@cornerstoneservice.ca>
List: netbsd-bugs
Date: 08/14/2007 09:16:20
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"