== Series Details == Series: Refactor MST DSC Determination Policy (rev3) URL : https://patchwork.freedesktop.org/series/140832/ State : warning == Summary == Error: dim checkpatch failed 4af0868951f8 drm/display/dsc: Refactor DRM MST DSC Determination Policy -:11: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?) #11: in drm_dp_mst_is_virtual_dpcd() is not always correct. There are DP1.4 products -:186: WARNING:LONG_LINE: line length of 101 exceeds 100 columns #186: FILE: drivers/gpu/drm/display/drm_dp_mst_topology.c:6064: + drm_info(mgr->dev, "MST_DSC check port %p for dsc decompression capability\n", port); -:198: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #198: FILE: drivers/gpu/drm/display/drm_dp_mst_topology.c:6074: + drm_info(mgr->dev, "MST_DSC check immediate_upstream_port %p for dsc passthrough capability\n", + immediate_upstream_port); -:226: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #226: FILE: drivers/gpu/drm/display/drm_dp_mst_topology.c:6087: + if (endpoint_dsc & DP_DSC_DECOMPRESSION_IS_SUPPORTED && + upstream_dsc & DP_DSC_PASSTHROUGH_IS_SUPPORTED) { -:257: CHECK:BRACES: braces {} should be used on all arms of this statement #257: FILE: drivers/gpu/drm/display/drm_dp_mst_topology.c:6111: + if (endpoint_dsc & DP_DSC_DECOMPRESSION_IS_SUPPORTED) [...] + else { [...] -:259: CHECK:BRACES: Unbalanced braces around else statement #259: FILE: drivers/gpu/drm/display/drm_dp_mst_topology.c:6113: + else { -:269: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #269: FILE: drivers/gpu/drm/display/drm_dp_mst_topology.c:6121: + drm_info(mgr->dev, "MST_DSC topology with multiple mstbs, dsc decompression at immediate_upstream_port %p\n", + dsc_port); -:282: WARNING:LONG_LINE_COMMENT: line length of 109 exceeds 100 columns #282: FILE: drivers/gpu/drm/display/drm_dp_mst_topology.c:6134: + /* read fec cap one more time in case fec not capable return from enum path result */ -:283: WARNING:LONG_LINE: line length of 102 exceeds 100 columns #283: FILE: drivers/gpu/drm/display/drm_dp_mst_topology.c:6135: + if ((drm_dp_dpcd_read(&fec_port->aux, DP_FEC_CAPABILITY, &fec_cap, 1) != 1) || -:284: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #284: FILE: drivers/gpu/drm/display/drm_dp_mst_topology.c:6136: + if ((drm_dp_dpcd_read(&fec_port->aux, DP_FEC_CAPABILITY, &fec_cap, 1) != 1) || + !(fec_cap & DP_FEC_CAPABLE)) { -:285: WARNING:LONG_LINE: line length of 112 exceeds 100 columns #285: FILE: drivers/gpu/drm/display/drm_dp_mst_topology.c:6137: + drm_err(mgr->dev, "MST_DSC Failed to retrieve fec caps at port %p\n", fec_port); -:303: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #303: FILE: drivers/gpu/drm/display/drm_dp_mst_topology.c:6148: + if ((drm_dp_dpcd_read(mgr->aux, DP_FEC_CAPABILITY, &fec_cap, 1) != 1) || + !(fec_cap & DP_FEC_CAPABLE)) { total: 0 errors, 5 warnings, 7 checks, 297 lines checked c7a7ec814c6e drm/display/dsc: MST DSC Interface Change -:7: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?) #7: Starting from dp2 where dsc passthrough is introduced, it is required to identify -:132: WARNING:LONG_LINE: line length of 101 exceeds 100 columns #132: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c:262: + if (drm_dp_dpcd_read(aconnector->mst_output_port->dsc_aux, DP_DSC_SUPPORT, dsc_caps, 16) < 0) -:137: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #137: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c:266: + if (drm_dp_dpcd_read(aconnector->mst_output_port->dsc_aux, DP_DSC_BRANCH_OVERALL_THROUGHPUT_0, dsc_branch_dec_caps_raw, 3) == 3) -:149: WARNING:LONG_LINE: line length of 121 exceeds 100 columns #149: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c:285: + if (drm_dp_dpcd_read(aconnector->mst_output_port->dsc_aux, DP_DOWNSTREAMPORT_PRESENT, &ds_port_present, 1) < 0) { total: 0 errors, 3 warnings, 1 checks, 271 lines checked
