[PATCH weston 03/68] compositor-drm: Comment struct members

Armin Krezović krezovic.armin at gmail.com
Fri Dec 9 20:38:44 UTC 2016


On 09.12.2016 20:57, Daniel Stone wrote:
> Clarify the difference between crtc_id (DRM object) and pipe (index into
> drmModeRes->crtcs array, possible_crtcs bitmask).
> 
> Signed-off-by: Daniel Stone <daniels at collabora.com>
> Reviewed-by: Quentin Glidic <sardemff7+git at sardemff7.net>
> Differential Revision: https://phabricator.freedesktop.org/D1405

Documenting structure members is always welcome.

Reviewed-by: Armin Krezović <krezovic.armin at gmail.com>

(this time it's me!)

> ---
>  libweston/compositor-drm.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libweston/compositor-drm.c b/libweston/compositor-drm.c
> index a899213..268117d 100644
> --- a/libweston/compositor-drm.c
> +++ b/libweston/compositor-drm.c
> @@ -155,8 +155,8 @@ struct drm_output {
>  	struct weston_output base;
>  	drmModeConnector *connector;
>  
> -	uint32_t crtc_id;
> -	int pipe;
> +	uint32_t crtc_id; /* object ID to pass to DRM functions */
> +	int pipe; /* index of CRTC in resource array / bitmasks */
>  	uint32_t connector_id;
>  	drmModeCrtcPtr original_crtc;
>  	struct drm_edid edid;
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 870 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20161209/e68490b2/attachment.sig>


More information about the wayland-devel mailing list