Skip to content

Commit

Permalink
[media] staging: as102: Unconditionally compile code dependent on DVB…
Browse files Browse the repository at this point in the history
…_CORE

The driver depends on DVB_CORE so there is no need for conditional
compilation of parts of the code depending on CONFIG_DVB_CORE as
the driver is never compiled with CONFIG_DVB_CORE* disabled.

Cc: Devin Heitmueller <dheitmueller@kernellabs.com>
Signed-off-by: Sylwester Nawrocki <snjw23@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
  • Loading branch information
Sylwester Nawrocki authored and Mauro Carvalho Chehab committed Nov 3, 2011
1 parent 8be62e7 commit ff7029f
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 23 deletions.
12 changes: 1 addition & 11 deletions drivers/staging/media/as102/as102_drv.c
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,7 @@
/* header file for Usb device driver*/
#include "as102_drv.h"
#include "as102_fw.h"

#if defined(CONFIG_DVB_CORE) || defined(CONFIG_DVB_CORE_MODULE)
#include "dvbdev.h"
#else
#warning >>> DVB_CORE not defined !!! <<<
#endif

int debug;
module_param_named(debug, debug, int, 0644);
Expand Down Expand Up @@ -65,7 +60,6 @@ MODULE_PARM_DESC(elna_enable, "Activate eLNA (default: on)");
DVB_DEFINE_MOD_OPT_ADAPTER_NR(adapter_nr);
#endif

#if defined(CONFIG_DVB_CORE) || defined(CONFIG_DVB_CORE_MODULE)
static void as102_stop_stream(struct as102_dev_t *dev)
{
struct as102_bus_adapter_t *bus_adap;
Expand Down Expand Up @@ -200,14 +194,12 @@ static int as102_dvb_dmx_stop_feed(struct dvb_demux_feed *dvbdmxfeed)
LEAVE();
return 0;
}
#endif

int as102_dvb_register(struct as102_dev_t *as102_dev)
{
int ret = 0;
ENTER();

#if defined(CONFIG_DVB_CORE) || defined(CONFIG_DVB_CORE_MODULE)
ret = dvb_register_adapter(&as102_dev->dvb_adap,
as102_dev->name,
THIS_MODULE,
Expand Down Expand Up @@ -260,7 +252,6 @@ int as102_dvb_register(struct as102_dev_t *as102_dev)
__func__, ret);
goto failed;
}
#endif

/* init bus mutex for token locking */
mutex_init(&as102_dev->bus_adap.lock);
Expand Down Expand Up @@ -288,7 +279,6 @@ void as102_dvb_unregister(struct as102_dev_t *as102_dev)
{
ENTER();

#if defined(CONFIG_DVB_CORE) || defined(CONFIG_DVB_CORE_MODULE)
/* unregister as102 frontend */
as102_dvb_unregister_fe(&as102_dev->dvb_fe);

Expand All @@ -298,7 +288,7 @@ void as102_dvb_unregister(struct as102_dev_t *as102_dev)

/* unregister dvb adapter */
dvb_unregister_adapter(&as102_dev->dvb_adap);
#endif

LEAVE();
}

Expand Down
6 changes: 0 additions & 6 deletions drivers/staging/media/as102/as102_drv.h
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,9 @@ extern struct usb_driver as102_usb_driver;
extern struct spi_driver as102_spi_driver;
#endif

#if defined(CONFIG_DVB_CORE) || defined(CONFIG_DVB_CORE_MODULE)
#include "dvb_demux.h"
#include "dvb_frontend.h"
#include "dmxdev.h"
#endif

#define DRIVER_FULL_NAME "Abilis Systems as10x usb driver"
#define DRIVER_NAME "as10x_usb"
Expand Down Expand Up @@ -112,12 +110,10 @@ struct as102_dev_t {
struct kref kref;
unsigned long minor;

#if defined(CONFIG_DVB_CORE) || defined(CONFIG_DVB_CORE_MODULE)
struct dvb_adapter dvb_adap;
struct dvb_frontend dvb_fe;
struct dvb_demux dvb_dmx;
struct dmxdev dvb_dmxdev;
#endif

/* demodulator stats */
struct as10x_demod_stats demod_stats;
Expand All @@ -139,9 +135,7 @@ struct as102_dev_t {
int as102_dvb_register(struct as102_dev_t *dev);
void as102_dvb_unregister(struct as102_dev_t *dev);

#if defined(CONFIG_DVB_CORE) || defined(CONFIG_DVB_CORE_MODULE)
int as102_dvb_register_fe(struct as102_dev_t *dev, struct dvb_frontend *fe);
int as102_dvb_unregister_fe(struct dvb_frontend *dev);
#endif

/* EOF - vim: set textwidth=80 ts=8 sw=8 sts=8 noet: */
2 changes: 0 additions & 2 deletions drivers/staging/media/as102/as102_fe.c
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@

extern int elna_enable;

#if defined(CONFIG_DVB_CORE) || defined(CONFIG_DVB_CORE_MODULE)
static void as10x_fe_copy_tps_parameters(struct dvb_frontend_parameters *dst,
struct as10x_tps *src);

Expand Down Expand Up @@ -672,6 +671,5 @@ static void as102_fe_copy_tune_parameters(struct as10x_tune_args *tune_args,
as102_fe_get_code_rate(params->u.ofdm.code_rate_HP);
}
}
#endif

/* EOF - vim: set textwidth=80 ts=8 sw=8 sts=8 noet: */
4 changes: 0 additions & 4 deletions drivers/staging/media/as102/as102_usb_drv.c
Original file line number Diff line number Diff line change
Expand Up @@ -214,13 +214,9 @@ void as102_urb_stream_irq(struct urb *urb)
struct as102_dev_t *as102_dev = urb->context;

if (urb->actual_length > 0) {
#if defined(CONFIG_DVB_CORE) || defined(CONFIG_DVB_CORE_MODULE)
dvb_dmx_swfilter(&as102_dev->dvb_dmx,
urb->transfer_buffer,
urb->actual_length);
#else
/* do nothing ? */
#endif
} else {
if (urb->actual_length == 0)
memset(urb->transfer_buffer, 0, AS102_USB_BUF_SIZE);
Expand Down

0 comments on commit ff7029f

Please sign in to comment.