ChangeSet 1.2231.1.213, 2005/03/28 20:15:33-08:00, [EMAIL PROTECTED]

        [PATCH] dvb: cleanups, make stuff static
        
        This patch contains the following possible cleanups:
        o make needlessly global code static
        o remove the following EXPORT_SYMBOL'ed but unused function:
          - bt8xx/bt878.c: bt878_find_by_i2c_adap
        o remove the following unused global functions:
          - dvb-core/dvb_demux.c: dmx_get_demuxes
          - dvb-core/dvb_demux.c: dvb_set_crc32
        o remove the following unneeded EXPORT_SYMBOL's:
          - dvb-core/dvb_demux.c: dvb_dmx_swfilter_packet
          - dvb-core/dvb_demux.c: dvbdmx_connect_frontend
          - dvb-core/dvb_demux.c: dvbdmx_disconnect_frontend
          - dvb-core/dvbdev.c: dvb_class
        
        Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>
        Signed-off-by: Johannes Stezenbach <[EMAIL PROTECTED]>
        Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
        Signed-off-by: Linus Torvalds <[EMAIL PROTECTED]>



 b2c2/b2c2-common.c         |    2 +-
 b2c2/skystar2.c            |    2 +-
 bt8xx/bt878.c              |   15 ---------------
 bt8xx/dst_priv.h           |    1 -
 dibusb/dvb-dibusb-core.c   |    2 +-
 dibusb/dvb-dibusb-fe-i2c.c |    2 +-
 dibusb/dvb-dibusb.h        |    2 --
 dvb-core/demux.h           |    4 ----
 dvb-core/dvb_demux.c       |   36 ++++++------------------------------
 dvb-core/dvb_demux.h       |    4 ----
 dvb-core/dvb_net.c         |    4 ++--
 frontends/tda10021.c       |    2 +-
 ttpci/budget-av.c          |    4 ++--
 ttpci/budget-patch.c       |    2 +-
 14 files changed, 16 insertions(+), 66 deletions(-)


diff -Nru a/drivers/media/dvb/b2c2/b2c2-common.c 
b/drivers/media/dvb/b2c2/b2c2-common.c
--- a/drivers/media/dvb/b2c2/b2c2-common.c      2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/b2c2/b2c2-common.c      2005-03-28 21:55:00 -08:00
@@ -158,7 +158,7 @@
        return 0;
 }
 
-int samsung_tdtc9251dh0_pll_set(struct dvb_frontend* fe, struct 
dvb_frontend_parameters* params, u8* pllbuf)
+static int samsung_tdtc9251dh0_pll_set(struct dvb_frontend* fe, struct 
dvb_frontend_parameters* params, u8* pllbuf)
 {
        u32 div;
        unsigned char bs = 0;
diff -Nru a/drivers/media/dvb/b2c2/skystar2.c 
b/drivers/media/dvb/b2c2/skystar2.c
--- a/drivers/media/dvb/b2c2/skystar2.c 2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/b2c2/skystar2.c 2005-03-28 21:55:00 -08:00
@@ -2352,7 +2352,7 @@
        return 0;
 }
 
