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 760982BE03C for ; Wed, 26 Aug 2026 03:18:54 +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=1787714335; cv=none; b=gc+I5CIh7sh1+om/KWSyFtmBtXsBxIIfQHn1BuYucNk3jEFSA69IyCukmKsBbSaVjUXpcJx/CHFEYiwCmve0OTRcRIUd/vToWhrBJawzu3uRCrGIkb+V2xS2bV1znK0T0pKhKy2vgS5kguabTkssZZ5J56qMH6CvvrSXBvEIWMk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787714335; c=relaxed/simple; bh=mViQhBv9Io1wKUsVU4M2w5Iku9EsAue2Mbo3ELpm93I=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=PQn+yzoeCswHh+brywjcT58wey6Unc0ToqyyAC/SxrBOlwBG+mao57W6UPRbAeLoydyOROCRq+MuhNsu0JIozLi3/YBXAJNqaM0i3sDxlAhwRWuTHzXdq5ba8RD6sgwqoTCzv0q6K6hxydyT4/kARjp3fjD6DkRsJdMgs0nfd8M= 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=Th5GLWzs; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=LTH1GESZ; 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="Th5GLWzs"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="LTH1GESZ" 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 67Q3HGhl030887 for ; Wed, 26 Aug 2026 03:18:53 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=ZUIYAbbI0PQ2rPV+8Y0oOuMg GdJMz2un1mMUt72E1EY=; b=Th5GLWzsnn+IqCdRXiEH3rw4Se6lZXWmMeh/7slz cKtLjgNoTgRIwONH+XD3Zl6T/BkYilIaouYOHQoPNA7vobAg9cUyr7eC8hZQgVAH eWb9FNZX7jK70Q/YnRMbWOShTb63es8iLynKfJXxRrYc8c/CgGJVyAItvUQgz1d3 tLQJr/HeZZZG4eaQIRLXM9kh/HTEoemaO3HgSzl0tr9cUgm3dmisNS5vpfu6E8vO 4bCUuaXGnqCBC8aA7Rf/HXR+CG1yyPEfN1o+nY1FjEKCi/vnYaR6BPcC+0FRIYB4 hGxL6KUEami98e9Zjl0q3xbTQq8JJ04TzMrmVh5ukDJryg== Received: from mail-pj1-f71.google.com (mail-pj1-f71.google.com [209.85.216.71]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4g9r1gg0np-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Wed, 26 Aug 2026 03:18:52 +0000 (GMT) Received: by mail-pj1-f71.google.com with SMTP id 98e67ed59e1d1-395543dc382so700234a91.0 for ; Tue, 25 Aug 2026 20:18:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1787714332; x=1788319132; 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=ZUIYAbbI0PQ2rPV+8Y0oOuMgGdJMz2un1mMUt72E1EY=; b=LTH1GESZC/s5FWjL7wHEK7zPYt36xwhtjplWbfnyhGCuY9QKA5d0Qq8HZep3d/Mfdw ZPUyr7D41ACt6pOlijgZ0eBQeQT5/Y3649i0NKrttOFRBGbe0UKiD0q8QQe/Yfs5jMM/ fVyx2HNcUk8a7C/7XIfhh7Ei2oa3pcDTC7EAOQjmptxh8OKUq0/+PW38wGFBeZTtkrHR Gf6X976cYKXsu8rXLfyYnNHvAGLVxHQZ2EnKpct638DBgniUNTfjp7SNLSu15QgdpTk1 rfqRv9u3LlA+vU+v3kjKKzkjVK/6zcb5LgH/+mz9X/uHuEFnm9dzz+IPm6r0bgpsRRJI pIkg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787714332; x=1788319132; 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=ZUIYAbbI0PQ2rPV+8Y0oOuMgGdJMz2un1mMUt72E1EY=; b=Sp+MchiQqxRu0x9c3W92ZhTNiaPL/+p25IY9yvKzsie6pUTxhALsDrFZpVRnl2B7rc ULsE5gK2j19paUnveJ8xhGjnTUqR9S0lhQ5G3M8Rm6FdwC7WAtBToc5sFrt6OYgaTzW2 5KWLZxUqqeJYwXwcEogBLQnpgXx0jMF+wLQ0kI9/bkuG/MeqzVxzTyfBpssuAPNsie4K 3McJM71bMf+CyQJWISH79FHU178tac6AX4mTr7M9DS45cnMhYB3zeQ7wufaF39gXUq0o BHA3wLiplcWUVSTTYidjW0f0vGh78Mzo6tUlcj8/UumuV77v9B0fNRR6FiQBlOx4sFvW /92w== X-Forwarded-Encrypted: i=1; AHgh+RrUJwW23P1/uE0pVQFmh38P3nsaMfUSntBBaPDog5PaXCifEsEI384QPYnITEIL0HCRjVVoi6TIWRMCXx3SJnF/@vger.kernel.org X-Gm-Message-State: AFuF++nVE0YQAWqu9ugPbx8+rYHUsB1pBHRQxy6gEn4J/X37fZFMozlP bgDB1AuMft7OZxapjKqV+ZVs6BnYF3n2sQUDAJCbEJ7AAveVXUXbSJD6Y914hFv0+wBa5F3Ac3F PKClv9/E8Z0hiwKeXPtiD7v9nbqG4RsGea6JA9WY+XI1LNdRDrPg7g54C9IKYVSjyqCCprR1t X-Gm-Gg: AR+sD11UXQbJUIqhlZNCdTba6q60gjoe6tfJXiNhBVGuNrE672omzV+qdeeZ9nCijjz G0wd7+G0aBzk+hC0BfZ4c3KDTdiKPsrKmAEZCaABKgHZUi7LhAkqcgacTR2+JuNaTfuKGmXb7DJ j19PFgCEKRT9caTjVUViL4f/okJGNcTZn03smqle3++CPBORiHanUAnfY4AXDAAldJjQr/NRVO8 FP2EC9E0m/LPrxX1z+EBM4jXqHGvTUfvuu+c0okpDiIMU8CT2yS5xbuz+w6pynu78M3FsZeMOLc ewzpiITN5ecp5Ijoo01vrrUe6JtmJ2fzUdaOI9uaealPeRFmqQD/EQ3PWOt4ZIdfyE79hycutiF tukj6AK/8Clb1HuHTxpsSQV7kUhU92rTsKEJeRuqBHHg= X-Received: by 2002:a17:90b:3890:b0:395:bbfe:fc6f with SMTP id 98e67ed59e1d1-3966d401db6mr7594135a91.3.1787714331957; Tue, 25 Aug 2026 20:18:51 -0700 (PDT) X-Received: by 2002:a17:90b:3890:b0:395:bbfe:fc6f with SMTP id 98e67ed59e1d1-3966d401db6mr7594070a91.3.1787714331567; Tue, 25 Aug 2026 20:18:51 -0700 (PDT) Received: from QCOM-aGQu4IUr3Y (i-global052.qualcomm.com. [199.106.103.52]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-3283d885fb6sm3670178eec.14.2026.08.25.20.18.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 25 Aug 2026 20:18:51 -0700 (PDT) Date: Wed, 26 Aug 2026 11:18:45 +0800 From: Shawn Guo To: Konrad Dybcio Cc: Bjorn Andersson , Mathieu Poirier , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Bartosz Golaszewski , Mukesh Ojha , linux-arm-msm@vger.kernel.org, linux-remoteproc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 5/8] remoteproc: qcom: pas: Track HPASS ADSP cluster membership Message-ID: References: <20260815132541.1575121-1-shengchao.guo@oss.qualcomm.com> <20260815132541.1575121-6-shengchao.guo@oss.qualcomm.com> <1699019c-510c-4658-843d-e1ecd7364c5b@oss.qualcomm.com> Precedence: bulk X-Mailing-List: linux-remoteproc@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: <1699019c-510c-4658-843d-e1ecd7364c5b@oss.qualcomm.com> X-Proofpoint-GUID: 5_P637V43j9qiuRA3v8lTFFiQUe8lLj6 X-Authority-Analysis: v=2.4 cv=FPMrAeos c=1 sm=1 tr=0 ts=6a8e5b1c cx=c_pps a=UNFcQwm+pnOIJct1K4W+Mw==:117 a=b9+bayejhc3NMeqCNyeLQQ==:17 a=kj9zAlcOel0A:10 a=Sv0fKeRqtYgA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=3WHJM1ZQz_JShphwDgj5:22 a=NRRTt12i9hMZvaXXc5UA:9 a=CjuIK1q_8ugA:10 a=uKXjsCUrEbL0IQVhDsJ9:22 X-Proofpoint-Spam-Info: AW1haW4tMjYwODI2MDAyNCBTYWx0ZWRfX+wylZZZkND8C VexV9he2nG/x6gFlllIwFIOBwqZSU4XRwz/ltuWjB7cmtGiW9ve2+rMvU5+FJBM4ZArNCIMq6dq 4zfVBLVLRlxP/TxTngtdj+s23HGM0hc= X-Proofpoint-ORIG-GUID: 5_P637V43j9qiuRA3v8lTFFiQUe8lLj6 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwODI2MDAyNCBTYWx0ZWRfX/8BbE11e2hXc QnVgdNTDvF0ooEHu/9BxH5W/wjHBRHopn9nG7NSi8eD95eheXHwAAkBimKjZ0eTyJvDVTvpBC25 N4uvDnT5fxcd7ZU8zHKHfjPf2nYLkUvMBMkQx92QSAOVphXqhnuCKOD5lDNdZW2BTOP+wURI7fJ x3mVd4xruN5lYa8SgSEavX61M2Wcg9bpuWuAsS9EDFEZJrsbgTZiGFxr9RJ4VLhu+r4TSzr0BuO khGDVZkiTul66ZMBJxeEAE/QbVuG8sHHSkZGczTc82ik2OOeye9e1RHixvPq+NUBwlMv5sw9jtw vyM52cHqR07dqHGtlEEltqLkc6D1bnriv10Rqxfm8Ov3v5G4d355JjVUC9xujRi2vyfeP/vOsE1 wpCMcH23Ervqz7JOSrvRgIdSWqNoMfmb9ehFWws2RpmWLrggpeezvBSn/DLYl0vI3TwkKtx5yly ad8TPTE2OMcv1lVpnzw== 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-08-26_01,2026-08-24_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 malwarescore=0 priorityscore=1501 clxscore=1015 lowpriorityscore=0 suspectscore=0 adultscore=0 spamscore=0 phishscore=0 bulkscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2608260024 On Mon, Aug 17, 2026 at 03:18:30PM +0200, Konrad Dybcio wrote: > On 8/15/26 3:25 PM, Shawn Guo wrote: > > On Nord, ADSP0/1/2 share HPASS-domain resources (PLLs, AG_NOC, RSCp, > > CESTA, THROTTLE, QTMR) owned by ADSP0, and their firmware cannot > > tolerate one member being started or stopped independently of the > > others. Modeling that requires each PAS instance to know which other > > instances it is grouped with. > > [...] > > > +static struct qcom_pas_cluster *qcom_pas_cluster_get(struct device_node *node) > > +{ > > + struct qcom_pas_cluster *cluster; > > + > > + mutex_lock(&qcom_pas_cluster_list_lock); > > Use scoped mutex guards Will do in the next version, thanks! > > [...] > > > +static int qcom_pas_cluster_init(struct qcom_pas *pas, struct device_node *np) > > +{ > > + struct device_node *root_node; > > + bool is_root; > > + > > + root_node = of_parse_phandle(np, "qcom,cluster-root", 0); > > + if (!root_node) > > + return 0; > > + > > + is_root = root_node == np; > > + > > + /* > > + * A non-root member is useless without its root: it can never be > > + * booted, since its boot has to be sequenced after the root's. Reject > > + * it here rather than at first boot, so that a DT enabling a dependent > > + * DSP but not the one owning the shared resources fails loudly and > > + * early. > > + */ > > + if (!is_root && !of_device_is_available(root_node)) { > > Let's use fwnode_device_is_available() instead, even though there's > megatons of OF-specific calls across the framework Noted! But this will probably be gone anyway with Krzysztof's comment that property "qcom,cluster-root" is not really necessary. Shawn