Re: [PULL] http://linuxtv.org/hg/~awalls/v4l-dvb-misc

2010-01-16 Thread Mauro Carvalho Chehab
Andy Walls wrote:
>> The better is to rely on input_dev stuff, since they can easily be used by 
>> ir-core
>> sysfs to provide device naming for loading keytables from userspace during 
>> udev
>> handling.
> 
> OK, I don't see struct input_dev providing any storage for name and phys
> strings.  From vanilla-2.6.31-rc8/include/linux/input.h:
> 
>   struct input_dev {
>   const char *name;
>   const char *phys;
>   [...]
> 
> 
> and vanilla-2.6.31-rc8/drivers/input/input.c:input_register_device()
> doesn't do any allocation.  In fact it spits out useless default strings
> like "Unspecified device" if input_dev->name is NULL.
> 
> Also vanilla-2.6.31-rc8/drivers/input/input.c:input_devices_seq_show(),
> doesn't print any useful information if they are not set:
> 
>   seq_printf(seq, "N: Name=\"%s\"\n", dev->name ? dev->name : "");
>   seq_printf(seq, "P: Phys=%s\n", dev->phys ? dev->phys : "");
> 
> So I don't see where the input layer is providing anything: storage
> space nor automatic string generation.
> 
That's true, but the drivers should be dynamically allocating the memory:
ir->phys = kasprintf(GFP_KERNEL, "pci-%s/ir0", pci_name(pci));
or
ir->phys = kasprintf(GFP_KERNEL, "usb-%s-%s", dev->bus->bus_name,
  dev->devpath);

For the IR name, probably this would be enough:
ir->name = dev->name;

That's said, I agree that the better is to provide automatic string generation
at the ir-core, for both PCI and USB cases.

Of course, those names should be freed when unregistering the device.

> 
> Some of the V4L drivers use struct card_ir from
> linux/include/media/ir-common.h:
> 
>   struct card_ir {
>   struct input_dev*dev;
>   struct ir_input_state   ir;
>   charname[32];
>   charphys[32];
>   [...]

Yes, i know. Legacy code. We should really remove name/phys from there
and use the pointers at input.h. As input layer changed, we should have
better patched the IR drivers to not duplicate data.

> which has string storage that is too small for the card names in
> cx23885-cards.c and also has a group of fields that don't make a lot of
> sense for the RC-5 and NEC decoding state machines I've implemented.
> 
> In fact linux/drivers/media/video/cx88/cx88-input.c uses it's own struct
> cx88_IR instead of card_IR:
> 
>   struct cx88_IR {
>   struct cx88_core *core;
>   struct input_dev *input;
>   struct ir_input_state ir;
>   char name[32];
>   char phys[32];
>   [...]
> 
> 
> So what is it you want me to do with this change?  I don't know what
> storage space for strings you want me to reuse. 
> 
> Regards,
> Andy
> 
>> 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: [PULL] http://linuxtv.org/hg/~awalls/v4l-dvb-misc

2010-01-14 Thread Andy Walls
On Sun, 2010-01-10 at 11:52 -0200, Mauro Carvalho Chehab wrote:
> Andy Walls wrote:
> > On Sun, 2010-01-10 at 09:35 -0200, Mauro Carvalho Chehab wrote:
> >> Andy Walls wrote:
> >>> Mauro,
> >>>
> >>> If no one has any objections, please pull from
> >>>
> >>>  http://linuxtv.org/hg/~awalls/v4l-dvb-misc
> >>>
> >>> for the following 12 changesets.
> >>>
> >>> Of note:
> >>> 02-04 are from Jean Delvare and fix up the cx23885 i2c routines
> >>> 05-17 and 12 add and use a new v4l2_subdev core op for configuring I/O 
> >>> pin muxes
> >>> 08-10 are some minor cx23885 ir fixes noted when trying to get the TeVii 
> >>> S470 working
> >>>
> >>> 10/12: cx23885: Convert from struct card_ir to struct cx23885_ir_input 
> >>> for IR Rx
> >>> http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=aa62944baa92
> >> Hmm... This doesn't sound right:
> >>
> >> +struct cx23885_ir_input {
> >> +   struct input_dev*dev;
> >> +   struct ir_input_state   ir;
> >> +   charname[48];
> >> +   charphys[48];
> >> +
> >> +   /* Cooked code processing */
> >> +   int start;   /* Allowed start symbols */
> >> +   u32 addr;/* Expected remote address */
> >> +   u32 last_code;   /* last good cooked code seen 
> >> */
> >> +   int key_timeout; /* ms until we force a key up 
> >> */
> >> +   struct timer_list   timer_keyup; /* timer for key release */
> >> +
> >> +   /* Raw code collection and construction */
> >> +   int active; /* building code */
> >> +   int last_bit;   /* last bit seen */
> >> +   u32 code;   /* code under construction */
> >> +};
> >>
> >> Why are you creating a name[] and phys[] chars here? It should be using 
> >> the names already
> >> defined at struct input_dev.
> > 
> > Well two reasons:
> > 
> > 1. That's what the previous, common "card ir" struct did.  (Not a good
> > reason of course.)  When I needed to reimplement specific fields (in
> > anticipation of NEC decoding for the TeVii S470) I just carried them
> > over.
> > 
> > 2. The strings in the old card ir struct were too short: the card names
> > in cx23885-cards.c are pretty long and would get truncated.
> > 
> > 
> > I'll reexamine if the strings in input_dev are long enough to do the
> > job, and get back to you.
> 
> The better is to rely on input_dev stuff, since they can easily be used by 
> ir-core
> sysfs to provide device naming for loading keytables from userspace during 
> udev
> handling.

OK, I don't see struct input_dev providing any storage for name and phys
strings.  From vanilla-2.6.31-rc8/include/linux/input.h:

struct input_dev {
const char *name;
const char *phys;
[...]


and vanilla-2.6.31-rc8/drivers/input/input.c:input_register_device()
doesn't do any allocation.  In fact it spits out useless default strings
like "Unspecified device" if input_dev->name is NULL.

Also vanilla-2.6.31-rc8/drivers/input/input.c:input_devices_seq_show(),
doesn't print any useful information if they are not set:

seq_printf(seq, "N: Name=\"%s\"\n", dev->name ? dev->name : "");
seq_printf(seq, "P: Phys=%s\n", dev->phys ? dev->phys : "");

So I don't see where the input layer is providing anything: storage
space nor automatic string generation.


Some of the V4L drivers use struct card_ir from
linux/include/media/ir-common.h:

struct card_ir {
struct input_dev*dev;
struct ir_input_state   ir;
charname[32];
charphys[32];
[...]

which has string storage that is too small for the card names in
cx23885-cards.c and also has a group of fields that don't make a lot of
sense for the RC-5 and NEC decoding state machines I've implemented.

In fact linux/drivers/media/video/cx88/cx88-input.c uses it's own struct
cx88_IR instead of card_IR:

struct cx88_IR {
struct cx88_core *core;
struct input_dev *input;
struct ir_input_state ir;
char name[32];
char phys[32];
[...]


So what is it you want me to do with this change?  I don't know what
storage space for strings you want me to reuse. 

Regards,
Andy

> 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: [PULL] http://linuxtv.org/hg/~awalls/v4l-dvb-misc

2010-01-11 Thread Andy Walls
On Sun, 2010-01-10 at 11:52 -0200, Mauro Carvalho Chehab wrote:
> Andy Walls wrote:
> > On Sun, 2010-01-10 at 09:35 -0200, Mauro Carvalho Chehab wrote:
> >> Andy Walls wrote:
> >>> Mauro,
> >>>
> >>> If no one has any objections, please pull from
> >>>
> >>>  http://linuxtv.org/hg/~awalls/v4l-dvb-misc
> >>>
> >>> for the following 12 changesets.
> >>>
> >>> Of note:
> >>> 02-04 are from Jean Delvare and fix up the cx23885 i2c routines
> >>> 05-17 and 12 add and use a new v4l2_subdev core op for configuring I/O 
> >>> pin muxes
> >>> 08-10 are some minor cx23885 ir fixes noted when trying to get the TeVii 
> >>> S470 working
> >>>
> >>> 10/12: cx23885: Convert from struct card_ir to struct cx23885_ir_input 
> >>> for IR Rx
> >>> http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=aa62944baa92
> >> Hmm... This doesn't sound right:
> >>
> >> +struct cx23885_ir_input {
> >> +   struct input_dev*dev;
> >> +   struct ir_input_state   ir;
> >> +   charname[48];
> >> +   charphys[48];
> >> +
> >> +   /* Cooked code processing */
> >> +   int start;   /* Allowed start symbols */
> >> +   u32 addr;/* Expected remote address */
> >> +   u32 last_code;   /* last good cooked code seen 
> >> */
> >> +   int key_timeout; /* ms until we force a key up 
> >> */
> >> +   struct timer_list   timer_keyup; /* timer for key release */
> >> +
> >> +   /* Raw code collection and construction */
> >> +   int active; /* building code */
> >> +   int last_bit;   /* last bit seen */
> >> +   u32 code;   /* code under construction */
> >> +};
> >>
> >> Why are you creating a name[] and phys[] chars here? It should be using 
> >> the names already
> >> defined at struct input_dev.
> > 
> > Well two reasons:
> > 
> > 1. That's what the previous, common "card ir" struct did.  (Not a good
> > reason of course.)  When I needed to reimplement specific fields (in
> > anticipation of NEC decoding for the TeVii S470) I just carried them
> > over.
> > 
> > 2. The strings in the old card ir struct were too short: the card names
> > in cx23885-cards.c are pretty long and would get truncated.
> > 
> > 
> > I'll reexamine if the strings in input_dev are long enough to do the
> > job, and get back to you.
> 
> The better is to rely on input_dev stuff, since they can easily be used by 
> ir-core
> sysfs to provide device naming for loading keytables from userspace during 
> udev
> handling.

OK.  Hold off on that whole pull request.  That whole pull request was
the stable/ready part of what is in my cx23885-ir tree.

However, I just found what was wrong with my cx23885-ir tree code for IR
from an actual CX23885 chip. I'll just rework and update the whole
series, once I also get a change prepared and tested for the TeVii S470.

Regards,
Andy

> 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: [PULL] http://linuxtv.org/hg/~awalls/v4l-dvb-misc

2010-01-10 Thread Mauro Carvalho Chehab
Andy Walls wrote:
> On Sun, 2010-01-10 at 09:35 -0200, Mauro Carvalho Chehab wrote:
>> Andy Walls wrote:
>>> Mauro,
>>>
>>> If no one has any objections, please pull from
>>>
>>>  http://linuxtv.org/hg/~awalls/v4l-dvb-misc
>>>
>>> for the following 12 changesets.
>>>
>>> Of note:
>>> 02-04 are from Jean Delvare and fix up the cx23885 i2c routines
>>> 05-17 and 12 add and use a new v4l2_subdev core op for configuring I/O pin 
>>> muxes
>>> 08-10 are some minor cx23885 ir fixes noted when trying to get the TeVii 
>>> S470 working
>>>
>>> 10/12: cx23885: Convert from struct card_ir to struct cx23885_ir_input for 
>>> IR Rx
>>> http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=aa62944baa92
>> Hmm... This doesn't sound right:
>>
>> +struct cx23885_ir_input {
>> +   struct input_dev*dev;
>> +   struct ir_input_state   ir;
>> +   charname[48];
>> +   charphys[48];
>> +
>> +   /* Cooked code processing */
>> +   int start;   /* Allowed start symbols */
>> +   u32 addr;/* Expected remote address */
>> +   u32 last_code;   /* last good cooked code seen */
>> +   int key_timeout; /* ms until we force a key up */
>> +   struct timer_list   timer_keyup; /* timer for key release */
>> +
>> +   /* Raw code collection and construction */
>> +   int active; /* building code */
>> +   int last_bit;   /* last bit seen */
>> +   u32 code;   /* code under construction */
>> +};
>>
>> Why are you creating a name[] and phys[] chars here? It should be using the 
>> names already
>> defined at struct input_dev.
> 
> Well two reasons:
> 
> 1. That's what the previous, common "card ir" struct did.  (Not a good
> reason of course.)  When I needed to reimplement specific fields (in
> anticipation of NEC decoding for the TeVii S470) I just carried them
> over.
> 
> 2. The strings in the old card ir struct were too short: the card names
> in cx23885-cards.c are pretty long and would get truncated.
> 
> 
> I'll reexamine if the strings in input_dev are long enough to do the
> job, and get back to you.

The better is to rely on input_dev stuff, since they can easily be used by 
ir-core
sysfs to provide device naming for loading keytables from userspace during udev
handling.

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: [PULL] http://linuxtv.org/hg/~awalls/v4l-dvb-misc

2010-01-10 Thread Andy Walls
On Sun, 2010-01-10 at 09:35 -0200, Mauro Carvalho Chehab wrote:
> Andy Walls wrote:
> > Mauro,
> > 
> > If no one has any objections, please pull from
> > 
> >  http://linuxtv.org/hg/~awalls/v4l-dvb-misc
> > 
> > for the following 12 changesets.
> > 
> > Of note:
> > 02-04 are from Jean Delvare and fix up the cx23885 i2c routines
> > 05-17 and 12 add and use a new v4l2_subdev core op for configuring I/O pin 
> > muxes
> > 08-10 are some minor cx23885 ir fixes noted when trying to get the TeVii 
> > S470 working
> > 
> > 10/12: cx23885: Convert from struct card_ir to struct cx23885_ir_input for 
> > IR Rx
> > http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=aa62944baa92
> 
> Hmm... This doesn't sound right:
> 
> +struct cx23885_ir_input {
> +   struct input_dev*dev;
> +   struct ir_input_state   ir;
> +   charname[48];
> +   charphys[48];
> +
> +   /* Cooked code processing */
> +   int start;   /* Allowed start symbols */
> +   u32 addr;/* Expected remote address */
> +   u32 last_code;   /* last good cooked code seen */
> +   int key_timeout; /* ms until we force a key up */
> +   struct timer_list   timer_keyup; /* timer for key release */
> +
> +   /* Raw code collection and construction */
> +   int active; /* building code */
> +   int last_bit;   /* last bit seen */
> +   u32 code;   /* code under construction */
> +};
> 
> Why are you creating a name[] and phys[] chars here? It should be using the 
> names already
> defined at struct input_dev.

Well two reasons:

1. That's what the previous, common "card ir" struct did.  (Not a good
reason of course.)  When I needed to reimplement specific fields (in
anticipation of NEC decoding for the TeVii S470) I just carried them
over.

2. The strings in the old card ir struct were too short: the card names
in cx23885-cards.c are pretty long and would get truncated.


I'll reexamine if the strings in input_dev are long enough to do the
job, and get back to you.

Regards,
Andy

> 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: [PULL] http://linuxtv.org/hg/~awalls/v4l-dvb-misc

2010-01-10 Thread Mauro Carvalho Chehab
Andy Walls wrote:
> Mauro,
> 
> If no one has any objections, please pull from
> 
>  http://linuxtv.org/hg/~awalls/v4l-dvb-misc
> 
> for the following 12 changesets.
> 
> Of note:
> 02-04 are from Jean Delvare and fix up the cx23885 i2c routines
> 05-17 and 12 add and use a new v4l2_subdev core op for configuring I/O pin 
> muxes
> 08-10 are some minor cx23885 ir fixes noted when trying to get the TeVii S470 
> working
> 
> 10/12: cx23885: Convert from struct card_ir to struct cx23885_ir_input for IR 
> Rx
> http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=aa62944baa92

Hmm... This doesn't sound right:

+struct cx23885_ir_input {
+   struct input_dev*dev;
+   struct ir_input_state   ir;
+   charname[48];
+   charphys[48];
+
+   /* Cooked code processing */
+   int start;   /* Allowed start symbols */
+   u32 addr;/* Expected remote address */
+   u32 last_code;   /* last good cooked code seen */
+   int key_timeout; /* ms until we force a key up */
+   struct timer_list   timer_keyup; /* timer for key release */
+
+   /* Raw code collection and construction */
+   int active; /* building code */
+   int last_bit;   /* last bit seen */
+   u32 code;   /* code under construction */
+};

Why are you creating a name[] and phys[] chars here? It should be using the 
names already
defined at struct input_dev.

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


[PULL] http://linuxtv.org/hg/~awalls/v4l-dvb-misc

2010-01-01 Thread Andy Walls
On Sun, 2009-12-20 at 21:34 -0500, Andy Walls wrote:
> Mauro,
> 
> If no one has any objections, please pull from
> 
>  http://linuxtv.org/hg/~awalls/v4l-dvb-misc
> 
> for the following 12 changesets.
> 
> Of note:
> 02-04 are from Jean Delvare and fix up the cx23885 i2c routines
> 05-17 and 12 add and use a new v4l2_subdev core op for configuring I/O pin 
> muxes
> 08-10 are some minor cx23885 ir fixes noted when trying to get the TeVii S470 
> working
[snip]

Mauro,

I'm adding one patch to this pull request.  The new patch adds in kernel
IR Rx support for the HVR-1270 boards. (The HVR-1270 I have was sold as
a HVR-1250 but it has a CX23888 chip vs. a CX23885).

So, please pull from http://linuxtv.org/hg/~awalls/v4l-dvb-misc

for the following 13 changesets:

01/13: ivtv: Add a card entry to identify ProVideo PV-947D cards
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=a45c59b269d1

02/13: cx23885: Return -ENXIO on slave nack
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=e8788f94fe89

03/13: cx23885: Check for slave nack on all transactions
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=ce19c9683fd1

04/13: cx23885: i2c_wait_done returns 0 or 1, don't check for < 0 return value
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=714516d12a8f

05/13: v4l2_subdev: Add s_io_pin_config to v4l2_subdev_core_ops
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=08647cb79215

06/13: cx25840: Add s_io_pin_config core subdev ops for the CX2388[578]
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=bc272f3b7e15

07/13: cx23885: When using CX23888 IR, configure the IO pin mux IR pins 
explcitly
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=3f61484274ab

08/13: v4l2_subdev, cx23885: Differentiate IR carrier sense and I/O pin 
inversion
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=72507c5a253b

09/13: cx23885: Set IR Tx output pin drive properly for HVR-1850 and HVR-1290
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=bbb8e2feee04

10/13: cx23885: Convert from struct card_ir to struct cx23885_ir_input for IR Rx
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=aa62944baa92

11/13: cx23885: Add a v4l2_subdev group id for the CX2388[578] integrated AV 
core
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=cdb0930a7d0e

12/13: cx25840: Add an IO pad control for the IRQ out of the CX2388[578] AV Core
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=5df16bc74746

13/13: cx23885: Add IR Rx support for the HVR-1270 cards
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=7527decc9161


 drivers/media/video/cx23885/cx23885-cards.c |   33 +-
 drivers/media/video/cx23885/cx23885-i2c.c   |   27 ++--
 drivers/media/video/cx23885/cx23885-input.c |   50 ++---
 drivers/media/video/cx23885/cx23885.h   |   25 
 drivers/media/video/cx23885/cx23888-ir.c|   32 -
 drivers/media/video/cx25840/cx25840-core.c  |  153 
 drivers/media/video/ivtv/ivtv-cards.c   |   62 +++
 drivers/media/video/ivtv/ivtv-cards.h   |4 
 drivers/media/video/ivtv/ivtv-driver.c  |1 
 include/media/cx25840.h |   75 +
 include/media/v4l2-subdev.h |   28 -
 11 files changed, 447 insertions(+), 43 deletions(-)

Thanks,
Andy

--
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


[PULL] http://linuxtv.org/hg/~awalls/v4l-dvb-misc

2009-12-20 Thread Andy Walls
Mauro,

If no one has any objections, please pull from

 http://linuxtv.org/hg/~awalls/v4l-dvb-misc

for the following 12 changesets.

Of note:
02-04 are from Jean Delvare and fix up the cx23885 i2c routines
05-17 and 12 add and use a new v4l2_subdev core op for configuring I/O pin muxes
08-10 are some minor cx23885 ir fixes noted when trying to get the TeVii S470 
working

01/12: ivtv: Add a card entry to identify ProVideo PV-947D cards
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=a45c59b269d1

02/12: cx23885: Return -ENXIO on slave nack
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=e8788f94fe89

03/12: cx23885: Check for slave nack on all transactions
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=ce19c9683fd1

04/12: cx23885: i2c_wait_done returns 0 or 1, don't check for < 0 return value
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=714516d12a8f

05/12: v4l2_subdev: Add s_io_pin_config to v4l2_subdev_core_ops
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=08647cb79215

06/12: cx25840: Add s_io_pin_config core subdev ops for the CX2388[578]
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=bc272f3b7e15

07/12: cx23885: When using CX23888 IR, configure the IO pin mux IR pins 
explcitly
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=3f61484274ab

08/12: v4l2_subdev, cx23885: Differentiate IR carrier sense and I/O pin 
inversion
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=72507c5a253b

09/12: cx23885: Set IR Tx output pin drive properly for HVR-1850 and HVR-1290
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=bbb8e2feee04

10/12: cx23885: Convert from struct card_ir to struct cx23885_ir_input for IR Rx
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=aa62944baa92

11/12: cx23885: Add a v4l2_subdev group id for the CX2388[578] integrated AV 
core
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=cdb0930a7d0e

12/12: cx25840: Add an IO pad control for the IRQ out of the CX2388[578] AV Core
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=5df16bc74746


 drivers/media/video/cx23885/cx23885-cards.c |   28 -
 drivers/media/video/cx23885/cx23885-i2c.c   |   27 ++--
 drivers/media/video/cx23885/cx23885-input.c |   44 +---
 drivers/media/video/cx23885/cx23885.h   |   25 
 drivers/media/video/cx23885/cx23888-ir.c|   32 -
 drivers/media/video/cx25840/cx25840-core.c  |  153 
 drivers/media/video/ivtv/ivtv-cards.c   |   62 +++
 drivers/media/video/ivtv/ivtv-cards.h   |4 
 drivers/media/video/ivtv/ivtv-driver.c  |1 
 include/media/cx25840.h |   75 +
 include/media/v4l2-subdev.h |   28 -
 11 files changed, 437 insertions(+), 42 deletions(-)

Thanks.

Regards,
Andy



--
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


[PULL] http://linuxtv.org/hg/~awalls/v4l-dvb-misc

2009-11-21 Thread Andy Walls
Mauro,

Please pull from http://linuxtv.org/hg/~awalls/v4l-dvb-misc

for the following 7 changesets:

01/07: ivtv: Update the cards definitions to add another AVerMedia M113 variant
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=b7e065c84fc7

02/07: ivtv: Added FM radio support to Avermedia AVerTV MCE 116 Plus card
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=77af5cddacb5

03/07: ivtv: Add module parameter to adjust I2C SCL clock period per board
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=ab8a84fe29ff

04/07: ivtv: Defer legacy I2C IR probing until after setup of known I2C devices
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=0b2cae0e938f

05/07: ivtv: Add explicit IR controller initialization for the AVerTV M116
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=9ca00802e126

06/07: cx18: Use per cx18 instance init data for ir-kbd-i2c instead of const 
data
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=46e56e52b8ae

07/07: ivtv: Add more explicit detection of known IR devices for Hauppauge cards
http://linuxtv.org/hg/~awalls/v4l-dvb-misc?cmd=changeset;node=2659ce7e3123


 cx18/cx18-cards.h  |3 
 cx18/cx18-driver.h |7 ++
 cx18/cx18-i2c.c|   31 +++--
 ivtv/ivtv-cards.c  |   16 +++-
 ivtv/ivtv-cards.h  |   87 +++--
 ivtv/ivtv-driver.c |   25 +++
 ivtv/ivtv-driver.h |   10 ++
 ivtv/ivtv-i2c.c|  180 -
 ivtv/ivtv-i2c.h|1 
 9 files changed, 259 insertions(+), 101 deletions(-)

Thanks,
Andy

--
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