-int samsung_tdtc9251dh0_pll_set(struct dvb_frontend* fe, struct 
dvb_frontend_parameters* params, u8* pllbuf)
+static int samsung_tdtc9251dh0_pll_set(struct dvb_frontend* fe, struct 
dvb_frontend_parameters* params, u8* pllbuf)
 {
        u32 div;
        unsigned char bs = 0;
diff -Nru a/drivers/media/dvb/bt8xx/bt878.c b/drivers/media/dvb/bt8xx/bt878.c
--- a/drivers/media/dvb/bt8xx/bt878.c   2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/bt8xx/bt878.c   2005-03-28 21:55:00 -08:00
@@ -381,21 +381,6 @@
 
 EXPORT_SYMBOL(bt878_device_control);
 
-struct bt878 *bt878_find_by_i2c_adap(struct i2c_adapter *adapter)
-{
-       unsigned int card_nr;
-       
-       printk("bt878 find by dvb adap: checking \"%s\"\n",adapter->name);
-       for (card_nr = 0; card_nr < bt878_num; card_nr++) {
-               if (bt878[card_nr].adapter == adapter)
-                       return &bt878[card_nr];
-       }
-       printk("bt878 find by dvb adap: NOT found \"%s\"\n",adapter->name);
-       return NULL;
-}
-
-EXPORT_SYMBOL(bt878_find_by_i2c_adap);
-
 /***********************/
 /* PCI device handling */
 /***********************/
diff -Nru a/drivers/media/dvb/bt8xx/dst_priv.h 
b/drivers/media/dvb/bt8xx/dst_priv.h
--- a/drivers/media/dvb/bt8xx/dst_priv.h        2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/bt8xx/dst_priv.h        2005-03-28 21:55:00 -08:00
@@ -34,4 +34,3 @@
 
 int bt878_device_control(struct bt878 *bt, unsigned int cmd, union 
dst_gpio_packet *mp);
 
-struct bt878 *bt878_find_by_i2c_adap(struct i2c_adapter *adap);
diff -Nru a/drivers/media/dvb/dibusb/dvb-dibusb-core.c 
b/drivers/media/dvb/dibusb/dvb-dibusb-core.c
--- a/drivers/media/dvb/dibusb/dvb-dibusb-core.c        2005-03-28 21:55:00 
-08:00
+++ b/drivers/media/dvb/dibusb/dvb-dibusb-core.c        2005-03-28 21:55:00 
-08:00
@@ -505,7 +505,7 @@
 }
 
 /* usb specific object needed to register this driver with the usb subsystem */
