== Series Details == Series: New DRM properties for output color format URL : https://patchwork.freedesktop.org/series/128826/ State : warning == Summary == Error: dim checkpatch failed 73e5d5241200 drm/amd/display: Remove unnecessary SIGNAL_TYPE_HDMI_TYPE_A check -:35: CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #35: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:5578: if (drm_mode_is_420_only(info, mode_in) + || (drm_mode_is_420_also(info, mode_in) && aconnector->force_yuv420_output)) total: 0 errors, 0 warnings, 1 checks, 12 lines checked bfe7e92f2b36 drm/uAPI: Add "force color format" drm property as setting for userspace 0ee65f6d9c49 drm/amd/display: Add handling for new "force color format" property -:11: WARNING:BAD_SIGN_OFF: 'Co-developed-by:' is the preferred signature form #11: Co-Developed-by: Andri Yngvason <andri@xxxxxxxxxxx> -:28: CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #28: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:5578: + if (connector_state + && (connector_state->force_color_format == DRM_COLOR_FORMAT_YCBCR420 -:29: CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #29: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:5579: + && (connector_state->force_color_format == DRM_COLOR_FORMAT_YCBCR420 + || aconnector->force_yuv420_output) && drm_mode_is_420(info, mode_in)) -:34: CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #34: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:5582: + else if (connector_state + && connector_state->force_color_format == DRM_COLOR_FORMAT_YCBCR444 -:35: CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #35: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:5583: + && connector_state->force_color_format == DRM_COLOR_FORMAT_YCBCR444 + && connector->display_info.color_formats & DRM_COLOR_FORMAT_YCBCR444) -:39: CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #39: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:5586: + else if (connector_state + && connector_state->force_color_format == DRM_COLOR_FORMAT_RGB444 -:40: CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #40: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:5587: + && connector_state->force_color_format == DRM_COLOR_FORMAT_RGB444 + && !drm_mode_is_420_only(info, mode_in)) -:51: CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #51: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:5598: + else if ((connector->display_info.color_formats & DRM_COLOR_FORMAT_YCBCR444) + && stream->signal == SIGNAL_TYPE_HDMI_TYPE_A) -:70: WARNING:LINE_SPACING: Missing a blank line after declarations #70: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:6713: + enum dc_pixel_encoding encoding = PIXEL_ENCODING_UNDEFINED; + switch (drm_state->force_color_format) { total: 0 errors, 2 warnings, 7 checks, 101 lines checked e847a2e61678 drm/i915/display: Add handling for new "force color format" property -:11: WARNING:BAD_SIGN_OFF: 'Co-developed-by:' is the preferred signature form #11: Co-Developed-by: Andri Yngvason <andri@xxxxxxxxxxx> total: 0 errors, 1 warnings, 0 checks, 136 lines checked