| From 38b88722bce07b6a5927f45fbf7a9a85e834572c Mon Sep 17 00:00:00 2001 |
| From: Stephan Gerhold <stephan.gerhold@linaro.org> |
| Date: Wed, 12 Feb 2025 18:03:50 +0100 |
| Subject: arm64: dts: qcom: sm8650: Add missing properties for cryptobam |
| |
| From: Stephan Gerhold <stephan.gerhold@linaro.org> |
| |
| commit 38b88722bce07b6a5927f45fbf7a9a85e834572c upstream. |
| |
| num-channels and qcom,num-ees are required for BAM nodes without clock, |
| because the driver cannot ensure the hardware is powered on when trying to |
| obtain the information from the hardware registers. Specifying the node |
| without these properties is unsafe and has caused early boot crashes for |
| other SoCs before [1, 2]. |
| |
| Add the missing information from the hardware registers to ensure the |
| driver can probe successfully without causing crashes. |
| |
| [1]: https://lore.kernel.org/r/CY01EKQVWE36.B9X5TDXAREPF@fairphone.com/ |
| [2]: https://lore.kernel.org/r/20230626145959.646747-1-krzysztof.kozlowski@linaro.org/ |
| |
| Cc: stable@vger.kernel.org |
| Fixes: 10e024671295 ("arm64: dts: qcom: sm8650: add interconnect dependent device nodes") |
| Signed-off-by: Stephan Gerhold <stephan.gerhold@linaro.org> |
| Link: https://lore.kernel.org/r/20250212-bam-dma-fixes-v1-4-f560889e65d8@linaro.org |
| Signed-off-by: Bjorn Andersson <andersson@kernel.org> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| arch/arm64/boot/dts/qcom/sm8650.dtsi | 2 ++ |
| 1 file changed, 2 insertions(+) |
| |
| --- a/arch/arm64/boot/dts/qcom/sm8650.dtsi |
| +++ b/arch/arm64/boot/dts/qcom/sm8650.dtsi |
| @@ -2533,6 +2533,8 @@ |
| <&apps_smmu 0x481 0>; |
| |
| qcom,ee = <0>; |
| + qcom,num-ees = <4>; |
| + num-channels = <20>; |
| qcom,controlled-remotely; |
| }; |
| |
