== Series Details == Series: drm/i915: Audio stuff URL : https://patchwork.freedesktop.org/series/110188/ State : warning == Summary == Error: dim checkpatch failed 89816a330d11 drm/i915/audio: s/dev_priv/i915/ -:1033: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "i915->display.audio.lpe.platdev" #1033: FILE: drivers/gpu/drm/i915/display/intel_audio.c:1405: + if (i915->display.audio.lpe.platdev != NULL) total: 0 errors, 0 warnings, 1 checks, 980 lines checked 8325b0ad65e4 drm/i915/audio: Nuke leftover ROUNDING_FACTOR d094a7c24e7b drm/i915/audio: Remove CL/BLC audio stuff 54d4a3711cf8 drm/i915/audio: Extract struct ilk_audio_regs 4f45f3bb14c8 drm/i915/audio: Use REG_BIT() & co. -:159: WARNING:LONG_LINE: line length of 103 exceeds 100 columns #159: FILE: drivers/gpu/drm/i915/display/intel_audio_regs.h:64: +#define AUD_CONFIG_N(n) (REG_FIELD_PREP(AUD_CONFIG_UPPER_N_MASK, (n) >> 12) | \ -:159: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'n' - possible side-effects? #159: FILE: drivers/gpu/drm/i915/display/intel_audio_regs.h:64: +#define AUD_CONFIG_N(n) (REG_FIELD_PREP(AUD_CONFIG_UPPER_N_MASK, (n) >> 12) | \ + REG_FIELD_PREP(AUD_CONFIG_LOWER_N_MASK, (n) & 0xfff)) -:160: WARNING:LONG_LINE: line length of 102 exceeds 100 columns #160: FILE: drivers/gpu/drm/i915/display/intel_audio_regs.h:65: + REG_FIELD_PREP(AUD_CONFIG_LOWER_N_MASK, (n) & 0xfff)) total: 0 errors, 2 warnings, 1 checks, 171 lines checked 06c9bf190dac drm/i915/audio: Unify register bit naming 6d602f53455a drm/i915/audio: Protect singleton register with a lock 1073b7c305e4 drm/i915/audio: Nuke intel_eld_uptodate() 697922c27a1f drm/i915/audio: Read ELD buffer size from hardware 8cbdd1858525 drm/i915/audio: Make sure we write the whole ELD buffer c1c46ff8ff02 drm/i915/audio: Use u32* for ELD 0b75f88aab20 drm/i915/audio: Use intel_de_rmw() for most audio registers 5f2b90c200c2 drm/i915/audio: Split "ELD valid" vs. audio PD on hsw+ 1ec9053b637e drm/i915/audio: Do the vblank waits 672420cdf531 drm/i915/sdvo: Extract intel_sdvo_has_audio()