Re: v4l2_subdev GPIO and Pin Control ops (Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T)

2009-06-29 Thread Steven Toth
On 6/26/09 2:19 PM, Andy Walls wrote: On Fri, 2009-06-26 at 12:25 -0400, Steven Toth wrote: On 6/26/09 12:12 PM, Andy Walls wrote: My plan was to add the necessary support to the cx25840 module for setting up the cx23885 pin control multiplexers (subdev config time), the GPIO 23-19 directions

Re: v4l2_subdev GPIO and Pin Control ops (Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T)

2009-06-26 Thread Andy Walls
On Fri, 2009-06-26 at 12:25 -0400, Steven Toth wrote: > On 6/26/09 12:12 PM, Andy Walls wrote: > > On Fri, 2009-06-26 at 10:37 -0400, Steven Toth wrote: > >> On 6/25/09 7:15 AM, Andy Walls wrote: > >>> On Thu, 2009-06-25 at 08:39 +0200, Hans Verkuil wrote: > On Thursday 25 June 2009 04:40:11 A

Re: v4l2_subdev GPIO and Pin Control ops (Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T)

2009-06-26 Thread Andy Walls
On Fri, 2009-06-26 at 12:25 -0400, Steven Toth wrote: > On 6/26/09 12:12 PM, Andy Walls wrote: > > > > My plan was to add the necessary support to the cx25840 module for > > setting up the cx23885 pin control multiplexers (subdev config time), > > the GPIO 23-19 directions (subdev config time), an

Re: v4l2_subdev GPIO and Pin Control ops (Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T)

2009-06-26 Thread Steven Toth
On 6/26/09 12:12 PM, Andy Walls wrote: On Fri, 2009-06-26 at 10:37 -0400, Steven Toth wrote: On 6/25/09 7:15 AM, Andy Walls wrote: On Thu, 2009-06-25 at 08:39 +0200, Hans Verkuil wrote: On Thursday 25 June 2009 04:40:11 Andy Walls wrote: On Tue, 2009-06-23 at 14:33 +0200, Hans Verkuil wrote:

Re: v4l2_subdev GPIO and Pin Control ops (Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T)

2009-06-26 Thread Andy Walls
On Fri, 2009-06-26 at 10:37 -0400, Steven Toth wrote: > On 6/25/09 7:15 AM, Andy Walls wrote: > > On Thu, 2009-06-25 at 08:39 +0200, Hans Verkuil wrote: > >> On Thursday 25 June 2009 04:40:11 Andy Walls wrote: > >>> On Tue, 2009-06-23 at 14:33 +0200, Hans Verkuil wrote: > > On Tue, 2009-06-23 a

Re: v4l2_subdev GPIO and Pin Control ops (Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T)

2009-06-26 Thread Steven Toth
On 6/25/09 7:15 AM, Andy Walls wrote: On Thu, 2009-06-25 at 08:39 +0200, Hans Verkuil wrote: On Thursday 25 June 2009 04:40:11 Andy Walls wrote: On Tue, 2009-06-23 at 14:33 +0200, Hans Verkuil wrote: On Tue, 2009-06-23 at 11:39 +0800, Terry Wu wrote: There is already an s_gpio in the core op

Re: v4l2_subdev GPIO and Pin Control ops (Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T)

2009-06-25 Thread Andy Walls
On Thu, 2009-06-25 at 08:39 +0200, Hans Verkuil wrote: > On Thursday 25 June 2009 04:40:11 Andy Walls wrote: > > On Tue, 2009-06-23 at 14:33 +0200, Hans Verkuil wrote: > > > > On Tue, 2009-06-23 at 11:39 +0800, Terry Wu wrote: > > > > > > > > > There is already an s_gpio in the core ops. It would

Re: v4l2_subdev GPIO and Pin Control ops (Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T)

2009-06-24 Thread Hans Verkuil
On Thursday 25 June 2009 04:40:11 Andy Walls wrote: > On Tue, 2009-06-23 at 14:33 +0200, Hans Verkuil wrote: > > > On Tue, 2009-06-23 at 11:39 +0800, Terry Wu wrote: > > > > > > There is already an s_gpio in the core ops. It would be simple to add a > > g_gpio as well if needed. > > Hans, > > As

v4l2_subdev GPIO and Pin Control ops (Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T)

2009-06-24 Thread Andy Walls
On Tue, 2009-06-23 at 14:33 +0200, Hans Verkuil wrote: > > On Tue, 2009-06-23 at 11:39 +0800, Terry Wu wrote: > > > There is already an s_gpio in the core ops. It would be simple to add a > g_gpio as well if needed. Hans, As you probably know int (*s_gpio)(v4l2_subdev *sd, u32 val); is

Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T

2009-06-23 Thread Andy Walls
On Tue, 2009-06-23 at 21:10 +0800, Terry Wu wrote: > 2. Also, I can not find GPIO functions in the cx25840-core.c > > Something missing or unfinished ? Missing: they are not implemented in the cx25840 driver. I will at least implement a change to the cx25840 module so the v4l bridge driver (cx

Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T

2009-06-23 Thread Andy Walls
On Tue, 2009-06-23 at 14:33 +0200, Hans Verkuil wrote: > > On Tue, 2009-06-23 at 11:39 +0800, Terry Wu wrote: > >> Hi, > >> > >> I add the following codes in the cx23885_initialize() of > >> cx25840-core.c: > >>/* Drive GPIO2 (GPIO 19~23) direction and values for DVB-T */ > >>cx25840_an

Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T

2009-06-23 Thread Terry Wu
Hi, 1. The CX23885's < GPIO 23 thru 19 - On the cx25840 a/v core> is not implemented yet. You can find the following codes in the cx23885-core.c : /* Mask represents 32 different GPIOs, GPIO's are split into multiple * registers depending on the board configuration (and whether the * 417 encoder

Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T

2009-06-23 Thread Hans Verkuil
> On Tue, 2009-06-23 at 11:39 +0800, Terry Wu wrote: >> Hi, >> >> I add the following codes in the cx23885_initialize() of >> cx25840-core.c: >> /* Drive GPIO2 (GPIO 19~23) direction and values for DVB-T */ >> cx25840_and_or(client, 0x160, 0x1d, 0x00); >> cx25840_write(client, 0

Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T

2009-06-23 Thread claude
Terry Wu a écrit : Hi, I add the following codes in the cx23885_initialize() of cx25840-core.c: /* Drive GPIO2 (GPIO 19~23) direction and values for DVB-T */ cx25840_and_or(client, 0x160, 0x1d, 0x00); cx25840_write(client, 0x164, 0x00); Before that, the tuning st

Re: PxDVR3200 H LinuxTV v4l-dvb patch : Pull GPIO-20 low for DVB-T

2009-06-23 Thread Andy Walls
On Tue, 2009-06-23 at 11:39 +0800, Terry Wu wrote: > Hi, > > I add the following codes in the cx23885_initialize() of cx25840-core.c: > /* Drive GPIO2 (GPIO 19~23) direction and values for DVB-T */ > cx25840_and_or(client, 0x160, 0x1d, 0x00); > cx25840_write(client, 0x164, 0x