<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"><style type="text/css"><!-- DIV {margin:0px;}--></style><div style="font-family: times new roman,new york,times,serif; font-size: 12pt;"><div style="font-family: times new roman,new york,times,serif; font-size: 12pt;">Finished testing the remote under Windows.&nbsp;&nbsp; The repeat function does not work under Windows either.&nbsp; So I'm going to remove the repeat key check in the patch.&nbsp;&nbsp; Weird....&nbsp; It's the first time I have seen a remote not respond when a key is held.<br><br>Michael, I'll fix up the white space, repeat key and the two remote keys missing&nbsp;issues then submit&nbsp; a signed-off patch.<br><br>Now, can anyone suggest a workaround for the KNC One device which share the same 0x30 value?&nbsp; The patch in it's existing form is going to break the KNC One device.&nbsp;
 In&nbsp;ir-kbd-i2c.c with&nbsp;ir_attach the KNC One device is already using 0x30.&nbsp; I'm not sure how to handle this situation.&nbsp; I took a
 look at the rest of ir-kbd-i2c.c and the situation does not come up.<br><br>case 0x30:<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;name&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;= "Kworld ATSC110";<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ir_type&nbsp;&nbsp;&nbsp;&nbsp; = IR_TYPE_OTHER;<br>-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;ir_codes&nbsp;&nbsp;&nbsp;&nbsp;= ir_codes_empty;<br>+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;ir_codes&nbsp;&nbsp;&nbsp;&nbsp;= ir_codes_kworld_atsc110;<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; break;<br><br><div style="font-family: times new roman,new york,times,serif; font-size: 12pt;">----- Original Message ----<br>From: Michael Krufky &lt;mkrufky@linuxtv.org&gt;<br>To: Linux and Kernel Video
 &lt;video4linux-list@redhat.com&gt;<br>Cc: Dwaine Garden &lt;dwainegarden@rogers.com&gt;; Bob &lt;yagobob@gmail.com&gt;<br>Sent: Friday, September 22, 2006 10:55:13 AM<br>Subject: Re: Kworld HDTV ATSC110 IR-REMOTE Patch<br><br><div>Dwaine Garden wrote:<br>&gt; I clean up the original patch that was submitted a couple of weeks ago.&nbsp;&nbsp;&nbsp;&nbsp;Missing where two channels.&nbsp;&nbsp;I have added them.<br>&gt; All the keys have been confirmed working.&nbsp;&nbsp; The only outstanding issue would be the repeating key problem.<br>&gt; <br>&gt; I still have an issue with the patch because the KNC1 device has the same values as this device.&nbsp;&nbsp;So the get_key_knc1 generates<br>&gt; a warning becasue the case statement was altered for the kworld atsc110 device.<br>&gt; <br>[patch snipped]<br><br>Bob wrote:<br>&gt; good job, dwaine.&nbsp;&nbsp;i haven't looked at the code, but isn't there some<br>&gt; workaround to support both knc1 and atsc110?<br>&gt;<br>&gt; as
 for repeating keys, one could consider this a feature 'request',<br>&gt; not a bug, and it could be added at a later date.&nbsp;&nbsp;i'm worried that if<br>&gt; this atsc110 remote support doesn't get checked in soon, it might get<br>&gt; lost......<br><br><br>Patches without sign-off's will not get applied.&nbsp;&nbsp;If the patch gets<br>resubmitted with a description and sign-off within the next few days,<br>then it will still be in time for 2.6.19 ...<br><br>There are also whitespace problems in this patch.&nbsp;&nbsp;It would be nice for<br>those to be taken care of as well, but I would be quicker to fix<br>something like that myself, as opposed to letting the patch get lost<br>forever.&nbsp;&nbsp;Even still, my hands are tied without an S-O-B.<br><br>Cheers,<br><br>Mike<br>-- <br>Michael Krufky<br><br></div></div><br></div></div></div></body></html>