From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (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 68A1A3DD86F for ; Tue, 13 Oct 2026 03:09:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791860948; cv=none; b=GmVebw4M3JQPFNXGKiygvZMfiiHW2QmcDP8d4KiMz7R4nl6pPFpfDGKJ3GSHdcUrSOvwWYTLaDRztTnTKESylwSDtxZfKXpCZwhWxXiDUeLtrDii59BlUkgocPglYspgI1TKigSm8OGGFWBvNvmc2NXFwvJ/OcmhA3UTqF4dza4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791860948; c=relaxed/simple; bh=2DaEZecYbqzdkows3wlJT/6rHLkdPtQeCKBsVysIRa8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=I5eR08cNLyZwdJSOLMPge412vcB168O9jfPNeKfMW0JJq4UfauYamsk8PQVsQi/BBnPGZpEQFVY/2VsnwNFFK1MeDnEzMiCS7Cpp7m6mYMXWLN3Eo5r4sL+6m+1ZsnzixyyIjVqC+pxkRiqD4WFyQudPBFI58y+d7+eAG2IQeG4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=ONMnEjSd; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=TnyhHrjL; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="ONMnEjSd"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="TnyhHrjL" Received: from pps.filterd (m0279868.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 69D21E493752532 for ; Tue, 13 Oct 2026 03:09:06 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to; s=qcppdkim1; bh=5ROHOyfkd+2sJ+PYo6ghhU3d bvpfdOhK3QHU5GiExA8=; b=ONMnEjSdkFsg84qr8BwvQxnBWKmAH3jxBFV5+zi2 I3OBTb2gsYOCkOLQsza4JVDkYskdHF30qLwAb/hI0/CqzEJL3csUFxrhYEuNRPDG HgoUrx1HXk7ZMVnZPYwmaF65tVltAKC4TfuUwzAcFai9QdTuI6MaPEiGsXxhVSN5 zz00O28Ui/H8kuhETuzmALvIJc3Z5e73qRpcbLYTxuP+y50YaaqJ5z/JOXspGa4Y YFYqnfZ37y36/8sHn4nYRD8acy6lKxbbRMXl7xPoRRv3RVhcA99qNjf79Kc8lxKD D05k0SZMRBA5TsO5IY85KYCOwCuIUlXA6HGfzSC+HVYe0g== Received: from mail-dy1-f199.google.com (mail-dy1-f199.google.com [74.125.82.199]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4h940w1w44-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Tue, 13 Oct 2026 03:09:06 +0000 (GMT) Received: by mail-dy1-f199.google.com with SMTP id 5a478bee46e88-3548cdfd2fbso4051316eec.1 for ; Mon, 12 Oct 2026 20:09:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1791860945; x=1792465745; darn=vger.kernel.org; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=5ROHOyfkd+2sJ+PYo6ghhU3dbvpfdOhK3QHU5GiExA8=; b=TnyhHrjL8Ugndz5knosCKFfIEdMCpHImqUFhIWqkwxW5rFGPK9drcfEE3hyYtAP8PK wCMVxy8MfJ3zWPVtzKWAWtI+Awgms76+IBE84AA3rLgt2DbmnnZyY267hAXKfk+Wg1BL RLvD59pvO6VFSAo0fYJj+u35yUj5R42AqzgrYsjWV15nRuQCB6zLhds2hocTFVe7sSGM hg54yT6H3/cvjKn/qT4Jn/UwhlKLTlH0w2VWwNxvnZriMXIDVy4RWU6vI3aEghs8/URt ebh/XVCo+QNOMhk33JX4JQ1iWcKhC/DM/7dpZ9rrBWJcwvTRsVlRoLHD5mvvVn5SVTfi gsWQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1791860945; x=1792465745; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=5ROHOyfkd+2sJ+PYo6ghhU3dbvpfdOhK3QHU5GiExA8=; b=0riMtlI/IBFfzrzPikoQ7g/M/w+pnaKl+ZR5BEJThbobZD8bp8b9RG4jHCPYirxSWS m4JwPN6r4yujVftKyGM3olgJa66IaF6WBROf5Ag+ePTAsBVOA7kuGySq0xWPOadrDelb exppaZe478unhKMkaN7uRqUyOOXs/vhk3p2rQC9ajP+YJjC6ndC81ACHLHmnNQnr9KIw C0zvIHlpe8+JGm1C6Me0v7OI9mKnm6BkazH2ckhVXBdH2adhWExK7ej0E5k2aeB+H06U XrQ9nXz8EdAsHUii0fDBm9+AvPNkSQQDs27KpEuzyOidood8r8eBM+XOXcePoIr82LNq iCFA== X-Forwarded-Encrypted: i=1; AKwUvBxddoHHym/a+jcWaZGq7xoE7A4rOS7w0n8Mi9ZRRdwA7JnHX4H+1RE6DnjMg7H+9+tDk5k/ytj+k00p@vger.kernel.org X-Gm-Message-State: AFq9FYIVmZN6wQ/cVKnHtUUiVe54Ny02sAl9v/rFLJsUHMwaPHfxq/of OQsGwqrOfG8SHEghrcrd/ESmMnsiqgF823YWappr0a9VuGdJL/cvGfP24igIjhpBSh1IxeYfbWF Q0LbZ+CmcscE1fG3YdGHFCvvf8PCpz2gLnWGv6Rw76qAwjLT4MvPIdh/zAmAJOzCk X-Gm-Gg: AYBFou0bWHx41UoNj8x7yEaTeSK6xyYUSX31W8Z8vhvdxvZlMgRmZn7roi5kLTGpXNI llGGXwQ4b5wPL3lTfyAYqhti321zWfKD6MN9k8QkbJfqGCQE3IXzmlaus7FjR4/+Yqx5Wb8MQzl ea166yJ4I0BwzgOU2ZPmSJw+4y13AX52VYsTqQs+tDBstXPLbaq9h42RjA8gS7bXQSkz1Gykmq8 l2SIsJcqFOMW5+fdX6+fatR11MlICJVgAcL39VwsbzUh6Azpl445V9f1Q1+4DZPh8YuFS1eqj+i skNg7lImxvpRDeXJr9kvbcfDEovpXqkHARjMhWp61Fg/sGi8xqje6GR5+Ooojb38md5w9Y7G0AN VcRhnW8iR8tZScgDudG+U6U/dKyDTd80m1iH+TKIJ1w== X-Received: by 2002:a05:701b:4202:10b0:143:50a0:56be with SMTP id a92af1059eb24-16a5fc910e2mr14712376c88.14.1791860944801; Mon, 12 Oct 2026 20:09:04 -0700 (PDT) X-Received: by 2002:a05:701b:4202:10b0:143:50a0:56be with SMTP id a92af1059eb24-16a5fc910e2mr14712344c88.14.1791860944031; Mon, 12 Oct 2026 20:09:04 -0700 (PDT) Received: from QCOM-aGQu4IUr3Y (i-global052.qualcomm.com. [199.106.103.52]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-3537ca6d202sm37060349eec.13.2026.10.12.20.08.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 12 Oct 2026 20:09:03 -0700 (PDT) Date: Tue, 13 Oct 2026 11:08:56 +0800 From: Shawn Guo To: Taniya Das Cc: Bjorn Andersson , Michael Turquette , Stephen Boyd , Brian Masney , Bartosz Golaszewski , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Neil Armstrong , Konrad Dybcio , Ajit Pandey , Imran Shaik , Jagadeesh Kona , linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Konrad Dybcio , Alexandre Mergnat Subject: Re: [PATCH v3 2/6] clk: qcom: negcc-nord: keep GPU2 CFG clock enabled via critical CBCR Message-ID: References: <20260724-nords_mm_v1-v3-0-32b45232217f@oss.qualcomm.com> <20260724-nords_mm_v1-v3-2-32b45232217f@oss.qualcomm.com> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260724-nords_mm_v1-v3-2-32b45232217f@oss.qualcomm.com> X-Proofpoint-GUID: vLaWCqIFpGN5SqUrgupROlrdFHolsluS X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYxMDEzMDAxMiBTYWx0ZWRfX8hOom7giuQN5 P6IOvxKse1ayfo6nyIq7Uw5W5jZ/tkxTs+C8hISUpoBbzVs//lpzIf2ey2v3ClbEt59O2RnsDjd agDIa0u6VoSZ1XN7pDbraRIZ2rM3QZGBu3HO8fur7hgjHVgrOs87EroJOsrm5zbfBkSCNZD746L Loj04waGVnLH64aiqTO+Njg679TKgEIjf4gkSPXW8hr6F6ImC+fefZGu+W7xZTE4mHBMxJXT19q Alo66aWHq/jf98V9flNPZF/wuuDDbQG2Mw1vDCJ35sbYf2puEuTuXnrE0dMNuikHleAqpprdxRo T0wG6EmI0W72Q4m3UkEhZgcHVui+pBCMKLTVTo5ltXhYLitJEfeLCp//ELFzXyFp2jepyCu+Niy EUtWfOPXKEGETmIbjsVBxfSb2v3BGk32YkR1FLTlZ3OlocYuIXT2rrOjVtywvFlAedwT/Ybs1yA qLAtzEAfcOtt3+WAwGg== X-Proofpoint-Spam-Info: AW1haW4tMjYxMDEzMDAxMiBTYWx0ZWRfX3e0EBKfJeEHL tqCt3HKtAV5OJixLeJwQsiQkv9RbGpUUwJV5byP7sx1lwyE8MIDNwVvrjqrr5RCdLv6D+4ARsYe JjL3cE6LMzNN+mdvSJV684/D0EFZFmk= X-Authority-Analysis: v=2.4 cv=ftZJ914f c=1 sm=1 tr=0 ts=6acda0d2 cx=c_pps a=cFYjgdjTJScbgFmBucgdfQ==:117 a=b9+bayejhc3NMeqCNyeLQQ==:17 a=kj9zAlcOel0A:10 a=660iZSQnnn4A:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=ZpdpYltYx_vBUK5n70dp:22 a=EUspDBNiAAAA:8 a=IpJZQVW2AAAA:8 a=yUjrznqbfBiV5e72-0MA:9 a=CjuIK1q_8ugA:10 a=scEy_gLbYbu1JhEsrz4S:22 a=IawgGOuG5U0WyFbmm1f5:22 X-Proofpoint-ORIG-GUID: vLaWCqIFpGN5SqUrgupROlrdFHolsluS X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1176,Hydra:6.1.134,FMLib:17.12.100.49 definitions=2026-10-13_01,2026-10-09_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 adultscore=0 malwarescore=0 bulkscore=0 lowpriorityscore=0 priorityscore=1501 impostorscore=0 suspectscore=0 spamscore=0 clxscore=1015 phishscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2610080000 definitions=main-2610130012 On Fri, Jul 24, 2026 at 04:25:06PM +0530, Taniya Das wrote: > The GPU2 CFG clock must remain enabled for correct operation and > should not be exposed as a controllable clk_branch. Hi Taniya, Could you share a bit more information why this is the case? I do not seem to get any problem when testing without this change. Shawn > > Remove the clk_branch and mark its CBCR as critical instead to > prevent unintended gating. This follows the same approach as > 'nw_gcc_gpu_cfg_ahb_clk' and aligns with other always-on clocks in > Qualcomm CC drivers. > > Fixes: a4f780cd5c7a ("clk: qcom: gcc: Add multiple global clock controller driver for Nord SoC") > Reviewed-by: Konrad Dybcio > Reviewed-by: Alexandre Mergnat > Signed-off-by: Taniya Das > --- > drivers/clk/qcom/negcc-nord.c | 22 ++++++---------------- > 1 file changed, 6 insertions(+), 16 deletions(-) > > diff --git a/drivers/clk/qcom/negcc-nord.c b/drivers/clk/qcom/negcc-nord.c > index 2e653ef0fe0e304f09ed354d5578fdbed9151d20..bc4d478d5606ad7478431573f98b62f6f9b7d9fb 100644 > --- a/drivers/clk/qcom/negcc-nord.c > +++ b/drivers/clk/qcom/negcc-nord.c > @@ -952,21 +952,6 @@ static struct clk_branch ne_gcc_gp2_clk = { > }, > }; > > -static struct clk_branch ne_gcc_gpu_2_cfg_clk = { > - .halt_reg = 0x34004, > - .halt_check = BRANCH_HALT_VOTED, > - .hwcg_reg = 0x34004, > - .hwcg_bit = 1, > - .clkr = { > - .enable_reg = 0x34004, > - .enable_mask = BIT(0), > - .hw.init = &(const struct clk_init_data) { > - .name = "ne_gcc_gpu_2_cfg_clk", > - .ops = &clk_branch2_ops, > - }, > - }, > -}; > - > static struct clk_branch ne_gcc_gpu_2_gpll0_clk_src = { > .halt_check = BRANCH_HALT_DELAY, > .clkr = { > @@ -1817,7 +1802,6 @@ static struct clk_regmap *ne_gcc_nord_clocks[] = { > [NE_GCC_GPLL0] = &ne_gcc_gpll0.clkr, > [NE_GCC_GPLL0_OUT_EVEN] = &ne_gcc_gpll0_out_even.clkr, > [NE_GCC_GPLL2] = &ne_gcc_gpll2.clkr, > - [NE_GCC_GPU_2_CFG_CLK] = &ne_gcc_gpu_2_cfg_clk.clkr, > [NE_GCC_GPU_2_GPLL0_CLK_SRC] = &ne_gcc_gpu_2_gpll0_clk_src.clkr, > [NE_GCC_GPU_2_GPLL0_DIV_CLK_SRC] = &ne_gcc_gpu_2_gpll0_div_clk_src.clkr, > [NE_GCC_GPU_2_HSCNOC_GFX_CLK] = &ne_gcc_gpu_2_hscnoc_gfx_clk.clkr, > @@ -1946,10 +1930,16 @@ static void clk_nord_regs_configure(struct device *dev, struct regmap *regmap) > qcom_branch_set_force_mem_core(regmap, ne_gcc_ufs_phy_axi_clk, true); > } > > +static const u32 ne_gcc_nord_critical_cbcrs[] = { > + 0x34004, /* NE_GCC_GPU_2_CFG_CLK */ > +}; > + > static const struct qcom_cc_driver_data ne_gcc_nord_driver_data = { > .dfs_rcgs = ne_gcc_nord_dfs_clocks, > .num_dfs_rcgs = ARRAY_SIZE(ne_gcc_nord_dfs_clocks), > .clk_regs_configure = clk_nord_regs_configure, > + .clk_cbcrs = ne_gcc_nord_critical_cbcrs, > + .num_clk_cbcrs = ARRAY_SIZE(ne_gcc_nord_critical_cbcrs), > }; > > static const struct qcom_cc_desc ne_gcc_nord_desc = { > > -- > 2.34.1 > >