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

platform/x86: ISST: Use in_range() to check package ID validity 

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



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

Intel-SIG: commit e4e365b43460 platform/x86: ISST: Use in_range() to check package ID validity.
Backport Intel speed select ISST driver support on TPMI.

Use in_range() macro to simplify range check.

No functional impact is expected.

Signed-off-by: default avatarSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Suggested-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Link: https://lore.kernel.org/r/20240423204619.3946901-9-srinivas.pandruvada@linux.intel.com


Reviewed-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
[ Yingbao Jia: amend commit log ]
Signed-off-by: default avatarYingbao Jia <yingbao.jia@intel.com>
parent 10bf0052

drivers/platform/x86/intel/speed_select_if/isst_tpmi_core.c

+1 −2
Original line number Diff line number Diff line
@@ -531,8 +531,7 @@ static struct tpmi_per_power_domain_info *get_instance(int pkg_id, int power_dom 
	struct tpmi_sst_struct *sst_inst;

	u8 part;



	if (pkg_id < 0 || pkg_id > isst_common.max_index ||

	    pkg_id >= topology_max_packages())

	if (!in_range(pkg_id, 0, topology_max_packages()) || pkg_id > isst_common.max_index)

		return NULL;



	sst_inst = isst_common.sst_inst[pkg_id];