Commit 607e5ad8 authored by Kuninori Morimoto's avatar Kuninori Morimoto Committed by Mauro Carvalho Chehab

V4L/DVB (13655): tw9910: Add power control

Power down internal blocks when not streaming.
Signed-off-by: default avatarKuninori Morimoto <morimoto.kuninori@renesas.com>
Signed-off-by: default avatarGuennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 06f292e4
...@@ -181,6 +181,14 @@ ...@@ -181,6 +181,14 @@
* but all register content remain unchanged. * but all register content remain unchanged.
* This bit is self-resetting. * This bit is self-resetting.
*/ */
#define ACNTL1_PDN_MASK 0x0e
#define CLK_PDN 0x08 /* system clock power down */
#define Y_PDN 0x04 /* Luma ADC power down */
#define C_PDN 0x02 /* Chroma ADC power down */
/* ACNTL2 */
#define ACNTL2_PDN_MASK 0x40
#define PLL_PDN 0x40 /* PLL power down */
/* VBICNTL */ /* VBICNTL */
...@@ -478,10 +486,31 @@ static int tw9910_mask_set(struct i2c_client *client, u8 command, ...@@ -478,10 +486,31 @@ static int tw9910_mask_set(struct i2c_client *client, u8 command,
static void tw9910_reset(struct i2c_client *client) static void tw9910_reset(struct i2c_client *client)
{ {
i2c_smbus_write_byte_data(client, ACNTL1, SRESET); tw9910_mask_set(client, ACNTL1, SRESET, SRESET);
msleep(1); msleep(1);
} }
static int tw9910_power(struct i2c_client *client, int enable)
{
int ret;
u8 acntl1;
u8 acntl2;
if (enable) {
acntl1 = 0;
acntl2 = 0;
} else {
acntl1 = CLK_PDN | Y_PDN | C_PDN;
acntl2 = PLL_PDN;
}
ret = tw9910_mask_set(client, ACNTL1, ACNTL1_PDN_MASK, acntl1);
if (ret < 0)
return ret;
return tw9910_mask_set(client, ACNTL2, ACNTL2_PDN_MASK, acntl2);
}
static const struct tw9910_scale_ctrl* static const struct tw9910_scale_ctrl*
tw9910_select_norm(struct soc_camera_device *icd, u32 width, u32 height) tw9910_select_norm(struct soc_camera_device *icd, u32 width, u32 height)
{ {
...@@ -520,8 +549,8 @@ static int tw9910_s_stream(struct v4l2_subdev *sd, int enable) ...@@ -520,8 +549,8 @@ static int tw9910_s_stream(struct v4l2_subdev *sd, int enable)
{ {
struct i2c_client *client = sd->priv; struct i2c_client *client = sd->priv;
struct tw9910_priv *priv = to_tw9910(client); struct tw9910_priv *priv = to_tw9910(client);
u8 val; u8 val;
int ret;
if (!enable) { if (!enable) {
switch (priv->revision) { switch (priv->revision) {
...@@ -549,7 +578,11 @@ static int tw9910_s_stream(struct v4l2_subdev *sd, int enable) ...@@ -549,7 +578,11 @@ static int tw9910_s_stream(struct v4l2_subdev *sd, int enable)
priv->scale->height); priv->scale->height);
} }
return tw9910_mask_set(client, OPFORM, OEN_TRI_SEL_MASK, val); ret = tw9910_mask_set(client, OPFORM, OEN_TRI_SEL_MASK, val);
if (ret < 0)
return ret;
return tw9910_power(client, enable);
} }
static int tw9910_set_bus_param(struct soc_camera_device *icd, static int tw9910_set_bus_param(struct soc_camera_device *icd,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment