From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.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 E1301390C93 for ; Fri, 13 Mar 2026 11:59:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773403193; cv=none; b=h3BaO9rS7xWaJcQlZGfNLMSjDV9bS1z1v7uU9VvU2wzyev6Tkcfdsa8boyB5nSyNiSGAXQ3nJqdL0ai49CvpA28iIeZnFBBSjri110Sdsl13Dn4/WmenefekskOm42ZpwibaA3CDa/cX4aWdvY0IAs8Hf/8MX6BstE5qSkjFHzI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773403193; c=relaxed/simple; bh=z0sVzBAcv13vC9l86gQlsXV87u5QGlJTtmJZRwdqpj4=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=oGFhmrOU3SBPPtujTEMbt+iRprklNnUHNe5beBK9LrGJXrIaMGhDpm645gcYgBewXSDNXRg57uhOLlFIQbwo5bE3pIUd1Uef+xZbvKXCXNMxLR9V5QhmtY+Rn1FBMsNSFouVUqn/tgxMi8BlZRNLYDv2ygQqMZXAI8oZYljVAB4= 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=BtF5t7GB; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=PlmaDAl8; arc=none smtp.client-ip=205.220.168.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="BtF5t7GB"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="PlmaDAl8" Received: from pps.filterd (m0279864.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 62DA8BAk1749668 for ; Fri, 13 Mar 2026 11:59:51 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= dAPyYNhQ7Nilvok6h06J0gKQb9GOg5VVQtkr6R/ZE0I=; b=BtF5t7GBYwJUfunD K43XSiJjqXqPrGS9E8flXCr/CG5vIzZB66CCfxSHHBaYu0uwieMm12oxuvkNOvfn ja1fqd3U2vZCiW3Y2q84CzAew3ijeJVFVuXE8q02dxQeLmSC/U35nIun4/hmsow+ nJ+GO+Q1e83yw5oaj2VXlWSDhAIR+ibLLDcmtaDgISq5duMFyJCELzm4PVV4UY4B SV8LUlYila5uJookHMR4p4SvrL6NiWFc3JWjX0wSHLWFPXNJJIAK4BQlerda9t1B ZkjFryzFs6FJxSgd7s57853V0JBinDvp3wrKXg1lOh3G0zcuEbVTvDDA1BhnosAh AIyQAg== Received: from mail-qk1-f197.google.com (mail-qk1-f197.google.com [209.85.222.197]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4cvgj6ga70-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Fri, 13 Mar 2026 11:59:51 +0000 (GMT) Received: by mail-qk1-f197.google.com with SMTP id af79cd13be357-8cd8e89d19bso114413385a.2 for ; Fri, 13 Mar 2026 04:59:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1773403190; x=1774007990; 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=dAPyYNhQ7Nilvok6h06J0gKQb9GOg5VVQtkr6R/ZE0I=; b=PlmaDAl8vrOd2KP38BpdysJzWxfF+75kyVkAxGmJEHGPnKNRdH6k/G63rNmIKdIb3B kpqmfloClxBuLzhSuEBjE9mtlr5mgJN5F++0B7/j7Ze/2yl9eiLpPaXqQyIjLcy0LSXC //BO/Dcewl3WzuhdQa/yY4AcfatDnTOxKS2QyOXpHcU7102cULuIx/plhz6ovMx2gRxn D3RZMiQppcRnTY14InkrUrcoq0pg98sIiMZv8t4hQlez2+5ZIUA+3c8mfMHs3Yr9+IT5 NmHUaFcAP4hsM0IRUqlbDztR3wssT5K44vLRd8OSrQejJ4LM3pfKDVHKnkOzJy8NM/E1 uCkw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1773403190; x=1774007990; 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=dAPyYNhQ7Nilvok6h06J0gKQb9GOg5VVQtkr6R/ZE0I=; b=kgNlLLK+SF13bk8/pzuiV3ZUnTGRYDJBLtzYmicGtctA2CaD9GT58cIP5NkYqePYrK HA3SyUEKNthbLk+8/IurMZLDPdK5md/UV54RysQGvKkUjawGKZsWTcqO61O+YRU2QsT3 bgf+Fh49MTJmaoWf8T7WKXdNz9vOepxYsMi+XNwKoRtytre2B1wwO/8jl12i5OJkj8Zi IHY1bsPws2I7AET5gcsM5CoTEE/vhvaQ7PdsQP4peGQOicyvsXb7M7/4V8B5q4RR8QfR nmVYv/st6VQOAizB17PqeUqJdJ5Nd9GSx/mXZayRTCpHKOJgIz/Mx6r/kjQnwHb8z7xq PcjA== X-Forwarded-Encrypted: i=1; AJvYcCX+0ILqCYUBIBsv/UUXbwcIHDrIhtH9+qcFqAOSxUaf0T2m1C2ZUSGT/gw0mAi0vyM9j2Iaffk1Pvai@vger.kernel.org X-Gm-Message-State: AOJu0Yw8nGrxMyIBI8fJKVOD8D3x6zQAjCeFB6w3Db7oA7rYu8lBYdGg sc2WLqAdL3t5TSRzBriIs7seyL4AFnMjP6hUpTX8RhvA8xafh+1fL88+srbydot6v8FNxSCHoHy sDwJRHBf4eWT9gCB6fg2OpvcisDe7B2bUWlUlLDtoXdVeTXksjjsIgBaMsnEhMT2O X-Gm-Gg: ATEYQzz4rcu6BMt3xJS1bL1ywBV62R0pNCo1hgCS16iCUvDoLd/qx7Y3i/qUuZrhGSt MkwPQEZe6y5q/Oh/KXQ5akpkUF9i72cu239/sIDffC82/CyZgft2B+86//eFmxiIYHXGK2wfef/ XjuXgzLk++MYly3JEyJ9xVtmg+iXTrpHGbU1lV1fPMTMEmaRq29jPupMWDVRE1UjYBITFck36e+ +wckXY3uATHHHNDbk61E3EFMZluU+/+e6+QW+4qYpIm3q4LYVrj8JFjXn6d3C9D/J01KLJeDVRq M4ypLOFzOk/zCdNXjbc/0Op0VvJgE58Oy6UvWXOHm6AcvfLNjs7oLsD9UtIdyP+R/ez9FEIfRxO ixmpGeBG7AhErkpZ4FhIu6UoMDUPiiIPXw4cDHGxXtqenF1msJbiuttKhRt/hkwcy/iKbnQSMMm YJxXU= X-Received: by 2002:a05:620a:1990:b0:8cb:3fa7:c4f0 with SMTP id af79cd13be357-8cdb5b42e6amr292516585a.5.1773403190152; Fri, 13 Mar 2026 04:59:50 -0700 (PDT) X-Received: by 2002:a05:620a:1990:b0:8cb:3fa7:c4f0 with SMTP id af79cd13be357-8cdb5b42e6amr292513185a.5.1773403189637; Fri, 13 Mar 2026 04:59:49 -0700 (PDT) Received: from [192.168.119.254] (078088045245.garwolin.vectranet.pl. [78.88.45.245]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b976cf49fa6sm39747866b.58.2026.03.13.04.59.47 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 13 Mar 2026 04:59:48 -0700 (PDT) Message-ID: <90b3a7df-cd02-4878-b614-1499589f0906@oss.qualcomm.com> Date: Fri, 13 Mar 2026 12:59:46 +0100 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 1/5] arm64: dts: qcom: x1e80100: Remove interconnect from SCM device To: "Maulik Shah (mkshah)" , Dmitry Baryshkov Cc: Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Thomas Gleixner , Linus Walleij , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, Sneh Mankad References: <20260312-hamoa_pdc-v1-0-760c8593ce50@oss.qualcomm.com> <20260312-hamoa_pdc-v1-1-760c8593ce50@oss.qualcomm.com> <198ccf60-a4b9-438b-ad92-bc4d2cc84b83@oss.qualcomm.com> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <198ccf60-a4b9-438b-ad92-bc4d2cc84b83@oss.qualcomm.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwMzEzMDA5NSBTYWx0ZWRfX/MKfZLNZ247c izMDSGGwq+j3wxq9xC/2LpYXkjxXydSeh+gL41I+qQAUSoB3OOSJgvMcrn679vNCvDt5fFvkn5i x09MkRAY8H2aDv47Ird/6yjQIMoVXEhSw9hxdykrtcYPN0IZpd6pdhCvFc9zgVcQuzlePFKy1e/ U6XPCK0gBvxTnNg9bwXJXCoals4raxiInbylyqRWl7crSpUTfIPDqPY63cbPv3EPGyR0p3T8Vw6 4RGelTma/AMGWwW79YbgZHf6AjHVomaz+eavIhXOLc5x9mioy2Rp1i6sUaCTKaMfUkiiBZSkd4l bO7Jw+++DdD8Zlih4qj/H10ZhDIPCtCNTRTlsa2zj9KE58JbVkawTlzlZQlfnu4+mSAmFitT6Bz ph2Bp3eODQ8Nbyczz3fY5MFF0j7nntr8Liz63ggaJFvDhJqC+ufccRcCB2/Lo+Hw4JTarPTlE06 Zo/j5wrpdClrae99IpA== X-Proofpoint-GUID: it8qgSORgVumMS90UYnEcXX8KxNg_SoQ X-Proofpoint-ORIG-GUID: it8qgSORgVumMS90UYnEcXX8KxNg_SoQ X-Authority-Analysis: v=2.4 cv=H+vWAuYi c=1 sm=1 tr=0 ts=69b3fc37 cx=c_pps a=50t2pK5VMbmlHzFWWp8p/g==:117 a=FpWmc02/iXfjRdCD7H54yg==:17 a=IkcTkHD0fZMA:10 a=Yq5XynenixoA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=DJpcGTmdVt4CTyJn9g5Z:22 a=D3rGKGdzeZ1DfF1dGIIA:9 a=QEXdDO2ut3YA:10 a=IoWCM6iH3mJn3m4BftBB:22 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.51,FMLib:17.12.100.49 definitions=2026-03-13_02,2026-03-13_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 lowpriorityscore=0 phishscore=0 clxscore=1015 malwarescore=0 adultscore=0 impostorscore=0 priorityscore=1501 bulkscore=0 suspectscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2603050001 definitions=main-2603130095 On 3/13/26 11:12 AM, Maulik Shah (mkshah) wrote: > > > On 3/13/2026 7:41 AM, Dmitry Baryshkov wrote: >> On Thu, Mar 12, 2026 at 09:26:35PM +0530, Maulik Shah wrote: >>> Interconnect from SCM device are optional and were added to get >>> additional performance benefit. These nodes however delays the >>> SCM firmware device probe due to dependency on interconnect and >>> results in NULL pointer dereference for the users of SCM device >>> driver APIs, such as PDC driver. >> >> This sounds like a bug in the PDC driver. It should reject being probed >> before SCM is available. > > The SCM driver provides no way to check if its ready or not to decide to reject/defer the probe. > A new API like below would be needed here, There is, qcom_scm_is_available() > Let me know any preferences from below options or any other. > > a) Add the API like qcom_scm_ready(), this has been tested and works fine. > b) Move interconnects from SCM to remoteproc PAS driver for all devices > Take the vote before invoking SCM API and release after return. I think this is not the right decision. The crypto path is only necessary, because cryptographic checks must be carried out in the TZ in order to (dis)allow a certain firmware binary. This is not a characteristic of the remoteprocs themselves, as with a non-prudent TZ, the firmware loading would amount to a memcpy() (and some SMMU/XPU configs via reg writes) > c) Remove the interconnects from SCM and rely on crypto driver already > placing the vote, Route the remote proc to SCM call via crypto API, > This would ensure crpyto is being used and it would have placed the required vote. I think this would make things even worse, because instead of waiting on the interconnect driver, we'd now have to wait on the interconnect driver, the clock driver and the crypto driver > d) Add separate SCM child device (with interconnects) under SoC. We'd then have to probe it as an aux device or something, which would either delay the probing of SCM, or introduce the need to ping-pong for PAS availability between the API provider and consumer, since some calls work perfectly fine without the ICC path, while others could really use it Konrad