(Adding Mauro to CC has he is the maintainer) On Thu, Aug 16, 2012 at 10:09:32AM +0200, Matthijs Kooijman wrote: > Hi folks, > > > I'm currently compiling a 3.5 kernel with just the rdev initialization > > moved up to see if this will fix my problem at all, but I'd like your > > view on this in the meantime as well. > Ok, this seems to fix my problem: > > --- a/drivers/media/rc/nuvoton-cir.c > +++ b/drivers/media/rc/nuvoton-cir.c > @@ -1066,6 +1066,7 @@ > /* tx bits */ > rdev->tx_resolution = XYZ; > #endif > + nvt->rdev = rdev; This makes sense to me. Note however that there are more drivers with a similar problem (e.g., fintek-cir.c). > > ret = -EBUSY; /* now claim resources */ @@ -1090,7 +1091,6 > @@ goto failure5; > > device_init_wakeup(&pdev->dev, true); > - nvt->rdev = rdev; > nvt_pr(KERN_NOTICE, "driver has been successfully loaded\n"); > if (debug) { > cir_dump_regs(nvt); > > > I'm still not sure if the rc_register_device shouldn't also be moved up. It > seems this doesn't trigger a problem right now, but if there is a problem, I > suspect its trigger window is a lot smaller than with the rdev initialization > problem... I'm not sure as well, I'm not very familiar with this code. However, it looks like the IRQ request should actually be one of the last things to do here. Cheers, -- Luis -- 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