If there aren't any objections, I'll follow the i2c layout and create a drivers/w1/busses folder and put the driver there. Should I also create a few patches to move the matrox_w1 and ds_w1_bridge stuff to that folder? Should I also create a drivers/w1/chips folder and move the family drivers? > > +static u8 ds2482_w1_touch_bit(unsigned long data, u8 bit) > > +{ > > + struct ds2482_w1_chan *pchan = (struct ds2482_w1_chan *)data; > > That's an ugly cast. Can't it be avoided? Sure, if we change the w1 subsystem to use 'void *' as the user-data type instead of 'unsigned long'. Eugeniy - are you OK with that change? Thanks, Ben