On 23 July 2011 20:02, <alexdeucher@xxxxxxxxx> wrote: > From: Alex Deucher <alexander.deucher@xxxxxxx> > > Those chips have crt2_ddc bus. Maybe adding a comment in the code'll make sure nobody will ever "clean up" or "optimize" the tests by replacing them back by "else if (rdev->family >= CHIP_R200)"... > - } else if (rdev->family >= CHIP_R200) { > + } else if ((rdev->family == CHIP_R200) || > + (rdev->family >= CHIP_R300)) { > /* 0x68 */ > i2c = combios_setup_i2c_bus(rdev, DDC_MONID, 0, 0); > rdev->i2c_bus[3] = radeon_i2c_create(dev, &i2c, "MONID"); My 2 cents _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel