[media] ngene: Codingstyle fixes
authorOliver Endriss <o.endriss@gmx.de>
Sun, 3 Jul 2011 16:56:28 +0000 (13:56 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 27 Jul 2011 20:55:43 +0000 (17:55 -0300)
Codingstyle fixes

Signed-off-by: Oliver Endriss <o.endriss@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/ngene/ngene-cards.c
drivers/media/dvb/ngene/ngene-core.c
drivers/media/dvb/ngene/ngene-dvb.c
drivers/media/dvb/ngene/ngene.h

index ca2e146f1b8f912ae63992823c47b040a99f9484..0d550a950e5337ae5940f8971f64e6692098cb8f 100644 (file)
@@ -40,6 +40,7 @@
 #include "lnbh24.h"
 #include "lgdt330x.h"
 #include "mt2131.h"
+#include "tda18271c2dd.h"
 #include "drxk.h"
 
 
@@ -99,9 +100,6 @@ static int drxk_gate_ctrl(struct dvb_frontend *fe, int enable)
        return status;
 }
 
-struct dvb_frontend *tda18271c2dd_attach(struct dvb_frontend *fe,
-                                        struct i2c_adapter *i2c, u8 adr);
-
 static int tuner_attach_tda18271(struct ngene_channel *chan)
 {
        struct i2c_adapter *i2c;
@@ -114,7 +112,7 @@ static int tuner_attach_tda18271(struct ngene_channel *chan)
        if (chan->fe->ops.i2c_gate_ctrl)
                chan->fe->ops.i2c_gate_ctrl(chan->fe, 0);
        if (!fe) {
-               printk("No TDA18271 found!\n");
+               printk(KERN_ERR "No TDA18271 found!\n");
                return -ENODEV;
        }
 
@@ -180,7 +178,7 @@ static void cineS2_tuner_i2c_lock(struct dvb_frontend *fe, int lock)
 static int i2c_read(struct i2c_adapter *adapter, u8 adr, u8 *val)
 {
        struct i2c_msg msgs[1] = {{.addr = adr,  .flags = I2C_M_RD,
-                                  .buf  = val,  .len   = 1 }};
+                                  .buf  = val,  .len   = 1 } };
        return (i2c_transfer(adapter, msgs, 1) == 1) ? 0 : -1;
 }
 
@@ -191,7 +189,7 @@ static int i2c_read_reg16(struct i2c_adapter *adapter, u8 adr,
        struct i2c_msg msgs[2] = {{.addr = adr, .flags = 0,
                                   .buf  = msg, .len   = 2},
                                  {.addr = adr, .flags = I2C_M_RD,
-                                  .buf  = val, .len   = 1}};
+                                  .buf  = val, .len   = 1} };
        return (i2c_transfer(adapter, msgs, 2) == 2) ? 0 : -1;
 }
 
@@ -209,25 +207,22 @@ static int port_has_drxk(struct i2c_adapter *i2c, int port)
 
        if (i2c_read(i2c, 0x29+port, &val) < 0)
                return 0;
