Re: [PATCH 0/6] Large scancode handling
On Wed, 8 Sep 2010, Jarod Wilson wrote: It'll conflict a little bith with the tivo slide patch I posted yesterday, but mostly just minor context changes. I can redo that patch on top of these changes if that's preferred. I can handle those context changes when merging the patches at linux-next and when merging upstream. We just need to sync in a way that Dmitry send his patch series before mine when sending them to Linus, and I'll take care of fixing the merge conflicts. Ah, the specific conflicts I was referring here are confined to drivers/hid/hid-input.c, and I sent the patch thinking it would go in via the hid tree. It *is* for a remote, but its a pure HID device in this case. Umm, what patch are you talking about please? I don't seem to have anything from you in my queue. Gah. I suck. Forgot to cc you on it. http://www.spinics.net/lists/linux-input/msg11007.html Can resend and/or bounce you a copy if need be. No need to resend, I'll dig it out from linux-input@ archives where I have overlooked it and will start a discussion in that thread if needed, so that we don't pollute this one. Thanks, -- Jiri Kosina SUSE Labs, Novell Inc. -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Wed, Sep 8, 2010 at 11:34 AM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: On Wed, Sep 08, 2010 at 10:31:21AM -0400, Jarod Wilson wrote: On Wed, Sep 08, 2010 at 12:41:38AM -0700, Dmitry Torokhov wrote: ... Ville, do you still have the hardware to try our ati_remote2 changes? If not, I've got the hardware. (Speaking of which, porting ati_remote* over to {ir,rc}-core is also on the TODO list, albeit with very very low priority at the moment). Ok, then we'' pencil you in for testing if we do not hear from Ville ;) We've heard from Ville, but I went ahead and did some testing anyway. My RWII with the default mode (didn't try any of the others) works just fine after applying this patch atop 2.6.36-rc3-git1, as do my imon and mceusb receivers. Ran into some problems with streamzap, but I'm fairly sure they're not the fault of this patchset. -- Jarod Wilson ja...@wilsonet.com -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Mon, Sep 13, 2010 at 01:48:14PM -0400, Jarod Wilson wrote: On Wed, Sep 8, 2010 at 11:34 AM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: On Wed, Sep 08, 2010 at 10:31:21AM -0400, Jarod Wilson wrote: On Wed, Sep 08, 2010 at 12:41:38AM -0700, Dmitry Torokhov wrote: ... Ville, do you still have the hardware to try our ati_remote2 changes? If not, I've got the hardware. (Speaking of which, porting ati_remote* over to {ir,rc}-core is also on the TODO list, albeit with very very low priority at the moment). Ok, then we'' pencil you in for testing if we do not hear from Ville ;) We've heard from Ville, but I went ahead and did some testing anyway. My RWII with the default mode (didn't try any of the others) works just fine after applying this patch atop 2.6.36-rc3-git1, as do my imon and mceusb receivers. Ran into some problems with streamzap, but I'm fairly sure they're not the fault of this patchset. Thank you Jarod, I'll add Tested-by: you then. -- Dmitry -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Wed, 8 Sep 2010, Dmitry Torokhov wrote: Hi Mauro, I guess I better get off my behind and commit the changes to support large scancodes, or they will not make to 2.6.37 either... There isn't much changes, except I followed David's suggestion and changed boolean index field into u8 flags field. Still, please glance it over once again and shout if you see something you do not like. Jiri, how do you want to handle the changes to HID? I could either push them through my tree together with the first patch or you can push through yours once the first change hits mainline. I think that there will unlikely be any conflict in .37 merge window in this area (and if there were, I'll sort it out). So please add Acked-by: Jiri Kosina jkos...@suse.cz to the hid-input.c bits and feel free to take it through your tree, if it is convenient for you. Thanks, -- Jiri Kosina SUSE Labs, Novell Inc. -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Wed, Sep 08, 2010 at 11:48:50AM +0200, Jiri Kosina wrote: On Wed, 8 Sep 2010, Dmitry Torokhov wrote: Hi Mauro, I guess I better get off my behind and commit the changes to support large scancodes, or they will not make to 2.6.37 either... There isn't much changes, except I followed David's suggestion and changed boolean index field into u8 flags field. Still, please glance it over once again and shout if you see something you do not like. Jiri, how do you want to handle the changes to HID? I could either push them through my tree together with the first patch or you can push through yours once the first change hits mainline. I think that there will unlikely be any conflict in .37 merge window in this area (and if there were, I'll sort it out). So please add Acked-by: Jiri Kosina jkos...@suse.cz to the hid-input.c bits and feel free to take it through your tree, if it is convenient for you. It'll conflict a little bith with the tivo slide patch I posted yesterday, but mostly just minor context changes. I can redo that patch on top of these changes if that's preferred. -- Jarod Wilson ja...@redhat.com -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Wed, Sep 08, 2010 at 12:41:38AM -0700, Dmitry Torokhov wrote: Hi Mauro, I guess I better get off my behind and commit the changes to support large scancodes, or they will not make to 2.6.37 either... There isn't much changes, except I followed David's suggestion and changed boolean index field into u8 flags field. Still, please glance it over once again and shout if you see something you do not like. Jiri, how do you want to handle the changes to HID? I could either push them through my tree together with the first patch or you can push through yours once the first change hits mainline. Mauro, the same question goes for media/IR patch. David, I suppose you still have the winbond remote so you could test changes to winbond-cir driver. This'll be fun. :) David recently posted a patchset that among other things, ports winbond-cir over to {ir,rc}-core to the tune of ~700 less lines in winbond-cir.c. It also touches a fair amount of core bits, as does a patchset posted by Maxim... I suspect this series should probably go in first though. Ville, do you still have the hardware to try our ati_remote2 changes? If not, I've got the hardware. (Speaking of which, porting ati_remote* over to {ir,rc}-core is also on the TODO list, albeit with very very low priority at the moment). -- Jarod Wilson ja...@redhat.com -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
Em 08-09-2010 11:24, Jarod Wilson escreveu: On Wed, Sep 08, 2010 at 11:48:50AM +0200, Jiri Kosina wrote: On Wed, 8 Sep 2010, Dmitry Torokhov wrote: Hi Mauro, I guess I better get off my behind and commit the changes to support large scancodes, or they will not make to 2.6.37 either... There isn't much changes, except I followed David's suggestion and changed boolean index field into u8 flags field. Still, please glance it over once again and shout if you see something you do not like. Jiri, how do you want to handle the changes to HID? I could either push them through my tree together with the first patch or you can push through yours once the first change hits mainline. I think that there will unlikely be any conflict in .37 merge window in this area (and if there were, I'll sort it out). So please add Acked-by: Jiri Kosina jkos...@suse.cz to the hid-input.c bits and feel free to take it through your tree, if it is convenient for you. It'll conflict a little bith with the tivo slide patch I posted yesterday, but mostly just minor context changes. I can redo that patch on top of these changes if that's preferred. I can handle those context changes when merging the patches at linux-next and when merging upstream. We just need to sync in a way that Dmitry send his patch series before mine when sending them to Linus, and I'll take care of fixing the merge conflicts. Cheers, mauro. -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Wed, Sep 08, 2010 at 12:15:06PM -0300, Mauro Carvalho Chehab wrote: Em 08-09-2010 11:24, Jarod Wilson escreveu: On Wed, Sep 08, 2010 at 11:48:50AM +0200, Jiri Kosina wrote: On Wed, 8 Sep 2010, Dmitry Torokhov wrote: Hi Mauro, I guess I better get off my behind and commit the changes to support large scancodes, or they will not make to 2.6.37 either... There isn't much changes, except I followed David's suggestion and changed boolean index field into u8 flags field. Still, please glance it over once again and shout if you see something you do not like. Jiri, how do you want to handle the changes to HID? I could either push them through my tree together with the first patch or you can push through yours once the first change hits mainline. I think that there will unlikely be any conflict in .37 merge window in this area (and if there were, I'll sort it out). So please add Acked-by: Jiri Kosina jkos...@suse.cz to the hid-input.c bits and feel free to take it through your tree, if it is convenient for you. It'll conflict a little bith with the tivo slide patch I posted yesterday, but mostly just minor context changes. I can redo that patch on top of these changes if that's preferred. I can handle those context changes when merging the patches at linux-next and when merging upstream. We just need to sync in a way that Dmitry send his patch series before mine when sending them to Linus, and I'll take care of fixing the merge conflicts. Ah, the specific conflicts I was referring here are confined to drivers/hid/hid-input.c, and I sent the patch thinking it would go in via the hid tree. It *is* for a remote, but its a pure HID device in this case. -- Jarod Wilson ja...@redhat.com -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
Em 08-09-2010 04:41, Dmitry Torokhov escreveu: Hi Mauro, I guess I better get off my behind and commit the changes to support large scancodes, or they will not make to 2.6.37 either... There isn't much changes, except I followed David's suggestion and changed boolean index field into u8 flags field. Still, please glance it over once again and shout if you see something you do not like. Jiri, how do you want to handle the changes to HID? I could either push them through my tree together with the first patch or you can push through yours once the first change hits mainline. Mauro, the same question goes for media/IR patch. It seems easier if you apply them on your tree. I'm sure we won't have any major conflicts, if we don't apply the big ir-rc renaming patch. I'll postpone such patch to be applied after the merge upstream. The patches look sane to me. Acked-by: Mauro Carvalho Chehab mche...@redhat.com Cheers, Mauro -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Wed, Sep 08, 2010 at 10:31:21AM -0400, Jarod Wilson wrote: On Wed, Sep 08, 2010 at 12:41:38AM -0700, Dmitry Torokhov wrote: Hi Mauro, I guess I better get off my behind and commit the changes to support large scancodes, or they will not make to 2.6.37 either... There isn't much changes, except I followed David's suggestion and changed boolean index field into u8 flags field. Still, please glance it over once again and shout if you see something you do not like. Jiri, how do you want to handle the changes to HID? I could either push them through my tree together with the first patch or you can push through yours once the first change hits mainline. Mauro, the same question goes for media/IR patch. David, I suppose you still have the winbond remote so you could test changes to winbond-cir driver. This'll be fun. :) David recently posted a patchset that among other things, ports winbond-cir over to {ir,rc}-core to the tune of ~700 less lines in winbond-cir.c. It also touches a fair amount of core bits, as does a patchset posted by Maxim... I suspect this series should probably go in first though. Hmm, the only reason for conversion is that I want to do the back converstion get/setkeycode_new-get/setkeycode sometimes around 2.6.37-rc2. So it should be perfectly fine for David's changes to go in and I will just drop my patch completely. Ville, do you still have the hardware to try our ati_remote2 changes? If not, I've got the hardware. (Speaking of which, porting ati_remote* over to {ir,rc}-core is also on the TODO list, albeit with very very low priority at the moment). Ok, then we'' pencil you in for testing if we do not hear from Ville ;) -- Dmitry -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Wed, Sep 08, 2010 at 05:25:13PM +0200, Jiri Kosina wrote: On Wed, 8 Sep 2010, Jarod Wilson wrote: It'll conflict a little bith with the tivo slide patch I posted yesterday, but mostly just minor context changes. I can redo that patch on top of these changes if that's preferred. I can handle those context changes when merging the patches at linux-next and when merging upstream. We just need to sync in a way that Dmitry send his patch series before mine when sending them to Linus, and I'll take care of fixing the merge conflicts. Ah, the specific conflicts I was referring here are confined to drivers/hid/hid-input.c, and I sent the patch thinking it would go in via the hid tree. It *is* for a remote, but its a pure HID device in this case. Umm, what patch are you talking about please? I don't seem to have anything from you in my queue. Depending on the extent of the patch in question I could take it through my tree as well to avoid too much interdependencies between trees at merge window time... -- Dmitry -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Wed, Sep 08, 2010 at 11:48:50AM +0200, Jiri Kosina wrote: On Wed, 8 Sep 2010, Dmitry Torokhov wrote: Hi Mauro, I guess I better get off my behind and commit the changes to support large scancodes, or they will not make to 2.6.37 either... There isn't much changes, except I followed David's suggestion and changed boolean index field into u8 flags field. Still, please glance it over once again and shout if you see something you do not like. Jiri, how do you want to handle the changes to HID? I could either push them through my tree together with the first patch or you can push through yours once the first change hits mainline. I think that there will unlikely be any conflict in .37 merge window in this area (and if there were, I'll sort it out). So please add Acked-by: Jiri Kosina jkos...@suse.cz to the hid-input.c bits and feel free to take it through your tree, if it is convenient for you. Thanks Jiri. -- Dmitry -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Wed, Sep 08, 2010 at 05:25:13PM +0200, Jiri Kosina wrote: On Wed, 8 Sep 2010, Jarod Wilson wrote: It'll conflict a little bith with the tivo slide patch I posted yesterday, but mostly just minor context changes. I can redo that patch on top of these changes if that's preferred. I can handle those context changes when merging the patches at linux-next and when merging upstream. We just need to sync in a way that Dmitry send his patch series before mine when sending them to Linus, and I'll take care of fixing the merge conflicts. Ah, the specific conflicts I was referring here are confined to drivers/hid/hid-input.c, and I sent the patch thinking it would go in via the hid tree. It *is* for a remote, but its a pure HID device in this case. Umm, what patch are you talking about please? I don't seem to have anything from you in my queue. Gah. I suck. Forgot to cc you on it. http://www.spinics.net/lists/linux-input/msg11007.html Can resend and/or bounce you a copy if need be. -- Jarod Wilson ja...@redhat.com -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Wed, Sep 08, 2010 at 12:09:08PM -0400, Jarod Wilson wrote: On Wed, Sep 08, 2010 at 05:25:13PM +0200, Jiri Kosina wrote: On Wed, 8 Sep 2010, Jarod Wilson wrote: It'll conflict a little bith with the tivo slide patch I posted yesterday, but mostly just minor context changes. I can redo that patch on top of these changes if that's preferred. I can handle those context changes when merging the patches at linux-next and when merging upstream. We just need to sync in a way that Dmitry send his patch series before mine when sending them to Linus, and I'll take care of fixing the merge conflicts. Ah, the specific conflicts I was referring here are confined to drivers/hid/hid-input.c, and I sent the patch thinking it would go in via the hid tree. It *is* for a remote, but its a pure HID device in this case. Umm, what patch are you talking about please? I don't seem to have anything from you in my queue. Gah. I suck. Forgot to cc you on it. http://www.spinics.net/lists/linux-input/msg11007.html Can resend and/or bounce you a copy if need be. Hmm, I do not see anything in there that would conflict with my changes... -- Dmitry -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/6] Large scancode handling
On Wed, Sep 08, 2010 at 09:56:07AM -0700, Dmitry Torokhov wrote: On Wed, Sep 08, 2010 at 12:09:08PM -0400, Jarod Wilson wrote: On Wed, Sep 08, 2010 at 05:25:13PM +0200, Jiri Kosina wrote: On Wed, 8 Sep 2010, Jarod Wilson wrote: It'll conflict a little bith with the tivo slide patch I posted yesterday, but mostly just minor context changes. I can redo that patch on top of these changes if that's preferred. I can handle those context changes when merging the patches at linux-next and when merging upstream. We just need to sync in a way that Dmitry send his patch series before mine when sending them to Linus, and I'll take care of fixing the merge conflicts. Ah, the specific conflicts I was referring here are confined to drivers/hid/hid-input.c, and I sent the patch thinking it would go in via the hid tree. It *is* for a remote, but its a pure HID device in this case. Umm, what patch are you talking about please? I don't seem to have anything from you in my queue. Gah. I suck. Forgot to cc you on it. http://www.spinics.net/lists/linux-input/msg11007.html Can resend and/or bounce you a copy if need be. Hmm, I do not see anything in there that would conflict with my changes... Sorry, yeah, it *should* be solely context conflicts due to line offsets and other misc drift, no functional conflicts with your changes. -- Jarod Wilson ja...@redhat.com -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html