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 268852459D4 for ; Tue, 18 Nov 2025 04:48:53 +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=1763441336; cv=none; b=mFlcqYBim29KtPi+p+8RQcloOLt8K3BTb/K7F7AE2tgUJ6AW/DordZVDdTkGjWhn1ZOYj/ZFfVEcHow208kwUZ1sn8ekcy3gf5fP9qvrrLAajA7JGU4RWeDxOv10Ww459lmRCFUnzcWImkbHoSGGF0VGEkbmXQmtMXxCjtniLEA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763441336; c=relaxed/simple; bh=iIFQHjofGlzBMdAV/7VJaucvzmg9LLWDEE3IRoLoW2k=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=Az7lDMAaqmvDqf62oNGqzM0+jFOwbRRuCOla7gpg9RF6crPfo4kko/Je20Moi4t1BUP+lqhnhsb2mc74w29dopnPxaaAJeP0B5PgrX4mrwbE9jQUqARSdDi4xVJKOd0vR5M7vBrsoU9S+SmP8pmbNJk+VHvXva7AtTJAa9LbvTc= 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=OfFQJBsx; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=GN9xW2eb; 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="OfFQJBsx"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="GN9xW2eb" Received: from pps.filterd (m0279871.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 5AHJYR4s4107460 for ; Tue, 18 Nov 2025 04:48:53 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= 6EWfBA1U81NlKI70VVXPQFaenMKF6YCLlfEM2N9rP5c=; b=OfFQJBsxtXWvNmGr bYHHE9YmuzTjkQY0CIR25Q/Zy2aJ0qt33pJ3wAlmInz49rKGqzi2uEsU2ctAEgmq E1VCOyy4RXKHiOjdqFOfD5XNA45zSpk0+V5HOOZ/JCgVci6bHdhn92nPL8uhymrx xS86LYLy6HYyZ5hsj/JqVv9GB2KBSQpEUXEKrAxRsqA+eGTVLU3nGMoBdRD5+m5q 6PPsDI9W3eUqbGssB++7sjEKrXJMwwZjNR1h0FgOdWPZSF9saoUZWWwXPzTe+1AY vrRyR/sxTXh/4ScRVPSbHLDE97gKvbLuYU1m9HVxLXHQgoTpkzmSDaiZtNxNtaAO SqcInA== Received: from mail-pj1-f70.google.com (mail-pj1-f70.google.com [209.85.216.70]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4ag2hpar66-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Tue, 18 Nov 2025 04:48:52 +0000 (GMT) Received: by mail-pj1-f70.google.com with SMTP id 98e67ed59e1d1-3436d81a532so11423179a91.3 for ; Mon, 17 Nov 2025 20:48:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1763441332; x=1764046132; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=6EWfBA1U81NlKI70VVXPQFaenMKF6YCLlfEM2N9rP5c=; b=GN9xW2ebfYI3fohtCLDFnDeed9jcf3A5K8UkeSIeCjyk2Eby1TRU4pjm0C/woID5m1 /px0GAfLrUQ+jOs9Xd3wfJK6ZG7IybzkQiM72znhdoENtyGdhRM0TmC9XpaKuSDPznPN aQSGCIwBgrIf9GZWGZpocPNmJD16A+/U5YIwFOgot2n7cT8poUsu26cj2NMZikBk2krP /f5iAHloXmao0Awix7lN9pKoP0NT47XP6VzLmHuYm9AUA4h6a/IDoVOQADAJ6rHH82LE CVZ1P7Pto1Pdu9HYFv1IPrzZXdzSROyDkNuD87hrEcdB8I0K3HQstub+t7PCymLwqOfj JYUA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763441332; x=1764046132; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=6EWfBA1U81NlKI70VVXPQFaenMKF6YCLlfEM2N9rP5c=; b=Ehv6jepqBAZA8vbkTlue7CAawY425bkQBPsiLvmfMAABOpQa1/WxRKgjM8OhFaDR1B 4jh4rjOr5PWxdDnRBBGMz27AodfktwsikQ0PKT+qsx4bIoinQlVHMU4/g0ws5xCGuazF VHdElZK08seVPNRmoYJMFFg15YJJ6Ckj6qLJxdT8E+u+9DTJQ+jeni+YKyKieNOAF3Q4 LQiM4rWqKnxlEt0hKqvB2Eg6pcS3YDSl2RjaEHy482PGnf3ICPVYP23YMguG1vlc4bYH V24H/YgwDjLtD3IHXb0xzCuJ5pyQe0qe2Fr731ej92iyud0O3bGOOKNIqKDFBqGF81j6 r+3A== X-Forwarded-Encrypted: i=1; AJvYcCXPSxP8EBdp99l+YXZRdxZmjFSY4ZxO4Yzr47PQjDFynNb7weJs0uVOIK5sT3Auk++k69jIJAaGMDyC@vger.kernel.org X-Gm-Message-State: AOJu0YwIxTuER95WoLkhLIMs38RycdhrfyUzlWI0YrB3CJE3qDnq/VkT 4FCoolYTZgt51XE2vWm8e6Q4pUzNwhMdyiGWtPwU3mkoFba6hyyC/agJD99tV+EDXhmVnldHDLI GtBjeca/G/DkiW/hWIVvglgpvcz6JS9mzEONE7jHbzjplCuskLMUcyupCun37/GIS X-Gm-Gg: ASbGnct+UpVIC7O1/LYsN0p6Iajww4Yt88Ovutm4rHre9Lj7lboBfsUOAXnPdy1m3ez ecPDYY/wKC+zvu8umcBDMljW9Sns00CoNrBc5WqgPj3fn5wlafSL0N9KLabgCGSW+pWgCl7oAuT f6WvkQHiw5v743KqCL7oCZy7PuiX4A5pqjRi8BK5qprE9XbcpL7IXU7tWMMDgcpceqSvHyL1cC3 mcQXZEnMFJOXnIJX2SriINrdzeXwmYFPIiBgnVdjUtqh39nkzrEtRYDnQNErip35PbHe9m3JO0m FJBeifl+QwT0t5WWLXftZinN5R8yRu165ScgY33YhrgcdOXecr4+FZHwhz6j95BwpNxxMD8fN/I YyWB7kf+0o+Bxh5diFLK5cwd5/du8c63/PSH0bg== X-Received: by 2002:a17:90b:2ccc:b0:340:ba29:d3b6 with SMTP id 98e67ed59e1d1-343f9e92724mr16204363a91.6.1763441331911; Mon, 17 Nov 2025 20:48:51 -0800 (PST) X-Google-Smtp-Source: AGHT+IGx0fVzlAnUq4dpOWrVnMfPjWnKeWw+SIEXnH5mFBLo1+TOoZKSzxI6/PYJd2KildgZH1jJZA== X-Received: by 2002:a17:90b:2ccc:b0:340:ba29:d3b6 with SMTP id 98e67ed59e1d1-343f9e92724mr16204340a91.6.1763441331437; Mon, 17 Nov 2025 20:48:51 -0800 (PST) Received: from [10.217.217.147] ([202.46.22.19]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-343e0794e56sm20397887a91.10.2025.11.17.20.48.46 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 17 Nov 2025 20:48:50 -0800 (PST) Message-ID: Date: Tue, 18 Nov 2025 10:18:44 +0530 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 4/6] clk: qcom: rpmh: Add support for Kaanapali rpmh clocks To: Dmitry Baryshkov Cc: Bjorn Andersson , Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , jingyi.wang@oss.qualcomm.com, aiqun.yu@oss.qualcomm.com, Ajit Pandey , Imran Shaik , Jagadeesh Kona , linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20251030-gcc_kaanapali-v2-v2-0-a774a587af6f@oss.qualcomm.com> <20251030-gcc_kaanapali-v2-v2-4-a774a587af6f@oss.qualcomm.com> <380aa79f-f334-44db-9527-85247f9735af@oss.qualcomm.com> <2jfvrt4r7ddagh2ztbad7qnjpcvulcrtr3ekul6i26qg6zcd44@dxfhl3265okx> Content-Language: en-US From: Taniya Das In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Proofpoint-GUID: kzgAeBdkDHEZgVAJmxQy3AsfFDeUOKdQ X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUxMTE4MDAzNSBTYWx0ZWRfX69UcHO7MjbKc 6aDlPT5AZbvLlriB1ekyb78ts6eUvKG49CBppyvpzqcEl9CtYANFdH/vT1cOZHY3mEUS04S7mw3 NEA3ihDGl1vLlw70qLnx02FFnlCaDlepRQuWO1GktFmkobC+GP5ez+tt5GLag3Ms5HrMDHVlKSg NvsdrabqKXpgqAfkyYGPyTdn2Y/hf+9ImUQ651rs3yMjcFPo3pkoaOGRjnY3Z7x55tba+3smcye 7wDGu/8AlQgvWsqQp/B4HBhYPIaBaJoGQDSvg/Hnm9yNQWzpNfXBqsHkCHURRfDKCqHLBP2WzLl kccR4XqvRqMQo1/8Ni+XhyKV0tcOoEesueDrVd9M/u6teqkzUOF9hIz3JNz3jUh3kxLEDcJClyl XrJICH/fgGgTHw8OyQKW/nPTcPf92Q== X-Proofpoint-ORIG-GUID: kzgAeBdkDHEZgVAJmxQy3AsfFDeUOKdQ X-Authority-Analysis: v=2.4 cv=Y5L1cxeN c=1 sm=1 tr=0 ts=691bfab4 cx=c_pps a=0uOsjrqzRL749jD1oC5vDA==:117 a=fChuTYTh2wq5r3m49p7fHw==:17 a=IkcTkHD0fZMA:10 a=6UeiqGixMTsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=EUspDBNiAAAA:8 a=A39G7UXKOQFEQjqAOJYA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=mQ_c8vxmzFEMiUWkPHU9:22 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.9,FMLib:17.12.100.49 definitions=2025-11-17_04,2025-11-13_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 suspectscore=0 spamscore=0 bulkscore=0 phishscore=0 priorityscore=1501 malwarescore=0 impostorscore=0 lowpriorityscore=0 adultscore=0 clxscore=1015 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2510240001 definitions=main-2511180035 On 11/18/2025 10:04 AM, Dmitry Baryshkov wrote: > On Mon, Nov 17, 2025 at 01:28:46PM +0530, Taniya Das wrote: >> >> >> On 11/14/2025 4:38 PM, Dmitry Baryshkov wrote: >>> On Fri, Nov 14, 2025 at 02:13:49PM +0530, Taniya Das wrote: >>>> >>>> >>>> On 11/11/2025 4:16 PM, Dmitry Baryshkov wrote: >>>>> On Thu, Oct 30, 2025 at 04:39:07PM +0530, Taniya Das wrote: >>>>>> Add the RPMH clocks present in Kaanapali SoC. >>>>>> >>>>>> Signed-off-by: Jingyi Wang >>>>>> Signed-off-by: Taniya Das >>>>>> --- >>>>>> drivers/clk/qcom/clk-rpmh.c | 42 ++++++++++++++++++++++++++++++++++++++++++ >>>>>> 1 file changed, 42 insertions(+) >>>>>> >>>>>> diff --git a/drivers/clk/qcom/clk-rpmh.c b/drivers/clk/qcom/clk-rpmh.c >>>>>> index 1a98b3a0c528c24b600326e6b951b2edb6dcadd7..fd0fe312a7f2830a27e6effc0c0bd905d9d5ebed 100644 >>>>>> --- a/drivers/clk/qcom/clk-rpmh.c >>>>>> +++ b/drivers/clk/qcom/clk-rpmh.c >>>>>> @@ -395,6 +395,19 @@ DEFINE_CLK_RPMH_VRM(clk4, _a, "C4A_E0", 1); >>>>>> DEFINE_CLK_RPMH_VRM(clk5, _a, "C5A_E0", 1); >>>>>> DEFINE_CLK_RPMH_VRM(clk8, _a, "C8A_E0", 1); >>>>>> >>>>>> +DEFINE_CLK_RPMH_VRM(ln_bb_clk1, _a2_e0, "C6A_E0", 2); >>>>>> +DEFINE_CLK_RPMH_VRM(ln_bb_clk2, _a2_e0, "C7A_E0", 2); >>>>>> +DEFINE_CLK_RPMH_VRM(ln_bb_clk3, _a2_e0, "C8A_E0", 2); >>>>>> + >>>>>> +DEFINE_CLK_RPMH_VRM(rf_clk1, _a_e0, "C1A_E0", 1); >>>>>> +DEFINE_CLK_RPMH_VRM(rf_clk2, _a_e0, "C2A_E0", 1); >>>>> >>>>> What is the difference between these clocks and clk[3458] defined few >>>>> lines above? Why are they named differently? If the other name is >>>>> incorrect, please fix it. >>>>> >>>> >>>> Dmitry, my intention was to make a clear distinction between the ‘rf’ >>>> clocks and the ‘ln’ clocks. Since there could be overlap in the >>>> numbering, I added prefixes for clarity. I should have applied the same >>>> approach to clk[3458] as well. I will add the fix-up for the same. >>> >>> Why do we need to distinguish between them here? The resources in CMD-DB >>> don't have such a difference. You'll select whether the clock is RF or >>> LN when describing the platform data. >>> >> >> It is more for readibility and maintain a direct mapping with the PMIC >> clock grid. This way we can immediately identify the clock type without >> cross-referencing desc as the clock mapping here would indicate the type >> of clock. Yes, the CMD-DB name does not reflect anything with the names >> here. Please do let me know your suggestion. > > I'd prefer if variables reflect CMD-DB resource names rather than the > actual clock usage. In the end, platform data does exactly that - it > maps usage to CMD-DB resources. > Actually the grid does not reflect the cmd-db resource name at all, so I do not think that is the right approach. Let me move to completely remove the 'ln' and 'rf' references and refer them to 'clkN'. > Also, if we follow your proposal, we will end up with ln_bb_clk and > rf_clk referencing exactly the same resource, causing possible > confusion. > Sure, Dmitry, I will move to use clkN and remove the suffixes. -- Thanks, Taniya Das