Skip to content

Commit

Permalink
[media] rtl28xxu: do not refcount rtl2832_sdr module
Browse files Browse the repository at this point in the history
This driver, rtl28xxu, offers frontend service for rtl2832_sdr
module, thus we are producer and rtl2832_sdr module is consumer.
Due to that, reference counting should be done in way rtl2832_sdr
takes refrence to rtl28xxu. Remove wrong refcount.

Signed-off-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
  • Loading branch information
Antti Palosaari authored and Mauro Carvalho Chehab committed Feb 3, 2015
1 parent a6f6243 commit 6a53fa2
Showing 1 changed file with 2 additions and 8 deletions.
10 changes: 2 additions & 8 deletions drivers/media/usb/dvb-usb-v2/rtl28xxu.c
Original file line number Diff line number Diff line change
Expand Up @@ -1142,16 +1142,12 @@ static int rtl2832u_tuner_attach(struct dvb_usb_adapter *adap)
pdata.v4l2_subdev = subdev;

request_module("%s", "rtl2832_sdr");
pdev = platform_device_register_data(&priv->i2c_client_demod->dev,
pdev = platform_device_register_data(&d->intf->dev,
"rtl2832_sdr",
PLATFORM_DEVID_AUTO,
&pdata, sizeof(pdata));
if (pdev == NULL || pdev->dev.driver == NULL)
break;
if (!try_module_get(pdev->dev.driver->owner)) {
platform_device_unregister(pdev);
break;
}
priv->platform_device_sdr = pdev;
break;
default:
Expand All @@ -1175,10 +1171,8 @@ static int rtl2832u_tuner_detach(struct dvb_usb_adapter *adap)

/* remove platform SDR */
pdev = priv->platform_device_sdr;
if (pdev) {
module_put(pdev->dev.driver->owner);
if (pdev)
platform_device_unregister(pdev);
}

/* remove I2C tuner */
client = priv->i2c_client_tuner;
Expand Down

0 comments on commit 6a53fa2

Please sign in to comment.