| Message ID | 20230123071654.73139-1-jagan@amarulasolutions.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2,1/4] arm64: dts: rockchip: Fix compatible for Radxa CM3 | expand |
On 23/01/2023 08:16, Jagan Teki wrote: > The compatible string "radxa,radxa-cm3" referring the product name > as "Radxa Radxa CM3" but the actual product name is "Radxa CM3". > > Fix the compatible strings. > > Fixes: 24a28d3eb07d ("dt-bindings: arm: rockchip: Add Radxa Compute > Module 3") > Fixes: 7469ab529bca ("arm64: dts: rockchip: Add rk3566 based Radxa > Compute Module 3") > Fixes: 096ebfb74b19 ("arm64: dts: rockchip: Add Radxa Compute Module > 3 IO board") > Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> Don't wrap the tags. Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Mon, 23 Jan 2023 12:46:50 +0530, Jagan Teki wrote: > The compatible string "radxa,radxa-cm3" referring the product name > as "Radxa Radxa CM3" but the actual product name is "Radxa CM3". > > Fix the compatible strings. > > Applied, thanks! [1/4] arm64: dts: rockchip: Fix compatible for Radxa CM3 commit: 8f19828844f20b22182719cf53be64f8c955aee8 [2/4] arm64: dts: rockchip: Add missing CM3i fallback compatible for Radxa E25 commit: c4d2b02d63ee38b381fbc886c02eecfec4f981cc [3/4] arm64: dts: rockchip: Drop unneeded model for Radxa CM3i commit: 421c059d413812444318d27c1b4d6e71f1c1134c [4/4] arm64: dts: rockchip: Correct the model name for Radxa E25 commit: ef9134d9bbce071c9e4ebdcbb6f8fb1a5dd0a67e Best regards,
diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml index 98afd7c6b8e6..35f74eda30ae 100644 --- a/Documentation/devicetree/bindings/arm/rockchip.yaml +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml @@ -624,8 +624,8 @@ properties: - description: Radxa Compute Module 3(CM3) items: - enum: - - radxa,radxa-cm3-io - - const: radxa,radxa-cm3 + - radxa,cm3-io + - const: radxa,cm3 - const: rockchip,rk3566 - description: Radxa CM3 Industrial diff --git a/arch/arm64/boot/dts/rockchip/rk3566-radxa-cm3-io.dts b/arch/arm64/boot/dts/rockchip/rk3566-radxa-cm3-io.dts index 3e6578f27ec0..4f8ac9fbbf7c 100644 --- a/arch/arm64/boot/dts/rockchip/rk3566-radxa-cm3-io.dts +++ b/arch/arm64/boot/dts/rockchip/rk3566-radxa-cm3-io.dts @@ -11,7 +11,7 @@ / { model = "Radxa Compute Module 3(CM3) IO Board"; - compatible = "radxa,radxa-cm3-io", "radxa,radxa-cm3", "rockchip,rk3566"; + compatible = "radxa,cm3-io", "radxa,cm3", "rockchip,rk3566"; aliases { mmc1 = &sdmmc0; diff --git a/arch/arm64/boot/dts/rockchip/rk3566-radxa-cm3.dtsi b/arch/arm64/boot/dts/rockchip/rk3566-radxa-cm3.dtsi index 23a85a13311a..0bc747997c14 100644 --- a/arch/arm64/boot/dts/rockchip/rk3566-radxa-cm3.dtsi +++ b/arch/arm64/boot/dts/rockchip/rk3566-radxa-cm3.dtsi @@ -8,7 +8,7 @@ #include <dt-bindings/leds/common.h> / { - compatible = "radxa,radxa-cm3", "rockchip,rk3566"; + compatible = "radxa,cm3", "rockchip,rk3566"; aliases { mmc0 = &sdhci;
The compatible string "radxa,radxa-cm3" referring the product name as "Radxa Radxa CM3" but the actual product name is "Radxa CM3". Fix the compatible strings. Fixes: 24a28d3eb07d ("dt-bindings: arm: rockchip: Add Radxa Compute Module 3") Fixes: 7469ab529bca ("arm64: dts: rockchip: Add rk3566 based Radxa Compute Module 3") Fixes: 096ebfb74b19 ("arm64: dts: rockchip: Add Radxa Compute Module 3 IO board") Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> --- Changes for v2: - new patch Documentation/devicetree/bindings/arm/rockchip.yaml | 4 ++-- arch/arm64/boot/dts/rockchip/rk3566-radxa-cm3-io.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3566-radxa-cm3.dtsi | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-)