Re: [patch v2] [media] stv090x: remove indent levels in stv090x_get_coldlock()

2014-02-21 Thread walter harms
Am 21.02.2014 09:50, schrieb Dan Carpenter: > 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. > >

[patch v2] [media] stv090x: remove indent levels in stv090x_get_coldlock()

2014-02-21 Thread Dan Carpenter
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 >= 1000