drm/i915/pps: split intel_pps_reset_all() to vlv and bxt variants
authorJani Nikula <jani.nikula@intel.com>
Thu, 19 Sep 2024 09:04:27 +0000 (12:04 +0300)
committerJani Nikula <jani.nikula@intel.com>
Mon, 23 Sep 2024 08:21:17 +0000 (11:21 +0300)
commit39bc6d24f01f4a62b098f6531533dc72d1ecc99c
tree3a5c1c7fefee5d996a61b3eeb421b8d43c6f4eda
parentc6be231c9f98ec9e07884dc39e28d45123840958
drm/i915/pps: split intel_pps_reset_all() to vlv and bxt variants

The intel_pps_reset_all() function does similar but not quite the same
things for VLV/CHV and BXT/GLK. Observe that it's called from platform
specific code only, and a split to two functions vlv_pps_reset_all() and
bxt_pps_reset_all() is natural.

Remove the platform checks and warnings from the functions. We don't
usually have them, unless we're unsure. To make this easier to reason
about for BXT/GLK, change the condition on caller side from "!PCH" to
"BXT || GLK".

Suggested-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240919090427.1859032-1-jani.nikula@intel.com
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/intel_display_power_well.c
drivers/gpu/drm/i915/display/intel_pps.c
drivers/gpu/drm/i915/display/intel_pps.h