[media] ak881x: simplify standard checks
authorHans Verkuil <hans.verkuil@cisco.com>
Mon, 15 Jun 2015 11:33:32 +0000 (08:33 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Tue, 11 Aug 2015 12:50:01 +0000 (09:50 -0300)
Simplify confusing conditions. This also swaps the checks for NTSC and PAL:
to be consistent with other drivers check for NTSC first. So if the user
sets both NTSC and PAL bits, then NTSC wins.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/i2c/ak881x.c

index 29846245aa3bb6762f06087379d089ad779c5d98..d3b965ec3bbc5597245a9df0eb9d415bb376993e 100644 (file)
@@ -156,12 +156,12 @@ static int ak881x_s_std_output(struct v4l2_subdev *sd, v4l2_std_id std)
        } else if (std == V4L2_STD_PAL_60) {
                vp1 = 7;
                ak881x->lines = 480;
-       } else if (std && !(std & ~V4L2_STD_PAL)) {
-               vp1 = 0xf;
-               ak881x->lines = 576;
-       } else if (std && !(std & ~V4L2_STD_NTSC)) {
+       } else if (std & V4L2_STD_NTSC) {
                vp1 = 0;
                ak881x->lines = 480;
+       } else if (std & V4L2_STD_PAL) {
+               vp1 = 0xf;
+               ak881x->lines = 576;
        } else {
                /* No SECAM or PAL_N/Nc supported */
                return -EINVAL;