diff mbox series

drm: Remove drmm_add_final_kfree() declaration from public headers

Message ID 20201204092932.21636-1-laurent.pinchart+renesas@ideasonboard.com (mailing list archive)
State New, archived
Headers show
Series drm: Remove drmm_add_final_kfree() declaration from public headers | expand

Commit Message

Laurent Pinchart Dec. 4, 2020, 9:29 a.m. UTC
The drmm_add_final_kfree() function is declared in the
include/drm/drm_managed.h public header, but has become an internal API
not exposed to drivers. Drop it from drm_managed.h as it's already
declared in drm_internal.h.

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
---
 include/drm/drm_managed.h | 2 --
 1 file changed, 2 deletions(-)

Comments

Thomas Zimmermann Dec. 4, 2020, 9:47 a.m. UTC | #1
Am 04.12.20 um 10:29 schrieb Laurent Pinchart:
> The drmm_add_final_kfree() function is declared in the
> include/drm/drm_managed.h public header, but has become an internal API
> not exposed to drivers. Drop it from drm_managed.h as it's already
> declared in drm_internal.h.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>

Acked-by: Thomas Zimmermann <tzimmermann@suse.de>

> ---
>   include/drm/drm_managed.h | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/include/drm/drm_managed.h b/include/drm/drm_managed.h
> index ca4114633bf9..b45c6fbf53ac 100644
> --- a/include/drm/drm_managed.h
> +++ b/include/drm/drm_managed.h
> @@ -44,8 +44,6 @@ int __must_check __drmm_add_action_or_reset(struct drm_device *dev,
>   					    drmres_release_t action,
>   					    void *data, const char *name);
>   
> -void drmm_add_final_kfree(struct drm_device *dev, void *container);
> -
>   void *drmm_kmalloc(struct drm_device *dev, size_t size, gfp_t gfp) __malloc;
>   
>   /**
>
Sam Ravnborg Dec. 5, 2020, 7:02 p.m. UTC | #2
Hi Laurent,

On Fri, Dec 04, 2020 at 11:29:32AM +0200, Laurent Pinchart wrote:
> The drmm_add_final_kfree() function is declared in the
> include/drm/drm_managed.h public header, but has become an internal API
> not exposed to drivers. Drop it from drm_managed.h as it's already
> declared in drm_internal.h.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>

Applied to drm-misc-next, thanks.

	Sam
diff mbox series

Patch

diff --git a/include/drm/drm_managed.h b/include/drm/drm_managed.h
index ca4114633bf9..b45c6fbf53ac 100644
--- a/include/drm/drm_managed.h
+++ b/include/drm/drm_managed.h
@@ -44,8 +44,6 @@  int __must_check __drmm_add_action_or_reset(struct drm_device *dev,
 					    drmres_release_t action,
 					    void *data, const char *name);
 
-void drmm_add_final_kfree(struct drm_device *dev, void *container);
-
 void *drmm_kmalloc(struct drm_device *dev, size_t size, gfp_t gfp) __malloc;
 
 /**