* [PATCH 2/4] ARM: dts: qcom: sdx55: use generic node names for USB
2023-06-19 17:01 [PATCH 1/4] ARM: dts: qcom: ipq4019: use generic node names for USB Krzysztof Kozlowski
@ 2023-06-19 17:01 ` Krzysztof Kozlowski
2023-06-19 17:15 ` Konrad Dybcio
2023-06-19 17:01 ` [PATCH 3/4] ARM: dts: qcom: ipq8064: drop spi-max-frequency from controller Krzysztof Kozlowski
` (3 subsequent siblings)
4 siblings, 1 reply; 9+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-19 17:01 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, devicetree,
linux-kernel
Cc: Krzysztof Kozlowski
Device node names should be generic which is also expected by USB
bindings:
qcom-sdx55-t55.dtb: dwc3@a600000: $nodename:0: 'dwc3@a600000' does not match '^usb(@.*)?'
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm/boot/dts/qcom-sdx55.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/qcom-sdx55.dtsi b/arch/arm/boot/dts/qcom-sdx55.dtsi
index df3cd9c4ffb9..55ce87b75253 100644
--- a/arch/arm/boot/dts/qcom-sdx55.dtsi
+++ b/arch/arm/boot/dts/qcom-sdx55.dtsi
@@ -603,7 +603,7 @@ usb: usb@a6f8800 {
resets = <&gcc GCC_USB30_BCR>;
- usb_dwc3: dwc3@a600000 {
+ usb_dwc3: usb@a600000 {
compatible = "snps,dwc3";
reg = <0x0a600000 0xcd00>;
interrupts = <GIC_SPI 133 IRQ_TYPE_LEVEL_HIGH>;
--
2.34.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 2/4] ARM: dts: qcom: sdx55: use generic node names for USB
2023-06-19 17:01 ` [PATCH 2/4] ARM: dts: qcom: sdx55: " Krzysztof Kozlowski
@ 2023-06-19 17:15 ` Konrad Dybcio
0 siblings, 0 replies; 9+ messages in thread
From: Konrad Dybcio @ 2023-06-19 17:15 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, devicetree,
linux-kernel
On 19.06.2023 19:01, Krzysztof Kozlowski wrote:
> Device node names should be generic which is also expected by USB
> bindings:
>
> qcom-sdx55-t55.dtb: dwc3@a600000: $nodename:0: 'dwc3@a600000' does not match '^usb(@.*)?'
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
> arch/arm/boot/dts/qcom-sdx55.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/qcom-sdx55.dtsi b/arch/arm/boot/dts/qcom-sdx55.dtsi
> index df3cd9c4ffb9..55ce87b75253 100644
> --- a/arch/arm/boot/dts/qcom-sdx55.dtsi
> +++ b/arch/arm/boot/dts/qcom-sdx55.dtsi
> @@ -603,7 +603,7 @@ usb: usb@a6f8800 {
>
> resets = <&gcc GCC_USB30_BCR>;
>
> - usb_dwc3: dwc3@a600000 {
> + usb_dwc3: usb@a600000 {
> compatible = "snps,dwc3";
> reg = <0x0a600000 0xcd00>;
> interrupts = <GIC_SPI 133 IRQ_TYPE_LEVEL_HIGH>;
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 3/4] ARM: dts: qcom: ipq8064: drop spi-max-frequency from controller
2023-06-19 17:01 [PATCH 1/4] ARM: dts: qcom: ipq4019: use generic node names for USB Krzysztof Kozlowski
2023-06-19 17:01 ` [PATCH 2/4] ARM: dts: qcom: sdx55: " Krzysztof Kozlowski
@ 2023-06-19 17:01 ` Krzysztof Kozlowski
2023-06-19 17:16 ` Konrad Dybcio
2023-06-19 17:01 ` [PATCH 4/4] ARM: dts: qcom: msm8960: " Krzysztof Kozlowski
` (2 subsequent siblings)
4 siblings, 1 reply; 9+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-19 17:01 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, devicetree,
linux-kernel
Cc: Krzysztof Kozlowski
spi-max-frequency is a property of SPI device, not SPI controller. Drop
it from the controller nodes. No functional impact expected because
child SPI device already defines spi-max-frequency. This fixes
dtbs_check warnings like:
qcom-ipq8064-ap148.dtb: spi@1a280000: Unevaluated properties are not allowed ('spi-max-frequency' was unexpected)
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm/boot/dts/qcom-ipq8064-rb3011.dts | 1 -
arch/arm/boot/dts/qcom-ipq8064-v1.0.dtsi | 1 -
2 files changed, 2 deletions(-)
diff --git a/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts b/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts
index 104eb729c2d6..1796ded31d17 100644
--- a/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts
+++ b/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts
@@ -282,7 +282,6 @@ gsbi5: gsbi@1a200000 {
spi4: spi@1a280000 {
status = "okay";
- spi-max-frequency = <50000000>;
pinctrl-0 = <&spi_pins>;
pinctrl-names = "default";
diff --git a/arch/arm/boot/dts/qcom-ipq8064-v1.0.dtsi b/arch/arm/boot/dts/qcom-ipq8064-v1.0.dtsi
index c5abe7151f14..17f65e140e02 100644
--- a/arch/arm/boot/dts/qcom-ipq8064-v1.0.dtsi
+++ b/arch/arm/boot/dts/qcom-ipq8064-v1.0.dtsi
@@ -30,7 +30,6 @@ gsbi5: gsbi@1a200000 {
spi4: spi@1a280000 {
status = "okay";
- spi-max-frequency = <50000000>;
pinctrl-0 = <&spi_pins>;
pinctrl-names = "default";
--
2.34.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 3/4] ARM: dts: qcom: ipq8064: drop spi-max-frequency from controller
2023-06-19 17:01 ` [PATCH 3/4] ARM: dts: qcom: ipq8064: drop spi-max-frequency from controller Krzysztof Kozlowski
@ 2023-06-19 17:16 ` Konrad Dybcio
0 siblings, 0 replies; 9+ messages in thread
From: Konrad Dybcio @ 2023-06-19 17:16 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, devicetree,
linux-kernel
On 19.06.2023 19:01, Krzysztof Kozlowski wrote:
> spi-max-frequency is a property of SPI device, not SPI controller. Drop
> it from the controller nodes. No functional impact expected because
> child SPI device already defines spi-max-frequency. This fixes
> dtbs_check warnings like:
>
> qcom-ipq8064-ap148.dtb: spi@1a280000: Unevaluated properties are not allowed ('spi-max-frequency' was unexpected)
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
> arch/arm/boot/dts/qcom-ipq8064-rb3011.dts | 1 -
> arch/arm/boot/dts/qcom-ipq8064-v1.0.dtsi | 1 -
> 2 files changed, 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts b/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts
> index 104eb729c2d6..1796ded31d17 100644
> --- a/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts
> +++ b/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts
> @@ -282,7 +282,6 @@ gsbi5: gsbi@1a200000 {
>
> spi4: spi@1a280000 {
> status = "okay";
> - spi-max-frequency = <50000000>;
>
> pinctrl-0 = <&spi_pins>;
> pinctrl-names = "default";
> diff --git a/arch/arm/boot/dts/qcom-ipq8064-v1.0.dtsi b/arch/arm/boot/dts/qcom-ipq8064-v1.0.dtsi
> index c5abe7151f14..17f65e140e02 100644
> --- a/arch/arm/boot/dts/qcom-ipq8064-v1.0.dtsi
> +++ b/arch/arm/boot/dts/qcom-ipq8064-v1.0.dtsi
> @@ -30,7 +30,6 @@ gsbi5: gsbi@1a200000 {
>
> spi4: spi@1a280000 {
> status = "okay";
> - spi-max-frequency = <50000000>;
>
> pinctrl-0 = <&spi_pins>;
> pinctrl-names = "default";
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 4/4] ARM: dts: qcom: msm8960: drop spi-max-frequency from controller
2023-06-19 17:01 [PATCH 1/4] ARM: dts: qcom: ipq4019: use generic node names for USB Krzysztof Kozlowski
2023-06-19 17:01 ` [PATCH 2/4] ARM: dts: qcom: sdx55: " Krzysztof Kozlowski
2023-06-19 17:01 ` [PATCH 3/4] ARM: dts: qcom: ipq8064: drop spi-max-frequency from controller Krzysztof Kozlowski
@ 2023-06-19 17:01 ` Krzysztof Kozlowski
2023-06-19 17:16 ` Konrad Dybcio
2023-06-19 17:15 ` [PATCH 1/4] ARM: dts: qcom: ipq4019: use generic node names for USB Konrad Dybcio
2023-07-10 5:07 ` (subset) " Bjorn Andersson
4 siblings, 1 reply; 9+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-19 17:01 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, devicetree,
linux-kernel
Cc: Krzysztof Kozlowski
spi-max-frequency is a property of SPI device, not SPI controller. Drop
it from the controller nodes. No functional impact expected, although
qcom-msm8960-samsung-expressatt board does not have any child SPI
devices thus the property disappears. This fixes dtbs_check warnings
like:
qcom-msm8960-cdp.dtb: spi@16080000: Unevaluated properties are not allowed ('spi-max-frequency' was unexpected)
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm/boot/dts/qcom-msm8960.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi b/arch/arm/boot/dts/qcom-msm8960.dtsi
index fa2013388d99..d13080fcbeea 100644
--- a/arch/arm/boot/dts/qcom-msm8960.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8960.dtsi
@@ -364,7 +364,6 @@ gsbi1_spi: spi@16080000 {
#size-cells = <0>;
reg = <0x16080000 0x1000>;
interrupts = <GIC_SPI 147 IRQ_TYPE_LEVEL_HIGH>;
- spi-max-frequency = <24000000>;
cs-gpios = <&msmgpio 8 0>;
clocks = <&gcc GSBI1_QUP_CLK>, <&gcc GSBI1_H_CLK>;
--
2.34.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 4/4] ARM: dts: qcom: msm8960: drop spi-max-frequency from controller
2023-06-19 17:01 ` [PATCH 4/4] ARM: dts: qcom: msm8960: " Krzysztof Kozlowski
@ 2023-06-19 17:16 ` Konrad Dybcio
0 siblings, 0 replies; 9+ messages in thread
From: Konrad Dybcio @ 2023-06-19 17:16 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, devicetree,
linux-kernel
On 19.06.2023 19:01, Krzysztof Kozlowski wrote:
> spi-max-frequency is a property of SPI device, not SPI controller. Drop
> it from the controller nodes. No functional impact expected, although
> qcom-msm8960-samsung-expressatt board does not have any child SPI
> devices thus the property disappears. This fixes dtbs_check warnings
> like:
>
> qcom-msm8960-cdp.dtb: spi@16080000: Unevaluated properties are not allowed ('spi-max-frequency' was unexpected)
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
> arch/arm/boot/dts/qcom-msm8960.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi b/arch/arm/boot/dts/qcom-msm8960.dtsi
> index fa2013388d99..d13080fcbeea 100644
> --- a/arch/arm/boot/dts/qcom-msm8960.dtsi
> +++ b/arch/arm/boot/dts/qcom-msm8960.dtsi
> @@ -364,7 +364,6 @@ gsbi1_spi: spi@16080000 {
> #size-cells = <0>;
> reg = <0x16080000 0x1000>;
> interrupts = <GIC_SPI 147 IRQ_TYPE_LEVEL_HIGH>;
> - spi-max-frequency = <24000000>;
> cs-gpios = <&msmgpio 8 0>;
>
> clocks = <&gcc GSBI1_QUP_CLK>, <&gcc GSBI1_H_CLK>;
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/4] ARM: dts: qcom: ipq4019: use generic node names for USB
2023-06-19 17:01 [PATCH 1/4] ARM: dts: qcom: ipq4019: use generic node names for USB Krzysztof Kozlowski
` (2 preceding siblings ...)
2023-06-19 17:01 ` [PATCH 4/4] ARM: dts: qcom: msm8960: " Krzysztof Kozlowski
@ 2023-06-19 17:15 ` Konrad Dybcio
2023-07-10 5:07 ` (subset) " Bjorn Andersson
4 siblings, 0 replies; 9+ messages in thread
From: Konrad Dybcio @ 2023-06-19 17:15 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, devicetree,
linux-kernel
On 19.06.2023 19:01, Krzysztof Kozlowski wrote:
> Device node names should be generic which is also expected by USB
> bindings:
>
> qcom-ipq4018-jalapeno.dtb: dwc3@6000000: $nodename:0: 'dwc3@6000000' does not match '^usb(@.*)?'
>
> Override also the DWC3 node in qcom-ipq4018-ap120c-ac.dtsi by
> label/phandle, not via node path, because it is less error-prone and
> makes the overriding node-name independent.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
> arch/arm/boot/dts/qcom-ipq4018-ap120c-ac.dtsi | 7 ++++---
> arch/arm/boot/dts/qcom-ipq4019.dtsi | 14 +++++++-------
> 2 files changed, 11 insertions(+), 10 deletions(-)
>
> diff --git a/arch/arm/boot/dts/qcom-ipq4018-ap120c-ac.dtsi b/arch/arm/boot/dts/qcom-ipq4018-ap120c-ac.dtsi
> index d90b4f4c63af..da67d55fa557 100644
> --- a/arch/arm/boot/dts/qcom-ipq4018-ap120c-ac.dtsi
> +++ b/arch/arm/boot/dts/qcom-ipq4018-ap120c-ac.dtsi
> @@ -262,10 +262,11 @@ &usb3_hs_phy {
> &usb3 {
> status = "okay";
>
> - dwc3@8a00000 {
> - phys = <&usb3_hs_phy>;
> - phy-names = "usb2-phy";
> };
> +
> +&usb3_dwc {
> + phys = <&usb3_hs_phy>;
> + phy-names = "usb2-phy";
> };
>
> &usb2_hs_phy {
> diff --git a/arch/arm/boot/dts/qcom-ipq4019.dtsi b/arch/arm/boot/dts/qcom-ipq4019.dtsi
> index f0ef86fadc9d..13388e5c1b4b 100644
> --- a/arch/arm/boot/dts/qcom-ipq4019.dtsi
> +++ b/arch/arm/boot/dts/qcom-ipq4019.dtsi
> @@ -621,7 +621,7 @@ ethphy4: ethernet-phy@4 {
> };
> };
>
> - usb3_ss_phy: ssphy@9a000 {
> + usb3_ss_phy: usb-phy@9a000 {
> compatible = "qcom,usb-ss-ipq4019-phy";
> #phy-cells = <0>;
> reg = <0x9a000 0x800>;
> @@ -631,7 +631,7 @@ usb3_ss_phy: ssphy@9a000 {
> status = "disabled";
> };
>
> - usb3_hs_phy: hsphy@a6000 {
> + usb3_hs_phy: usb-phy@a6000 {
> compatible = "qcom,usb-hs-ipq4019-phy";
> #phy-cells = <0>;
> reg = <0xa6000 0x40>;
> @@ -641,7 +641,7 @@ usb3_hs_phy: hsphy@a6000 {
> status = "disabled";
> };
>
> - usb3: usb3@8af8800 {
> + usb3: usb@8af8800 {
> compatible = "qcom,ipq4019-dwc3", "qcom,dwc3";
> reg = <0x8af8800 0x100>;
> #address-cells = <1>;
> @@ -653,7 +653,7 @@ usb3: usb3@8af8800 {
> ranges;
> status = "disabled";
>
> - dwc3@8a00000 {
> + usb3_dwc: usb@8a00000 {
> compatible = "snps,dwc3";
> reg = <0x8a00000 0xf8000>;
> interrupts = <GIC_SPI 132 IRQ_TYPE_LEVEL_HIGH>;
> @@ -663,7 +663,7 @@ dwc3@8a00000 {
> };
> };
>
> - usb2_hs_phy: hsphy@a8000 {
> + usb2_hs_phy: usb-phy@a8000 {
> compatible = "qcom,usb-hs-ipq4019-phy";
> #phy-cells = <0>;
> reg = <0xa8000 0x40>;
> @@ -673,7 +673,7 @@ usb2_hs_phy: hsphy@a8000 {
> status = "disabled";
> };
>
> - usb2: usb2@60f8800 {
> + usb2: usb@60f8800 {
> compatible = "qcom,ipq4019-dwc3", "qcom,dwc3";
> reg = <0x60f8800 0x100>;
> #address-cells = <1>;
> @@ -685,7 +685,7 @@ usb2: usb2@60f8800 {
> ranges;
> status = "disabled";
>
> - dwc3@6000000 {
> + usb@6000000 {
> compatible = "snps,dwc3";
> reg = <0x6000000 0xf8000>;
> interrupts = <GIC_SPI 136 IRQ_TYPE_LEVEL_HIGH>;
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: (subset) [PATCH 1/4] ARM: dts: qcom: ipq4019: use generic node names for USB
2023-06-19 17:01 [PATCH 1/4] ARM: dts: qcom: ipq4019: use generic node names for USB Krzysztof Kozlowski
` (3 preceding siblings ...)
2023-06-19 17:15 ` [PATCH 1/4] ARM: dts: qcom: ipq4019: use generic node names for USB Konrad Dybcio
@ 2023-07-10 5:07 ` Bjorn Andersson
4 siblings, 0 replies; 9+ messages in thread
From: Bjorn Andersson @ 2023-07-10 5:07 UTC (permalink / raw)
To: Andy Gross, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-arm-msm, devicetree, linux-kernel,
Krzysztof Kozlowski
On Mon, 19 Jun 2023 19:01:48 +0200, Krzysztof Kozlowski wrote:
> Device node names should be generic which is also expected by USB
> bindings:
>
> qcom-ipq4018-jalapeno.dtb: dwc3@6000000: $nodename:0: 'dwc3@6000000' does not match '^usb(@.*)?'
>
> Override also the DWC3 node in qcom-ipq4018-ap120c-ac.dtsi by
> label/phandle, not via node path, because it is less error-prone and
> makes the overriding node-name independent.
>
> [...]
Applied, thanks!
[1/4] ARM: dts: qcom: ipq4019: use generic node names for USB
commit: 7caf09215ca32f1020df1559027d77770ca2e901
[2/4] ARM: dts: qcom: sdx55: use generic node names for USB
commit: 9a3b29c33b5d3d3dd446c1fa314a79f7a905886a
[3/4] ARM: dts: qcom: ipq8064: drop spi-max-frequency from controller
commit: 1bfeee1aeef0e6070e9ca2f06d310eb1c3058464
[4/4] ARM: dts: qcom: msm8960: drop spi-max-frequency from controller
commit: 594ccb8d24726c89dd6601b2322b399648da7a8c
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 9+ messages in thread