[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <fe9bb128563bea9798573ac9eec1d214@codeaurora.org>
Date: Tue, 17 Mar 2020 21:22:21 +0530
From: Sibi Sankar <sibis@...eaurora.org>
To: bjorn.andersson@...aro.org, robh+dt@...nel.org, joro@...tes.org,
robin.murphy@....com
Cc: linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
linux-arm-msm@...r.kernel.org, iommu@...ts.linux-foundation.org,
linux-remoteproc@...r.kernel.org, ohad@...ery.com,
agross@...nel.org, dianders@...omium.org,
saiprakash.ranjan@...eaurora.org
Subject: Re: [PATCH v2 0/3] Request direct mapping for modem device
On 2020-03-17 20:39, Sibi Sankar wrote:
> The Q6 modem sub-system has direct access to DDR through memnoc and
> an indirect access routed through a SMMU which MSS CE (crypto engine
> sub-component of MSS) uses during out of reset sequence. Request direct
> mapping for the modem device since smmu is not expected to provide
> access
> control/translation for these SIDs (sandboxing of the modem is achieved
> through XPUs engaged using SMC calls). This is done on platforms which
> don't have TrustZone (which programs the modem SIDs) to prevent the
> following global faults seen on Cheza/Trogdor:
>
> Cheza:
> arm-smmu 15000000.iommu: Unexpected global fault, this could be serious
> arm-smmu 15000000.iommu: GFSR 0x80000002, GFSYNR0 0x00000000,
> GFSYNR1 0x00000781, GFSYNR2 0x00000000
>
> Trogdor:
> arm-smmu 15000000.iommu: Unexpected global fault, this could be serious
> arm-smmu 15000000.iommu: GFSR 0x80000002, GFSYNR0 0x00000000,
> GFSYNR1 0x00000461, GFSYNR2 0x00000000
>
> V2:
> * Request direct mapping from SoC-specific corner of the SMMU
> driver [Robin]
> * Add iommu property to remoteproc modem node on Cheza
>
> Depends on:
> https://lore.kernel.org/patchwork/cover/1183528/
>
> Sibi Sankar (3):
> dt-bindings: remoteproc: qcom: Add iommus property
> remoteproc: qcom_q6v5_mss: Request direct mapping for modem device
iommu: arm-smmu-qcom: Request direct mapping for modem device
sry should have been ^^ instead
> arm64: dts: qcom: sdm845-cheza: Add iommus property
>
> Documentation/devicetree/bindings/remoteproc/qcom,q6v5.txt | 3 +++
> arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi | 4 ++++
> drivers/iommu/arm-smmu-qcom.c | 6 ++++++
> 3 files changed, 13 insertions(+)
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project.
Powered by blists - more mailing lists