> agreed. > Khali? OK, done. > Greg KH wrote: > > On Fri, Aug 29, 2003 at 06:47:51PM +0200, Jean Delvare wrote: > > > >>>On Sat, Aug 23, 2003 at 06:58:47PM +0200, Kronos wrote: > >>> > >>>>Hi Greg, > >>>>I forgot to cc you when I sent the radeonfb-i2c patch to fbdev. I > >>>>need a new driver-id: > >>>> > >>>> > >>>>--- linux-2.6.vanilla/include/linux/i2c-id.h Tue Jul 15 14:27:55 > >>>>2003+++ linux-2.6/include/linux/i2c-id.h Wed Aug 20 19:22:58 2003 > >>>>@@ -99,6 +99,7 @@ > >>>> #define I2C_DRIVERID_24LC32A 51 /* Microchip 24LC32A 32k EEPROM */ > >>>> #define I2C_DRIVERID_STM41T00 52 /* real time clock */ > >>>> #define I2C_DRIVERID_UDA1342 53 /* UDA1342 audio codec */ > >>>>+#define I2C_DRIVERID_RADEON 54 /* I2C bus on Radeon boards */ > >>> > >>>Thanks, I've given you the id of 55, as someone else is already > >using>>54. I'll send the patch on to Linus in a few days. > >> > >>Hey, wait a minute. *We* (the LM Sensors group) are the authority > >for>giving the IDs. And we gave 54 for Radeon. Whoever else is using > >54 has>no right to do so. So, please use 54 for Radeon in 2.6 too, > >and ask the>other person to contact us to get an ID (which will > >probably be 55). > > > > > > Ah, the joys of trying to control a codebase that is out of your > > control :) > > > > It looks like the big zoran driver update that went into the 2.6 > > kernel tree added an id for number 54: > > #define I2C_DRIVERID_ADV7170 54 /* video encoder */ > > > > As that driver also has 3 other ids, I guess they got there first. > > > > I've cced the person who sent in the patch to Linus for this driver. > > > > Personally, I think we should just make the Radeon #55 as I've done > > in the 2.6 tree and move on, we all have better things to do with > > our time... > > > > thanks, > > > > greg k-h -- Jean Delvare http://www.ensicaen.ismra.fr/~delvare/
