On Wed, Jan 16, 2013 at 04:36:08PM +0300, Dan Carpenter wrote: > You've added the writes for GO7007_BOARDID_ADLINK_MPG24 but removed > them for GO7007_BOARDID_XMEN and GO7007_BOARDID_XMEN_III. Won't > that break those boards? > I don`t remove code for GO7007_BOARDID_XMEN and GO7007_BOARDID_XMEN_III. case there are auto reusing for XMen and XMen-III: look old code: if ((go->board_id == GO7007_BOARDID_XMEN || go->board_id == GO7007_BOARDID_XMEN_III) && go->i2c_adapter_online) { union i2c_smbus_data data; /* Check to see if register 0x0A is 0x76 */ i2c_smbus_xfer(&go->i2c_adapter, 0x21, I2C_CLIENT_SCCB, I2C_SMBUS_READ, 0x0A, I2C_SMBUS_BYTE_DATA, &data); if (data.byte != 0x76) { if (assume_endura) { go->board_id = GO7007_BOARDID_ENDURA; usb->board = board = &board_endura; go->board_info = &board->main_info; strncpy(go->name, "Pelco Endura", sizeof(go->name)); } else { u16 channel; - /* set GPIO5 to be an output, currently low */ - go7007_write_addr(go, 0x3c82, 0x0000); - go7007_write_addr(go, 0x3c80, 0x00df); /* read channel number from GPIO[1:0] */ go7007_read_addr(go, 0x3c81, &channel); channel &= 0x3; >>> go->board_id = GO7007_BOARDID_ADLINK_MPG24; Here any XMen or XMen-III will reassigned as Adlink-mpg24 id so any i2c initialization will reassigned to that id and we can use that id in init_encoder. usb->board = board = &board_adlink_mpg24; go->board_info = &board->main_info; go->channel_number = channel; snprintf(go->name, sizeof(go->name), "Adlink PCI-MPG24, channel #%d", channel); } } } > regards, > dan carpenter > > _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel