Re: [PATCHv2] media: add tuner standby op, use where needed

2018-02-21 Thread Hans Verkuil
On 02/21/2018 08:21 PM, Laurent Pinchart wrote: > Hi Hans, > > On Wednesday, 21 February 2018 15:11:36 EET Hans Verkuil wrote: >> On 02/21/18 13:37, Laurent Pinchart wrote: >>> On Wednesday, 21 February 2018 09:40:29 EET Hans Verkuil wrote: On 02/21/2018 01:02 AM, Laurent Pinchart wrote:

Re: [PATCHv2] media: add tuner standby op, use where needed

2018-02-21 Thread Laurent Pinchart
Hi Hans, On Wednesday, 21 February 2018 15:11:36 EET Hans Verkuil wrote: > On 02/21/18 13:37, Laurent Pinchart wrote: > > On Wednesday, 21 February 2018 09:40:29 EET Hans Verkuil wrote: > >> On 02/21/2018 01:02 AM, Laurent Pinchart wrote: > >>> On Tuesday, 20 February 2018 11:44:20 EET Hans

Re: [PATCHv2] media: add tuner standby op, use where needed

2018-02-21 Thread Hans Verkuil
On 02/21/18 13:37, Laurent Pinchart wrote: > Hi Hans, > > On Wednesday, 21 February 2018 09:40:29 EET Hans Verkuil wrote: >> On 02/21/2018 01:02 AM, Laurent Pinchart wrote: >>> On Tuesday, 20 February 2018 11:44:20 EET Hans Verkuil wrote: The v4l2_subdev core s_power op was used for two

Re: [PATCHv2] media: add tuner standby op, use where needed

2018-02-21 Thread Laurent Pinchart
Hi Hans, On Wednesday, 21 February 2018 09:40:29 EET Hans Verkuil wrote: > On 02/21/2018 01:02 AM, Laurent Pinchart wrote: > > On Tuesday, 20 February 2018 11:44:20 EET Hans Verkuil wrote: > >> The v4l2_subdev core s_power op was used for two different things: power > >> on/off sensors or video

Re: [PATCHv2] media: add tuner standby op, use where needed

2018-02-20 Thread Hans Verkuil
On 02/21/2018 01:02 AM, Laurent Pinchart wrote: > Hi Hans, > > Thank you for the patch. > > On Tuesday, 20 February 2018 11:44:20 EET Hans Verkuil wrote: >> The v4l2_subdev core s_power op was used for two different things: power >> on/off sensors or video decoders/encoders and to put a tuner in

Re: [PATCHv2] media: add tuner standby op, use where needed

2018-02-20 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Tuesday, 20 February 2018 11:44:20 EET Hans Verkuil wrote: > The v4l2_subdev core s_power op was used for two different things: power > on/off sensors or video decoders/encoders and to put a tuner in standby > (and only the tuner!). There is no 'tuner wakeup'

[PATCHv2] media: add tuner standby op, use where needed

2018-02-20 Thread Hans Verkuil
The v4l2_subdev core s_power op was used for two different things: power on/off sensors or video decoders/encoders and to put a tuner in standby (and only the tuner!). There is no 'tuner wakeup' op, that's done automatically when the tuner is accessed. The danger with calling (s_power, 0) to put