== Series Details == Series: series starting with [v2,1/3] drm/i915: move *_crtc_clock_get() to intel_dpll.c (rev3) URL : https://patchwork.freedesktop.org/series/126388/ State : warning == Summary == Error: dim checkpatch failed 35cbbdaf6d00 drm/i915: move *_crtc_clock_get() to intel_dpll.c -:346: CHECK:BRACES: braces {} should be used on all arms of this statement #346: FILE: drivers/gpu/drm/i915/display/intel_dpll.c:458: + if (dpll & PLL_P1_DIVIDE_BY_TWO) [...] + else { [...] -:348: CHECK:BRACES: Unbalanced braces around else statement #348: FILE: drivers/gpu/drm/i915/display/intel_dpll.c:460: + else { total: 0 errors, 0 warnings, 2 checks, 424 lines checked 1dbaeebedb89 drm/i915: add vlv_pipe_to_phy() helper to replace DPIO_PHY() -:57: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations #57: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:120: } +static inline enum dpio_phy vlv_pipe_to_phy(enum pipe pipe) total: 0 errors, 0 warnings, 1 checks, 71 lines checked 4d743e43c3b8 drm/i915: convert vlv_dpio_read()/write() from pipe to phy -:362: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #362: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.c:953: + vlv_dpio_write(dev_priv, phy, CHV_TX_DW14(ch, i), data << DPIO_UPAR_SHIFT); -:437: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #437: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.c:1062: + vlv_dpio_write(dev_priv, phy, VLV_TX_DW2(port), uniqtranscale_reg_value); -:466: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #466: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.c:1088: + vlv_dpio_write(dev_priv, phy, VLV_PCS_DW0(port), DPIO_PCS_TX_LANE2_RESET | -:470: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #470: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.c:1091: + vlv_dpio_write(dev_priv, phy, VLV_PCS_DW1(port), DPIO_PCS_CLK_CRI_RXEB_EIOS_EN | -:668: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #668: FILE: drivers/gpu/drm/i915/display/intel_dpll.c:1899: + vlv_dpio_write(dev_priv, phy, VLV_PLL_DW10(pipe), 0x009f0003); -:672: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #672: FILE: drivers/gpu/drm/i915/display/intel_dpll.c:1902: + vlv_dpio_write(dev_priv, phy, VLV_PLL_DW10(pipe), 0x00d0000f); -:679: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #679: FILE: drivers/gpu/drm/i915/display/intel_dpll.c:1908: + vlv_dpio_write(dev_priv, phy, VLV_PLL_DW5(pipe), 0x0df40000); -:683: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #683: FILE: drivers/gpu/drm/i915/display/intel_dpll.c:1911: + vlv_dpio_write(dev_priv, phy, VLV_PLL_DW5(pipe), 0x0df70000); -:689: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #689: FILE: drivers/gpu/drm/i915/display/intel_dpll.c:1916: + vlv_dpio_write(dev_priv, phy, VLV_PLL_DW5(pipe), 0x0df70000); -:693: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #693: FILE: drivers/gpu/drm/i915/display/intel_dpll.c:1919: + vlv_dpio_write(dev_priv, phy, VLV_PLL_DW5(pipe), 0x0df40000); -:723: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #723: FILE: drivers/gpu/drm/i915/display/intel_dpll.c:1997: + vlv_dpio_write(dev_priv, phy, CHV_CMN_DW13(port), 5 << DPIO_CHV_S1_DIV_SHIFT | -:735: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #735: FILE: drivers/gpu/drm/i915/display/intel_dpll.c:2007: + vlv_dpio_write(dev_priv, phy, CHV_PLL_DW1(port), DPIO_CHV_M1_DIV_BY_2 | -:783: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #783: FILE: drivers/gpu/drm/i915/display/intel_dpll.c:2062: + vlv_dpio_write(dev_priv, phy, CHV_CMN_DW14(port), + vlv_dpio_read(dev_priv, phy, CHV_CMN_DW14(port)) | total: 0 errors, 0 warnings, 13 checks, 805 lines checked
