netbsd-bugs@netbsd.org
[Top] [All Lists]

Re: kern/36759 (patch: AMD PCI product id updates)

Subject: Re: kern/36759 patch: AMD PCI product id updates
From: "Christoph Egger"
Date: Tue, 14 Aug 2007 16:39:06 UTC
Newsgroups: fa.netbsd.bugs


> The following reply was made to PR kern/36759; it has been noted by GNATS.
> 
> From: jnemeth@xxxxxxxxxxxxxxxxxxxxx (John Nemeth)
> To: "Christoph Egger" <Christoph_Egger@xxxxxx>,
>       gnats-bugs@xxxxxxxxxx, netbsd-bugs@xxxxxxxxxx
> 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@xxxxxx>
>  } > To: gnats-bugs@xxxxxxxxxx, jnemeth@xxxxxxxxxx,
>  } >  gnats-admin@xxxxxxxxxx, netbsd-bugs@xxxxxxxxxx,
>  } >  kern-bug-people@xxxxxxxxxx
>  } > 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@xxxxxxxxxx
>  } >  > 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



<Prev in Thread] Current Thread [Next in Thread>