Re: [PATCH v7 04/54] drm/bridge: Document the drm_encoder.bridge_chain field as private

[Boris Brezillon <boris.brezillon@collabora.com>]

On Sat, 22 Feb 2020 17:00:16 +0200
Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote:

> The drm_encoder.bridge_chain is not meant to be touched manually by
> drivers. Make this clear in the documentation.

Thanks for this clarification:

Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>

> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
>  include/drm/drm_encoder.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/include/drm/drm_encoder.h b/include/drm/drm_encoder.h
> index 5623994b6e9e..4370e039c015 100644
> --- a/include/drm/drm_encoder.h
> +++ b/include/drm/drm_encoder.h
> @@ -174,7 +174,8 @@ struct drm_encoder {
>  	struct drm_crtc *crtc;
>  
>  	/**
> -	 * @bridge_chain: Bridges attached to this encoder.
> +	 * @bridge_chain: Bridges attached to this encoder. Drivers shall not
> +	 * access this field directly.
>  	 */
>  	struct list_head bridge_chain;
>  

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel