Commit bd16168d authored by David Härdeman's avatar David Härdeman Committed by Mauro Carvalho Chehab

[media] lirc_zilog: remove module parameter minor

Always let the kernel decide what minor the lirc chardev gets.
Signed-off-by: default avatarDavid Härdeman <david@hardeman.nu>
Signed-off-by: default avatarSean Young <sean@mess.org>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent 56481f00
...@@ -156,7 +156,6 @@ static struct mutex tx_data_lock; ...@@ -156,7 +156,6 @@ static struct mutex tx_data_lock;
/* module parameters */ /* module parameters */
static bool debug; /* debug output */ static bool debug; /* debug output */
static bool tx_only; /* only handle the IR Tx function */ static bool tx_only; /* only handle the IR Tx function */
static int minor = -1; /* minor number */
/* struct IR reference counting */ /* struct IR reference counting */
...@@ -184,10 +183,11 @@ static void release_ir_device(struct kref *ref) ...@@ -184,10 +183,11 @@ static void release_ir_device(struct kref *ref)
* ir->open_count == 0 - happens on final close() * ir->open_count == 0 - happens on final close()
* ir_lock, tx_ref_lock, rx_ref_lock, all released * ir_lock, tx_ref_lock, rx_ref_lock, all released
*/ */
if (ir->l.minor >= 0 && ir->l.minor < MAX_IRCTL_DEVICES) { if (ir->l.minor >= 0) {
lirc_unregister_driver(ir->l.minor); lirc_unregister_driver(ir->l.minor);
ir->l.minor = MAX_IRCTL_DEVICES; ir->l.minor = -1;
} }
if (kfifo_initialized(&ir->rbuf.fifo)) if (kfifo_initialized(&ir->rbuf.fifo))
lirc_buffer_free(&ir->rbuf); lirc_buffer_free(&ir->rbuf);
list_del(&ir->list); list_del(&ir->list);
...@@ -1597,12 +1597,11 @@ static int ir_probe(struct i2c_client *client, const struct i2c_device_id *id) ...@@ -1597,12 +1597,11 @@ static int ir_probe(struct i2c_client *client, const struct i2c_device_id *id)
} }
/* register with lirc */ /* register with lirc */
ir->l.minor = minor; /* module option: user requested minor number */
ir->l.minor = lirc_register_driver(&ir->l); ir->l.minor = lirc_register_driver(&ir->l);
if (ir->l.minor < 0 || ir->l.minor >= MAX_IRCTL_DEVICES) { if (ir->l.minor < 0) {
dev_err(tx->ir->l.dev, dev_err(tx->ir->l.dev,
"%s: \"minor\" must be between 0 and %d (%d)!\n", "%s: lirc_register_driver() failed: %i\n",
__func__, MAX_IRCTL_DEVICES-1, ir->l.minor); __func__, ir->l.minor);
ret = -EBADRQC; ret = -EBADRQC;
goto out_put_xx; goto out_put_xx;
} }
...@@ -1674,9 +1673,6 @@ MODULE_LICENSE("GPL"); ...@@ -1674,9 +1673,6 @@ MODULE_LICENSE("GPL");
/* for compat with old name, which isn't all that accurate anymore */ /* for compat with old name, which isn't all that accurate anymore */
MODULE_ALIAS("lirc_pvr150"); MODULE_ALIAS("lirc_pvr150");
module_param(minor, int, 0444);
MODULE_PARM_DESC(minor, "Preferred minor device number");
module_param(debug, bool, 0644); module_param(debug, bool, 0644);
MODULE_PARM_DESC(debug, "Enable debugging messages"); MODULE_PARM_DESC(debug, "Enable debugging messages");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment