✗ Fi.CI.CHECKPATCH: warning for New DRM properties for output color format

Patchwork patchwork at emeril.freedesktop.org
Tue Jan 16 13:08:15 UTC 2024


== 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 at yngvason.is>

-: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 at yngvason.is>

total: 0 errors, 1 warnings, 0 checks, 136 lines checked




More information about the Intel-gfx mailing list