From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: Konrad Dybcio <konrad.dybcio@linaro.org>,
Bhupesh Sharma <bhupesh.sharma@linaro.org>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org
Cc: agross@kernel.org, linux-kernel@vger.kernel.org,
linux-crypto@vger.kernel.org, andersson@kernel.org,
bhupesh.linux@gmail.com, robh+dt@kernel.org,
vladimir.zapolskiy@linaro.org, rfoss@kernel.org,
neil.armstrong@linaro.org, djakov@kernel.org,
stephan@gerhold.net, Anders Roxell <anders.roxell@linaro.org>,
Linux Kernel Functional Testing <lkft@linaro.org>
Subject: Re: [PATCH v8 10/11] arm64: dts: qcom: sm8350: Add Crypto Engine support
Date: Sat, 17 Jun 2023 09:28:21 +0200 [thread overview]
Message-ID: <83807d70-c35c-6b66-e7f9-521bdfc6d1b4@linaro.org> (raw)
In-Reply-To: <11c3eb6c-823d-9688-ec53-e05c7bb557c5@linaro.org>
On 17/06/2023 01:33, Konrad Dybcio wrote:
> On 16.06.2023 19:36, Krzysztof Kozlowski wrote:
>> On 26/05/2023 21:22, Bhupesh Sharma wrote:
>>> Add crypto engine (CE) and CE BAM related nodes and definitions to
>>> 'sm8350.dtsi'.
>>>
>>> Tested-by: Anders Roxell <anders.roxell@linaro.org>
>>> Tested-by: Linux Kernel Functional Testing <lkft@linaro.org>
>>> Co-developed-by and Signed-off-by: Robert Foss <rfoss@kernel.org>
>>> [Bhupesh: Switch to '#interconnect-cells = <2>', available since commit 4f287e31ff5f]
>>> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
>>> ---
>>
>> #regzbot introduced: f1040a7fe8f069d2259ab3dab9190210005ceb33
>> #regzbot title: HDK8350 silently crashes early on boot
>>
>> Hi, this landed in the next but unfortunately it causes silent crash
>> (and reboot) of HDK8350. Reverting this commit helps.
> Downstream also references the following SIDs:
>
> iommus = <&apps_smmu 0x592 0>,
> <&apps_smmu 0x598 0>,
> <&apps_smmu 0x599 0>,
> <&apps_smmu 0x59F 0>;
I already tried iommus from downstream from:
1. qcrypto node (0x584, 0x594)
2. qcedev (0x586, 0x596)
3. qcom_cedev_ns_cb (0x592, 0x598, 0x599, 0x59F), although with 0x0011
last argument.
Same results, but indeed iommu would be nice reason here.
I also double checked the version of block (BAM DMA is v1.7.4) and other
properties. When I disabled crypto but left BAM DMA, the result was the
same, thus it is maybe the BAM who causes abort.
Best regards,
Krzysztof
next prev parent reply other threads:[~2023-06-17 7:28 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-26 19:21 [PATCH v8 00/11] arm64: qcom: Enable Crypto Engine for a few Qualcomm SoCs Bhupesh Sharma
2023-05-26 19:22 ` [PATCH v8 01/11] dt-bindings: dma: Add support for SM6115 and QCM2290 SoCs Bhupesh Sharma
2023-05-29 6:13 ` Bhupesh Sharma
2023-06-06 8:14 ` Bhupesh Sharma
2023-06-08 12:32 ` Vinod Koul
2023-06-30 8:30 ` Bhupesh Sharma
2023-05-26 19:22 ` [PATCH v8 02/11] dt-bindings: dma: Increase iommu maxItems for BAM DMA Bhupesh Sharma
2023-06-18 8:36 ` Krzysztof Kozlowski
2023-06-18 8:39 ` Krzysztof Kozlowski
2023-06-30 8:35 ` Bhupesh Sharma
2023-05-26 19:22 ` [PATCH v8 03/11] arm64: dts: qcom: sdm8550: Fix the BAM DMA engine compatible string Bhupesh Sharma
2023-05-26 19:22 ` [PATCH v8 04/11] arm64: dts: qcom: sdm845: Fix the slimbam " Bhupesh Sharma
2023-05-26 19:22 ` [PATCH v8 05/11] dt-bindings: qcom-qce: Fix compatible combinations for SM8150 and IPQ4019 SoCs Bhupesh Sharma
2023-05-29 6:15 ` Bhupesh Sharma
2023-06-02 10:26 ` Herbert Xu
2023-05-26 19:22 ` [PATCH v8 06/11] dt-bindings: qcom-qce: Add compatibles for SM6115 and QCM2290 Bhupesh Sharma
2023-05-26 19:22 ` [PATCH v8 07/11] arm64: dts: qcom: sm6115: Add Crypto Engine support Bhupesh Sharma
2023-05-26 20:49 ` Konrad Dybcio
2023-05-26 19:22 ` [PATCH v8 08/11] arm64: dts: qcom: sm8150: " Bhupesh Sharma
2023-05-26 19:22 ` [PATCH v8 09/11] arm64: dts: qcom: sm8250: " Bhupesh Sharma
2023-05-26 19:22 ` [PATCH v8 10/11] arm64: dts: qcom: sm8350: " Bhupesh Sharma
2023-06-16 17:36 ` Krzysztof Kozlowski
2023-06-16 23:33 ` Konrad Dybcio
2023-06-17 7:28 ` Krzysztof Kozlowski [this message]
2023-06-18 6:49 ` Bhupesh Sharma
2023-06-26 10:51 ` Linux regression tracking #update (Thorsten Leemhuis)
2023-05-26 19:22 ` [PATCH v8 11/11] arm64: dts: qcom: sm8450: add crypto nodes Bhupesh Sharma
2023-05-26 20:50 ` Konrad Dybcio
2023-05-27 1:07 ` (subset) [PATCH v8 00/11] arm64: qcom: Enable Crypto Engine for a few Qualcomm SoCs Bjorn Andersson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=83807d70-c35c-6b66-e7f9-521bdfc6d1b4@linaro.org \
--to=krzysztof.kozlowski@linaro.org \
--cc=agross@kernel.org \
--cc=anders.roxell@linaro.org \
--cc=andersson@kernel.org \
--cc=bhupesh.linux@gmail.com \
--cc=bhupesh.sharma@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=djakov@kernel.org \
--cc=konrad.dybcio@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-crypto@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lkft@linaro.org \
--cc=neil.armstrong@linaro.org \
--cc=rfoss@kernel.org \
--cc=robh+dt@kernel.org \
--cc=stephan@gerhold.net \
--cc=vladimir.zapolskiy@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).