media: pvrusb2: use semicolons rather than commas to separate statements
authorJulia Lawall <Julia.Lawall@inria.fr>
Tue, 29 Sep 2020 13:14:32 +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/pvrusb2/pvrusb2-devattr.c

index 1fcf6321888590402bc081d6e5dcaaa36f6549c5..d1b984ec757defad844fc13353458dc273131f7d 100644 (file)
@@ -594,7 +594,7 @@ static int pvr2_lgdt3306a_attach(struct pvr2_dvb_adapter *adap)
        lgdt3306a_config.mpeg_mode = LGDT3306A_MPEG_PARALLEL;
        lgdt3306a_config.tpclk_edge = LGDT3306A_TPCLK_FALLING_EDGE;
        lgdt3306a_config.tpvalid_polarity = LGDT3306A_TP_VALID_LOW;
-       lgdt3306a_config.xtalMHz = 25, /* demod clock MHz; 24/25 supported */
+       lgdt3306a_config.xtalMHz = 25; /* demod clock MHz; 24/25 supported */
 
        adap->i2c_client_demod[0] = dvb_module_probe("lgdt3306a", NULL,
                                                     &adap->channel.hdw->i2c_adap,