media: gspca: ov534-ov772x: use semicolons rather than commas to separate statements
authorJulia Lawall <Julia.Lawall@inria.fr>
Tue, 29 Sep 2020 13:14:29 +0000 (15:14 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Mon, 16 Nov 2020 09:31:06 +0000 (10:31 +0100)
Replace commas with semicolons.  Commas introduce unnecessary
variability in the code structure and are hard to see.  What is done
is essentially described by the following Coccinelle semantic patch
(http://coccinelle.lip6.fr/):

// <smpl>
@@ expression e1,e2; @@
e1
-,
+;
e2
... when any
// </smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/usb/gspca/ov534.c

index 9a11158..8b6a57f 100644 (file)
@@ -1220,9 +1220,9 @@ static int sd_init_controls(struct gspca_dev *gspca_dev)
        int hflip_def;
 
        if (sd->sensor == SENSOR_OV767x) {
-               saturation_min = 0,
-               saturation_max = 6,
-               saturation_def = 3,
+               saturation_min = 0;
+               saturation_max = 6;
+               saturation_def = 3;
                brightness_min = -127;
                brightness_max = 127;
                brightness_def = 0;
@@ -1233,9 +1233,9 @@ static int sd_init_controls(struct gspca_dev *gspca_dev)
                exposure_def = 0x13;
                hflip_def = 1;
        } else {
-               saturation_min = 0,
-               saturation_max = 255,
-               saturation_def = 64,
+               saturation_min = 0;
+               saturation_max = 255;
+               saturation_def = 64;
                brightness_min = 0;
                brightness_max = 255;
                brightness_def = 0;