diff mbox series

[1/3] ARM: dts: qcom: ipq8064-rb3011: fix nand node validation

Message ID 20221013190657.48499-1-luca@z3ntu.xyz
State Accepted
Commit 6f917ec31d3eb0f2c657f36d299d39bd8d051e03
Headers show
Series [1/3] ARM: dts: qcom: ipq8064-rb3011: fix nand node validation | expand

Commit Message

Luca Weiss Oct. 13, 2022, 7:06 p.m. UTC
The devicetree documentation for the nand node requires the subnode be
called nand@ and no compatible is needed.

Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
---
 arch/arm/boot/dts/qcom-ipq8064-rb3011.dts | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Krzysztof Kozlowski Oct. 13, 2022, 9:16 p.m. UTC | #1
On 13/10/2022 15:06, Luca Weiss wrote:
> The devicetree documentation for the nand node requires the subnode be
> called nand@ and no compatible is needed.
> 
> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> ---
>  arch/arm/boot/dts/qcom-ipq8064-rb3011.dts | 3 +--


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
Krzysztof Kozlowski Oct. 13, 2022, 9:17 p.m. UTC | #2
On 13/10/2022 15:06, Luca Weiss wrote:
> There's a typo missing the arm, prefix of arm,coresight-etb10. Fix it to
> make devicetree validation happier.
> 

Fixes: 7a5c275fd821 ("ARM: dts: qcom: Add apq8064 CoreSight components")

> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>

With fixes tag:

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts b/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts
index 5a65cce2500c..86f895db9894 100644
--- a/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts
+++ b/arch/arm/boot/dts/qcom-ipq8064-rb3011.dts
@@ -264,8 +264,7 @@  &hs_phy_1 {
 &nand {
 	status = "okay";
 
-	nandcs@0 {
-		compatible = "qcom,nandcs";
+	nand@0 {
 		reg = <0>;
 
 		nand-ecc-strength = <4>;