Re: [git:v4l-dvb/for_v3.7] [media] media: rc: Introduce RX51 IR transmitter driver

2012-08-16 Thread Timo Kokkonen
On 08/16/12 19:34, Sakari Ailus wrote: > Hi Sebastian, > > On Thu, Aug 16, 2012 at 01:21:04PM +0200, Sebastian Reichel wrote: >> Hi, >> It was an requirement back then that this driver needs to be a module as 99% of the N900 owners still don't even know they have this kind of capabi

Re: [git:v4l-dvb/for_v3.7] [media] media: rc: Introduce RX51 IR transmitter driver

2012-08-16 Thread Sakari Ailus
Hi Sebastian, On Thu, Aug 16, 2012 at 01:21:04PM +0200, Sebastian Reichel wrote: > Hi, > > > > It was an requirement back then that this driver needs to be a module as > > > 99% of the N900 owners still don't even know they have this kind of > > > capability on their devices, so it doesn't make s

Re: [git:v4l-dvb/for_v3.7] [media] media: rc: Introduce RX51 IR transmitter driver

2012-08-16 Thread Sebastian Reichel
Hi, > > It was an requirement back then that this driver needs to be a module as > > 99% of the N900 owners still don't even know they have this kind of > > capability on their devices, so it doesn't make sense to keep the module > > loaded unless the user actually needs it. > > I don't think tha

Re: [git:v4l-dvb/for_v3.7] [media] media: rc: Introduce RX51 IR transmitter driver

2012-08-16 Thread Sakari Ailus
Moi, On Wed, Aug 15, 2012 at 10:46:43PM +0300, Timo Kokkonen wrote: > Long time no see. Indeed! Better late than never I guess. :) > On 08/15/12 19:06, Sakari Ailus wrote: > > Heippa, > > > > Thanks for the patch! I know Mauro has already applied this so any changes > > would make a separate pa

Re: [git:v4l-dvb/for_v3.7] [media] media: rc: Introduce RX51 IR transmitter driver

2012-08-15 Thread Timo Kokkonen
Terve Sakari, Long time no see. On 08/15/12 19:06, Sakari Ailus wrote: > Heippa, > > Thanks for the patch! I know Mauro has already applied this so any changes > would make a separate patch. > Patches are cheap :) I'll have also couple of changes from Sean's comments. > I have tested this up

Re: [git:v4l-dvb/for_v3.7] [media] media: rc: Introduce RX51 IR transmitter driver

2012-08-15 Thread Sakari Ailus
Heippa, Thanks for the patch! I know Mauro has already applied this so any changes would make a separate patch. I have tested this up to the point I can see that the IR LED blinks --- using my phone's camera. :-) But I have no receivers so the testing ends to this. On Mon, Aug 13, 2012 at 09:53