Re: [PATCH 5/9] drm/atomic: add connector mask to drm_crtc_state.

Thierry Reding
2015-12-07T10:24:39Z

[-- Attachment #1.1: Type: text/plain, Size: 439 bytes --]

On Tue, Nov 24, 2015 at 10:34:32AM +0100, Maarten Lankhorst wrote:
> Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> ---
> drivers/gpu/drm/drm_atomic.c | 11 +++++++++++
> include/drm/drm_crtc.h | 3 +++
> 2 files changed, 14 insertions(+)

I think this could use a proper commit message. As it is, it's
completely unclear as to what this mask will be useful for.

Other than that looks good to me.

Thierry

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]