-struct usb_driver dibusb_driver = {
+static struct usb_driver dibusb_driver = {
        .owner          = THIS_MODULE,
        .name           = DRIVER_DESC,
        .probe          = dibusb_probe,
diff -Nru a/drivers/media/dvb/dibusb/dvb-dibusb-fe-i2c.c 
b/drivers/media/dvb/dibusb/dvb-dibusb-fe-i2c.c
--- a/drivers/media/dvb/dibusb/dvb-dibusb-fe-i2c.c      2005-03-28 21:55:00 
-08:00
+++ b/drivers/media/dvb/dibusb/dvb-dibusb-fe-i2c.c      2005-03-28 21:55:00 
-08:00
@@ -14,7 +14,7 @@
 
 #include <linux/usb.h>
 
-int dibusb_i2c_msg(struct usb_dibusb *dib, u8 addr,
+static int dibusb_i2c_msg(struct usb_dibusb *dib, u8 addr,
                          u8 *wbuf, u16 wlen, u8 *rbuf, u16 rlen)
 {
        u8 sndbuf[wlen+4]; /* lead(1) devaddr,direction(1) addr(2) data(wlen) 
(len(2) (when reading)) */
diff -Nru a/drivers/media/dvb/dibusb/dvb-dibusb.h 
b/drivers/media/dvb/dibusb/dvb-dibusb.h
--- a/drivers/media/dvb/dibusb/dvb-dibusb.h     2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/dibusb/dvb-dibusb.h     2005-03-28 21:55:00 -08:00
@@ -210,8 +210,6 @@
 int dibusb_remote_init(struct usb_dibusb *dib);
 
 /* dvb-dibusb-fe-i2c.c */
-int dibusb_i2c_msg(struct usb_dibusb *dib, u8 addr,
-               u8 *wbuf, u16 wlen, u8 *rbuf, u16 rlen);
 int dibusb_fe_init(struct usb_dibusb* dib);
 int dibusb_fe_exit(struct usb_dibusb *dib);
 int dibusb_i2c_init(struct usb_dibusb *dib);
diff -Nru a/drivers/media/dvb/dvb-core/demux.h 
b/drivers/media/dvb/dvb-core/demux.h
--- a/drivers/media/dvb/dvb-core/demux.h        2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/dvb-core/demux.h        2005-03-28 21:55:00 -08:00
@@ -298,8 +298,4 @@
 
 #define DMX_DIR_ENTRY(list) list_entry(list, struct dmx_demux, reg_list)
 
-int dmx_register_demux (struct dmx_demux* demux);
-int dmx_unregister_demux (struct dmx_demux* demux);
-struct list_head* dmx_get_demuxes (void);
-
 #endif /* #ifndef __DEMUX_H */
diff -Nru a/drivers/media/dvb/dvb-core/dvb_demux.c 
b/drivers/media/dvb/dvb-core/dvb_demux.c
--- a/drivers/media/dvb/dvb-core/dvb_demux.c    2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/dvb-core/dvb_demux.c    2005-03-28 21:55:00 -08:00
@@ -39,17 +39,17 @@
 // #define DVB_DEMUX_SECTION_LOSS_LOG
 
 
-LIST_HEAD(dmx_muxs);
+static LIST_HEAD(dmx_muxs);
 
 
-int dmx_register_demux(struct dmx_demux *demux) 
+static int dmx_register_demux(struct dmx_demux *demux)
 {
        demux->users = 0;
        list_add(&demux->reg_list, &dmx_muxs);
        return 0;
 }
 
-int dmx_unregister_demux(struct dmx_demux* demux)
+static int dmx_unregister_demux(struct dmx_demux* demux)
 {
        struct list_head *pos, *n, *head=&dmx_muxs;
 
@@ -66,14 +66,6 @@
 }
 
 
-struct list_head *dmx_get_demuxes(void)
-{
-       if (list_empty(&dmx_muxs))
-               return NULL;
-
-       return &dmx_muxs;
-}
-
 /******************************************************************************
  * static inlined helper functions
  
******************************************************************************/
@@ -105,19 +97,6 @@
 }
 
 
-void dvb_set_crc32(u8 *data, int length)
-{
-       u32 crc;
-
-       crc = crc32_be(~0, data, length);
-
-       data[length]   = (crc >> 24) & 0xff;
-       data[length+1] = (crc >> 16) & 0xff;
-       data[length+2] = (crc >>  8) & 0xff;
-       data[length+3] = (crc)       & 0xff;
-}
-
-
 static u32 dvb_dmx_crc32 (struct dvb_demux_feed *f, const u8 *src, size_t len)
 {
        return (f->feed.sec.crc_val = crc32_be (f->feed.sec.crc_val, src, len));
@@ -424,7 +403,7 @@
        ((f)->feed.ts.is_filtering) &&                                  \
        (((f)->ts_type & (TS_PACKET|TS_PAYLOAD_ONLY)) == TS_PACKET))
 
-void dvb_dmx_swfilter_packet(struct dvb_demux *demux, const u8 *buf)
+static void dvb_dmx_swfilter_packet(struct dvb_demux *demux, const u8 *buf)
 {
        struct dvb_demux_feed *feed;
        struct list_head *pos, *head=&demux->feed_list;
@@ -452,7 +431,6 @@
                        feed->cb.ts(buf, 188, NULL, 0, &feed->feed.ts, DMX_OK);
        }
 }
-EXPORT_SYMBOL(dvb_dmx_swfilter_packet);
 
 void dvb_dmx_swfilter_packets(struct dvb_demux *demux, const u8 *buf, size_t 
count)
 {
@@ -1190,7 +1168,7 @@
 }
 
 
-int dvbdmx_connect_frontend(struct dmx_demux *demux, struct dmx_frontend 
*frontend)
+static int dvbdmx_connect_frontend(struct dmx_demux *demux, struct 
dmx_frontend *frontend)
 {
        struct dvb_demux *dvbdemux = (struct dvb_demux *) demux;
 
@@ -1204,10 +1182,9 @@
        up(&dvbdemux->mutex);
        return 0;
 }
-EXPORT_SYMBOL(dvbdmx_connect_frontend);
 
 
-int dvbdmx_disconnect_frontend(struct dmx_demux *demux)
+static int dvbdmx_disconnect_frontend(struct dmx_demux *demux)
 {
        struct dvb_demux *dvbdemux = (struct dvb_demux *) demux;
 
@@ -1218,7 +1195,6 @@
        up(&dvbdemux->mutex);
        return 0;
 }
-EXPORT_SYMBOL(dvbdmx_disconnect_frontend);
 
 
 static int dvbdmx_get_pes_pids(struct dmx_demux *demux, u16 *pids)
diff -Nru a/drivers/media/dvb/dvb-core/dvb_demux.h 
b/drivers/media/dvb/dvb-core/dvb_demux.h
--- a/drivers/media/dvb/dvb-core/dvb_demux.h    2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/dvb-core/dvb_demux.h    2005-03-28 21:55:00 -08:00
@@ -138,12 +138,8 @@
 
 int dvb_dmx_init(struct dvb_demux *dvbdemux);
 int dvb_dmx_release(struct dvb_demux *dvbdemux);
-void dvb_dmx_swfilter_packet(struct dvb_demux *dvbdmx, const u8 *buf);
 void dvb_dmx_swfilter_packets(struct dvb_demux *dvbdmx, const u8 *buf, size_t 
count);
 void dvb_dmx_swfilter(struct dvb_demux *demux, const u8 *buf, size_t count);
 void dvb_dmx_swfilter_204(struct dvb_demux *demux, const u8 *buf, size_t 
count);
-
-int dvbdmx_connect_frontend(struct dmx_demux *demux, struct dmx_frontend 
*frontend);
-int dvbdmx_disconnect_frontend(struct dmx_demux *demux);
 
 #endif /* _DVB_DEMUX_H_ */
diff -Nru a/drivers/media/dvb/dvb-core/dvb_net.c 
b/drivers/media/dvb/dvb-core/dvb_net.c
--- a/drivers/media/dvb/dvb-core/dvb_net.c      2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/dvb-core/dvb_net.c      2005-03-28 21:55:00 -08:00
@@ -217,12 +217,12 @@
 #define ULE_TEST       0
 #define ULE_BRIDGED    1
 
-int ule_test_sndu( struct dvb_net_priv *p )
+static int ule_test_sndu( struct dvb_net_priv *p )
 {
        return -1;
 }
 
-int ule_bridged_sndu( struct dvb_net_priv *p )
+static int ule_bridged_sndu( struct dvb_net_priv *p )
 {
        /* BRIDGE SNDU handling sucks in draft-ietf-ipdvb-ule-03.txt.
         * This has to be the last extension header, otherwise it won't work.
diff -Nru a/drivers/media/dvb/frontends/tda10021.c 
b/drivers/media/dvb/frontends/tda10021.c
--- a/drivers/media/dvb/frontends/tda10021.c    2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/frontends/tda10021.c    2005-03-28 21:55:00 -08:00
@@ -61,7 +61,7 @@
 
 #define FIN (XIN >> 4)
 
-int tda10021_inittab_size = 0x40;
+static int tda10021_inittab_size = 0x40;
 static u8 tda10021_inittab[0x40]=
 {
        0x73, 0x6a, 0x23, 0x0a, 0x02, 0x37, 0x77, 0x1a,
diff -Nru a/drivers/media/dvb/ttpci/budget-av.c 
b/drivers/media/dvb/ttpci/budget-av.c
--- a/drivers/media/dvb/ttpci/budget-av.c       2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/ttpci/budget-av.c       2005-03-28 21:55:00 -08:00
@@ -59,7 +59,7 @@
        struct dvb_ca_en50221 ca;
 };
 
-int enable_ci = 0;
+static int enable_ci = 0;
 
 
 /****************************************************************************
@@ -658,7 +658,7 @@
        return request_firmware(fw, name, &budget->dev->pci->dev);
 }
 
-struct tda1004x_config philips_tu1216_config = {
+static struct tda1004x_config philips_tu1216_config = {
 
        .demod_address = 0x8,
        .invert = 1,
diff -Nru a/drivers/media/dvb/ttpci/budget-patch.c 
b/drivers/media/dvb/ttpci/budget-patch.c
--- a/drivers/media/dvb/ttpci/budget-patch.c    2005-03-28 21:55:00 -08:00
+++ b/drivers/media/dvb/ttpci/budget-patch.c    2005-03-28 21:55:00 -08:00
@@ -406,7 +406,7 @@
        return 0;
 }
 
-struct tda8083_config grundig_29504_451_config = {
+static struct tda8083_config grundig_29504_451_config = {
        .demod_address = 0x68,
        .pll_set = grundig_29504_451_pll_set,
 };
-
To unsubscribe from this list: send the line "unsubscribe bk-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to