Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-28 Thread Oliver Endriss
Johannes Stezenbach wrote: On Sun, Aug 19, 2007, Oliver Endriss wrote: Questions: - Why should dvb_shutdown_timeout==0 disable sleep mode? The use case was to watch video without any software running. Just program the hardware once and let it do it's job. Some people want that

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-19 Thread Oliver Endriss
Manu Abraham wrote: Oliver Endriss wrote: Does anyone know, why dvb_shutdown_timeout was introduced initially? It was originally introduced looong time back by Obi. Some operations still incomplete on close() was the reason stated, IIRC. It had something to do with the FF cards ?, dunno.

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-19 Thread Andreas Oberritter
Manu Abraham wrote: Oliver Endriss wrote: Does anyone know, why dvb_shutdown_timeout was introduced initially? It was originally introduced looong time back by Obi. Really? It's been five years since then, so I don't remember exactly where it came from. CVS indicates that it's Holger's

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-19 Thread Manu Abraham
Andreas Oberritter wrote: Manu Abraham wrote: Oliver Endriss wrote: Does anyone know, why dvb_shutdown_timeout was introduced initially? It was originally introduced looong time back by Obi. Really? It's been five years since then, so I don't remember exactly where it came from. CVS

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-19 Thread Johannes Stezenbach
On Sun, Aug 19, 2007, Oliver Endriss wrote: Does anyone know, why dvb_shutdown_timeout was introduced initially? That was a (IMHO stupid) hack by Holger with the purpose of speeding up tuning with szap a little bit by avoiding the FE_INIT after re-opening the frontend device. I.e. Holger's

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-19 Thread Johannes Stezenbach
On Sun, Aug 19, 2007, Oliver Endriss wrote: Questions: - Why should dvb_shutdown_timeout==0 disable sleep mode? The use case was to watch video without any software running. Just program the hardware once and let it do it's job. Some people want that although I don't think it's really useful.

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-18 Thread Trent Piepho
On Fri, 17 Aug 2007, Markus Rechberger wrote: as I wrote earlier the thread can be idle/closed even before the node gets closed (you can test that with kaffeine, and you can test the other case with the scan utility) How can this happen? Afaics the fe thread may continue to exist

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-18 Thread Manu Abraham
ts_bus_ctrl does a kind of reference counting. For readers: - fe-ops.ts_bus_ctrl(fe,1) is called during open - fe-ops.ts_bus_ctrl(fe,0) is called during close For the one and only writer: - fe-ops.ts_bus_ctrl(fe,1) is called during open - fe-ops.ts_bus_ctrl(fe,0) is called when the

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-18 Thread Manu Abraham
Hi Obi, Andreas Oberritter wrote: Hi Oliver, Oliver Endriss wrote: ts_bus_ctrl does a kind of reference counting. For readers: - fe-ops.ts_bus_ctrl(fe,1) is called during open - fe-ops.ts_bus_ctrl(fe,0) is called during close For the one and only writer: - fe-ops.ts_bus_ctrl(fe,1) is

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-18 Thread Andreas Oberritter
Hi Manu, Manu Abraham wrote: Andreas Oberritter wrote: I think that the option dvb_shutdown_timeout must be removed to fix the problem, where the frontend thread can be idle after close(). IMHO the thread should always be terminated when the last writer closes the device. The release

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-18 Thread Oliver Endriss
Steven Toth wrote: Trent Piepho wrote: On Fri, 17 Aug 2007, Markus Rechberger wrote: as I wrote earlier the thread can be idle/closed even before the node gets closed (you can test that with kaffeine, and you can test the other case with the scan utility) How can this

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-18 Thread Manu Abraham
Oliver Endriss wrote: Does anyone know, why dvb_shutdown_timeout was introduced initially? It was originally introduced looong time back by Obi. Some operations still incomplete on close() was the reason stated, IIRC. It had something to do with the FF cards ?, dunno. I don't have a problem

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-17 Thread Markus Rechberger
On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Steven Toth wrote: Steven Toth wrote: Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-17 Thread Oliver Endriss
Markus Rechberger wrote: On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Steven Toth wrote: The ts_bus_ctrl function pointer / callback is already in the mainline, check dvb_frontend.c for more details. You shouldn't need a patch from me. ACK, should be enough to do this kind of

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-17 Thread Markus Rechberger
On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Steven Toth wrote: The ts_bus_ctrl function pointer / callback is already in the mainline, check dvb_frontend.c for more details. You shouldn't need a

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-17 Thread Oliver Endriss
Markus Rechberger wrote: On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Steven Toth wrote: The ts_bus_ctrl function pointer / callback is already in the mainline, check dvb_frontend.c for more

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-17 Thread Markus Rechberger
On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Steven Toth wrote: The ts_bus_ctrl function pointer / callback is

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-17 Thread Markus Rechberger
On 8/17/07, Markus Rechberger [EMAIL PROTECTED] wrote: On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/17/07, Oliver Endriss [EMAIL PROTECTED] wrote: Steven Toth

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-17 Thread Andreas Oberritter
Hi Oliver, Oliver Endriss wrote: ts_bus_ctrl does a kind of reference counting. For readers: - fe-ops.ts_bus_ctrl(fe,1) is called during open - fe-ops.ts_bus_ctrl(fe,0) is called during close For the one and only writer: - fe-ops.ts_bus_ctrl(fe,1) is called during open -

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-17 Thread Markus Rechberger
On 8/17/07, Andreas Oberritter [EMAIL PROTECTED] wrote: Hi Oliver, Oliver Endriss wrote: ts_bus_ctrl does a kind of reference counting. For readers: - fe-ops.ts_bus_ctrl(fe,1) is called during open - fe-ops.ts_bus_ctrl(fe,0) is called during close For the one and only writer: -

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-16 Thread Oliver Endriss
Steven Toth wrote: Steven Toth wrote: Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote:

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-15 Thread Steven Toth
Steven Toth wrote: Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: Following patch adds a rather

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-15 Thread Markus Rechberger
On 8/15/07, Steven Toth [EMAIL PROTECTED] wrote: Steven Toth wrote: Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: Following patch adds a

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-14 Thread Markus Rechberger
On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: Following patch adds a rather primitive way to temporary lock dvb devicenodes, this can be useful for hybrid devices which use the

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-14 Thread Steven Toth
Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: Following patch adds a rather primitive way to temporary lock dvb devicenodes, this can be

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-11 Thread Markus Rechberger
On 8/11/07, Trent Piepho [EMAIL PROTECTED] wrote: On Thu, 9 Aug 2007, Markus Rechberger wrote: On 8/9/07, Markus Rechberger [EMAIL PROTECTED] wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: Following patch adds a rather primitive way to temporary lock

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-10 Thread Trent Piepho
On Thu, 9 Aug 2007, Markus Rechberger wrote: On 8/9/07, Markus Rechberger [EMAIL PROTECTED] wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: Following patch adds a rather primitive way to temporary lock dvb devicenodes, this can be useful for hybrid

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-10 Thread Manu Abraham
On 8/11/07, Trent Piepho [EMAIL PROTECTED] wrote: On Thu, 9 Aug 2007, Markus Rechberger wrote: On 8/9/07, Markus Rechberger [EMAIL PROTECTED] wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: Following patch adds a rather primitive way to temporary lock

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-09 Thread Steven Toth
Markus Rechberger wrote: Following patch adds a rather primitive way to temporary lock dvb devicenodes, this can be useful for hybrid devices which use the video4linux framework for the analogue TV part and the dvb framework for digital TV if only one mode can be accessed at a time.

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-09 Thread Markus Rechberger
On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: Following patch adds a rather primitive way to temporary lock dvb devicenodes, this can be useful for hybrid devices which use the video4linux framework for the analogue TV part and the dvb framework for digital TV

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-09 Thread Markus Rechberger
On 8/9/07, Markus Rechberger [EMAIL PROTECTED] wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: Following patch adds a rather primitive way to temporary lock dvb devicenodes, this can be useful for hybrid devices which use the video4linux framework for

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-09 Thread Steven Toth
Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: Following patch adds a rather primitive way to temporary lock dvb devicenodes, this can be useful for hybrid devices which use the video4linux framework for the analogue TV part and the

Re: [linux-dvb] [PATCH] add device node locking possibility to dvbcore

2007-08-09 Thread Markus Rechberger
On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: On 8/9/07, Steven Toth [EMAIL PROTECTED] wrote: Markus Rechberger wrote: Following patch adds a rather primitive way to temporary lock dvb devicenodes, this can be useful for hybrid devices which use the