Re: [PATCH] drm: update the documentation of drm_framebuffer_unregister_private

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 31, 2016 at 07:59:56PM +0800, Rongrong Zou wrote:
> Add obvious description to drm_framebuffer_unregister_private()
> to explain it is deprecated.
> 
> Signed-off-by: Rongrong Zou <zourongrong@xxxxxxxxx>
> ---
>  drivers/gpu/drm/drm_framebuffer.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/gpu/drm/drm_framebuffer.c b/drivers/gpu/drm/drm_framebuffer.c
> index 398efd6..d2b0507 100644
> --- a/drivers/gpu/drm/drm_framebuffer.c
> +++ b/drivers/gpu/drm/drm_framebuffer.c
> @@ -751,6 +751,11 @@ struct drm_framebuffer *drm_framebuffer_lookup(struct drm_device *dev,
>   * those used for fbdev. Note that the caller must hold a reference of it's own,
>   * i.e. the object may not be destroyed through this call (since it'll lead to a
>   * locking inversion).
> + *
> + * NOTE: This function is deprecated. For driver-private framebuffers it is not
> + * recommended to embed a framebuffer struct info fbdev struct, instead, a
> + * framebuffer pointer is preferred and drm_framebuffer_unreference() should be
> + * called when the framebuffer is to be cleaned up.

Awesome, kerneldoc updates for stuff we discussed, I really like this.
Thanks for submitting your patch, applied to drm-misc.
-Daniel

>   */
>  void drm_framebuffer_unregister_private(struct drm_framebuffer *fb)
>  {
> -- 
> 1.9.1
> 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux