Hi Daniel,
On Mon, Aug 05, 2019 at 06:28:15PM +0200, Daniel Vetter wrote:
> 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/
DDC is carried by two signals (at least for DDC2, the mostly
unimplemented DDC1 used a single signal), so "carries DDC signals" seems
appropriate to me.
> > + * 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
