diff mbox series

arm64: dts: imx93-tqma9352-mba93: Fix USB hub node name

Message ID 20240808123206.192906-1-festevam@gmail.com (mailing list archive)
State New, archived
Headers show
Series arm64: dts: imx93-tqma9352-mba93: Fix USB hub node name | expand

Commit Message

Fabio Estevam Aug. 8, 2024, 12:32 p.m. UTC 
From: Fabio Estevam <festevam@denx.de>

According to microchip,usb2514.yaml, the node name must contain
'usb-hub'.

Change it accordingly to fix the following dt-schema warning:
 
hub@1: $nodename:0: 'hub@1' does not match '^usb(@.*)?'

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts | 2 +-
 arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Alexander Stein Aug. 12, 2024, 8:05 a.m. UTC | #1 
Hi,

thanks for the patch.

Am Donnerstag, 8. August 2024, 14:32:06 CEST schrieb Fabio Estevam:
> From: Fabio Estevam <festevam@denx.de>
> 
> According to microchip,usb2514.yaml, the node name must contain
> 'usb-hub'.
> 
> Change it accordingly to fix the following dt-schema warning:
>  
> hub@1: $nodename:0: 'hub@1' does not match '^usb(@.*)?'
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>

> ---
>  arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts | 2 +-
>  arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
> index 852dd3d2eac7..9673b93ba470 100644
> --- a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
> +++ b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
> @@ -533,7 +533,7 @@ &usbotg2 {
>  	samsung,picophy-dc-vol-level-adjust = <7>;
>  	status = "okay";
>  
> -	hub_2_0: hub@1 {
> +	hub_2_0: usb-hub@1 {
>  		compatible = "usb424,2517";
>  		reg = <1>;
>  		reset-gpios = <&expander1 2 GPIO_ACTIVE_LOW>;
> diff --git a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
> index da8f19a646a9..f39905b4a2d2 100644
> --- a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
> +++ b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
> @@ -486,7 +486,7 @@ &usbotg2 {
>  	samsung,picophy-dc-vol-level-adjust = <7>;
>  	status = "okay";
>  
> -	hub_2_0: hub@1 {
> +	hub_2_0: usb-hub@1 {
>  		compatible = "usb424,2517";
>  		reg = <1>;
>  		reset-gpios = <&expander1 2 GPIO_ACTIVE_LOW>;
>
Shawn Guo Aug. 13, 2024, 8:47 a.m. UTC | #2 
On Thu, Aug 08, 2024 at 09:32:06AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> According to microchip,usb2514.yaml, the node name must contain
> 'usb-hub'.
> 
> Change it accordingly to fix the following dt-schema warning:
>  
> hub@1: $nodename:0: 'hub@1' does not match '^usb(@.*)?'
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Applied, thanks!
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
index 852dd3d2eac7..9673b93ba470 100644
--- a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
+++ b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
@@ -533,7 +533,7 @@  &usbotg2 {
 	samsung,picophy-dc-vol-level-adjust = <7>;
 	status = "okay";
 
-	hub_2_0: hub@1 {
+	hub_2_0: usb-hub@1 {
 		compatible = "usb424,2517";
 		reg = <1>;
 		reset-gpios = <&expander1 2 GPIO_ACTIVE_LOW>;
diff --git a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
index da8f19a646a9..f39905b4a2d2 100644
--- a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
+++ b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
@@ -486,7 +486,7 @@  &usbotg2 {
 	samsung,picophy-dc-vol-level-adjust = <7>;
 	status = "okay";
 
-	hub_2_0: hub@1 {
+	hub_2_0: usb-hub@1 {
 		compatible = "usb424,2517";
 		reg = <1>;
 		reset-gpios = <&expander1 2 GPIO_ACTIVE_LOW>;