On Jan 10, 2011, at 2:50 PM, Jarod Wilson wrote: > On Jan 10, 2011, at 9:25 AM, Jason Gauthier wrote: > >>>> >>>> I did simply try changing: >>>> >>>> /* until i2c is working properly */ >>>> retval = 0; /* hdpvr_register_i2c_ir(dev); */ >>>> if (retval < 0) >>>> >>>> so that it would register with i2c. >>>> Doing so returns a positive registration with I2C, but the lirc_zilog >>>> driver doesn't see the chip when it loads. (The lirc_zilog is now in >>>> the kernel, yay) >> >>> There's a bit more to it than just the one line change. Here's the patch we're carrying in the Fedora kernels to enable it: >> >>> http://wilsonet.com/jarod/lirc_misc/hdpvr-ir/hdpvr-ir-enable.patch >> >>> Janne, I've heard many success stories w/the hdpvr IR lately, and almost no reports of lockups, so I'm thinking a firmware update may have helped out >here, and thus, maybe its time we just go ahead and push this patch along upstream? We still require someone to load lirc_zilog manually, so it seems like a >fairly low-risk thing to do. >> >> Thanks. What source tree is this against? I see the patch is dated 09/2009. Manually comparing to my .37 source tree it does not appear to match up. >> I don't mind bringing in a third source tree to compare against to see if I can make this work on .37, I just don't know which one! > > Bah. Yeah, sorry, that wasn't the current patch in Fedora 14. This is: > > http://wilsonet.com/jarod/lirc_misc/hdpvr-ir/hdpvr-ir-enable-2.patch > > Its atop the F14 2.6.35.10 kernel, which has a fairly recent v4l/dvb > backport on top of it, so it should be pretty close to matching the > current v4l/dvb code... With the help of Andy Walls and Jean Delvare, I think we've hashed out an updated patch that will work sitting atop the current v4l/dvb hdpvr code, but I'm only just now getting around to compile-testing it, and its past my bedtime, so it'll be tomorrow before I can do any sort of functional testing (but hey, due to the snow, I'll be working from home tomorrow, where my hdpvr happens to be...). -- Jarod Wilson jarod@xxxxxxxxxxxx -- 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
