Commit 0dfa6303 authored by Srinivas Pandruvada's avatar Srinivas Pandruvada Committed by Jia, Yingbao
Browse files

platform/x86: ISST: Use only TPMI interface when present 

mainline inclusion
from mainline-v6.11
commit 2f9514f005530502452c34295e77bdfb395b5bc6
category: bugfix
bugzilla: https://gitee.com/openeuler/intel-kernel/issues/IB6QD3
CVE: NA
Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=2f9514f005530502452c34295e77bdfb395b5bc6



-------------------------------------------------

Intel-SIG: commit 2f9514f00553 platform/x86: ISST: Use only TPMI interface when present.
Backport Intel speed select ISST driver bugfix from upstream 6.11.

When the TPMI interface is present, use this interface instead of legacy.
On some systems legacy IO device is also present. Using both interfaces
together is confusing and may set the hardware in inconsistent state.

When TPMI interface is present, don't load legacy drivers.

Signed-off-by: default avatarSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Reviewed-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: default avatarZhang Rui <rui.zhang@intel.com>
Reviewed-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Link: https://lore.kernel.org/r/20240531083554.1313110-4-srinivas.pandruvada@linux.intel.com


Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
[ Yingbao Jia: amend commit log ]
Signed-off-by: default avatarYingbao Jia <yingbao.jia@intel.com>
parent 1daa90d3

drivers/platform/x86/intel_speed_select_if/isst_if_common.c

+3 −0
Original line number Diff line number Diff line
@@ -777,6 +777,9 @@ int isst_if_cdev_register(int device_type, struct isst_if_cmd_cb *cb) 
	if (device_type >= ISST_IF_DEV_MAX)

		return -EINVAL;



	if (device_type < ISST_IF_DEV_TPMI && isst_hpm_support)

		return -ENODEV;



	mutex_lock(&punit_misc_dev_open_lock);

	/* Device is already open, we don't want to add new callbacks */

	if (misc_device_open) {