== Series Details ==

Series: drm/i915/cx0_phy: Update HDMI TMDS C20 algorithm value (rev5)
URL   : https://patchwork.freedesktop.org/series/141280/
State : warning

== Summary ==

Error: dim checkpatch failed
dbfd52a793e2 drm/i915/display: Add MTL subplatforms definition
-:70: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in 
parentheses
#70: FILE: include/drm/intel/pciids.h:814:
+#define INTEL_MTL_U_IDS(MACRO__, ...) \
        MACRO__(0x7D40, ## __VA_ARGS__), \
+       MACRO__(0x7D45, ## __VA_ARGS__)

-:70: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'MACRO__' - possible 
side-effects?
#70: FILE: include/drm/intel/pciids.h:814:
+#define INTEL_MTL_U_IDS(MACRO__, ...) \
        MACRO__(0x7D40, ## __VA_ARGS__), \
+       MACRO__(0x7D45, ## __VA_ARGS__)

-:75: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in 
parentheses
#75: FILE: include/drm/intel/pciids.h:818:
+#define INTEL_MTL_IDS(MACRO__, ...) \
+       INTEL_MTL_U_IDS(MACRO__, ## __VA_ARGS__), \
        MACRO__(0x7D55, ## __VA_ARGS__), \
        MACRO__(0x7D60, ## __VA_ARGS__), \
        MACRO__(0x7DD5, ## __VA_ARGS__)

-:75: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'MACRO__' - possible 
side-effects?
#75: FILE: include/drm/intel/pciids.h:818:
+#define INTEL_MTL_IDS(MACRO__, ...) \
+       INTEL_MTL_U_IDS(MACRO__, ## __VA_ARGS__), \
        MACRO__(0x7D55, ## __VA_ARGS__), \
        MACRO__(0x7D60, ## __VA_ARGS__), \
        MACRO__(0x7DD5, ## __VA_ARGS__)

total: 2 errors, 0 warnings, 2 checks, 46 lines checked
2b9ddd4f3e1b drm/i915/cx0_phy: Update HDMI TMDS C20 algorithm value
-:119: WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#119: FILE: drivers/gpu/drm/i915/display/intel_cx0_phy_regs.h:301:
+#define   C20_PHY_TX_DCC_CAL_RANGE_MASK ^IREG_GENMASK16(11, 8)$

-:141: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'id' - possible side-effects?
#141: FILE: drivers/gpu/drm/i915/display/intel_display_device.h:242:
+#define IS_ARROWLAKE_S_BY_HOST_BRIDGE_ID(id)  \
+       (((id) == ARLS_HOST_BRIDGE_PCI_ID1) || \
+        ((id) == ARLS_HOST_BRIDGE_PCI_ID2) || \
+        ((id) == ARLS_HOST_BRIDGE_PCI_ID3) || \
+        ((id) == ARLS_HOST_BRIDGE_PCI_ID4))

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


Reply via email to