== Series Details ==

Series: DRM helpers for Display Stream Compression PPS infoframes (rev2)
URL   : https://patchwork.freedesktop.org/series/42968/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
bf8fa6314dd0 drm/dp: Define payload size for DP SDP PPS packet
feef4201c5b3 drm/dsc: Define Display Stream Compression PPS infoframe
-:19: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does 
MAINTAINERS need updating?
#19: 
new file mode 100644

-:24: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier 
tag in line 1
#24: FILE: include/drm/drm_dsc.h:1:
+/*

-:423: WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each 
line
#423: FILE: include/drm/drm_dsc.h:400:
+/**
+* struct drm_dsc_pps_infoframe - DSC infoframe carrying the Picture Parameter 
Set Metadata

total: 0 errors, 3 warnings, 0 checks, 412 lines checked
662aa7ec9b08 drm/dsc: Define VESA Display Stream Compression Capabilities
71dd1304b3aa drm/dsc: Add helpers for DSC picture parameter set infoframes
-:36: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does 
MAINTAINERS need updating?
#36: 
new file mode 100644

-:41: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier 
tag in line 1
#41: FILE: drivers/gpu/drm/drm_dsc.c:1:
+/*

-:177: WARNING:LONG_LINE: line over 100 characters
#177: FILE: drivers/gpu/drm/drm_dsc.c:137:
+       pps_sdp->pps_payload.scale_increment_interval = 
DSC_PPS_SWAP_BYTES(dsc_cfg->scale_increment_interval,

-:181: WARNING:LONG_LINE: line over 100 characters
#181: FILE: drivers/gpu/drm/drm_dsc.c:141:
+       pps_sdp->pps_payload.scale_decrement_interval_high = 
(u8)((dsc_cfg->scale_decrement_interval &

-:182: WARNING:LONG_LINE: line over 100 characters
#182: FILE: drivers/gpu/drm/drm_dsc.c:142:
+                                                                  
DSC_PPS_SCALE_DEC_INT_HIGH_MASK) >>

-:240: WARNING:LONG_LINE: line over 100 characters
#240: FILE: drivers/gpu/drm/drm_dsc.c:200:
+                       (u16)((dsc_cfg->rc_range_params[i].range_min_qp << 
DSC_PPS_RC_RANGE_MINQP_SHIFT) |

-:241: WARNING:LONG_LINE: line over 100 characters
#241: FILE: drivers/gpu/drm/drm_dsc.c:201:
+                             (dsc_cfg->rc_range_params[i].range_max_qp << 
DSC_PPS_RC_RANGE_MAXQP_SHIFT) |

-:243: WARNING:LONG_LINE: line over 100 characters
#243: FILE: drivers/gpu/drm/drm_dsc.c:203:
+               pps_sdp->pps_payload.rc_range_parameters[i] = 
DSC_PPS_SWAP_BYTES(pps_sdp->pps_payload.rc_range_parameters[i],

-:259: WARNING:LONG_LINE: line over 100 characters
#259: FILE: drivers/gpu/drm/drm_dsc.c:219:
+       pps_sdp->pps_payload.second_line_offset_adj = 
DSC_PPS_SWAP_BYTES(dsc_cfg->second_line_offset_adj,

-:289: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'val' - possible 
side-effects?
#289: FILE: include/drm/drm_dsc.h:45:
+#define DSC_PPS_SWAP_BYTES(val, swap)          ((swap) ? val : 
cpu_to_be16(val))

total: 0 errors, 9 warnings, 1 checks, 264 lines checked

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to