devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] arm64: dts: marvell: armada-8040: Align GPIO hog name with bindings
@ 2025-01-15 21:17 Krzysztof Kozlowski
  2025-01-15 21:17 ` [PATCH 2/2] ARM: dts: marvell: kirkwood-openrd: " Krzysztof Kozlowski
  2025-03-07 16:49 ` ***SPAM*** [PATCH 1/2] arm64: dts: marvell: armada-8040: " Gregory CLEMENT
  0 siblings, 2 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-01-15 21:17 UTC (permalink / raw)
  To: Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, linux-arm-kernel, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

Bindings expect GPIO hog names to end with 'hog' suffix, so correct it
to fix dtbs_check warning:

  armada-8040-clearfog-gt-8k.dtb: sata_reset: $nodename:0: 'sata_reset' does not match '^.+-hog(-[0-9]+)?$'

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
index 225a54ab688d..90ae93274a16 100644
--- a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
+++ b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
@@ -371,25 +371,25 @@ &cp0_pcie0 {
 };
 
 &cp0_gpio2 {
-	sata_reset {
+	sata-reset-hog {
 		gpio-hog;
 		gpios = <1 GPIO_ACTIVE_HIGH>;
 		output-high;
 	};
 
-	lte_reset {
+	lte-reset-hog {
 		gpio-hog;
 		gpios = <2 GPIO_ACTIVE_LOW>;
 		output-low;
 	};
 
-	wlan_disable {
+	wlan_disable-hog {
 		gpio-hog;
 		gpios = <19 GPIO_ACTIVE_LOW>;
 		output-low;
 	};
 
-	lte_disable {
+	lte-disable-hog {
 		gpio-hog;
 		gpios = <21 GPIO_ACTIVE_LOW>;
 		output-low;
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 2/2] ARM: dts: marvell: kirkwood-openrd: Align GPIO hog name with bindings
  2025-01-15 21:17 [PATCH 1/2] arm64: dts: marvell: armada-8040: Align GPIO hog name with bindings Krzysztof Kozlowski
@ 2025-01-15 21:17 ` Krzysztof Kozlowski
  2025-03-07 16:50   ` Gregory CLEMENT
  2025-03-07 16:49 ` ***SPAM*** [PATCH 1/2] arm64: dts: marvell: armada-8040: " Gregory CLEMENT
  1 sibling, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-01-15 21:17 UTC (permalink / raw)
  To: Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, linux-arm-kernel, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

Bindings expect GPIO hog names to end with 'hog' suffix, so correct it
to fix dtbs_check warning:

  kirkwood-openrd-ultimate.dtb: p28: $nodename:0: 'p28' does not match '^.+-hog(-[0-9]+)?$'

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm/boot/dts/marvell/kirkwood-openrd.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/marvell/kirkwood-openrd.dtsi b/arch/arm/boot/dts/marvell/kirkwood-openrd.dtsi
index 47f03c69c55a..9d7cff4feada 100644
--- a/arch/arm/boot/dts/marvell/kirkwood-openrd.dtsi
+++ b/arch/arm/boot/dts/marvell/kirkwood-openrd.dtsi
@@ -53,7 +53,7 @@ mvsdio@90000 {
 			cd-gpios = <&gpio0 29 9>;
 		};
 		gpio@10100 {
-			p28 {
+			p28-hog {
 				gpio-hog;
 				gpios = <28 GPIO_ACTIVE_HIGH>;
 				/*
@@ -71,7 +71,7 @@ p28 {
 			};
 		};
 		gpio@10140 {
-			p2 {
+			p2-hog {
 				gpio-hog;
 				gpios = <2 GPIO_ACTIVE_HIGH>;
 				/*
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: ***SPAM*** [PATCH 1/2] arm64: dts: marvell: armada-8040: Align GPIO hog name with bindings
  2025-01-15 21:17 [PATCH 1/2] arm64: dts: marvell: armada-8040: Align GPIO hog name with bindings Krzysztof Kozlowski
  2025-01-15 21:17 ` [PATCH 2/2] ARM: dts: marvell: kirkwood-openrd: " Krzysztof Kozlowski
@ 2025-03-07 16:49 ` Gregory CLEMENT
  1 sibling, 0 replies; 4+ messages in thread
From: Gregory CLEMENT @ 2025-03-07 16:49 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Andrew Lunn, Sebastian Hesselbarth,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-kernel,
	devicetree, linux-kernel
  Cc: Krzysztof Kozlowski

Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> writes:

> Bindings expect GPIO hog names to end with 'hog' suffix, so correct it
> to fix dtbs_check warning:
>
>   armada-8040-clearfog-gt-8k.dtb: sata_reset: $nodename:0: 'sata_reset' does not match '^.+-hog(-[0-9]+)?$'
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Applied on mvebu/dt64

Thanks,

Gregory

> ---
>  .../arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> index 225a54ab688d..90ae93274a16 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> @@ -371,25 +371,25 @@ &cp0_pcie0 {
>  };
>  
>  &cp0_gpio2 {
> -	sata_reset {
> +	sata-reset-hog {
>  		gpio-hog;
>  		gpios = <1 GPIO_ACTIVE_HIGH>;
>  		output-high;
>  	};
>  
> -	lte_reset {
> +	lte-reset-hog {
>  		gpio-hog;
>  		gpios = <2 GPIO_ACTIVE_LOW>;
>  		output-low;
>  	};
>  
> -	wlan_disable {
> +	wlan_disable-hog {
>  		gpio-hog;
>  		gpios = <19 GPIO_ACTIVE_LOW>;
>  		output-low;
>  	};
>  
> -	lte_disable {
> +	lte-disable-hog {
>  		gpio-hog;
>  		gpios = <21 GPIO_ACTIVE_LOW>;
>  		output-low;
> -- 
> 2.43.0
>

-- 
Grégory CLEMENT, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 2/2] ARM: dts: marvell: kirkwood-openrd: Align GPIO hog name with bindings
  2025-01-15 21:17 ` [PATCH 2/2] ARM: dts: marvell: kirkwood-openrd: " Krzysztof Kozlowski
@ 2025-03-07 16:50   ` Gregory CLEMENT
  0 siblings, 0 replies; 4+ messages in thread
From: Gregory CLEMENT @ 2025-03-07 16:50 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Andrew Lunn, Sebastian Hesselbarth,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-kernel,
	devicetree, linux-kernel
  Cc: Krzysztof Kozlowski

Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> writes:

> Bindings expect GPIO hog names to end with 'hog' suffix, so correct it
> to fix dtbs_check warning:
>
>   kirkwood-openrd-ultimate.dtb: p28: $nodename:0: 'p28' does not match '^.+-hog(-[0-9]+)?$'
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Applied on mvebu/dt64

Thanks,

Gregory

> ---
>  arch/arm/boot/dts/marvell/kirkwood-openrd.dtsi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/marvell/kirkwood-openrd.dtsi b/arch/arm/boot/dts/marvell/kirkwood-openrd.dtsi
> index 47f03c69c55a..9d7cff4feada 100644
> --- a/arch/arm/boot/dts/marvell/kirkwood-openrd.dtsi
> +++ b/arch/arm/boot/dts/marvell/kirkwood-openrd.dtsi
> @@ -53,7 +53,7 @@ mvsdio@90000 {
>  			cd-gpios = <&gpio0 29 9>;
>  		};
>  		gpio@10100 {
> -			p28 {
> +			p28-hog {
>  				gpio-hog;
>  				gpios = <28 GPIO_ACTIVE_HIGH>;
>  				/*
> @@ -71,7 +71,7 @@ p28 {
>  			};
>  		};
>  		gpio@10140 {
> -			p2 {
> +			p2-hog {
>  				gpio-hog;
>  				gpios = <2 GPIO_ACTIVE_HIGH>;
>  				/*
> -- 
> 2.43.0
>

-- 
Grégory CLEMENT, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-03-07 16:50 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-01-15 21:17 [PATCH 1/2] arm64: dts: marvell: armada-8040: Align GPIO hog name with bindings Krzysztof Kozlowski
2025-01-15 21:17 ` [PATCH 2/2] ARM: dts: marvell: kirkwood-openrd: " Krzysztof Kozlowski
2025-03-07 16:50   ` Gregory CLEMENT
2025-03-07 16:49 ` ***SPAM*** [PATCH 1/2] arm64: dts: marvell: armada-8040: " Gregory CLEMENT

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).