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, netbsd-bugs@netbsd.org,
	gnats-admin@netbsd.org, jnemeth@NetBSD.org
Cc: 
Subject: Re: kern/36759 (patch: AMD PCI product id updates)
Date: Tue, 14 Aug 2007 18:38:40 +0200

 > 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"
 >  
 
 Yes, that's right. It's always good to have another pair of eyes looking
 over it. Thanks for applying this.
 
 
 
 -- 
 Greetings,
 
 Christoph