Em 17-08-2012 12:04, Luis Henriques escreveu:
> (Adding Mauro to CC has he is the maintainer)
I'm actually waiting for Jarod's comments on it, as he wrote this driver
and the similar ones.
Jarod?
Regards,
Mauro
>
> 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
>
--
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
