diff options
author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2009-12-10 08:47:13 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-12-15 21:17:57 -0500 |
commit | 50462eb065e12f45851a9959a90d46b758944552 (patch) | |
tree | fea5b79f1852183c5eaa3f079d5054dc78a828fa /drivers/media/video/cx88 | |
parent | 63b0d5ad20365edf8baf96cdbb8e7faf62501286 (diff) |
V4L/DVB (13555): v4l: Use video_device_node_name() instead of the minor number
Instead of using the minor number in kernel log messages, use the device
node name as returned by the video_device_node_name() function. This
makes debug, informational and error messages easier to understand for
end users.
[mchehab.redhat.com: removed tm6000 changes as tm6000 is not ready yet for submission even on staging]
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video/cx88')
-rw-r--r-- | drivers/media/video/cx88/cx88-blackbird.c | 4 | ||||
-rw-r--r-- | drivers/media/video/cx88/cx88-video.c | 5 |
2 files changed, 4 insertions, 5 deletions
diff --git a/drivers/media/video/cx88/cx88-blackbird.c b/drivers/media/video/cx88/cx88-blackbird.c index f0ef94150d1e..03e592e016fe 100644 --- a/drivers/media/video/cx88/cx88-blackbird.c +++ b/drivers/media/video/cx88/cx88-blackbird.c | |||
@@ -1048,7 +1048,7 @@ static int vidioc_s_std (struct file *file, void *priv, v4l2_std_id *id) | |||
1048 | 1048 | ||
1049 | static int mpeg_open(struct file *file) | 1049 | static int mpeg_open(struct file *file) |
1050 | { | 1050 | { |
1051 | int minor = video_devdata(file)->minor; | 1051 | struct video_device *vdev = video_devdata(file); |
1052 | struct cx8802_dev *dev = video_drvdata(file); | 1052 | struct cx8802_dev *dev = video_drvdata(file); |
1053 | struct cx8802_fh *fh; | 1053 | struct cx8802_fh *fh; |
1054 | struct cx8802_driver *drv = NULL; | 1054 | struct cx8802_driver *drv = NULL; |
@@ -1075,7 +1075,7 @@ static int mpeg_open(struct file *file) | |||
1075 | unlock_kernel(); | 1075 | unlock_kernel(); |
1076 | return -EINVAL; | 1076 | return -EINVAL; |
1077 | } | 1077 | } |
1078 | dprintk(1,"open minor=%d\n",minor); | 1078 | dprintk(1, "open dev=%s\n", video_device_node_name(vdev)); |
1079 | 1079 | ||
1080 | /* allocate + initialize per filehandle data */ | 1080 | /* allocate + initialize per filehandle data */ |
1081 | fh = kzalloc(sizeof(*fh),GFP_KERNEL); | 1081 | fh = kzalloc(sizeof(*fh),GFP_KERNEL); |
diff --git a/drivers/media/video/cx88/cx88-video.c b/drivers/media/video/cx88/cx88-video.c index 758399173a52..0013cb4d756a 100644 --- a/drivers/media/video/cx88/cx88-video.c +++ b/drivers/media/video/cx88/cx88-video.c | |||
@@ -749,7 +749,6 @@ static int get_ressource(struct cx8800_fh *fh) | |||
749 | 749 | ||
750 | static int video_open(struct file *file) | 750 | static int video_open(struct file *file) |
751 | { | 751 | { |
752 | int minor = video_devdata(file)->minor; | ||
753 | struct video_device *vdev = video_devdata(file); | 752 | struct video_device *vdev = video_devdata(file); |
754 | struct cx8800_dev *dev = video_drvdata(file); | 753 | struct cx8800_dev *dev = video_drvdata(file); |
755 | struct cx88_core *core; | 754 | struct cx88_core *core; |
@@ -773,8 +772,8 @@ static int video_open(struct file *file) | |||
773 | 772 | ||
774 | core = dev->core; | 773 | core = dev->core; |
775 | 774 | ||
776 | dprintk(1,"open minor=%d radio=%d type=%s\n", | 775 | dprintk(1, "open dev=%s radio=%d type=%s\n", |
777 | minor,radio,v4l2_type_names[type]); | 776 | video_device_node_name(vdev), radio, v4l2_type_names[type]); |
778 | 777 | ||
779 | /* allocate + initialize per filehandle data */ | 778 | /* allocate + initialize per filehandle data */ |
780 | fh = kzalloc(sizeof(*fh),GFP_KERNEL); | 779 | fh = kzalloc(sizeof(*fh),GFP_KERNEL); |