Commit 130e820d authored by Julia Lawall's avatar Julia Lawall Committed by Mauro Carvalho Chehab

media: msp3400: use semicolons rather than commas to separate statements

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: default avatarJulia Lawall <Julia.Lawall@inria.fr>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+huawei@kernel.org>
parent fa71ae71
...@@ -549,8 +549,10 @@ int msp3400c_thread(void *data) ...@@ -549,8 +549,10 @@ int msp3400c_thread(void *data)
val = msp_read_dsp(client, 0x1b); val = msp_read_dsp(client, 0x1b);
if (val > 32767) if (val > 32767)
val -= 65536; val -= 65536;
if (val1 < val) if (val1 < val) {
val1 = val, max1 = i; val1 = val;
max1 = i;
}
dev_dbg_lvl(&client->dev, 1, msp_debug, dev_dbg_lvl(&client->dev, 1, msp_debug,
"carrier1 val: %5d / %s\n", val, cd[i].name); "carrier1 val: %5d / %s\n", val, cd[i].name);
} }
...@@ -586,8 +588,10 @@ int msp3400c_thread(void *data) ...@@ -586,8 +588,10 @@ int msp3400c_thread(void *data)
val = msp_read_dsp(client, 0x1b); val = msp_read_dsp(client, 0x1b);
if (val > 32767) if (val > 32767)
val -= 65536; val -= 65536;
if (val2 < val) if (val2 < val) {
val2 = val, max2 = i; val2 = val;
max2 = i;
}
dev_dbg_lvl(&client->dev, 1, msp_debug, dev_dbg_lvl(&client->dev, 1, msp_debug,
"carrier2 val: %5d / %s\n", val, cd[i].name); "carrier2 val: %5d / %s\n", val, cd[i].name);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment