On Sun, Aug 04, 2019 at 03:26:53PM +0300, Laurent Pinchart wrote:
> Improve the documentation of the ddc field by using DDC and I2C
> consistently, and explaining more clearly what the field points to.
>
> Signed-off-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> ---
> include/drm/drm_connector.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h
> index fc5d08438333..0d01662aa40e 100644
> --- a/include/drm/drm_connector.h
> +++ b/include/drm/drm_connector.h
> @@ -1315,10 +1315,10 @@ struct drm_connector {
>
> /**
> * @ddc: associated ddc adapter.
> - * A connector usually has its associated ddc adapter. If a driver uses
> - * this field, then an appropriate symbolic link is created in connector
> - * sysfs directory to make it easy for the user to tell which i2c
> - * adapter is for a particular display.
> + * When the connector carries DDC signals, this field points to the I2C
s/DDC/a DDC/
> + * adapter connected to the DDC signals, if any. When this field is not
s/not NULL/is set/ otherwise parsing hiccup on the double negative :-)
Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch>
> + * NULL a symbolic link is created in the connector's sysfs directory
> + * to expose the I2C adapter used by the connector.
> *
> * The field should be set by calling drm_connector_init_with_ddc().
> */
> --
> Regards,
>
> Laurent Pinchart
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
