diff options
author | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-11-02 06:13:54 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-11-03 12:59:03 -0500 |
commit | fb1817e48f49b0c4a9da262d9f3db5299dfd9a1c (patch) | |
tree | 5d3a55decbdee0eae2fa8840ef099247dff19d92 | |
parent | ec2a387eaf859415038fce698d5954c8ae437854 (diff) |
[media] cx231xx: use dev_info() for extension load/unload
Now that we're using dev_foo, the logs become like:
usb 1-2: DVB: registering adapter 0 frontend 0 (Fujitsu mb86A20s)...
usb 1-2: Successfully loaded cx231xx-dvb
cx231xx: Cx231xx dvb Extension initialized
It is not clear, by the logs, that usb 1-2 name is an alias for
cx231xx. So, we also need to use dvb_info() at extension load/unload.
After the patch, it will print:
usb 1-2: Cx231xx dvb Extension initialized
With is coherent with the other logs.
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
-rw-r--r-- | drivers/media/usb/cx231xx/cx231xx-core.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/media/usb/cx231xx/cx231xx-core.c b/drivers/media/usb/cx231xx/cx231xx-core.c index 36c3ecf204c1..64e907f02a02 100644 --- a/drivers/media/usb/cx231xx/cx231xx-core.c +++ b/drivers/media/usb/cx231xx/cx231xx-core.c | |||
@@ -98,10 +98,10 @@ int cx231xx_register_extension(struct cx231xx_ops *ops) | |||
98 | 98 | ||
99 | mutex_lock(&cx231xx_devlist_mutex); | 99 | mutex_lock(&cx231xx_devlist_mutex); |
100 | list_add_tail(&ops->next, &cx231xx_extension_devlist); | 100 | list_add_tail(&ops->next, &cx231xx_extension_devlist); |
101 | list_for_each_entry(dev, &cx231xx_devlist, devlist) | 101 | list_for_each_entry(dev, &cx231xx_devlist, devlist) { |
102 | ops->init(dev); | 102 | ops->init(dev); |
103 | 103 | dev_info(&dev->udev->dev, "%s initialized\n", ops->name); | |
104 | printk(KERN_INFO DRIVER_NAME ": %s initialized\n", ops->name); | 104 | } |
105 | mutex_unlock(&cx231xx_devlist_mutex); | 105 | mutex_unlock(&cx231xx_devlist_mutex); |
106 | return 0; | 106 | return 0; |
107 | } | 107 | } |
@@ -112,11 +112,11 @@ void cx231xx_unregister_extension(struct cx231xx_ops *ops) | |||
112 | struct cx231xx *dev = NULL; | 112 | struct cx231xx *dev = NULL; |
113 | 113 | ||
114 | mutex_lock(&cx231xx_devlist_mutex); | 114 | mutex_lock(&cx231xx_devlist_mutex); |
115 | list_for_each_entry(dev, &cx231xx_devlist, devlist) | 115 | list_for_each_entry(dev, &cx231xx_devlist, devlist) { |
116 | ops->fini(dev); | 116 | ops->fini(dev); |
117 | dev_info(&dev->udev->dev, "%s removed\n", ops->name); | ||
118 | } | ||
117 | 119 | ||
118 | |||
119 | printk(KERN_INFO DRIVER_NAME ": %s removed\n", ops->name); | ||
120 | list_del(&ops->next); | 120 | list_del(&ops->next); |
121 | mutex_unlock(&cx231xx_devlist_mutex); | 121 | mutex_unlock(&cx231xx_devlist_mutex); |
122 | } | 122 | } |