arm64: dts: renesas: r8a77970: Align GPIO hog name with bindings

[PATCH] arm64: dts: renesas: r8a77970: Align GPIO hog name with bindings

Posted by Krzysztof Kozlowski 2 weeks, 6 days ago

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

  r8a77970-eagle-function-expansion.dtb: gpio@27: 'vin0_adv7612_en' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../boot/dts/renesas/r8a77970-eagle-function-expansion.dtso     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
index 9450d8ac94cb..0c005660d8dd 100644
--- a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
+++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
@@ -70,7 +70,7 @@ gpio@27 {
 		gpio-controller;
 		#gpio-cells = <2>;
 
-		vin0_adv7612_en {
+		vin0-adv7612-en-hog {
 			gpio-hog;
 			gpios = <3 GPIO_ACTIVE_LOW>;
 			output-high;
-- 
2.43.0

Re: [PATCH] arm64: dts: renesas: r8a77970: Align GPIO hog name with bindings

Posted by Geert Uytterhoeven 2 weeks, 1 day ago

On Wed, 15 Jan 2025 at 22:18, Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
> Bindings expect GPIO hog names to end with 'hog' suffix, so correct it
> to fix dtbs_check warning:
>
>   r8a77970-eagle-function-expansion.dtb: gpio@27: 'vin0_adv7612_en' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-devel for v6.15.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [PATCH] arm64: dts: renesas: r8a77970: Align GPIO hog name with bindings

Posted by Niklas Söderlund 2 weeks, 6 days ago

Hi Krzysztof,

Thanks for your patch.

On 2025-01-15 22:17:55 +0100, Krzysztof Kozlowski wrote:
> Bindings expect GPIO hog names to end with 'hog' suffix, so correct it
> to fix dtbs_check warning:
> 
>   r8a77970-eagle-function-expansion.dtb: gpio@27: 'vin0_adv7612_en' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>

> ---
>  .../boot/dts/renesas/r8a77970-eagle-function-expansion.dtso     | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
> index 9450d8ac94cb..0c005660d8dd 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
> +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
> @@ -70,7 +70,7 @@ gpio@27 {
>  		gpio-controller;
>  		#gpio-cells = <2>;
>  
> -		vin0_adv7612_en {
> +		vin0-adv7612-en-hog {
>  			gpio-hog;
>  			gpios = <3 GPIO_ACTIVE_LOW>;
>  			output-high;
> -- 
> 2.43.0
> 
> 

-- 
Kind Regards,
Niklas Söderlund