-       printk("DRXK@%02x\n", 0x29+port);
        return 1;
 }
 
 static int demod_attach_drxk(struct ngene_channel *chan,
                             struct i2c_adapter *i2c)
 {
-       struct dvb_frontend *fe;
-
-       chan->fe = fe = dvb_attach(drxk_attach,
+       chan->fe = dvb_attach(drxk_attach,
                                   i2c, 0x29 + (chan->number^2),
                                   &chan->fe2);
        if (!chan->fe) {
-               printk("No DRXK found!\n");
+               printk(KERN_ERR "No DRXK found!\n");
                return -ENODEV;
        }
-       fe->sec_priv = chan;
-       chan->gate_ctrl = fe->ops.i2c_gate_ctrl;
-       fe->ops.i2c_gate_ctrl = drxk_gate_ctrl;
+       chan->fe->sec_priv = chan;
+       chan->gate_ctrl = chan->fe->ops.i2c_gate_ctrl;
+       chan->fe->ops.i2c_gate_ctrl = drxk_gate_ctrl;
        return 0;
 }
 
@@ -246,19 +241,18 @@ static int cineS2_probe(struct ngene_channel *chan)
                i2c = &chan->dev->channel[1].i2c_adapter;
 
        if (port_has_stv0900(i2c, chan->number)) {
-               chan->demod_type=0;
+               chan->demod_type = 0;
                fe_conf = chan->dev->card_info->fe_config[chan->number];
                /* demod found, attach it */
                rc = demod_attach_stv0900(chan);
                if (rc < 0 || chan->number < 2)
                        return rc;
-  
+
                /* demod #2: reprogram outputs DPN1 & DPN2 */
                i2c_msg.addr = fe_conf->address;
                i2c_msg.len = 3;
                buf[0] = 0xf1;
-               switch (chan->number)
-               {
+               switch (chan->number) {
                case 2:
                        buf[1] = 0x5c;
                        buf[2] = 0xc2;
@@ -276,10 +270,10 @@ static int cineS2_probe(struct ngene_channel *chan)
                        return -EIO;
                }
        } else if (port_has_drxk(i2c, chan->number^2)) {
-               chan->demod_type=1;
+               chan->demod_type = 1;
                demod_attach_drxk(chan, i2c);
        } else {
-               printk("No demod found on chan %d\n", chan->number);
+               printk(KERN_ERR "No demod found on chan %d\n", chan->number);
        }
        return 0;
 }
@@ -497,7 +491,7 @@ MODULE_DEVICE_TABLE(pci, ngene_id_tbl);
 /****************************************************************************/
 
 static pci_ers_result_t ngene_error_detected(struct pci_dev *dev,
-                                           enum pci_channel_state state)
+                                            enum pci_channel_state state)
 {
        printk(KERN_ERR DEVICE_NAME ": PCI error\n");
        if (state == pci_channel_io_perm_failure)
index c59bf5057a406c257e3c5ef97af1c09a0d6a8138..fa4b3eb92bd0564d731d0ed1480fbbf75efb9862 100644 (file)
@@ -41,7 +41,7 @@
 
 #include "ngene.h"
 
-static int one_adapter = 0;
+static int one_adapter;
 module_param(one_adapter, int, 0444);
 MODULE_PARM_DESC(one_adapter, "Use only one adapter.");
 
@@ -1443,9 +1443,9 @@ static void release_channel(struct ngene_channel *chan)
                chan->ci_dev = NULL;
        }
 
-       if (chan->fe2) {
+       if (chan->fe2)
                dvb_unregister_frontend(chan->fe2);
-       }
+
        if (chan->fe) {
                dvb_unregister_frontend(chan->fe);
                dvb_frontend_detach(chan->fe);
@@ -1540,10 +1540,10 @@ static int init_channel(struct ngene_channel *chan)
        if (chan->fe2) {
                if (dvb_register_frontend(adapter, chan->fe2) < 0)
                        goto err;
-                       chan->fe2->tuner_priv=chan->fe->tuner_priv;
-                       memcpy(&chan->fe2->ops.tuner_ops,
-                              &chan->fe->ops.tuner_ops,
-                              sizeof(struct dvb_tuner_ops));
+               chan->fe2->tuner_priv = chan->fe->tuner_priv;
+               memcpy(&chan->fe2->ops.tuner_ops,
+                      &chan->fe->ops.tuner_ops,
+                      sizeof(struct dvb_tuner_ops));
        }
 
        if (chan->has_demux) {
index 0b4943233166e67824ff39cbebb5a859fe4b1497..ba209cb3e0b178637baf6649f049d78f2930bfdd 100644 (file)
@@ -133,9 +133,9 @@ void *tsin_exchange(void *priv, void *buf, u32 len, u32 clock, u32 flags)
                }
                return 0;
        }
-       if (chan->users > 0) {
+       if (chan->users > 0)
                dvb_dmx_swfilter(&chan->demux, buf, len);
-       }
+
        return NULL;
 }
 
index e8cd93deba5ab096f49f6350ade0945aff2988bc..90fa136850c4fc786cc4ea444cf8710ce0a535fe 100644 (file)
@@ -855,7 +855,7 @@ struct ngene_info {
 };
 
 #ifdef NGENE_V4L
-struct ngene_format{
+struct ngene_format {
        char *name;
        int   fourcc;          /* video4linux 2      */
        int   btformat;        /* BT848_COLOR_FMT_*  */