Received: by 2002:a05:7412:3b8b:b0:fc:a2b0:25d7 with SMTP id nd11csp58180rdb; Wed, 7 Feb 2024 21:19:32 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCX7PRsiwF9pdiYgWMWXixGj5dgugwn9J34R5LByqK/vSuq34nQN1Vf/HbVThW6+cTIKXi0CRNbYiNCbdV+5F1s9deJm91QJfeRP6VyK+g== X-Google-Smtp-Source: AGHT+IFCanxn6/ZQqN7Gv1hwE4Uqn3YOF0siGki9C02IGm7vb+FEhplN7zTSbYcHTxP6ynbnut52 X-Received: by 2002:a05:6808:3190:b0:3bf:d29c:3e95 with SMTP id cd16-20020a056808319000b003bfd29c3e95mr9874466oib.24.1707369572372; Wed, 07 Feb 2024 21:19:32 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1707369572; cv=pass; d=google.com; s=arc-20160816; b=z6JRQSyGLA1uxVOwRAfsNImsEJcaoJ87//kvd8P+K+714Wj+lQtXZduEd5PietYxMx bFrGnEPr3Cisk7R9cimS/Civir8LVsUaDt2A2UrNlEjNzFHFwbsT7dmZNc9gs/AwT+zG Eu6Qw85+bh9dmJ792t/W+64z5wnmZoiiYzLICtRWO/75aAqgnmq0dxgZdwErVsBqn81q jgao9c2j1Bd5zKfy4jwUTR+JFcHIOWcSR2XecC8/6mhq2uHTEs9s6Ft/sUulpzaQkD/J ouIMkXl+UsXeVmNJ1yTIWDdDW0xb2xhCJCLS6QElXnLeAfKoTysB1O6QKCf3CNMIRfy7 WSSw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:date:message-id:dkim-signature; bh=iivgMn/eZNoH9LLlfm9hxbr5iUHDmqmWjt66U1kf5qE=; fh=B0r0sYvmPJ79E0G1M2fhopdX3H9l/1/JICFbnqjmPjo=; b=kHmf7C6iXTkpmbme2kuvWK2PWVG0pCBL8NJ6Jh47Eu1TZ6ZGRnu7AbRYDqx85aNiH+ 2URzR4cIT0sUREHB+0nTFRnqG5H6bmEyA46pY+H7IZ1vhYoPFqRZRMIj3rI3YHAJ+0Ar 0tDyUqDky4eJhoPrzRUCa72OmgMvS9kWlOJUxdseFGhQTq/UIrkX7iEv4upQNDttwSiz ccjKepeS7VKmgfOMfPXHSq3yY9o1KXwF1XlExORMVum7WLESJ+TYQvEL8a/9Le5fY4aQ Tn7fIc9wyCU48APuOcEFiyH3SRRu9T6d2PgGFXwe/YhpJAgI6tB0BnXbFXf0bpGA1uy9 YJtA==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=Px45P5S3; arc=pass (i=1 spf=pass spfdomain=quicinc.com dkim=pass dkdomain=quicinc.com dmarc=pass fromdomain=quicinc.com); spf=pass (google.com: domain of linux-kernel+bounces-57473-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-57473-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com X-Forwarded-Encrypted: i=2; AJvYcCXcghdL3LykL1plwEzPaNkAIwB/cBXWhP2sOyOQ02IYWHxONRMqXzNodhTJiabzzJOE2UKz9Q36DvMlzOKZfAFWkYUPUXOUiDO1TjSxgw== Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id f3-20020a056a000b0300b006e0500ee05dsi3046692pfu.160.2024.02.07.21.19.32 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 07 Feb 2024 21:19:32 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-57473-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=Px45P5S3; arc=pass (i=1 spf=pass spfdomain=quicinc.com dkim=pass dkdomain=quicinc.com dmarc=pass fromdomain=quicinc.com); spf=pass (google.com: domain of linux-kernel+bounces-57473-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-57473-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 91C9F284769 for ; Thu, 8 Feb 2024 05:19:04 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 73B2E67C45; Thu, 8 Feb 2024 05:18:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=quicinc.com header.i=@quicinc.com header.b="Px45P5S3" 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 A361D67A07; Thu, 8 Feb 2024 05:18:52 +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=1707369534; cv=none; b=kq25GMA1fFskls56hzsZytki6i3eNIvb8+27ZTE3A8HOszIy4iEoekkOor6DYFHInlQwKPkvF4TVNM05ALxXqF08oMETaJEYYCJXauTkhWnhXz7LNLgvds3gRJbCLbb/9AxfCahztWVOtj88kiWC3ONWvL1tTCgoy+ewTS7cXG0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707369534; c=relaxed/simple; bh=iivgMn/eZNoH9LLlfm9hxbr5iUHDmqmWjt66U1kf5qE=; h=Message-ID:Date:MIME-Version:Subject:To:CC:References:From: In-Reply-To:Content-Type; b=ONcuz7TpCN0Iof4aLd/OmXtmQAjhetgApjkp1sZAEjSK2cmFBJ4u0NpUn+IARx17Dud70zss/jNvLhY4F9LJ//miOcJVZaBW69UNyge7giMAIgVBtMBn1h1UTIRFR6KzeaIad13e7TegBVYwd7bsxesc0ev71JvCokfWDGpjprw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=quicinc.com; spf=pass smtp.mailfrom=quicinc.com; dkim=pass (2048-bit key) header.d=quicinc.com header.i=@quicinc.com header.b=Px45P5S3; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=quicinc.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=quicinc.com Received: from pps.filterd (m0279869.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.17.1.24/8.17.1.24) with ESMTP id 4183s3oa031299; Thu, 8 Feb 2024 05:18:48 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; h= message-id:date:mime-version:subject:to:cc:references:from :in-reply-to:content-type:content-transfer-encoding; s= qcppdkim1; bh=iivgMn/eZNoH9LLlfm9hxbr5iUHDmqmWjt66U1kf5qE=; b=Px 45P5S3WXfrsEN4qDVQvkopGMe3KpSmiqelSlVz2XcrQiujk0Qvk+FK5cQ41YDk9e C/2tJCdZoefgIZUcbhEE8KONRPtGBh5T+iQEmm7eSr7ol388jfnmauQoWu1sneSl yZHp/sY+oXlxTJ0hq/MtG8gtKNPbF9JMTsro7ZJb3JRAs31UgbmgZrVF6R0PhsU/ hB8sXZ431C2D4pW9ued8URwuby8TdQNx4NAw7ex3G9wBZHx4bRbA1fsPkktZWQZn +2aEO9+CEvF/5Dd/ceBvJMMznxO09Yj+epf6VBmXYcRRojbks/CBd2oqxIajPflj /G40K8fivBbXqvhLrAqA== Received: from nalasppmta03.qualcomm.com (Global_NAT1.qualcomm.com [129.46.96.20]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 3w4pavg84h-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 08 Feb 2024 05:18:48 +0000 (GMT) Received: from nalasex01c.na.qualcomm.com (nalasex01c.na.qualcomm.com [10.47.97.35]) by NALASPPMTA03.qualcomm.com (8.17.1.5/8.17.1.5) with ESMTPS id 4185IlYq024107 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 8 Feb 2024 05:18:47 GMT Received: from [10.218.19.46] (10.80.80.8) by nalasex01c.na.qualcomm.com (10.47.97.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.40; Wed, 7 Feb 2024 21:18:42 -0800 Message-ID: <69250065-2168-0159-2ef9-986716d3adf8@quicinc.com> Date: Thu, 8 Feb 2024 10:48:39 +0530 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.9.0 Subject: Re: [PATCH 2/3] clk: qcom: gcc-sm8150: Add gcc_parents_0_ao support Content-Language: en-US To: Konrad Dybcio , Bjorn Andersson , Michael Turquette , "Stephen Boyd" , Rob Herring , "Krzysztof Kozlowski" , Conor Dooley , Taniya Das CC: , , , , Ajit Pandey , Imran Shaik , "Jagadeesh Kona" References: <20240123-gcc-ao-support-v1-0-6c18d5310874@quicinc.com> <20240123-gcc-ao-support-v1-2-6c18d5310874@quicinc.com> <77903574-696b-90f9-f136-be5c5d219ba1@quicinc.com> <5ae84692-b05d-4a43-aabb-4d2e7d9926d5@linaro.org> <04788b2c-a212-b8a2-1124-d904f2f61f5d@quicinc.com> <6fdde54e-12cb-442c-9dea-d1d6cbca340b@linaro.org> From: "Satya Priya Kakitapalli (Temp)" In-Reply-To: <6fdde54e-12cb-442c-9dea-d1d6cbca340b@linaro.org> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit X-ClientProxiedBy: nasanex01b.na.qualcomm.com (10.46.141.250) To nalasex01c.na.qualcomm.com (10.47.97.35) X-QCInternal: smtphost X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=5800 signatures=585085 X-Proofpoint-ORIG-GUID: 4TrP1OWOFRi7t024bkfVVs_bzsyXq1wC X-Proofpoint-GUID: 4TrP1OWOFRi7t024bkfVVs_bzsyXq1wC X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26 definitions=2024-02-08_01,2024-02-07_01,2023-05-22_02 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 priorityscore=1501 mlxscore=0 lowpriorityscore=0 malwarescore=0 mlxlogscore=999 phishscore=0 bulkscore=0 adultscore=0 clxscore=1015 suspectscore=0 impostorscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.19.0-2401310000 definitions=main-2402080025 On 2/5/2024 5:03 PM, Konrad Dybcio wrote: > On 5.02.2024 09:37, Satya Priya Kakitapalli (Temp) wrote: >> On 1/25/2024 3:25 PM, Konrad Dybcio wrote: >>> >>> On 1/25/24 06:49, Satya Priya Kakitapalli (Temp) wrote: >>>> On 1/23/2024 11:17 PM, Konrad Dybcio wrote: >>>>> >>>>> On 1/23/24 17:34, Satya Priya Kakitapalli wrote: >>>>>> Add active_only support for gcc_parents_0, this is needed because >>>>>> some of the clocks under it are critical which would vote on xo >>>>>> blocking the suspend. >>>>>> >>>>>> Signed-off-by: Satya Priya Kakitapalli >>>>>> --- >>>>> Is there a need to keep gcc_cpuss_ahb_clk_src around? Do we do any >>>>> ratesetting on it? Should we ever turn it off? >>>>> >>>> The branch clocks under gcc_cpuss_ahb_clk_src are critical clocks, which are running at 19.2Mhz causing vote on XO during suspend. As of now no rate setting is happening but this rcg is useful to get the exact rates from debugfs. Hence this change is needed to avoid XO shutdown issues. >>> So, if I underderstood you correctly, this clock serves no purpose other >>> than getting rate? >>> >>> In this case, I'd say we should de-register it from the clock driver and >>> use debugcc [1] (contributions welcome!) for precise measurements. >>> >> Although currently there is no rate-setting happening now, its better to keep the rcg modelling as is, considering that it might be needed if some use case arises in future. > We don't tend to solve imaginary problems, please lay out what it would be > used for, if at all. Then, we can assess whether it's necessary. Okay, I'll de-register this RCG.