The patch number 9255 was added via Steven Toth <[EMAIL PROTECTED]>
to http://linuxtv.org/hg/v4l-dvb master development tree.

Kernel patches in this development tree may be modified to be backward
compatible with older kernels. Compatibility modifications will be
removed before inclusion into the mainstream Kernel

If anyone has any objections, please let us know by sending a message to:
        [EMAIL PROTECTED]

------

From: Steven Toth  <[EMAIL PROTECTED]>
tda10048: Checkpatch compliance


tda10048: Checkpatch compliance

Priority: normal

Signed-off-by: Steven Toth <[EMAIL PROTECTED]>


---

 linux/drivers/media/dvb/frontends/tda10048.c |  100 +++++++++++++------
 1 file changed, 73 insertions(+), 27 deletions(-)

diff -r 240b32420266 -r 260ff9d74377 
linux/drivers/media/dvb/frontends/tda10048.c
--- a/linux/drivers/media/dvb/frontends/tda10048.c      Thu Oct 16 19:22:01 
2008 -0400
+++ b/linux/drivers/media/dvb/frontends/tda10048.c      Thu Oct 16 19:22:57 
2008 -0400
@@ -195,7 +195,7 @@ static int tda10048_writereg(struct tda1
 static int tda10048_writereg(struct tda10048_state *state, u8 reg, u8 data)
 {
        int ret;
-       u8 buf [] = { reg, data };
+       u8 buf[] = { reg, data };
        struct i2c_msg msg = {
                .addr = state->config->demod_address,
                .flags = 0, .buf = buf, .len = 2 };
@@ -213,9 +213,9 @@ static u8 tda10048_readreg(struct tda100
 static u8 tda10048_readreg(struct tda10048_state *state, u8 reg)
 {
        int ret;
-       u8 b0 [] = { reg };
-       u8 b1 [] = { 0 };
-       struct i2c_msg msg [] = {
+       u8 b0[] = { reg };
+       u8 b1[] = { 0 };
+       struct i2c_msg msg[] = {
                { .addr = state->config->demod_address,
                        .flags = 0, .buf = b0, .len = 1 },
                { .addr = state->config->demod_address,
@@ -393,43 +393,89 @@ static int tda10048_get_tps(struct tda10
 
        val = tda10048_readreg(state, TDA10048_OUT_CONF2);
        switch ((val & 0x60) >> 5) {
-       case 0: p->constellation =   QPSK; break;
-       case 1: p->constellation = QAM_16; break;
-       case 2: p->constellation = QAM_64; break;
+       case 0:
+               p->constellation = QPSK;
+               break;
+       case 1:
+               p->constellation = QAM_16;
+               break;
+       case 2:
+               p->constellation = QAM_64;
+               break;
        }
        switch ((val & 0x18) >> 3) {
-       case 0: p->hierarchy_information = HIERARCHY_NONE; break;
-       case 1: p->hierarchy_information =    HIERARCHY_1; break;
-       case 2: p->hierarchy_information =    HIERARCHY_2; break;
-       case 3: p->hierarchy_information =    HIERARCHY_4; break;
+       case 0:
+               p->hierarchy_information = HIERARCHY_NONE;
+               break;
+       case 1:
+               p->hierarchy_information = HIERARCHY_1;
+               break;
+       case 2:
+               p->hierarchy_information = HIERARCHY_2;
+               break;
+       case 3:
+               p->hierarchy_information = HIERARCHY_4;
+               break;
        }
        switch (val & 0x07) {
-       case 0: p->code_rate_HP = FEC_1_2; break;
-       case 1: p->code_rate_HP = FEC_2_3; break;
-       case 2: p->code_rate_HP = FEC_3_4; break;
-       case 3: p->code_rate_HP = FEC_5_6; break;
-       case 4: p->code_rate_HP = FEC_7_8; break;
+       case 0:
+               p->code_rate_HP = FEC_1_2;
+               break;
+       case 1:
+               p->code_rate_HP = FEC_2_3;
+               break;
+       case 2:
+               p->code_rate_HP = FEC_3_4;
+               break;
+       case 3:
+               p->code_rate_HP = FEC_5_6;
+               break;
+       case 4:
+               p->code_rate_HP = FEC_7_8;
+               break;
        }
 
        val = tda10048_readreg(state, TDA10048_OUT_CONF3);
        switch (val & 0x07) {
-       case 0: p->code_rate_LP = FEC_1_2; break;
-       case 1: p->code_rate_LP = FEC_2_3; break;
-       case 2: p->code_rate_LP = FEC_3_4; break;
-       case 3: p->code_rate_LP = FEC_5_6; break;
-       case 4: p->code_rate_LP = FEC_7_8; break;
+       case 0:
+               p->code_rate_LP = FEC_1_2;
+               break;
+       case 1:
+               p->code_rate_LP = FEC_2_3;
+               break;
+       case 2:
+               p->code_rate_LP = FEC_3_4;
+               break;
+       case 3:
+               p->code_rate_LP = FEC_5_6;
+               break;
+       case 4:
+               p->code_rate_LP = FEC_7_8;
+               break;
        }
 
        val = tda10048_readreg(state, TDA10048_OUT_CONF1);
        switch ((val & 0x0c) >> 2) {
-       case 0: p->guard_interval = GUARD_INTERVAL_1_32; break;
-       case 1: p->guard_interval = GUARD_INTERVAL_1_16; break;
-       case 2: p->guard_interval =  GUARD_INTERVAL_1_8; break;
-       case 3: p->guard_interval =  GUARD_INTERVAL_1_4; break;
+       case 0:
+               p->guard_interval = GUARD_INTERVAL_1_32;
+               break;
+       case 1:
+               p->guard_interval = GUARD_INTERVAL_1_16;
+               break;
+       case 2:
+               p->guard_interval =  GUARD_INTERVAL_1_8;
+               break;
+       case 3:
+               p->guard_interval =  GUARD_INTERVAL_1_4;
+               break;
        }
        switch (val & 0x02) {
-       case 0: p->transmission_mode = TRANSMISSION_MODE_2K; break;
-       case 1: p->transmission_mode = TRANSMISSION_MODE_8K; break;
+       case 0:
+               p->transmission_mode = TRANSMISSION_MODE_2K;
+               break;
+       case 1:
+               p->transmission_mode = TRANSMISSION_MODE_8K;
+               break;
        }
 
        return 0;


---

Patch is available at: 
http://linuxtv.org/hg/v4l-dvb/rev/260ff9d743773bde9b9fac53a3203dde7131276b

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to