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 1CA3A372EF6 for ; Thu, 9 Jul 2026 05:55:24 +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=1783576527; cv=none; b=GFyI4HghKWDqBpRv9p4Zjk7tM+E6Sqhg6C+DxZpe5NCUmGvsz5DFFulX+l+Qc8VDzfdI832Zr6rxrfcKJVuZUZQd63Q8lTGKJ8Ko4lAX+vrMNGsw89iA/2cVf9oHgo4WLKZbqKXsOg4UOB6UuE0ygkdbIdKKi3KaOw/hoH/z4H4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783576527; c=relaxed/simple; bh=mQux9bAenlDK11PX8bV/cumoIW+JAjvJmcG9schhrUc=; h=Message-ID:Date:MIME-Version:Subject:From:To:Cc:References: In-Reply-To:Content-Type; b=lRRLTEeNcqVHpFmCAHBshyhxTiORQVeru1nS+syv4lYZ0vCG4srfT5HVRX9kjJFdqi2VP0dE+wP26sb0jLAg0eyvbCTThahC53QdrbUUIuM/3agqkWucMYwqQfz1FpyCX3/dL8LzQrOMjqAwDWe/2xK/o6+PsH7NgDRORVlklVk= 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=N5x1AXfy; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=ElVM06mT; 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="N5x1AXfy"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="ElVM06mT" Received: from pps.filterd (m0279865.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 668N9Lmf085763 for ; Thu, 9 Jul 2026 05:55:24 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= 7FhovW18vcueRupJR1G9S2X38dNq/CQkczoYhUqon8g=; b=N5x1AXfympv/NUzf FTA060YFNUCbovFYC+Jc7Srsr80gfCuwJjS7KXylWdz3hILOFcCQpDAmhliLFRYZ ctLZOgbpme/de6jmwOvFWoVd+H+dXt3xJEWK8Xf2PBGIixtXm3tZbOIdQZ+B1d2e j0n2NOEdl3rUhrFN52+19Ey6JZlF8gbQd8mOkTrvwJ4x7Yo/svEmYUwocgMbtdQm WdEeF4PvfSjZ+aDjS0v6Wbi9g6xHWdihBwu63mJVqwd0v0jy2VZPESl3/Gi0qY9C YyMnbtPOOGdoWhl6Snz5SIBEI6lSTj4c7jLgJHD3SCDrOTHEU2ovHdSMpPQl4I7d KlfrkA== Received: from mail-pl1-f199.google.com (mail-pl1-f199.google.com [209.85.214.199]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4f9v4vt1r8-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Thu, 09 Jul 2026 05:55:24 +0000 (GMT) Received: by mail-pl1-f199.google.com with SMTP id d9443c01a7336-2cc8bde6318so11342415ad.3 for ; Wed, 08 Jul 2026 22:55:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1783576524; x=1784181324; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:content-language :references:cc:to:from:subject:user-agent:mime-version:date :message-id:from:to:cc:subject:date:message-id:reply-to:content-type; bh=7FhovW18vcueRupJR1G9S2X38dNq/CQkczoYhUqon8g=; b=ElVM06mTZSTXGNaCH+po3H3ib8cos0q9nl6tQkDNDm9hJ32BguX64nYOpgJaZft5Kv sICRBTGHs9pePdLIRYJi2SGDTrdclUc0arI0jdYsEYUfL77z+qfW/TcAhulLoUVg4pA3 YgLYqwtwryIYfgVq1QLSofV7v6KnwtPpTuJ4XGW7xIUkvepG8HnLGvY0D7q3Rvc/nkfB GtNbRY7yqLrRQjUf9oBKDi85gjrQES2id+s0f7WD7mkTGqUByFDVaDheC4O3H85gOaDf /YVaCkMJarUc0kEAaEBoU9FWi641OCS9FYcbDWUjy9V5z3V5f1kRNeZLrOtEeESPtFzJ ENgw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783576524; x=1784181324; h=content-transfer-encoding:content-type:in-reply-to:content-language :references:cc:to:from:subject:user-agent:mime-version:date :message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=7FhovW18vcueRupJR1G9S2X38dNq/CQkczoYhUqon8g=; b=rBC5NNEUONOMOwtr+4TM62VUqYA5AVMQpr0ow7vPxLjpp2VLUf3+C+wHQ/z55FM6Hy M6JZa5RqTpFdeQcVo97TZUKjUse9FObU8vW1Ep8FMJ0nJNqhpjqxFYViQ10mHGDLElab tK6tAv87CuzV8GCt96GvL8QXJDQG143+5tH3ZEWXDqjb//mBuposVYHHBzVuPhZqMtu+ E4xs5Yxceiubf+6joY2nMeTU6uz2+QPw16g5l7A0KyG/4xYuQ5Rvyba0KCguuVqrGhum U7SIHXtoKvdTCbe4XjWdmfhJ34mdL1OYM16BUg4Mbj/BZbqjVmW1GfJVnfIxM7F9i/ED KJJg== X-Forwarded-Encrypted: i=1; AHgh+RojztI5bGJihndDvXzH8iOu0IPHMv35Jm4H6tdRMysZPmMCJQW5d9SWLPV+JIWYLEyRgn4GUPgKeUwM@vger.kernel.org X-Gm-Message-State: AOJu0YztLr9AZJLVHq+5JDhURuwGQ/sjRRijb5CWN2uk6PtdW/xfpQ9T KTXF5PjQzoHfRWTz97z84+HjKB79PfruR1Vl6qQdhj850iyvnhv9mG+smNkAttNnCe/M9mEi5g2 L1M/Tjs232W3PrCxNfXTZelqIJrdsukXVYJTYejxpyHvOXewB/l8befYnugiYn5s7 X-Gm-Gg: AfdE7clAZjZusY2iW9S032O/WfVdiF3aCm8SDz7of+txvsWfyzDqkcXDmDub/5RCKkQ j/h4Ca7j70Z9rcpQtHaL3WhM5pS32DWMu2dsQ/QZrDi6N67Tyn2CgAlf0piY6F8ayqHBeEhVQ0b /R9HmulVXDHudCSQUyNcB6WvRfOY3QL8ySYn2PpPceoTikpMTfGMTyleV9qdBUDA9toE5nYA7iJ NQ1t2mmL+qld7U7+yUZpKdwt1EvM9rVSrLvLH3h5+Z29V0dWHazqm902Os2DWi1xJmkBctNaPED /yHbF13PDAX/wdpjlxQwrl3C0cImtiVfMyK1kIIS6a9f7JhQNg/wbN4zUXxdxkv30O9h8/uJf12 i/EIJy64zPcpcpWlloLlfj3WsnPxyceCVwmUEbXIJXa04RfcB X-Received: by 2002:a17:902:daca:b0:2c9:ff29:3f73 with SMTP id d9443c01a7336-2ccea45e5c0mr56146245ad.28.1783576523568; Wed, 08 Jul 2026 22:55:23 -0700 (PDT) X-Received: by 2002:a17:902:daca:b0:2c9:ff29:3f73 with SMTP id d9443c01a7336-2ccea45e5c0mr56146035ad.28.1783576523063; Wed, 08 Jul 2026 22:55:23 -0700 (PDT) Received: from [10.151.36.45] ([202.46.23.19]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2ccc9d3bc1fsm37683565ad.55.2026.07.08.22.55.18 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 08 Jul 2026 22:55:22 -0700 (PDT) Message-ID: <4a4e0e9c-8541-4fcc-8019-10a576840109@oss.qualcomm.com> Date: Thu, 9 Jul 2026 11:25:17 +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 v12 0/6] Add new driver for WCSS secure PIL loading From: Vignesh Viswanathan To: Bjorn Andersson Cc: Varadarajan Narayanan , mathieu.poirier@linaro.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, konradybcio@kernel.org, manikanta.mylavarapu@oss.qualcomm.com, linux-arm-msm@vger.kernel.org, linux-remoteproc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260505102310.2925956-1-varadarajan.narayanan@oss.qualcomm.com> Content-Language: en-US In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-GUID: yamTBpTs6zM_7t2FdvNZNlNhE348GrYE X-Proofpoint-ORIG-GUID: yamTBpTs6zM_7t2FdvNZNlNhE348GrYE X-Authority-Analysis: v=2.4 cv=GIg41ONK c=1 sm=1 tr=0 ts=6a4f37cc cx=c_pps a=JL+w9abYAAE89/QcEU+0QA==:117 a=j4ogTh8yFefVWWEFDRgCtg==:17 a=IkcTkHD0fZMA:10 a=RAioF0-LDSMA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=Um2Pa8k9VHT-vaBCBUpS:22 a=VwQbUJbxAAAA:8 a=COk6AnOGAAAA:8 a=LpQP-O61AAAA:8 a=EUspDBNiAAAA:8 a=UqCG9HQmAAAA:8 a=w873RiH1ZcoYVCw8PSIA:9 a=QEXdDO2ut3YA:10 a=324X-CrmTo6CU4MGRt3R:22 a=TjNXssC_j7lpFel5tvFf:22 a=pioyyrs4ZptJ924tMmac:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNzA5MDA1MyBTYWx0ZWRfX2ELqlMxb4BCt n7x2VPx3a/mDmc2ppTXk+8rBCaa6TRi8FRl/1BEwINjBqgLHb8EJAghUb4rUQpGX/Xl0W0VixOQ q3WgRoUs4oQIG4/PfJGqvc/2q3if85w9AiC7LDfHJFN8Zl4KR02H3lIK0dtyTFouMNCqBs3U34m Qe767AgqMLl9AjzWJPp0n9Wo4U7XwMg/CAcRe73h3sDRXEYgB+WaeTvG66cJ91w75Hv/RKBND7V xC+sw075Wsub2S3eo/bKy4D1hVo+6CLyNvY+NlSSYabTu+3ia+Dspo7jdMNFu/HvZiZveWKH8jC 15/urzL65dc5cdFWJvT+VGXHBlP5Do8OmazYEfjd98psgeFmYYN9ywQ8JauhCaJ1OkDbZjoJ4hu vG8ZHx2uW7Ki0iKXiNKD2eowpmdEQuisf/nd1mrGeIBg4/rk5ZgkFY3BfVfdTV59xa/dUmbFPhv vzMQR3HsLQeP3PW//8g== X-Proofpoint-Spam-Info: AW1haW4tMjYwNzA5MDA1MyBTYWx0ZWRfX6i0R7U3C2Ed1 f6W+1Tgo1WChtYfwfwo+dgmTMDQGjR39cBmMfpx9PEaEc2JPykzXm/BdsHk3uwgnKnFoSfSicSv /NKwjsHmSPF6BK8v9dL6ENBUsBVz7eg= X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.134,FMLib:17.12.100.49 definitions=2026-07-08_05,2026-07-08_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 phishscore=0 clxscore=1015 suspectscore=0 adultscore=0 priorityscore=1501 spamscore=0 malwarescore=0 bulkscore=0 impostorscore=0 lowpriorityscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2607090053 Hi Bjorn, Could you please help review and pick this series if it looks okay to you. Thanks, Vignesh On 5/13/2026 11:49 PM, Vignesh Viswanathan wrote: > Hi Bjorn, > > A gentle reminder to review this series. All the patches have picked > up R-b tags. If it looks okay to you, could you please help pick > it up in the current merge window. > > Thanks > Vignesh > > On 5/5/2026 3:53 PM, Varadarajan Narayanan wrote: >> As discussed in [4] posting this series after dropping IPQ5424 support >> to remove dependency on Sricharan's tmel-qmp mailbox driver series v4 [1]. >> >> Imported from 20251215-ipq5018-wifi-v7-0-ec4adba941b5@outlook.com. >> >> Imported from f20250417061245.497803-1-gokul.sriram.p@oss.qualcomm.com >> I've resumed Gokul's work as the last submission dates back April 2025. >> >> - Secure PIL is signed, firmware images which only TrustZone (TZ) >> can authenticate and load. Linux kernel will send a request to TZ to >> authenticate and load the PIL images. >> >> - When secure PIL support was added to the existing wcss PIL driver >> earlier in [2], Bjorn suggested not to overload the existing WCSS >> rproc driver, instead post a new driver for PAS based IPQ WCSS driver. >> This series adds a new secure PIL driver for the same. >> >> - Also adds changes to scm to pass metadata size as required for IPQ5332, >> reposted from [3]. >> >> [1] >> https://patchwork.kernel.org/project/linux-arm-msm/cover/20250327181750.3733881-1-quic_srichara@quicinc.com/ >> >> [2] >> https://patchwork.kernel.org/project/linux-arm-msm/patch/1611984013-10201-3-git-send-email-gokulsri@codeaurora.org/ >> >> [3] >> https://patchwork.kernel.org/project/linux-arm-msm/patch/20240820055618.267554-6-quic_gokulsri@quicinc.com/ >> >> [4] >> https://lore.kernel.org/linux-arm-msm/aUN7Aer%2FGG1d5Om9@hu-varada-blr.qualcomm.com/ >> >> Changes in v12: >> - No code change >> - Rebase to ToT >> - Change quicinc mail ids to oss.qualcomm.com mail ids >> - Link to v11: https://lore.kernel.org/linux-arm-msm/20260326043320.2507890-1-varadarajan.narayanan@oss.qualcomm.com/ >> >> Changes in v11: >> - Add R-b tags >> - Rebase the first patch ("firmware: qcom_scm: ipq5332: add support to pass metadata size") to top of tree >> - No change in other patches >> - Link to v10: https://lore.kernel.org/linux-arm-msm/20260113092021.1887980-1-varadarajan.narayanan@oss.qualcomm.com/ >> >> Changes in v10: >> - Add R-b tags >> - Remove unused fields from 'struct wcss_sec' >> - Remove glink and ssr subdev if wcss_sec_probe() fails >> - Link to v9: https://lore.kernel.org/linux-arm-msm/20260106105412.3529898-1-varadarajan.narayanan@oss.qualcomm.com/ >> >> Changes in v9: >> - Add R-b from Konrad for dts patches >> - Renamed qcom,wcss-sec-pil.yaml -> qcom,ipq5018-wcss-sec-pil.yaml >> - Restore clocks & clock-names in above yaml >> - Fix DCO on two patches >> - Link to v8: https://lore.kernel.org/linux-arm-msm/20251219031010.2919875-1-varadarajan.narayanan@oss.qualcomm.com/ >> >> Changes in v8: >> - Dropped Krzysztof's 'Reviewed-by' as the bindings file has changed significantly >> * IPQ5018 support added in v6 >> * IPQ5424 support dropped in v8 >> * Updated to use IPQ9574 as example >> - dt-bindings-check and dtbs-check passed >> - Dropped IPQ5424 support from drivers/remoteproc/qcom_q6v5_wcss_sec.c >> - Updated copyrights of drivers/remoteproc/qcom_q6v5_wcss_sec.c >> - Change 'qcom,smem-state-names' order to resolve dt-bindings-check error in ipq5018.dtsi >> - Dropped changes to ipq5424.dtsi >> - Link to v7: https://lore.kernel.org/linux-arm-msm/20251215-ipq5018-wifi-v7-0-ec4adba941b5@outlook.com/ >> >> Changes in v7: >> - correctly sorted QCOM_SCM_PIL_PAS_INIT_IMAGE_V2 by command ID >> - correctly sorted smp2p-wcss nodes in dtsi files >> - Link to v6: https://lore.kernel.org/r/20251208-ipq5018-wifi-v6-0-d0ce2facaa5f@outlook.com >> >> Changes in v6: >> - added patch to fix IPC register offset for ipq5424 >> - changed phandle description for mboxes property in dt-bindings >> - updated bindings to define the right clocks per SoC based on >> compatible. Ran make dt_binding_check for validation of all >> SoCs >> - use of more descriptive match data property (use_tmelcom) and >> added a condition in wcss_start to not error out if tmelcom >> isn't used >> - mitigated potential off-by-one >> - adopted use of of_reserved_mem_region_to_resource to acquire >> memory-region resource >> - added driver support for ipq5018 SoC >> - corrected size of reg properties as per Konrad's comments >> - added patch to bring up Q6 in ipq5018 dtsi >> - Link to v5: https://lore.kernel.org/r/20250417061245.497803-1-gokul.sriram.p@oss.qualcomm.com >> >> Changes in v5: >> - retained all the patches as in v3 and addressed comments in >> v3. >> - reverted changes to dt-bindings done in v4 and retained as in >> v3 and fixed firmware format from .mdt to .mbn and retained >> reviewed-by. >> - dropped 2 patches in v4 that adds support for q6 dtb loading. >> Will post them as a new series. >> >> Following tests were done: >> - checkpatch >> - dt_binding_check and dtbs_check >> - Link to v4: https://lore.kernel.org/r/20250327181750.3733881-1-quic_srichara@quicinc.com >> >> Changes in v4: >> - changed q6 firmware image format from .mdt to .mbn >> - corrected arrangement of variable assignemnts as per comments >> in qcom_scm.c >> - added scm call to get board machid >> - added support for q6 dtb loading with support for additional >> reserved memory for q6 dtb in .mbn format >> - updated dt-bindings to include new dts entry qcom,q6-dtb-info >> and additional item in memory-region for q6 dtb region. >> - removed unnecessary dependency for QCOM_Q6V5_WCSS_SEC in >> Kconfig >> - removed unwanted header files in qcom_q6v5_wcss_sec.c >> - removed repeated dtb parsing during runtime in qcom_q6v5_wcss_sec.c >> - added required check for using tmelcom, if available. Enabled >> fallback to scm based authentication, if tmelcom is unavailable. >> - added necessary padding for 8digt hex address in dts >> - Link to v3: https://lore.kernel.org/r/20250107101320.2078139-1-quic_gokulsri@quicinc.com >> >> Following tests were done: >> - checkpatch >> - kernel-doc >> - dt_binding_check and dtbs_check >> >> Changes in v3: >> - fixed copyright years and markings based on Jeff's comments. >> - replaced devm_ioremap_wc() with ioremap_wc() in >> wcss_sec_copy_segment(). >> - replaced rproc_alloc() and rproc_add() with their devres >> counterparts. >> - added mailbox call to tmelcom for secure image authentication >> as required for IPQ5424. Added ipq5424 APCS comatible required. >> - added changes to scm call to pass metadata size as required for >> IPQ5332. >> - Link to v2: https://lore.kernel.org/r/20240829134021.1452711-1-quic_gokulsri@quicinc.com >> >> Changes in v2: >> - Removed dependency of this series to q6 clock removal series >> as recommended by Krzysztof >> - Link to v1: https://lore.kernel.org/r/20240820085517.435566-1-quic_gokulsri@quicinc.com >> >> George Moussalem (1): >> arm64: dts: qcom: ipq5018: add nodes to bring up q6 >> >> Manikanta Mylavarapu (4): >> firmware: qcom_scm: ipq5332: add support to pass metadata size >> dt-bindings: remoteproc: qcom: document hexagon based WCSS secure PIL >> arm64: dts: qcom: ipq5332: add nodes to bring up q6 >> arm64: dts: qcom: ipq9574: add nodes to bring up q6 >> >> Vignesh Viswanathan (1): >> remoteproc: qcom: add hexagon based WCSS secure PIL driver >> >> .../remoteproc/qcom,ipq5018-wcss-sec-pil.yaml | 178 ++++++++++ >> arch/arm64/boot/dts/qcom/ipq5018.dtsi | 64 ++++ >> arch/arm64/boot/dts/qcom/ipq5332.dtsi | 64 +++- >> arch/arm64/boot/dts/qcom/ipq9574.dtsi | 60 +++- >> drivers/firmware/qcom/qcom_scm.c | 14 +- >> drivers/firmware/qcom/qcom_scm.h | 1 + >> drivers/remoteproc/Kconfig | 19 + >> drivers/remoteproc/Makefile | 1 + >> drivers/remoteproc/qcom_q6v5_wcss_sec.c | 325 ++++++++++++++++++ >> include/linux/remoteproc.h | 2 + >> 10 files changed, 723 insertions(+), 5 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,ipq5018-wcss-sec-pil.yaml >> create mode 100644 drivers/remoteproc/qcom_q6v5_wcss_sec.c >> >> >> base-commit: 9d0d467c3572e93c5faa2e5906a8bbcd70b24efd >