diff mbox series

[1/2] arm64: dts: qcom: msm8994: Drop spi-max-frequency from SPI host

Message ID 20221117105845.13644-1-konrad.dybcio@linaro.org
State Accepted
Commit d132d9e7c4e53a23d158a9ec3d59c372eb30e70e
Headers show
Series [1/2] arm64: dts: qcom: msm8994: Drop spi-max-frequency from SPI host | expand

Commit Message

Konrad Dybcio Nov. 17, 2022, 10:58 a.m. UTC
This is a device property, not a bus host one.

Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 arch/arm64/boot/dts/qcom/msm8994.dtsi | 2 --
 1 file changed, 2 deletions(-)

Comments

Bjorn Andersson Dec. 6, 2022, 6:19 p.m. UTC | #1
On Thu, 17 Nov 2022 11:58:44 +0100, Konrad Dybcio wrote:
> This is a device property, not a bus host one.
> 
> 

Applied, thanks!

[1/2] arm64: dts: qcom: msm8994: Drop spi-max-frequency from SPI host
      commit: d132d9e7c4e53a23d158a9ec3d59c372eb30e70e

Best regards,
Konrad Dybcio Feb. 17, 2023, 11:30 a.m. UTC | #2
On 17.11.2022 11:58, Konrad Dybcio wrote:
> Document the MSM8994 watchdog timer which is already used in DT.
> 
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
Looks like this never got applied?

Konrad
>  Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml b/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
> index d8ac0be36e6c..a1f17c9e02db 100644
> --- a/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
> @@ -17,6 +17,7 @@ properties:
>      oneOf:
>        - items:
>            - enum:
> +              - qcom,apss-wdt-msm8994
>                - qcom,apss-wdt-qcs404
>                - qcom,apss-wdt-sc7180
>                - qcom,apss-wdt-sc7280
Wim Van Sebroeck Feb. 18, 2023, 3 p.m. UTC | #3
Hi Konrad,

> On 17.11.2022 11:58, Konrad Dybcio wrote:
> > Document the MSM8994 watchdog timer which is already used in DT.
> > 
> > Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> > ---
> Looks like this never got applied?

Patience is a beautiful virtue.
Since Guenter reviewed it and since it is in Guenter's tree, I picked it up.

> Konrad
> >  Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml b/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
> > index d8ac0be36e6c..a1f17c9e02db 100644
> > --- a/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
> > +++ b/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
> > @@ -17,6 +17,7 @@ properties:
> >      oneOf:
> >        - items:
> >            - enum:
> > +              - qcom,apss-wdt-msm8994
> >                - qcom,apss-wdt-qcs404
> >                - qcom,apss-wdt-sc7180
> >                - qcom,apss-wdt-sc7280

Kind regards,
Wim.
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/msm8994.dtsi b/arch/arm64/boot/dts/qcom/msm8994.dtsi
index adcea31a0c68..9ff9d35496d2 100644
--- a/arch/arm64/boot/dts/qcom/msm8994.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8994.dtsi
@@ -554,7 +554,6 @@  blsp1_spi1: spi@f9923000 {
 			clocks = <&gcc GCC_BLSP1_QUP1_SPI_APPS_CLK>,
 				 <&gcc GCC_BLSP1_AHB_CLK>;
 			clock-names = "core", "iface";
-			spi-max-frequency = <19200000>;
 			dmas = <&blsp1_dma 12>, <&blsp1_dma 13>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
@@ -692,7 +691,6 @@  blsp2_spi4: spi@f9966000 {
 			clocks = <&gcc GCC_BLSP2_QUP4_SPI_APPS_CLK>,
 				 <&gcc GCC_BLSP2_AHB_CLK>;
 			clock-names = "core", "iface";
-			spi-max-frequency = <19200000>;
 			dmas = <&blsp2_dma 18>, <&blsp2_dma 19>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";