2022-09-23 16:21:09

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 05/11] arm64: dts: qcom: sdm845: drop unused slimbus dmas

Bindings document only two DMA channels. Linux driver also does not use
remaining rx2/tx2.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm64/boot/dts/qcom/sdm845.dtsi | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
index 2264bba69f84..1213f78a8b7a 100644
--- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
@@ -3828,9 +3828,8 @@ slim: [email protected] {
reg = <0 0x171c0000 0 0x2c000>;
interrupts = <GIC_SPI 163 IRQ_TYPE_LEVEL_HIGH>;

- dmas = <&slimbam 3>, <&slimbam 4>,
- <&slimbam 5>, <&slimbam 6>;
- dma-names = "rx", "tx", "tx2", "rx2";
+ dmas = <&slimbam 3>, <&slimbam 4>;
+ dma-names = "rx", "tx";

iommus = <&apps_smmu 0x1806 0x0>;
#address-cells = <1>;
--
2.34.1


2022-09-25 19:16:36

by Steev Klimaszewski

[permalink] [raw]
Subject: Re: [PATCH 05/11] arm64: dts: qcom: sdm845: drop unused slimbus dmas


On 9/23/22 11:14 AM, Krzysztof Kozlowski wrote:
> Bindings document only two DMA channels. Linux driver also does not use
> remaining rx2/tx2.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> arch/arm64/boot/dts/qcom/sdm845.dtsi | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> index 2264bba69f84..1213f78a8b7a 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> @@ -3828,9 +3828,8 @@ slim: [email protected] {
> reg = <0 0x171c0000 0 0x2c000>;
> interrupts = <GIC_SPI 163 IRQ_TYPE_LEVEL_HIGH>;
>
> - dmas = <&slimbam 3>, <&slimbam 4>,
> - <&slimbam 5>, <&slimbam 6>;
> - dma-names = "rx", "tx", "tx2", "rx2";
> + dmas = <&slimbam 3>, <&slimbam 4>;
> + dma-names = "rx", "tx";
>
> iommus = <&apps_smmu 0x1806 0x0>;
> #address-cells = <1>;

Tested on Lenovo Yoga C630

Tested-by: Steev Klimaszewski <[email protected]>