Hi,
This patchset addresses a number of potential undefined behaviors in
bit-shifting operations where the shift amount is derived from an enum
that can in theory hold a negative value (e.g. INVALID_TRANSCODER,
PHY_NONE), as reported by static analyzers.
None of these are actual problems with the current implementation,
because the values are always valid when the code is reached, but
it's more robust to make the shifts always well-defined.
Please review.
Cheers,
Luca.
Luca Coelho (4):
drm/i915/display: harden masks in HSW_AUD_PIN_ELD_CP_VLD macros
drm/i915/display: harden shifts in ICL_DPCLKA_CFGCR0_DDI_CLK_SEL
macros
drm/i915/display: harden shift in intel_ddi_compute_config_late()
drm/i915/display: harden shift in intel_pmdemand_update_phys_mask()
drivers/gpu/drm/i915/display/intel_audio_regs.h | 16 ++++++++++++----
drivers/gpu/drm/i915/display/intel_ddi.c | 4 ++--
.../gpu/drm/i915/display/intel_display_regs.h | 13 ++++++++++---
drivers/gpu/drm/i915/display/intel_pmdemand.c | 2 +-
4 files changed, 25 insertions(+), 10 deletions(-)
--
2.53.0
