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

index ad6d9d564a87e810e67aebf8eb73e9f389f49c5f..cfa4cdde99d8a89c5b4a2afc711b8e82ef858b1e 100644 (file)
@@ -1793,9 +1793,9 @@ static int m88ds3103_probe(struct i2c_client *client,
        dev->config.lnb_en_pol = pdata->lnb_en_pol;
        dev->cfg = &dev->config;
        /* create regmap */
-       dev->regmap_config.reg_bits = 8,
-       dev->regmap_config.val_bits = 8,
-       dev->regmap_config.lock_arg = dev,
+       dev->regmap_config.reg_bits = 8;
+       dev->regmap_config.val_bits = 8;
+       dev->regmap_config.lock_arg = dev;
        dev->regmap = devm_regmap_init_i2c(client, &dev->regmap_config);
        if (IS_ERR(dev->regmap)) {
                ret = PTR_ERR(dev->regmap);