On Fri, Jul 28, 2006 at 05:24:40PM -0700, Greg KH wrote: > On Fri, Jul 28, 2006 at 05:04:49PM -0700, Brandeburg, Jesse wrote:
> > The kernel driver advertises through our probe table and in our readme > > that we support this device (code is in e1000_main.c) but we don't load on > > it correctly due to an error when we submitted the patches (e1000_hw.c/h). > > > > Patch is very simple and low risk, but it is your call. > > Sorry, but for now, no. Users should be able to use the sysfs bind and > new_id interface to add new device support for your driver, if it's > written properly :) In this case it isn't an addition to the PCI ID table, but an actual added check in the code. You could make the argument that it is a bug since the PCI ID table says the device is supported, but it won't work w/o this additional check. Of course, the "if it's written properly" comment is still valid... :-) John -- John W. Linville [EMAIL PROTECTED] - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
