]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
[media] cx24110: Simplify error handling at cx24110_set_fec()
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>
Wed, 5 Nov 2014 12:00:45 +0000 (10:00 -0200)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Wed, 5 Nov 2014 12:00:45 +0000 (10:00 -0200)
move the return to happen before the logic. This way, we can
avoid one extra identation.

This also fixes an identation issue on this function.

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/dvb-frontends/cx24110.c

index 5a31b3f59306d55b2b4b1845f1aee7f6a554a708..7b510f2ae20f700b8ad5dd4bd8b81fbc9fcdfcdb 100644 (file)
@@ -177,10 +177,8 @@ static int cx24110_set_inversion (struct cx24110_state* state, fe_spectral_inver
        return 0;
 }
 
-static int cx24110_set_fec (struct cx24110_state* state, fe_code_rate_t fec)
+static int cx24110_set_fec(struct cx24110_state* state, fe_code_rate_t fec)
 {
-/* fixme (low): error handling */
-
        static const int rate[FEC_AUTO] = {-1,    1,    2,    3,    5,    7, -1};
        static const int g1[FEC_AUTO]   = {-1, 0x01, 0x02, 0x05, 0x15, 0x45, -1};
        static const int g2[FEC_AUTO]   = {-1, 0x01, 0x03, 0x06, 0x1a, 0x7a, -1};
@@ -208,16 +206,16 @@ static int cx24110_set_fec (struct cx24110_state* state, fe_code_rate_t fec)
        } else {
                cx24110_writereg(state, 0x37, cx24110_readreg(state, 0x37) | 0x20);
                /* set AcqVitDis bit */
-               if (rate[fec] > 0) {
-                       cx24110_writereg(state, 0x05, (cx24110_readreg(state, 0x05) & 0xf0) | rate[fec]);
-                       /* set nominal Viterbi rate */
-                       cx24110_writereg(state, 0x22, (cx24110_readreg(state, 0x22) & 0xf0) | rate[fec]);
-                       /* set current Viterbi rate */
-                       cx24110_writereg(state, 0x1a, g1[fec]);
-                       cx24110_writereg(state, 0x1b, g2[fec]);
-                       /* not sure if this is the right way: I always used AutoAcq mode */
-          } else
-                  return -EINVAL;
+               if (rate[fec] < 0)
+                       return -EINVAL;
+
+               cx24110_writereg(state, 0x05, (cx24110_readreg(state, 0x05) & 0xf0) | rate[fec]);
+               /* set nominal Viterbi rate */
+               cx24110_writereg(state, 0x22, (cx24110_readreg(state, 0x22) & 0xf0) | rate[fec]);
+               /* set current Viterbi rate */
+               cx24110_writereg(state, 0x1a, g1[fec]);
+               cx24110_writereg(state, 0x1b, g2[fec]);
+               /* not sure if this is the right way: I always used AutoAcq mode */
        }
        return 0;
 }