Messages in this thread | | | Date | Sun, 08 Jan 2006 11:07:08 +0400 | From | Manu Abraham <> | Subject | Re: [RFC: 2.6 patch] drivers/media/dvb/: possible cleanups |
| |
Adrian Bunk wrote:
>This patch contains the following possible cleanups: >- make needlessly global code static >- #if 0 the following unused global functions: > - b2c2/flexcop-dma.c: flexcop_dma_control_packet_irq() > - b2c2/flexcop-dma.c: flexcop_dma_config_packet_count() > >Please review which of these changes do make sense and which conflict >with pending patches. > > >Signed-off-by: Adrian Bunk <bunk@stusta.de> > > > u8 dst_check_sum(u8 * buf, u32 len); > struct dst_state* dst_attach(struct dst_state* state, struct dvb_adapter *dvb_adapter); > int dst_ca_attach(struct dst_state *state, struct dvb_adapter *dvb_adapter); >-int dst_gpio_outb(struct dst_state* state, u32 mask, u32 enbb, u32 outhigh, int delay); >- >-int dst_command(struct dst_state* state, u8 * data, u8 len); > > > #endif // DST_COMMON_H >--- linux-2.6.15-mm2-full/drivers/media/dvb/bt8xx/dst.c.old 2006-01-07 16:42:33.000000000 +0100 >+++ linux-2.6.15-mm2-full/drivers/media/dvb/bt8xx/dst.c 2006-01-07 16:54:56.000000000 +0100 >@@ -63,6 +63,7 @@ > } \ > } while(0) > >+static int dst_command(struct dst_state *state, u8 *data, u8 len); > > static void dst_packsize(struct dst_state *state, int psize) > { >@@ -72,7 +73,8 @@ > bt878_device_control(state->bt, DST_IG_TS, &bits); > } > >-int dst_gpio_outb(struct dst_state *state, u32 mask, u32 enbb, u32 outhigh, int delay) >+static int dst_gpio_outb(struct dst_state *state, u32 mask, u32 enbb, >+ u32 outhigh, int delay) > { > union dst_gpio_packet enb; > union dst_gpio_packet bits; >@@ -101,9 +103,8 @@ > > return 0; > } >-EXPORT_SYMBOL(dst_gpio_outb); > >-int dst_gpio_inb(struct dst_state *state, u8 *result) >+static int dst_gpio_inb(struct dst_state *state, u8 *result) > { > union dst_gpio_packet rd_packet; > int err; >@@ -117,7 +118,6 @@ > > return 0; > } >-EXPORT_SYMBOL(dst_gpio_inb); > > int rdc_reset_state(struct dst_state *state) > { >@@ -137,7 +137,7 @@ > } > EXPORT_SYMBOL(rdc_reset_state); > >-int rdc_8820_reset(struct dst_state *state) >+static int rdc_8820_reset(struct dst_state *state) > { > dprintk(verbose, DST_DEBUG, 1, "Resetting DST"); > if (dst_gpio_outb(state, RDC_8820_RESET, RDC_8820_RESET, 0, NO_DELAY) < 0) { >@@ -152,9 +152,8 @@ > > return 0; > } >-EXPORT_SYMBOL(rdc_8820_reset); > >-int dst_pio_enable(struct dst_state *state) >+static int dst_pio_enable(struct dst_state *state) > { > if (dst_gpio_outb(state, ~0, RDC_8820_PIO_0_ENABLE, 0, NO_DELAY) < 0) { > dprintk(verbose, DST_ERROR, 1, "dst_gpio_outb ERROR !"); >@@ -164,7 +163,6 @@ > > return 0; > } >-EXPORT_SYMBOL(dst_pio_enable); > > int dst_pio_disable(struct dst_state *state) > { >@@ -602,7 +600,7 @@ > > */ > >-struct dst_types dst_tlist[] = { >+static struct dst_types dst_tlist[] = { > { > .device_id = "200103A", > .offset = 0, >@@ -958,7 +956,7 @@ > return 0; > } > >-int dst_command(struct dst_state *state, u8 *data, u8 len) >+static int dst_command(struct dst_state *state, u8 *data, u8 len) > { > u8 reply; > >@@ -1021,7 +1019,6 @@ > return -EIO; > > } >-EXPORT_SYMBOL(dst_command); > > static int dst_get_signal(struct dst_state *state) > { > >
dst is currently a bit broken ATM and there are quite a bit changes that haven't yet gone into linuxtv.org CVS tree. So i would be of the opinion, that if you can hold off on the patches to the dst/dst_ca module till it's fixed, that would be quite helpful.
Thanks, Manu
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |