![]() | |||||||||||||
Messages in this thread |
Hi Jason, A few comments on your patch: > This patch adds DID support for ICH6 and 6300ESB to i2c-i801.c(SMBus). In > order to add this support I needed to patch pci_ids.h with the SMBus DID's. > To keep things orginized I renumbered the ICH6 and ESB entries in pci_ids.h. > I then patched the piix IDE and i810 audio drivers to reflect the updated > #define's. I also removed an error from irq.c; there was a reference to a > 6300ESB DID that does not exist. This patch is against the 2.6.6 kernel. To me, there are two different things here and you should split your patch accordingly for clarity. The renumbering of the entries should go in a first patch, then the new devices support in i2c-i801 in a second. I would also suggest that you follow the common habit to name the Intel 82801 chips after their numerical name (82801AA. 82801BA, etc...) and not their nickname (ICH, ICH2...) in both the entries and the comments. Not that it is absolulety better, but that's the way we did so far, so why change now? I would appreciate it if you could submit a similar patch for the 2.4 version of the i2c-i801 driver as held in the lm_sensors project. Thanks. Jean Delvare -- Jean "Khali" Delvare http://khali.linux-fr.org/ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | ||||||||||||
| Last update: 2005-03-22 13:03 [from the cache] ©2003-2008 | |||||||||||||