✗ CI.checkpatch: warning for drm/i915/display: convert to generic read_poll_timeout() (rev2)

Patchwork patchwork at emeril.freedesktop.org
Mon Jun 30 15:35:03 UTC 2025


== Series Details ==

Series: drm/i915/display: convert to generic read_poll_timeout() (rev2)
URL   : https://patchwork.freedesktop.org/series/150872/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
f8ff75ae1d2127635239b134695774ed4045d05b
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit b0e754cfcc1e976651b48ec3d75424ca72aed247
Author: Jani Nikula <jani.nikula at intel.com>
Date:   Fri Jun 27 17:15:29 2025 +0300

    drm/i915/ddi: prefer read_poll_timeout() over readx_poll_timeout()
    
    Unify on using read_poll_timeout() throughout instead of mixing with
    readx_poll_timeout(). While the latter can be ever so slightly simpler,
    they are both complicated enough that it's better to unify on one
    approach only.
    
    While at it, better separate the handling of error returns from
    drm_dp_dpcd_readb() and the actual status byte. This is best achieved by
    inlining the read_fec_detected_status() function, and switching to
    drm_dp_dpcd_read_byte().
    
    v2: Use drm_dp_dpcd_read_byte() (Imre)
    
    Cc: Imre Deak <imre.deak at intel.com>
    Signed-off-by: Jani Nikula <jani.nikula at intel.com>
+ /mt/dim checkpatch 51249df9fddf0b362b0e4d837e53830c22f3e73c drm-intel
be2f33918fc9 drm/i915/hdmi: use generic read_poll_timeout() instead of __wait_for()
a1201f345e4e drm/i915/hdcp: use generic read_poll_timeout() instead of __wait_for()
545fcbf162bb drm/i915/dsi: use generic read_poll_timeout() instead of wait_for_us()
c4753e481f5b drm/i915/dsi-pll: use generic read_poll_timeout() instead of wait_for()
2bcbde35ef70 drm/i915/gmbus: use generic read_poll_timeout*() instead of wait_for*()
3c5dd5b5b043 drm/i915/wm: use generic read_poll_timeout() instead of wait_for()
55a365ff9b86 drm/i915/cdclk: use generic read_poll_timeout() instead of wait_for()
-:67: WARNING:LONG_LINE: line length of 112 exceeds 100 columns
#67: FILE: drivers/gpu/drm/i915/display/intel_cdclk.c:728:
+					(val & CCK_FREQUENCY_STATUS) == (divider << CCK_FREQUENCY_STATUS_SHIFT),

total: 0 errors, 1 warnings, 0 checks, 77 lines checked
45af41b89398 drm/i915/power: use generic read_poll_timeout() instead of wait_for()
c188e1f35fd9 drm/i915/power-well: use generic read_poll_timeout() instead of wait_for() for DKL PHY
9ba503bf5a07 drm/i915/power-well: use generic read_poll_timeout() instead of wait_for() for VLV/CHV
230d57347667 drm/i915/dp: use generic read_poll_timeout() instead of wait_for()
b50b7f63f44b drm/i915/dp: use generic read_poll_timeout() instead of wait_for() in link training
87372a16ebf3 drm/i915/vblank: use generic read_poll_timeout() instead of wait_for()
0221d83ee8b6 drm/i915/tc: use generic read_poll_timeout() instead of wait_for()
e32ab41eadc3 drm/i915/dsb: use generic read_poll_timeout() instead of wait_for()
c7d73a2e4940 drm/i915/lspcon: use generic read_poll_timeout() instead of wait_for()
6ad8d2b51631 drm/i915/opregion: use generic read_poll_timeout() instead of wait_for()
b0e754cfcc1e drm/i915/ddi: prefer read_poll_timeout() over readx_poll_timeout()




More information about the Intel-xe mailing list