Commit c98afbfc authored by Philippe Rétornaz's avatar Philippe Rétornaz Committed by Mauro Carvalho Chehab

V4L/DVB (11035): mt9t031 bugfix

- The video device is not allocated when mt9t031_init() is called, don't use
it in debug printk.

- The clock polarity is inverted in mt9t031_set_bus_param(), use the correct
one.
Signed-off-by: default avatarPhilippe Rétornaz <philippe.retornaz@epfl.ch>
Signed-off-by: default avatarGuennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent d42574d1
...@@ -144,8 +144,6 @@ static int mt9t031_init(struct soc_camera_device *icd) ...@@ -144,8 +144,6 @@ static int mt9t031_init(struct soc_camera_device *icd)
int ret; int ret;
/* Disable chip output, synchronous option update */ /* Disable chip output, synchronous option update */
dev_dbg(icd->vdev->parent, "%s\n", __func__);
ret = reg_write(icd, MT9T031_RESET, 1); ret = reg_write(icd, MT9T031_RESET, 1);
if (ret >= 0) if (ret >= 0)
ret = reg_write(icd, MT9T031_RESET, 0); ret = reg_write(icd, MT9T031_RESET, 0);
...@@ -186,9 +184,9 @@ static int mt9t031_set_bus_param(struct soc_camera_device *icd, ...@@ -186,9 +184,9 @@ static int mt9t031_set_bus_param(struct soc_camera_device *icd,
return -EINVAL; return -EINVAL;
if (flags & SOCAM_PCLK_SAMPLE_FALLING) if (flags & SOCAM_PCLK_SAMPLE_FALLING)
reg_set(icd, MT9T031_PIXEL_CLOCK_CONTROL, 0x8000);
else
reg_clear(icd, MT9T031_PIXEL_CLOCK_CONTROL, 0x8000); reg_clear(icd, MT9T031_PIXEL_CLOCK_CONTROL, 0x8000);
else
reg_set(icd, MT9T031_PIXEL_CLOCK_CONTROL, 0x8000);
return 0; return 0;
} }
......
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