media: dib0700: use semicolons rather than commas to separate statements
authorJulia Lawall <Julia.Lawall@inria.fr>
Tue, 29 Sep 2020 13:14:40 +0000 (15:14 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Mon, 16 Nov 2020 09:31:07 +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/dvb-usb/dib0700_devices.c

index d3288c1079062b62dca9404d9b2ec83ab6a44d24..710c1afe3e85c25e994882cdf6d0513bc4dee5e1 100644 (file)
@@ -3074,8 +3074,8 @@ static int nim7090_tuner_attach(struct dvb_usb_adapter *adap)
        struct dib0700_adapter_state *st = adap->priv;
        struct i2c_adapter *tun_i2c = st->dib7000p_ops.get_i2c_tuner(adap->fe_adap[0].fe);
 
-       nim7090_dib0090_config.reset = st->dib7000p_ops.tuner_sleep,
-       nim7090_dib0090_config.sleep = st->dib7000p_ops.tuner_sleep,
+       nim7090_dib0090_config.reset = st->dib7000p_ops.tuner_sleep;
+       nim7090_dib0090_config.sleep = st->dib7000p_ops.tuner_sleep;
        nim7090_dib0090_config.get_adc_power = st->dib7000p_ops.get_adc_power;
 
        if (dvb_attach(dib0090_register, adap->fe_adap[0].fe, tun_i2c, &nim7090_dib0090_config) == NULL)