Skip to content

Commit

Permalink
[media] dvb_usb_v2: get rid of dvb_usb_device state
Browse files Browse the repository at this point in the history
Remove state from struct dvb_usb_device as we can live without.
Add some debug logs.

Signed-off-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
  • Loading branch information
Antti Palosaari authored and Mauro Carvalho Chehab committed Aug 4, 2012
1 parent ad26183 commit d70521a
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 26 deletions.
6 changes: 0 additions & 6 deletions drivers/media/dvb/dvb-usb/dvb_usb.h
Original file line number Diff line number Diff line change
Expand Up @@ -324,12 +324,6 @@ struct dvb_usb_device {
struct work_struct probe_work;
pid_t work_pid;
struct usb_interface *intf;

#define DVB_USB_STATE_INIT 0x000
#define DVB_USB_STATE_I2C 0x001
#define DVB_USB_STATE_DVB 0x002
int state;

int powered;

/* locking */
Expand Down
32 changes: 12 additions & 20 deletions drivers/media/dvb/dvb-usb/dvb_usb_init.c
Original file line number Diff line number Diff line change
Expand Up @@ -72,26 +72,24 @@ static int dvb_usbv2_i2c_init(struct dvb_usb_device *d)
{
int ret;

if (!d->props->i2c_algo) {
ret = 0;
goto err;
}
pr_debug("%s:\n", __func__);

if (!d->props->i2c_algo)
return 0;

strlcpy(d->i2c_adap.name, d->name, sizeof(d->i2c_adap.name));
d->i2c_adap.algo = d->props->i2c_algo;
d->i2c_adap.algo_data = NULL;
d->i2c_adap.dev.parent = &d->udev->dev;

i2c_set_adapdata(&d->i2c_adap, d);

ret = i2c_add_adapter(&d->i2c_adap);
if (ret < 0) {
pr_err("%s: i2c_add_adapter() failed\n", KBUILD_MODNAME);
d->i2c_adap.algo = NULL;
pr_err("%s: i2c_add_adapter() failed=%d\n", KBUILD_MODNAME,
ret);
goto err;
}

d->state |= DVB_USB_STATE_I2C;

return 0;
err:
pr_debug("%s: failed=%d\n", __func__, ret);
Expand All @@ -100,10 +98,10 @@ static int dvb_usbv2_i2c_init(struct dvb_usb_device *d)

static int dvb_usbv2_i2c_exit(struct dvb_usb_device *d)
{
if (d->state & DVB_USB_STATE_I2C)
i2c_del_adapter(&d->i2c_adap);
pr_debug("%s:\n", __func__);

d->state &= ~DVB_USB_STATE_I2C;
if (d->i2c_adap.algo)
i2c_del_adapter(&d->i2c_adap);

return 0;
}
Expand Down Expand Up @@ -282,7 +280,6 @@ static int dvb_usbv2_adapter_init(struct dvb_usb_device *d)
adap->dvb_adap.mfe_shared = 1;

d->num_adapters_initialized++;
d->state |= DVB_USB_STATE_DVB;
}

return 0;
Expand All @@ -304,20 +301,18 @@ static int dvb_usbv2_adapter_exit(struct dvb_usb_device *d)
}

d->num_adapters_initialized = 0;
d->state &= ~DVB_USB_STATE_DVB;

return 0;
}

/* general initialization functions */
static int dvb_usbv2_exit(struct dvb_usb_device *d)
{
pr_debug("%s: state before exiting everything: %x\n", __func__, d->state);
pr_debug("%s:\n", __func__);

dvb_usbv2_remote_exit(d);
dvb_usbv2_adapter_exit(d);
dvb_usbv2_i2c_exit(d);
pr_debug("%s: state should be zero now: %x\n", __func__, d->state);
d->state = DVB_USB_STATE_INIT;
kfree(d->priv);
kfree(d);

Expand All @@ -328,9 +323,6 @@ static int dvb_usbv2_init(struct dvb_usb_device *d)
{
int ret = 0;

d->state = DVB_USB_STATE_INIT;

/* check the capabilities and set appropriate variables */
dvb_usbv2_device_power_ctrl(d, 1);

if (d->props->read_config) {
Expand Down

0 comments on commit d70521a

Please sign in to comment.