* A gentle scrub
@ 2013-08-19 23:53 Damien Lespiau
2013-08-19 23:53 ` [PATCH 1/8] drm: Remove stale prototypes Damien Lespiau
` (8 more replies)
0 siblings, 9 replies; 12+ messages in thread
From: Damien Lespiau @ 2013-08-19 23:53 UTC (permalink / raw)
To: dri-devel
A small pass on drm headers to remove some stale prototypes/functions/defines
and to make static a few functions.
drivers/gpu/drm/drm_crtc.c | 38 +++----------------------------------
drivers/gpu/drm/drm_fb_cma_helper.c | 5 ++---
drivers/gpu/drm/drm_modes.c | 21 --------------------
drivers/gpu/drm/drm_platform.c | 7 +++----
include/drm/drmP.h | 8 --------
include/drm/drm_crtc.h | 17 -----------------
include/drm/drm_fb_cma_helper.h | 1 -
include/drm/drm_pciids.h | 30 -----------------------------
8 files changed, 8 insertions(+), 119 deletions(-)
--
Damien
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH 1/8] drm: Remove stale prototypes 2013-08-19 23:53 A gentle scrub Damien Lespiau @ 2013-08-19 23:53 ` Damien Lespiau 2013-08-19 23:53 ` [PATCH 2/8] drm: Remove drm_mode_create_dithering_property() Damien Lespiau ` (7 subsequent siblings) 8 siblings, 0 replies; 12+ messages in thread From: Damien Lespiau @ 2013-08-19 23:53 UTC (permalink / raw) To: dri-devel A few prototypes have been left in the headers, their function friends long gone. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> --- include/drm/drmP.h | 5 ----- include/drm/drm_crtc.h | 11 ----------- 2 files changed, 16 deletions(-) diff --git a/include/drm/drmP.h b/include/drm/drmP.h index 39911dc..046a7db 100644 --- a/include/drm/drmP.h +++ b/include/drm/drmP.h @@ -1371,7 +1371,6 @@ extern int drm_irq_uninstall(struct drm_device *dev); extern int drm_vblank_init(struct drm_device *dev, int num_crtcs); extern int drm_wait_vblank(struct drm_device *dev, void *data, struct drm_file *filp); -extern int drm_vblank_wait(struct drm_device *dev, unsigned int *vbl_seq); extern u32 drm_vblank_count(struct drm_device *dev, int crtc); extern u32 drm_vblank_count_and_time(struct drm_device *dev, int crtc, struct timeval *vblanktime); @@ -1491,10 +1490,6 @@ void drm_prime_destroy_file_private(struct drm_prime_file_private *prime_fpriv); int drm_prime_lookup_buf_handle(struct drm_prime_file_private *prime_fpriv, struct dma_buf *dma_buf, uint32_t *handle); void drm_prime_remove_buf_handle(struct drm_prime_file_private *prime_fpriv, struct dma_buf *dma_buf); -int drm_prime_add_dma_buf(struct drm_device *dev, struct drm_gem_object *obj); -int drm_prime_lookup_obj(struct drm_device *dev, struct dma_buf *buf, - struct drm_gem_object **obj); - #if DRM_DEBUG_CODE extern int drm_vma_info(struct seq_file *m, void *data); #endif diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index 45f1332..eb40a96 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h @@ -948,9 +948,6 @@ extern int drm_object_property_set_value(struct drm_mode_object *obj, extern int drm_object_property_get_value(struct drm_mode_object *obj, struct drm_property *property, uint64_t *value); -extern struct drm_display_mode *drm_crtc_mode_create(struct drm_device *dev); -extern void drm_framebuffer_set_object(struct drm_device *dev, - unsigned long handle); extern int drm_framebuffer_init(struct drm_device *dev, struct drm_framebuffer *fb, const struct drm_framebuffer_funcs *funcs); @@ -961,10 +958,6 @@ extern void drm_framebuffer_reference(struct drm_framebuffer *fb); extern void drm_framebuffer_remove(struct drm_framebuffer *fb); extern void drm_framebuffer_cleanup(struct drm_framebuffer *fb); extern void drm_framebuffer_unregister_private(struct drm_framebuffer *fb); -extern int drmfb_probe(struct drm_device *dev, struct drm_crtc *crtc); -extern int drmfb_remove(struct drm_device *dev, struct drm_framebuffer *fb); -extern void drm_crtc_probe_connector_modes(struct drm_device *dev, int maxX, int maxY); -extern bool drm_crtc_in_use(struct drm_crtc *crtc); extern void drm_object_attach_property(struct drm_mode_object *obj, struct drm_property *property, @@ -1039,10 +1032,6 @@ extern int drm_mode_getblob_ioctl(struct drm_device *dev, void *data, struct drm_file *file_priv); extern int drm_mode_connector_property_set_ioctl(struct drm_device *dev, void *data, struct drm_file *file_priv); -extern int drm_mode_hotplug_ioctl(struct drm_device *dev, - void *data, struct drm_file *file_priv); -extern int drm_mode_replacefb(struct drm_device *dev, - void *data, struct drm_file *file_priv); extern int drm_mode_getencoder(struct drm_device *dev, void *data, struct drm_file *file_priv); extern int drm_mode_gamma_get_ioctl(struct drm_device *dev, -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/8] drm: Remove drm_mode_create_dithering_property() 2013-08-19 23:53 A gentle scrub Damien Lespiau 2013-08-19 23:53 ` [PATCH 1/8] drm: Remove stale prototypes Damien Lespiau @ 2013-08-19 23:53 ` Damien Lespiau 2013-08-19 23:53 ` [PATCH 3/8] drm: Remove drm_mode_list_concat() Damien Lespiau ` (6 subsequent siblings) 8 siblings, 0 replies; 12+ messages in thread From: Damien Lespiau @ 2013-08-19 23:53 UTC (permalink / raw) To: dri-devel This was last used by nouveau, replaced by a driver-specific property in: commit de69185573586302ada2e59ba41835df36986277 Author: Ben Skeggs <bskeggs@redhat.com> Date: Mon Oct 17 12:23:41 2011 +1000 drm/nouveau: improve dithering properties, and implement proper auto mode Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> --- drivers/gpu/drm/drm_crtc.c | 31 ------------------------------- include/drm/drm_crtc.h | 1 - 2 files changed, 32 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index a691764..ffb791f 100644 --- a/drivers/gpu/drm/drm_crtc.c +++ b/drivers/gpu/drm/drm_crtc.c @@ -125,13 +125,6 @@ static const struct drm_prop_enum_list drm_scaling_mode_enum_list[] = { DRM_MODE_SCALE_ASPECT, "Full aspect" }, }; -static const struct drm_prop_enum_list drm_dithering_mode_enum_list[] = -{ - { DRM_MODE_DITHERING_OFF, "Off" }, - { DRM_MODE_DITHERING_ON, "On" }, - { DRM_MODE_DITHERING_AUTO, "Automatic" }, -}; - /* * Non-global properties, but "required" for certain connectors. */ @@ -1161,30 +1154,6 @@ int drm_mode_create_scaling_mode_property(struct drm_device *dev) EXPORT_SYMBOL(drm_mode_create_scaling_mode_property); /** - * drm_mode_create_dithering_property - create dithering property - * @dev: DRM device - * - * Called by a driver the first time it's needed, must be attached to desired - * connectors. - */ -int drm_mode_create_dithering_property(struct drm_device *dev) -{ - struct drm_property *dithering_mode; - - if (dev->mode_config.dithering_mode_property) - return 0; - - dithering_mode = - drm_property_create_enum(dev, 0, "dithering", - drm_dithering_mode_enum_list, - ARRAY_SIZE(drm_dithering_mode_enum_list)); - dev->mode_config.dithering_mode_property = dithering_mode; - - return 0; -} -EXPORT_SYMBOL(drm_mode_create_dithering_property); - -/** * drm_mode_create_dirty_property - create dirty property * @dev: DRM device * diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index eb40a96..781988f 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h @@ -982,7 +982,6 @@ extern int drm_mode_create_dvi_i_properties(struct drm_device *dev); extern int drm_mode_create_tv_properties(struct drm_device *dev, int num_formats, char *formats[]); extern int drm_mode_create_scaling_mode_property(struct drm_device *dev); -extern int drm_mode_create_dithering_property(struct drm_device *dev); extern int drm_mode_create_dirty_info_property(struct drm_device *dev); extern const char *drm_get_encoder_name(const struct drm_encoder *encoder); -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 3/8] drm: Remove drm_mode_list_concat() 2013-08-19 23:53 A gentle scrub Damien Lespiau 2013-08-19 23:53 ` [PATCH 1/8] drm: Remove stale prototypes Damien Lespiau 2013-08-19 23:53 ` [PATCH 2/8] drm: Remove drm_mode_create_dithering_property() Damien Lespiau @ 2013-08-19 23:53 ` Damien Lespiau 2013-08-19 23:53 ` [PATCH 4/8] drm: Make drm_mode_remove() static Damien Lespiau ` (5 subsequent siblings) 8 siblings, 0 replies; 12+ messages in thread From: Damien Lespiau @ 2013-08-19 23:53 UTC (permalink / raw) To: dri-devel The last user was removed in commit 575dc34ee0de867ba83abf25998e0963bff451fa Author: Dave Airlie <airlied@redhat.com> Date: Mon Sep 7 18:43:26 2009 +1000 drm/kms: remove old std mode fallback code. The new code adds modes in the helper, which makes more sense I disliked the non-driver code adding modes. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> --- drivers/gpu/drm/drm_modes.c | 21 --------------------- include/drm/drm_crtc.h | 2 -- 2 files changed, 23 deletions(-) diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c index 504a602..fc2adb6 100644 --- a/drivers/gpu/drm/drm_modes.c +++ b/drivers/gpu/drm/drm_modes.c @@ -596,27 +596,6 @@ void drm_mode_set_name(struct drm_display_mode *mode) EXPORT_SYMBOL(drm_mode_set_name); /** - * drm_mode_list_concat - move modes from one list to another - * @head: source list - * @new: dst list - * - * LOCKING: - * Caller must ensure both lists are locked. - * - * Move all the modes from @head to @new. - */ -void drm_mode_list_concat(struct list_head *head, struct list_head *new) -{ - - struct list_head *entry, *tmp; - - list_for_each_safe(entry, tmp, head) { - list_move_tail(entry, new); - } -} -EXPORT_SYMBOL(drm_mode_list_concat); - -/** * drm_mode_width - get the width of a mode * @mode: mode * diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index 781988f..39d84dc 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h @@ -927,8 +927,6 @@ extern int drm_mode_height(const struct drm_display_mode *mode); /* for us by fb module */ extern struct drm_display_mode *drm_mode_create(struct drm_device *dev); extern void drm_mode_destroy(struct drm_device *dev, struct drm_display_mode *mode); -extern void drm_mode_list_concat(struct list_head *head, - struct list_head *new); extern void drm_mode_validate_size(struct drm_device *dev, struct list_head *mode_list, int maxX, int maxY, int maxPitch); -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 4/8] drm: Make drm_mode_remove() static 2013-08-19 23:53 A gentle scrub Damien Lespiau ` (2 preceding siblings ...) 2013-08-19 23:53 ` [PATCH 3/8] drm: Remove drm_mode_list_concat() Damien Lespiau @ 2013-08-19 23:53 ` Damien Lespiau 2013-08-19 23:53 ` [PATCH 5/8] drm: Remove 2 unused defines Damien Lespiau ` (4 subsequent siblings) 8 siblings, 0 replies; 12+ messages in thread From: Damien Lespiau @ 2013-08-19 23:53 UTC (permalink / raw) To: dri-devel It's only used in drm_crtc.c. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> --- drivers/gpu/drm/drm_crtc.c | 7 +++---- include/drm/drm_crtc.h | 1 - 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index ffb791f..54b4169 100644 --- a/drivers/gpu/drm/drm_crtc.c +++ b/drivers/gpu/drm/drm_crtc.c @@ -686,20 +686,19 @@ void drm_mode_probed_add(struct drm_connector *connector, } EXPORT_SYMBOL(drm_mode_probed_add); -/** +/* * drm_mode_remove - remove and free a mode * @connector: connector list to modify * @mode: mode to remove * * Remove @mode from @connector's mode list, then free it. */ -void drm_mode_remove(struct drm_connector *connector, - struct drm_display_mode *mode) +static void drm_mode_remove(struct drm_connector *connector, + struct drm_display_mode *mode) { list_del(&mode->head); drm_mode_destroy(connector->dev, mode); } -EXPORT_SYMBOL(drm_mode_remove); /** * drm_connector_init - Init a preallocated connector diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index 39d84dc..014e491 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h @@ -910,7 +910,6 @@ extern struct edid *drm_get_edid(struct drm_connector *connector, struct i2c_adapter *adapter); extern int drm_add_edid_modes(struct drm_connector *connector, struct edid *edid); extern void drm_mode_probed_add(struct drm_connector *connector, struct drm_display_mode *mode); -extern void drm_mode_remove(struct drm_connector *connector, struct drm_display_mode *mode); extern void drm_mode_copy(struct drm_display_mode *dst, const struct drm_display_mode *src); extern struct drm_display_mode *drm_mode_duplicate(struct drm_device *dev, const struct drm_display_mode *mode); -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 5/8] drm: Remove 2 unused defines 2013-08-19 23:53 A gentle scrub Damien Lespiau ` (3 preceding siblings ...) 2013-08-19 23:53 ` [PATCH 4/8] drm: Make drm_mode_remove() static Damien Lespiau @ 2013-08-19 23:53 ` Damien Lespiau 2013-08-19 23:53 ` [PATCH 6/8] drm: Make drm_fb_cma_describe() static Damien Lespiau ` (3 subsequent siblings) 8 siblings, 0 replies; 12+ messages in thread From: Damien Lespiau @ 2013-08-19 23:53 UTC (permalink / raw) To: dri-devel These were introduced in the very first DRM commit: commit f453ba0460742ad027ae0c4c7d61e62817b3e7ef Author: Dave Airlie <airlied@redhat.com> Date: Fri Nov 7 14:05:41 2008 -0800 DRM: add mode setting support Add mode setting support to the DRM layer. But are unused. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> --- include/drm/drm_crtc.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index 014e491..0a9f73e 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h @@ -494,8 +494,6 @@ struct drm_encoder_funcs { void (*destroy)(struct drm_encoder *encoder); }; -#define DRM_CONNECTOR_MAX_UMODES 16 -#define DRM_CONNECTOR_LEN 32 #define DRM_CONNECTOR_MAX_ENCODER 3 /** -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 6/8] drm: Make drm_fb_cma_describe() static 2013-08-19 23:53 A gentle scrub Damien Lespiau ` (4 preceding siblings ...) 2013-08-19 23:53 ` [PATCH 5/8] drm: Remove 2 unused defines Damien Lespiau @ 2013-08-19 23:53 ` Damien Lespiau 2013-08-19 23:53 ` [PATCH 7/8] drm: Remove unused PCI ids Damien Lespiau ` (2 subsequent siblings) 8 siblings, 0 replies; 12+ messages in thread From: Damien Lespiau @ 2013-08-19 23:53 UTC (permalink / raw) To: dri-devel This function is only used in drm_fb_cma_helper.c. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> --- drivers/gpu/drm/drm_fb_cma_helper.c | 5 ++--- include/drm/drm_fb_cma_helper.h | 1 - 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/drm_fb_cma_helper.c b/drivers/gpu/drm/drm_fb_cma_helper.c index c385cc5..61b5a47 100644 --- a/drivers/gpu/drm/drm_fb_cma_helper.c +++ b/drivers/gpu/drm/drm_fb_cma_helper.c @@ -181,11 +181,11 @@ struct drm_gem_cma_object *drm_fb_cma_get_gem_obj(struct drm_framebuffer *fb, EXPORT_SYMBOL_GPL(drm_fb_cma_get_gem_obj); #ifdef CONFIG_DEBUG_FS -/** +/* * drm_fb_cma_describe() - Helper to dump information about a single * CMA framebuffer object */ -void drm_fb_cma_describe(struct drm_framebuffer *fb, struct seq_file *m) +static void drm_fb_cma_describe(struct drm_framebuffer *fb, struct seq_file *m) { struct drm_fb_cma *fb_cma = to_fb_cma(fb); int i, n = drm_format_num_planes(fb->pixel_format); @@ -199,7 +199,6 @@ void drm_fb_cma_describe(struct drm_framebuffer *fb, struct seq_file *m) drm_gem_cma_describe(fb_cma->obj[i], m); } } -EXPORT_SYMBOL_GPL(drm_fb_cma_describe); /** * drm_fb_cma_debugfs_show() - Helper to list CMA framebuffer objects diff --git a/include/drm/drm_fb_cma_helper.h b/include/drm/drm_fb_cma_helper.h index 4a3fc24..c54cf3d 100644 --- a/include/drm/drm_fb_cma_helper.h +++ b/include/drm/drm_fb_cma_helper.h @@ -24,7 +24,6 @@ struct drm_gem_cma_object *drm_fb_cma_get_gem_obj(struct drm_framebuffer *fb, unsigned int plane); #ifdef CONFIG_DEBUG_FS -void drm_fb_cma_describe(struct drm_framebuffer *fb, struct seq_file *m); int drm_fb_cma_debugfs_show(struct seq_file *m, void *arg); #endif -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 7/8] drm: Remove unused PCI ids 2013-08-19 23:53 A gentle scrub Damien Lespiau ` (5 preceding siblings ...) 2013-08-19 23:53 ` [PATCH 6/8] drm: Make drm_fb_cma_describe() static Damien Lespiau @ 2013-08-19 23:53 ` Damien Lespiau 2013-08-19 23:53 ` [PATCH 8/8] drm: Make drm_get_platform_dev() static Damien Lespiau 2013-08-20 14:10 ` A gentle scrub Alex Deucher 8 siblings, 0 replies; 12+ messages in thread From: Damien Lespiau @ 2013-08-19 23:53 UTC (permalink / raw) To: dri-devel Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> --- include/drm/drm_pciids.h | 30 ------------------------------ 1 file changed, 30 deletions(-) diff --git a/include/drm/drm_pciids.h b/include/drm/drm_pciids.h index 34efaf6..0a85e5c 100644 --- a/include/drm/drm_pciids.h +++ b/include/drm/drm_pciids.h @@ -690,29 +690,6 @@ {0x102b, 0x2527, PCI_ANY_ID, PCI_ANY_ID, 0, 0, MGA_CARD_TYPE_G550}, \ {0, 0, 0} -#define mach64_PCI_IDS \ - {0x1002, 0x4749, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4750, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4751, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4742, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4744, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4c49, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4c50, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4c51, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4c42, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4c44, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x474c, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x474f, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4752, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4753, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x474d, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x474e, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4c52, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4c53, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4c4d, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0x1002, 0x4c4e, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0, 0, 0} - #define sisdrv_PCI_IDS \ {0x1039, 0x0300, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ {0x1039, 0x5300, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ @@ -752,10 +729,6 @@ {0x8086, 0x1132, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ {0, 0, 0} -#define gamma_PCI_IDS \ - {0x3d3d, 0x0008, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, \ - {0, 0, 0} - #define savage_PCI_IDS \ {0x5333, 0x8a20, PCI_ANY_ID, PCI_ANY_ID, 0, 0, S3_SAVAGE3D}, \ {0x5333, 0x8a21, PCI_ANY_ID, PCI_ANY_ID, 0, 0, S3_SAVAGE3D}, \ @@ -781,6 +754,3 @@ {0x5333, 0x8d03, PCI_ANY_ID, PCI_ANY_ID, 0, 0, S3_PROSAVAGEDDR}, \ {0x5333, 0x8d04, PCI_ANY_ID, PCI_ANY_ID, 0, 0, S3_PROSAVAGEDDR}, \ {0, 0, 0} - -#define ffb_PCI_IDS \ - {0, 0, 0} -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 8/8] drm: Make drm_get_platform_dev() static 2013-08-19 23:53 A gentle scrub Damien Lespiau ` (6 preceding siblings ...) 2013-08-19 23:53 ` [PATCH 7/8] drm: Remove unused PCI ids Damien Lespiau @ 2013-08-19 23:53 ` Damien Lespiau 2013-08-20 14:56 ` Ian Romanick 2013-08-20 14:10 ` A gentle scrub Alex Deucher 8 siblings, 1 reply; 12+ messages in thread From: Damien Lespiau @ 2013-08-19 23:53 UTC (permalink / raw) To: dri-devel It's only used in drm_platform.c. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> --- drivers/gpu/drm/drm_platform.c | 7 +++---- include/drm/drmP.h | 3 --- 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/drm_platform.c b/drivers/gpu/drm/drm_platform.c index b8a282e..400024b 100644 --- a/drivers/gpu/drm/drm_platform.c +++ b/drivers/gpu/drm/drm_platform.c @@ -28,7 +28,7 @@ #include <linux/export.h> #include <drm/drmP.h> -/** +/* * Register. * * \param platdev - Platform device struture @@ -39,8 +39,8 @@ * Try and register, if we fail to register, backout previous work. */ -int drm_get_platform_dev(struct platform_device *platdev, - struct drm_driver *driver) +static int drm_get_platform_dev(struct platform_device *platdev, + struct drm_driver *driver) { struct drm_device *dev; int ret; @@ -107,7 +107,6 @@ err_g1: mutex_unlock(&drm_global_mutex); return ret; } -EXPORT_SYMBOL(drm_get_platform_dev); static int drm_platform_get_irq(struct drm_device *dev) { diff --git a/include/drm/drmP.h b/include/drm/drmP.h index 046a7db..1a7a78f 100644 --- a/include/drm/drmP.h +++ b/include/drm/drmP.h @@ -1657,9 +1657,6 @@ extern int drm_pcie_get_speed_cap_mask(struct drm_device *dev, u32 *speed_mask); extern int drm_platform_init(struct drm_driver *driver, struct platform_device *platform_device); extern void drm_platform_exit(struct drm_driver *driver, struct platform_device *platform_device); -extern int drm_get_platform_dev(struct platform_device *pdev, - struct drm_driver *driver); - /* returns true if currently okay to sleep */ static __inline__ bool drm_can_sleep(void) { -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 8/8] drm: Make drm_get_platform_dev() static 2013-08-19 23:53 ` [PATCH 8/8] drm: Make drm_get_platform_dev() static Damien Lespiau @ 2013-08-20 14:56 ` Ian Romanick 2013-08-20 16:17 ` Damien Lespiau 0 siblings, 1 reply; 12+ messages in thread From: Ian Romanick @ 2013-08-20 14:56 UTC (permalink / raw) To: Damien Lespiau; +Cc: dri-devel On 08/19/2013 04:53 PM, Damien Lespiau wrote: > It's only used in drm_platform.c. > > Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> > --- > drivers/gpu/drm/drm_platform.c | 7 +++---- > include/drm/drmP.h | 3 --- > 2 files changed, 3 insertions(+), 7 deletions(-) > > diff --git a/drivers/gpu/drm/drm_platform.c b/drivers/gpu/drm/drm_platform.c > index b8a282e..400024b 100644 > --- a/drivers/gpu/drm/drm_platform.c > +++ b/drivers/gpu/drm/drm_platform.c > @@ -28,7 +28,7 @@ > #include <linux/export.h> > #include <drm/drmP.h> > > -/** > +/* > * Register. > * > * \param platdev - Platform device struture Was this change intentional? The rest of the comment is doxygen still. > @@ -39,8 +39,8 @@ > * Try and register, if we fail to register, backout previous work. > */ > > -int drm_get_platform_dev(struct platform_device *platdev, > - struct drm_driver *driver) > +static int drm_get_platform_dev(struct platform_device *platdev, > + struct drm_driver *driver) > { > struct drm_device *dev; > int ret; > @@ -107,7 +107,6 @@ err_g1: > mutex_unlock(&drm_global_mutex); > return ret; > } > -EXPORT_SYMBOL(drm_get_platform_dev); > > static int drm_platform_get_irq(struct drm_device *dev) > { > diff --git a/include/drm/drmP.h b/include/drm/drmP.h > index 046a7db..1a7a78f 100644 > --- a/include/drm/drmP.h > +++ b/include/drm/drmP.h > @@ -1657,9 +1657,6 @@ extern int drm_pcie_get_speed_cap_mask(struct drm_device *dev, u32 *speed_mask); > extern int drm_platform_init(struct drm_driver *driver, struct platform_device *platform_device); > extern void drm_platform_exit(struct drm_driver *driver, struct platform_device *platform_device); > > -extern int drm_get_platform_dev(struct platform_device *pdev, > - struct drm_driver *driver); > - > /* returns true if currently okay to sleep */ > static __inline__ bool drm_can_sleep(void) > { > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 8/8] drm: Make drm_get_platform_dev() static 2013-08-20 14:56 ` Ian Romanick @ 2013-08-20 16:17 ` Damien Lespiau 0 siblings, 0 replies; 12+ messages in thread From: Damien Lespiau @ 2013-08-20 16:17 UTC (permalink / raw) To: Ian Romanick; +Cc: dri-devel On Tue, Aug 20, 2013 at 07:56:42AM -0700, Ian Romanick wrote: > On 08/19/2013 04:53 PM, Damien Lespiau wrote: > >It's only used in drm_platform.c. > > > >Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> > >--- > > drivers/gpu/drm/drm_platform.c | 7 +++---- > > include/drm/drmP.h | 3 --- > > 2 files changed, 3 insertions(+), 7 deletions(-) > > > >diff --git a/drivers/gpu/drm/drm_platform.c b/drivers/gpu/drm/drm_platform.c > >index b8a282e..400024b 100644 > >--- a/drivers/gpu/drm/drm_platform.c > >+++ b/drivers/gpu/drm/drm_platform.c > >@@ -28,7 +28,7 @@ > > #include <linux/export.h> > > #include <drm/drmP.h> > > > >-/** > >+/* > > * Register. > > * > > * \param platdev - Platform device struture > > Was this change intentional? The rest of the comment is doxygen still. It was indeed, just to stop documentation tools from picking up that symbol that is not exported anymore. The documentation blob still contains markup to tag the parameters though, might as well leave it there. -- Damien > > >@@ -39,8 +39,8 @@ > > * Try and register, if we fail to register, backout previous work. > > */ > > > >-int drm_get_platform_dev(struct platform_device *platdev, > >- struct drm_driver *driver) > >+static int drm_get_platform_dev(struct platform_device *platdev, > >+ struct drm_driver *driver) > > { > > struct drm_device *dev; > > int ret; > >@@ -107,7 +107,6 @@ err_g1: > > mutex_unlock(&drm_global_mutex); > > return ret; > > } > >-EXPORT_SYMBOL(drm_get_platform_dev); > > > > static int drm_platform_get_irq(struct drm_device *dev) > > { > >diff --git a/include/drm/drmP.h b/include/drm/drmP.h > >index 046a7db..1a7a78f 100644 > >--- a/include/drm/drmP.h > >+++ b/include/drm/drmP.h > >@@ -1657,9 +1657,6 @@ extern int drm_pcie_get_speed_cap_mask(struct drm_device *dev, u32 *speed_mask); > > extern int drm_platform_init(struct drm_driver *driver, struct platform_device *platform_device); > > extern void drm_platform_exit(struct drm_driver *driver, struct platform_device *platform_device); > > > >-extern int drm_get_platform_dev(struct platform_device *pdev, > >- struct drm_driver *driver); > >- > > /* returns true if currently okay to sleep */ > > static __inline__ bool drm_can_sleep(void) > > { > > > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: A gentle scrub 2013-08-19 23:53 A gentle scrub Damien Lespiau ` (7 preceding siblings ...) 2013-08-19 23:53 ` [PATCH 8/8] drm: Make drm_get_platform_dev() static Damien Lespiau @ 2013-08-20 14:10 ` Alex Deucher 8 siblings, 0 replies; 12+ messages in thread From: Alex Deucher @ 2013-08-20 14:10 UTC (permalink / raw) To: Damien Lespiau; +Cc: Maling list - DRI developers On Mon, Aug 19, 2013 at 7:53 PM, Damien Lespiau <damien.lespiau@intel.com> wrote: > A small pass on drm headers to remove some stale prototypes/functions/defines > and to make static a few functions. For the series: Reviewed-by: Alex Deucher <alexander.deucher@amd.com> > > drivers/gpu/drm/drm_crtc.c | 38 +++---------------------------------- > drivers/gpu/drm/drm_fb_cma_helper.c | 5 ++--- > drivers/gpu/drm/drm_modes.c | 21 -------------------- > drivers/gpu/drm/drm_platform.c | 7 +++---- > include/drm/drmP.h | 8 -------- > include/drm/drm_crtc.h | 17 ----------------- > include/drm/drm_fb_cma_helper.h | 1 - > include/drm/drm_pciids.h | 30 ----------------------------- > 8 files changed, 8 insertions(+), 119 deletions(-) > > -- > Damien > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2013-08-20 16:18 UTC | newest] Thread overview: 12+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-08-19 23:53 A gentle scrub Damien Lespiau 2013-08-19 23:53 ` [PATCH 1/8] drm: Remove stale prototypes Damien Lespiau 2013-08-19 23:53 ` [PATCH 2/8] drm: Remove drm_mode_create_dithering_property() Damien Lespiau 2013-08-19 23:53 ` [PATCH 3/8] drm: Remove drm_mode_list_concat() Damien Lespiau 2013-08-19 23:53 ` [PATCH 4/8] drm: Make drm_mode_remove() static Damien Lespiau 2013-08-19 23:53 ` [PATCH 5/8] drm: Remove 2 unused defines Damien Lespiau 2013-08-19 23:53 ` [PATCH 6/8] drm: Make drm_fb_cma_describe() static Damien Lespiau 2013-08-19 23:53 ` [PATCH 7/8] drm: Remove unused PCI ids Damien Lespiau 2013-08-19 23:53 ` [PATCH 8/8] drm: Make drm_get_platform_dev() static Damien Lespiau 2013-08-20 14:56 ` Ian Romanick 2013-08-20 16:17 ` Damien Lespiau 2013-08-20 14:10 ` A gentle scrub Alex Deucher
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.