Hi Mike, On Sat, 4 Apr 2009 10:51:01 -0500 (CDT), Mike Isely wrote: > > Nacked-by: Mike Isely <isely@xxxxxxxxx> > > This will interfere with the alternative use of LIRC drivers (which work > in more cases that ir-kbd). Why then is ir-kbd in the kernel tree and not LIRC drivers? > It will thus break some peoples' use of the driver. Do you think it will, or did you test and it actually does? If it indeed breaks, please explain why, so that a solution can be found. > Also we have better information on what i2c addresses needed to > be probed based on the model of the device This is excellent news. As I said in the header comment of the patch, avoiding probing when we know what the IR receiver is and at which address it sits is the way to go. Please send me all the information you have and I'll be happy to add a patch to the series, that skips probing whenever possible. Or write that patch yourself if you prefer. > - and some devices supported > by this device are not from Hauppauge so you are making a too-strong > assumption that IR should be probed this way in all cases. I didn't make any assumption, sorry. I simply copied the code from ir-kbd-i2c. If my code does the wrong thing for some devices, that was already the case before. And this will certainly be easier to fix after my changes than before. On top of that, the "Hauppauge trick" is really only the order in which the addresses are probed. Just because a specific order is better for Hauppauge boards, doesn't mean it won't work for non-Hauppauge boards. > Also, unless > ir-kbd has suddenly improved, this will not work at all for HVR-1950 > class devices nor MCE type PVR-24xxx devices (different incompatible IR > receiver). I'm sorry but you can't blame me for ir-kbd-i2c not supporting some devices. I updated the driver to make use of the new binding model, but that's about all I did. > This is why the pvrusb2 driver has never directly attempted to load > ir-kbd. The pvrusb2 driver however abuses the bttv driver's I2C adapter ID (I2C_HW_B_BT848) and was thus affected when ir-kbd-i2c is loaded. This is the only reason why my patch touches the pvrusb2 driver. If you tell me you want the ir-kbd-i2c driver to leave pvrusb2 alone, I can drop all the related changes from my patch, that's very easy. -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
