Vladimir Barinov wrote:
Hi Hans,
Hans Verkuil wrote:
+ */
+val = i2c_smbus_read_byte_data(client, STATUS_REG);
+if (val < 0)
+return val;
+
+priv->std = val & STATUS_NTSCPAL ? V4L2_STD_PAL : V4L2_STD_NTSC;
Shouldn't this be 50 Hz vs 60 Hz formats? There are
Hi Hans,
Hans Verkuil wrote:
+*/
+ val = i2c_smbus_read_byte_data(client, STATUS_REG);
+ if (val < 0)
+ return val;
+
+ priv->std = val & STATUS_NTSCPAL ? V4L2_STD_PAL : V4L2_STD_NTSC;
Shouldn't this be 50 Hz vs 60 Hz formats? There are 60 H
On Mon April 22 2013 10:39:42 Vladimir Barinov wrote:
> Hi Hans,
>
> Thank you for the review.
>
> Hans Verkuil wrote:
> >> +#include
> >>
> >
> > This include should be removed as well.
> >
> ok
> >
> >> +
> >> +static int ml86v7667_querystd(struct v4l2_subdev *sd, v4l2_std_id *std)
Hi Hans,
Thank you for the review.
Hans Verkuil wrote:
+#include
This include should be removed as well.
ok
+
+static int ml86v7667_querystd(struct v4l2_subdev *sd, v4l2_std_id *std)
+{
+ struct ml86v7667_priv *priv = to_ml86v7667(sd);
+
+ *std = priv->std;
Th
On Sun April 21 2013 20:40:30 Sergei Shtylyov wrote:
> From: Vladimir Barinov
>
> Add OKI Semiconductor ML86V7667 video decoder driver.
>
> Signed-off-by: Vladimir Barinov
> [Sergei: added v4l2_device_unregister_subdev() call to the error cleanup path
> of
> ml86v7667_probe(); some cleanup.]
>
From: Vladimir Barinov
Add OKI Semiconductor ML86V7667 video decoder driver.
Signed-off-by: Vladimir Barinov
[Sergei: added v4l2_device_unregister_subdev() call to the error cleanup path of
ml86v7667_probe(); some cleanup.]
Signed-off-by: Sergei Shtylyov
---
Changes since the original posting