aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Vetter <daniel.vetter@ffwll.ch>2013-11-03 16:24:37 -0500
committerDaniel Vetter <daniel.vetter@ffwll.ch>2014-04-23 04:32:53 -0400
commit9de1b51f1fae6476155350a0670dc637c762e718 (patch)
treeda8efba2fe0c63586aa3e789179f75cdbcbd44e4
parent5829d1834e5486e83547f36576b160023c9609c2 (diff)
drm: remove drm_bus->get_name
The only user is the info debugfs file, so we only need something human readable. Now for both pci and platform devices we've used the name of the underlying device driver, which matches the name of the drm driver in all cases. So we can just use that instead. The exception is usb, which used a generic "USB". Not to harmful with just one usb driver, but better to use "udl", too. With that converted we can rip out all the ->get_name implementations. Reviewed-by: Thierry Reding <treding@nvidia.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
-rw-r--r--drivers/gpu/drm/drm_info.c6
-rw-r--r--drivers/gpu/drm/drm_pci.c7
-rw-r--r--drivers/gpu/drm/drm_platform.c6
-rw-r--r--drivers/gpu/drm/drm_usb.c6
-rw-r--r--include/drm/drmP.h1
5 files changed, 2 insertions, 24 deletions
diff --git a/drivers/gpu/drm/drm_info.c b/drivers/gpu/drm/drm_info.c
index 7473035dd28b..86feedd5e6f6 100644
--- a/drivers/gpu/drm/drm_info.c
+++ b/drivers/gpu/drm/drm_info.c
@@ -47,18 +47,16 @@ int drm_name_info(struct seq_file *m, void *data)
47 struct drm_minor *minor = node->minor; 47 struct drm_minor *minor = node->minor;
48 struct drm_device *dev = minor->dev; 48 struct drm_device *dev = minor->dev;
49 struct drm_master *master = minor->master; 49 struct drm_master *master = minor->master;
50 const char *bus_name;
51 if (!master) 50 if (!master)
52 return 0; 51 return 0;
53 52
54 bus_name = dev->driver->bus->get_name(dev);
55 if (master->unique) { 53 if (master->unique) {
56 seq_printf(m, "%s %s %s\n", 54 seq_printf(m, "%s %s %s\n",
57 bus_name, 55 dev->driver->name,
58 dev_name(dev->dev), master->unique); 56 dev_name(dev->dev), master->unique);
59 } else { 57 } else {
60 seq_printf(m, "%s %s\n", 58 seq_printf(m, "%s %s\n",
61 bus_name, dev_name(dev->dev)); 59 dev->driver->name, dev_name(dev->dev));
62 } 60 }
63 return 0; 61 return 0;
64} 62}
diff --git a/drivers/gpu/drm/drm_pci.c b/drivers/gpu/drm/drm_pci.c
index 047c51046d94..1fbe22a254e1 100644
--- a/drivers/gpu/drm/drm_pci.c
+++ b/drivers/gpu/drm/drm_pci.c
@@ -137,12 +137,6 @@ static int drm_get_pci_domain(struct drm_device *dev)
137 return pci_domain_nr(dev->pdev->bus); 137 return pci_domain_nr(dev->pdev->bus);
138} 138}
139 139
140static const char *drm_pci_get_name(struct drm_device *dev)
141{
142 struct pci_driver *pdriver = dev->driver->kdriver.pci;
143 return pdriver->name;
144}
145
146static int drm_pci_set_busid(struct drm_device *dev, struct drm_master *master) 140static int drm_pci_set_busid(struct drm_device *dev, struct drm_master *master)
147{ 141{
148 int len, ret; 142 int len, ret;
@@ -287,7 +281,6 @@ void drm_pci_agp_destroy(struct drm_device *dev)
287} 281}
288 282
289static struct drm_bus drm_pci_bus = { 283static struct drm_bus drm_pci_bus = {
290 .get_name = drm_pci_get_name,
291 .set_busid = drm_pci_set_busid, 284 .set_busid = drm_pci_set_busid,
292}; 285};
293 286
diff --git a/drivers/gpu/drm/drm_platform.c b/drivers/gpu/drm/drm_platform.c
index b97b11ea2dc4..c7ec27bbe7c6 100644
--- a/drivers/gpu/drm/drm_platform.c
+++ b/drivers/gpu/drm/drm_platform.c
@@ -68,11 +68,6 @@ err_free:
68 return ret; 68 return ret;
69} 69}
70 70
71static const char *drm_platform_get_name(struct drm_device *dev)
72{
73 return dev->platformdev->name;
74}
75
76static int drm_platform_set_busid(struct drm_device *dev, struct drm_master *master) 71static int drm_platform_set_busid(struct drm_device *dev, struct drm_master *master)
77{ 72{
78 int len, ret, id; 73 int len, ret, id;
@@ -107,7 +102,6 @@ err:
107} 102}
108 103
109static struct drm_bus drm_platform_bus = { 104static struct drm_bus drm_platform_bus = {
110 .get_name = drm_platform_get_name,
111 .set_busid = drm_platform_set_busid, 105 .set_busid = drm_platform_set_busid,
112}; 106};
113 107
diff --git a/drivers/gpu/drm/drm_usb.c b/drivers/gpu/drm/drm_usb.c
index abdc265c9cc8..fae4aa4e1426 100644
--- a/drivers/gpu/drm/drm_usb.c
+++ b/drivers/gpu/drm/drm_usb.c
@@ -36,11 +36,6 @@ err_free:
36} 36}
37EXPORT_SYMBOL(drm_get_usb_dev); 37EXPORT_SYMBOL(drm_get_usb_dev);
38 38
39static const char *drm_usb_get_name(struct drm_device *dev)
40{
41 return "USB";
42}
43
44static int drm_usb_set_busid(struct drm_device *dev, 39static int drm_usb_set_busid(struct drm_device *dev,
45 struct drm_master *master) 40 struct drm_master *master)
46{ 41{
@@ -48,7 +43,6 @@ static int drm_usb_set_busid(struct drm_device *dev,
48} 43}
49 44
50static struct drm_bus drm_usb_bus = { 45static struct drm_bus drm_usb_bus = {
51 .get_name = drm_usb_get_name,
52 .set_busid = drm_usb_set_busid, 46 .set_busid = drm_usb_set_busid,
53}; 47};
54 48
diff --git a/include/drm/drmP.h b/include/drm/drmP.h
index 56b028d6bc98..493bbbb300e6 100644
--- a/include/drm/drmP.h
+++ b/include/drm/drmP.h
@@ -726,7 +726,6 @@ struct drm_master {
726#define DRM_SCANOUTPOS_ACCURATE (1 << 2) 726#define DRM_SCANOUTPOS_ACCURATE (1 << 2)
727 727
728struct drm_bus { 728struct drm_bus {
729 const char *(*get_name)(struct drm_device *dev);
730 int (*set_busid)(struct drm_device *dev, struct drm_master *master); 729 int (*set_busid)(struct drm_device *dev, struct drm_master *master);
731}; 730};
732 731