== Series Details == Series: drm/i915: Fix setting 10 bit color mode URL : https://patchwork.freedesktop.org/series/59246/ State : warning == Summary == $ dim checkpatch origin/drm-tip d76f6b458d07 drm/i915: Fix setting 10 bit color mode -:22: WARNING:BAD_SIGN_OFF: Non-standard signature: Co-authored-by: #22: Co-authored-by: Clint Taylor <clinton.a.taylor@xxxxxxxxx> -:49: ERROR:SPACING: space required before the open parenthesis '(' #49: FILE: drivers/gpu/drm/i915/intel_hdmi.c:2328: + if(pipe_config->pipe_bpp == 36) { -:49: CHECK:BRACES: braces {} should be used on all arms of this statement #49: FILE: drivers/gpu/drm/i915/intel_hdmi.c:2328: + if(pipe_config->pipe_bpp == 36) { [...] + } else if (pipe_config->pipe_bpp == 30) { [...] + } else [...] -:55: CHECK:BRACES: Unbalanced braces around else statement #55: FILE: drivers/gpu/drm/i915/intel_hdmi.c:2334: + } else -:62: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #62: FILE: drivers/gpu/drm/i915/intel_hdmi.c:2341: + hdmi_port_clock_valid(intel_hdmi, clock_bpc, true, force_dvi) == MODE_OK) { -:79: WARNING:LONG_LINE: line over 100 characters #79: FILE: drivers/gpu/drm/i915/intel_hdmi.c:2342: + DRM_DEBUG_KMS("picking bpc to %d for HDMI output\n",pipe_config->pipe_bpp/3); -:79: ERROR:SPACING: space required after that ',' (ctx:VxV) #79: FILE: drivers/gpu/drm/i915/intel_hdmi.c:2342: + DRM_DEBUG_KMS("picking bpc to %d for HDMI output\n",pipe_config->pipe_bpp/3); ^ -:79: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV) #79: FILE: drivers/gpu/drm/i915/intel_hdmi.c:2342: + DRM_DEBUG_KMS("picking bpc to %d for HDMI output\n",pipe_config->pipe_bpp/3); ^ total: 2 errors, 2 warnings, 4 checks, 60 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx