diff mbox series

[v2] ARM: dts: imx7-mba7: Use 'no-mmc' property

Message ID 20240325121404.1762825-1-festevam@gmail.com (mailing list archive)
State New, archived
Headers show
Series [v2] ARM: dts: imx7-mba7: Use 'no-mmc' property | expand

Commit Message

Fabio Estevam March 25, 2024, 12:14 p.m. UTC
From: Fabio Estevam <festevam@denx.de>

'no-emmc' is not a valid property.

The original intention was to use the 'no-mmc' property.

Change it accordingly to fix the following dt-schema warning:

imx7s-mba7.dtb: mmc@30b40000: Unevaluated properties are not allowed ('no-emmc' was unexpected)

Fixes: d430a7e0e181 ("ARM: dts: imx7-mba7: restrict usdhc interface modes")
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
Changes since v1:
Use no-mmc (Alexander).

 arch/arm/boot/dts/nxp/imx/imx7-mba7.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alexander Stein March 25, 2024, 12:56 p.m. UTC | #1
Hi Fabio,

thanks for the patch.

Am Montag, 25. März 2024, 13:14:04 CET schrieb Fabio Estevam:
> From: Fabio Estevam <festevam@denx.de>
> 
> 'no-emmc' is not a valid property.
> 
> The original intention was to use the 'no-mmc' property.
> 
> Change it accordingly to fix the following dt-schema warning:
> 
> imx7s-mba7.dtb: mmc@30b40000: Unevaluated properties are not allowed ('no-emmc' was unexpected)
> 
> Fixes: d430a7e0e181 ("ARM: dts: imx7-mba7: restrict usdhc interface modes")
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> ---
> Changes since v1:
> Use no-mmc (Alexander).
> 
>  arch/arm/boot/dts/nxp/imx/imx7-mba7.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/nxp/imx/imx7-mba7.dtsi b/arch/arm/boot/dts/nxp/imx/imx7-mba7.dtsi
> index 1235a71c6abe..52869e68f833 100644
> --- a/arch/arm/boot/dts/nxp/imx/imx7-mba7.dtsi
> +++ b/arch/arm/boot/dts/nxp/imx/imx7-mba7.dtsi
> @@ -666,7 +666,7 @@ &usdhc1 {
>  	bus-width = <4>;
>  	no-1-8-v;
>  	no-sdio;
> -	no-emmc;
> +	no-mmc;
>  	status = "okay";
>  };
>  
>
Shawn Guo April 2, 2024, 2:43 p.m. UTC | #2
On Mon, Mar 25, 2024 at 09:14:04AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> 'no-emmc' is not a valid property.
> 
> The original intention was to use the 'no-mmc' property.
> 
> Change it accordingly to fix the following dt-schema warning:
> 
> imx7s-mba7.dtb: mmc@30b40000: Unevaluated properties are not allowed ('no-emmc' was unexpected)
> 
> Fixes: d430a7e0e181 ("ARM: dts: imx7-mba7: restrict usdhc interface modes")
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Applied, thanks!
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/nxp/imx/imx7-mba7.dtsi b/arch/arm/boot/dts/nxp/imx/imx7-mba7.dtsi
index 1235a71c6abe..52869e68f833 100644
--- a/arch/arm/boot/dts/nxp/imx/imx7-mba7.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx7-mba7.dtsi
@@ -666,7 +666,7 @@  &usdhc1 {
 	bus-width = <4>;
 	no-1-8-v;
 	no-sdio;
-	no-emmc;
+	no-mmc;
 	status = "okay";
 };