]> git.baikalelectronics.ru Git - kernel.git/commit
[media] stv090x: remove indent levels in stv090x_get_coldlock()
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 21 Feb 2014 08:50:01 +0000 (05:50 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Mon, 3 Nov 2014 16:53:14 +0000 (14:53 -0200)
commit4d59a7d04ef46930321ddc65b45ce64cd8e46e90
tree70ffe6cc166ed339277071859381f934b82e1945
parent7ca5bc5bc16ce026ab3f9ccee4dd0985fa0c9815
[media] stv090x: remove indent levels in stv090x_get_coldlock()

This code is needlessly complicated and checkpatch.pl complains that we
go over the 80 characters per line limit.

If we flip the "if (!lock) {" test to "if (lock) return;" then we can
remove an indent level from the rest of the function.

We can add two returns in the "if (state->srate >= 10000000) {"
condition and move the else statement back an additional indent level.

There is another "if (!lock) {" check which can be removed since we have
already checked "lock" and know it is zero at this point.  This second
check on "lock" is also a problem because it sets off a static checker
warning.  I have reviewed this code for some time to see if something
else was intended, but have concluded that it was simply an oversight
and should be removed.  Removing this duplicative check gains us an
third indent level.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/dvb-frontends/stv090x.c