== 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
