<html><head><style type="text/css"><!-- DIV {margin:0px;} --></style></head><body><div style="font-family:times new roman, new york, times, serif;font-size:12pt"><DIV style="FONT-SIZE: 12pt; FONT-FAMILY: times new roman, new york, times, serif">Here&nbsp;is the polling patch from Henry....&nbsp;&nbsp; Does this improve the responsiveness of the remote.</DIV>
<DIV style="FONT-SIZE: 12pt; FONT-FAMILY: times new roman, new york, times, serif">Personally, I never had a problem with the default values that I had..<IMG src="http://us.i1.yimg.com/us.yimg.com/i/mesg/tsmileys2/04.gif"></DIV>
<DIV style="FONT-SIZE: 12pt; FONT-FAMILY: times new roman, new york, times, serif">&nbsp;</DIV>
<DIV style="FONT-SIZE: 12pt; FONT-FAMILY: times new roman, new york, times, serif">&nbsp;<BR></DIV>
<DIV style="FONT-SIZE: 12pt; FONT-FAMILY: times new roman, new york, times, serif">----- Forwarded Message ----<BR>From: Henry Wong &lt;v4l@stuffedcow.net&gt;<BR>To: Linux and Kernel Video &lt;video4linux-list@redhat.com&gt;<BR>Sent: Sunday, January 21, 2007 4:16:07 AM<BR>Subject: Re: [linux-dvb] 80th Attempt at Kworld ATSC110 IR Remote Patch - Please Review<BR><BR>
<DIV>I noticed some comments about the remote being "sluggish". <BR><BR>In my original patch, there were some changes to increase the GPIO polling rate for the TV@nywhere plus remote. I don't clearly recall the exact reason why I had to do it, but attached is what I wrote in my original post, and the relevant portions of my patch.<BR><BR>I didn't notice any changes in your patch that would change the polling interval? I'm not sure.<BR><BR>Could this be the cause of "sluggishness"?<BR><BR><BR><BR>-------------------------------------------------------------------------<BR>Excepts from "[patch] MSI TV@nywhere Plus IR remote", December 11, 2005:<BR><BR><BR><BR><BR>1. The IR controller doesn't like the default 100ms I2C polling <BR>interval. (See&nbsp;&nbsp;static void ir_work (void*)&nbsp;&nbsp;in ir-kbd-i2c.c)<BR><BR>I have added a static int polling_interval variable to ir-kbd-i2c.c. <BR>Objections?<BR><BR><BR><BR><BR>Index:
 linux/drivers/media/video/ir-kbd-i2c.c<BR>===================================================================<BR>RCS file: /cvs/video4linux/v4l-dvb/linux/drivers/media/video/ir-kbd-i2c.c,v<BR>retrieving revision 1.30<BR>diff -u -r1.30 ir-kbd-i2c.c<BR>--- linux/drivers/media/video/ir-kbd-i2c.c&nbsp;&nbsp;&nbsp;&nbsp;8 Dec 2005 21:02:56 -0000&nbsp;&nbsp;&nbsp;&nbsp;1.30<BR>+++ linux/drivers/media/video/ir-kbd-i2c.c&nbsp;&nbsp;&nbsp;&nbsp;12 Dec 2005 01:03:52 -0000<BR>@@ -95,6 +95,8 @@<BR>#define dprintk(level, fmt, arg...)&nbsp;&nbsp;&nbsp;&nbsp;if (debug &gt;= level) \<BR>&nbsp;&nbsp;&nbsp;&nbsp;printk(KERN_DEBUG DEVNAME ": " fmt , ## arg)<BR><BR>+static int polling_interval = 100; /* ms */<BR>+<BR>/* ----------------------------------------------------------------------- */<BR><BR>static int get_key_haup(struct IR_i2c *ir, u32 *ir_key, u32 *ir_raw)<BR>@@ -269,7 +271,7 @@<BR>{<BR>&nbsp;&nbsp;&nbsp;&nbsp;struct IR_i2c *ir =
 data;<BR>&nbsp;&nbsp;&nbsp;&nbsp;ir_key_poll(ir);<BR>-&nbsp;&nbsp;&nbsp;&nbsp;mod_timer(&amp;ir-&gt;timer, jiffies+HZ/10);<BR>+&nbsp;&nbsp;&nbsp;&nbsp;mod_timer(&amp;ir-&gt;timer, jiffies + polling_interval*HZ/1000);<BR>}<BR><BR>/* ----------------------------------------------------------------------- */<BR>@@ -340,6 +342,9 @@<BR>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;ir_codes&nbsp;&nbsp;&nbsp;&nbsp;= ir_codes_rc5_tv;<BR>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;break;<BR>&nbsp;&nbsp;&nbsp;&nbsp;case 0x30:<BR>+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if (adap-&gt;id == I2C_HW_SAA7134) /* Handled by saa7134-input */<BR>+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;polling_interval = 50; /* ms */<BR>+<BR>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;name&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;= "KNC One";<BR>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;ir-&gt;get_key =
 get_key_knc1;<BR>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;ir_type&nbsp;&nbsp;&nbsp;&nbsp; = IR_TYPE_OTHER;<BR><BR>--<BR>video4linux-list mailing list<BR>Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe<BR><A href="https://www.redhat.com/mailman/listinfo/video4linux-list" target=_blank>https://www.redhat.com/mailman/listinfo/video4linux-list</A></DIV></DIV>
<DIV style="FONT-SIZE: 12pt; FONT-FAMILY: times new roman, new york, times, serif"><BR></DIV></div></body></html>