Revert "drm/i915/hwmon: Enable PL1 power limit"
authorAshutosh Dixit <ashutosh.dixit@intel.com>
Wed, 8 Feb 2023 19:03:12 +0000 (11:03 -0800)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Wed, 8 Feb 2023 20:06:38 +0000 (15:06 -0500)
commit05d5562e401eb0aefab89788a24c0d4e776150d7
tree3ec89f3e6afda08b6f95745ca9f0abb598a27935
parenta0dcb06d29d9e477e1984dc3859e61568361fc1a
Revert "drm/i915/hwmon: Enable PL1 power limit"

This reverts commit 0349c41b05968befaffa5fbb7e73d0ee6004f610.

0349c41b0596 ("drm/i915/hwmon: Enable PL1 power limit") is incorrect and
caused a major regression on ATSM. The change enabled the PL1 power limit
but FW sets the default value of the PL1 limit to 0 which implies HW now
works at minimum power and therefore the lowest effective frequency. This
means all workloads now run slower resulting in even GuC FW load operations
timing out, rendering ATSM unusable.

A different solution to the original issue of the PL1 limit being disabled
on ATSM is needed but till that is developed, revert 0349c41b0596.

Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/8062
Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230208190312.1611335-1-ashutosh.dixit@intel.com
drivers/gpu/drm/i915/i915_hwmon.c