[mythtv-users] mce remote (lirc) fails after resume from standby

[Jarod Wilson]

On 06/30/2009 01:43 PM, Eloy Paris wrote:
> On Mon, Jun 29, 2009 at 11:52:36PM -0400, Jarod Wilson wrote:
>
>>>> >  >>root at altamira:/usr/src/lirc-0.8.4a/drivers/lirc_mceusb2#  diff -u lirc_mceusb2.c.orig lirc_mceusb2.c
>>>> >  >>--- lirc_mceusb2.c.orig 2009-01-28 16:56:52.000000000 -0500
>>>> >  >>+++ lirc_mceusb2.c      2009-01-28 16:29:26.000000000 -0500
>>>> >  >>@@ -1121,6 +1121,7 @@
>>>> >  >>  #if LINUX_VERSION_CODE>= KERNEL_VERSION(2, 6, 0)
>>>> >  >>         .suspend =      usb_remote_suspend,
>>>> >  >>         .resume =       usb_remote_resume,
>>>> >  >>+       .reset_resume = usb_remote_resume,
>>>> >  >>  #endif
>>>> >  >>         .id_table =     usb_remote_table
>>>> >  >>  };
>>>> >  >>
>>>> >  >>This was for lirc 0.8.4a. I just checked lirc 0.8.5 and they've
>>>> >  >>changed function names but the driver structure is still lacking a
>>>> >  >>.reset_resume field.
>>>> >  >>
>>>> >  >>I don't know if the above will fix your resume problem, but my
>>>> >  >>recommendation is that you try the above patch. As I said, resuming
>>> >  >>from S3 sleep by pressing the remote's power botton would not work
>>>> >  >>at all for me without the above little patch, and I've been running
>>>> >  >>for a few months with the above patch with no ill or side effects.
>> >
>> >  Eloy, would you mind submitting this upstream to the lirc list? I
>> >  can't recall if you already did and it was overlooked, or what, but
>> >  send it there again, and I'll keep an eye out for it and see if we
>> >  can't get it included properly.
>
> My apologies -- it's been in my to-do list to send this small patch to
> the LIRC maintainers but I keep neglecting to do that. I don't know if
> if the patch is correct, but without it LIRC stops working after S3
> resume for me. I'll submit it to the lirc list and let the maintainers
> decide if the patch makes sense, or if the problem needs to be handled
> in a different way.

Its in lirc cvs as of a few seconds ago, didn't see any reason not to 
put it in, since its confirmed to improve things for multiple users now.

-- 
Jarod Wilson
jarod at wilsonet.com