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

index e5bffaa..01dcc7f 100644 (file)
@@ -1056,13 +1056,13 @@ static int rtl2832_probe(struct i2c_client *client,
        dev->sleeping = true;
        INIT_DELAYED_WORK(&dev->i2c_gate_work, rtl2832_i2c_gate_work);
        /* create regmap */
-       dev->regmap_config.reg_bits =  8,
-       dev->regmap_config.val_bits =  8,
-       dev->regmap_config.volatile_reg = rtl2832_volatile_reg,
-       dev->regmap_config.max_register = 5 * 0x100,
-       dev->regmap_config.ranges = regmap_range_cfg,
-       dev->regmap_config.num_ranges = ARRAY_SIZE(regmap_range_cfg),
-       dev->regmap_config.cache_type = REGCACHE_NONE,
+       dev->regmap_config.reg_bits =  8;
+       dev->regmap_config.val_bits =  8;
+       dev->regmap_config.volatile_reg = rtl2832_volatile_reg;
+       dev->regmap_config.max_register = 5 * 0x100;
+       dev->regmap_config.ranges = regmap_range_cfg;
+       dev->regmap_config.num_ranges = ARRAY_SIZE(regmap_range_cfg);
+       dev->regmap_config.cache_type = REGCACHE_NONE;
        dev->regmap = regmap_init_i2c(client, &dev->regmap_config);
        if (IS_ERR(dev->regmap)) {
                ret = PTR_ERR(dev->regmap);