Re: [PATCH] drm/i915/pps: split intel_pps_reset_all() to vlv and bxt variants

2024-09-23 Thread Jani Nikula
On Thu, 19 Sep 2024, Ville Syrjälä wrote: > On Thu, Sep 19, 2024 at 12:04:27PM +0300, Jani Nikula wrote: >> 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 fun

Re: [PATCH] drm/i915/pps: split intel_pps_reset_all() to vlv and bxt variants

2024-09-19 Thread Ville Syrjälä
On Thu, Sep 19, 2024 at 12:04:27PM +0300, Jani Nikula wrote: > 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_a

[PATCH] drm/i915/pps: split intel_pps_reset_all() to vlv and bxt variants

2024-09-19 Thread Jani Nikula
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 fu