From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9EDF9497B90 for ; Mon, 5 Oct 2026 14:03:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791209032; cv=none; b=isjniwp/gscuaVFmZKfDOJulc+zz56KmvcVi9vUGQuy0iCT8XkxVwiiZgknsdYg47kKq3UYSuz+Cz2K/ndKKI4y6Log+2wakHPIoyitB+ElEsKTzhQ+WWrsK0QYsLRXy6CfDeLulHF6KZXbitoZ48WoJqfPwZtNJrrGPJT8V080= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791209032; c=relaxed/simple; bh=12efwERGA3ZMLV2Z4ZdtvR4HzHHN9sGcUtOossAMAHM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=C8EnwnPiorAc0cZ00SLvVDxzS1y5R7gOFLoDrurm6b/VpQQeDAJ1yUmlMDmV/a3Z57ZP2KRQeawn3s2KL33SKuYyHIz9njO7Y+N8wLil9nmPg8T4clnSp7pbjhtefBLP7LwHYi5gOzb9xZuLxWQo+pfX1B4HDnUPm2SUqs8pTzc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=blNpXK6W; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="blNpXK6W" Received: by smtp.kernel.org (Postfix) with ESMTPSA id A12011F00893; Mon, 5 Oct 2026 14:03:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1791209030; bh=MNMzhciw/HYcCDh8BxnUhOp+6/eRlqcmmsiBCVD785g=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=blNpXK6WwOYSpn1mbIuQdDp/pXUnsABBgG+dQPdQ7V5ZuTsZJWp6vvCN3Ltgqz5JY 5tWn9b6VY5B2+sA0c/v44v1Lap+QMtv0zbonEV0hjjT8DvySIJah2RBA9P1v1lWkwa s/Ob448WI3dYzIIdI0P52dC2DqJATiDoULXHthO3lzs8AkIo5gyHdqbPiU0YHUvApn o1DjXNgcrAQIGO5E4dseQhULyP+c2nA7JUkxR4r2ZFPP+cRaYBqaTDdN+UvODhEYUc h5VyP46ew35jEYzrJjTYrqE0lM575DQjHZxGhCbYbeIIu5idIT6N5WFzGPfeyaDjVr ygTKwjuPtAwtg== From: Sasha Levin To: stable@vger.kernel.org Cc: Viken Dadhaniya , stable@kernel.org, Konrad Dybcio , Greg Kroah-Hartman , Sasha Levin Subject: [PATCH 5.15.y] soc: qcom: geni-se: Correct QUP Core ICC vote constants Date: Mon, 5 Oct 2026 10:03:46 -0400 Message-ID: <20261005140331.2995000-1-sashal@kernel.org> X-Mailer: git-send-email 2.53.0 In-Reply-To: <2026100549-acetone-mannish-7172@gregkh> References: <2026100549-acetone-mannish-7172@gregkh> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Viken Dadhaniya [ Upstream commit 1cd9b1be5a9c56fd69e3f555b200a1844f9b873e ] The GENI_TO_CORE ("qup-core") ICC vote selects the QUP Core 2X clock rate. The CORE_2X_*_MHZ constants are expressed in Bps, but their values are several orders of magnitude too small. For example, the 50 MHz threshold is represented by 2500 rather than 25000000 Bps. As a result, clients using these constants can severely under-vote the QUP Core clock. Correct the constants to their intended Bps thresholds so that the ICC provider selects the corresponding QUP Core 2X clock rate. Drop the unused definitions. Fixes: 58ffbba6a399 ("soc: qcom: geni: Support for ICC voting") Cc: stable@kernel.org Signed-off-by: Viken Dadhaniya Reviewed-by: Konrad Dybcio Link: https://patch.msgid.link/20260915-correct-icc-bandwidth-vote-constants-v2-1-f78eb611204e@oss.qualcomm.com Signed-off-by: Greg Kroah-Hartman Signed-off-by: Sasha Levin --- include/linux/qcom-geni-se.h | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/include/linux/qcom-geni-se.h b/include/linux/qcom-geni-se.h index f5672785c0c4a..c54580c480768 100644 --- a/include/linux/qcom-geni-se.h +++ b/include/linux/qcom-geni-se.h @@ -265,17 +265,12 @@ struct geni_se { #define QUP_SE_VERSION_2_5 0x20050000 /* - * Define bandwidth thresholds that cause the underlying Core 2X interconnect - * clock to run at the named frequency. These baseline values are recommended - * by the hardware team, and are not dynamically scaled with GENI bandwidth - * beyond basic on/off. + * QUP Core 2X clock votes used by GENI clients through the "qup-core" ICC + * path. Values are in Bps and must be converted with Bps_to_icc() before + * setting avg_bw. */ -#define CORE_2X_19_2_MHZ 960 -#define CORE_2X_50_MHZ 2500 -#define CORE_2X_100_MHZ 5000 -#define CORE_2X_150_MHZ 7500 -#define CORE_2X_200_MHZ 10000 -#define CORE_2X_236_MHZ 16383 +#define CORE_2X_19_2_MHZ 9600000 +#define CORE_2X_50_MHZ 25000000 #define GENI_DEFAULT_BW Bps_to_icc(1000) -- 2.53.0