All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: at91: sam9x60ek: fix indentation for pinctrl sub-nodes
@ 2022-09-20  8:32 Dario Binacchi
  2022-09-20  8:43 ` Michael Nazzareno Trimarchi
  0 siblings, 1 reply; 3+ messages in thread
From: Dario Binacchi @ 2022-09-20  8:32 UTC (permalink / raw)
  To: u-boot
  Cc: Dario Binacchi, Andreas Bießmann, Eugen Hristev,
	Heiko Schocher, Sergiu Moga, Tom Rini, Tudor Ambarus,
	linux-amarula

The indentation went far on the right due to an extra tab for
each pinctrl sub-nodes.

Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
---

 arch/arm/dts/sam9x60ek.dts | 36 ++++++++++++++++++------------------
 1 file changed, 18 insertions(+), 18 deletions(-)

diff --git a/arch/arm/dts/sam9x60ek.dts b/arch/arm/dts/sam9x60ek.dts
index 54c694bd7848..eb44868a3e3e 100644
--- a/arch/arm/dts/sam9x60ek.dts
+++ b/arch/arm/dts/sam9x60ek.dts
@@ -80,26 +80,26 @@
 			};
 
 			pinctrl {
-					pinctrl_qspi: qspi {
-						atmel,pins =
-							<AT91_PIOB 19 AT91_PERIPH_A AT91_PINCTRL_NONE
-							 AT91_PIOB 20 AT91_PERIPH_A AT91_PINCTRL_NONE
-							 AT91_PIOB 21 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
-							 AT91_PIOB 22 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
-							 AT91_PIOB 23 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
-							 AT91_PIOB 24 AT91_PERIPH_A AT91_PINCTRL_PULL_UP>;
-					};
+				pinctrl_qspi: qspi {
+					atmel,pins =
+						<AT91_PIOB 19 AT91_PERIPH_A AT91_PINCTRL_NONE
+						 AT91_PIOB 20 AT91_PERIPH_A AT91_PINCTRL_NONE
+						 AT91_PIOB 21 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
+						 AT91_PIOB 22 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
+						 AT91_PIOB 23 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
+						 AT91_PIOB 24 AT91_PERIPH_A AT91_PINCTRL_PULL_UP>;
+				};
 
-					pinctrl_flx0: flx0_default {
-						atmel,pins =
-							<AT91_PIOA 0 AT91_PERIPH_A AT91_PINCTRL_NONE
-							 AT91_PIOA 1 AT91_PERIPH_A AT91_PINCTRL_NONE>;
-					};
+				pinctrl_flx0: flx0_default {
+					atmel,pins =
+						<AT91_PIOA 0 AT91_PERIPH_A AT91_PINCTRL_NONE
+						 AT91_PIOA 1 AT91_PERIPH_A AT91_PINCTRL_NONE>;
+				};
 
-					pinctrl_onewire_tm_default: onewire_tm_default {
-						atmel,pins =
-							<AT91_PIOD 14 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
-					};
+				pinctrl_onewire_tm_default: onewire_tm_default {
+					atmel,pins =
+						<AT91_PIOD 14 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
+				};
 
 			};
 		};
-- 
2.32.0


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

* Re: [PATCH] ARM: dts: at91: sam9x60ek: fix indentation for pinctrl sub-nodes
  2022-09-20  8:32 [PATCH] ARM: dts: at91: sam9x60ek: fix indentation for pinctrl sub-nodes Dario Binacchi
@ 2022-09-20  8:43 ` Michael Nazzareno Trimarchi
  2022-10-21  6:10   ` Eugen.Hristev
  0 siblings, 1 reply; 3+ messages in thread
From: Michael Nazzareno Trimarchi @ 2022-09-20  8:43 UTC (permalink / raw)
  To: Dario Binacchi
  Cc: u-boot, Andreas Bießmann, Eugen Hristev, Heiko Schocher,
	Sergiu Moga, Tom Rini, Tudor Ambarus, linux-amarula

Hi

On Tue, Sep 20, 2022 at 10:33 AM Dario Binacchi
<dario.binacchi@amarulasolutions.com> wrote:
>
> The indentation went far on the right due to an extra tab for
> each pinctrl sub-nodes.
>
> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> ---
>
>  arch/arm/dts/sam9x60ek.dts | 36 ++++++++++++++++++------------------
>  1 file changed, 18 insertions(+), 18 deletions(-)
>
> diff --git a/arch/arm/dts/sam9x60ek.dts b/arch/arm/dts/sam9x60ek.dts
> index 54c694bd7848..eb44868a3e3e 100644
> --- a/arch/arm/dts/sam9x60ek.dts
> +++ b/arch/arm/dts/sam9x60ek.dts
> @@ -80,26 +80,26 @@
>                         };
>
>                         pinctrl {
> -                                       pinctrl_qspi: qspi {
> -                                               atmel,pins =
> -                                                       <AT91_PIOB 19 AT91_PERIPH_A AT91_PINCTRL_NONE
> -                                                        AT91_PIOB 20 AT91_PERIPH_A AT91_PINCTRL_NONE
> -                                                        AT91_PIOB 21 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> -                                                        AT91_PIOB 22 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> -                                                        AT91_PIOB 23 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> -                                                        AT91_PIOB 24 AT91_PERIPH_A AT91_PINCTRL_PULL_UP>;
> -                                       };
> +                               pinctrl_qspi: qspi {
> +                                       atmel,pins =
> +                                               <AT91_PIOB 19 AT91_PERIPH_A AT91_PINCTRL_NONE
> +                                                AT91_PIOB 20 AT91_PERIPH_A AT91_PINCTRL_NONE
> +                                                AT91_PIOB 21 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> +                                                AT91_PIOB 22 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> +                                                AT91_PIOB 23 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> +                                                AT91_PIOB 24 AT91_PERIPH_A AT91_PINCTRL_PULL_UP>;
> +                               };
>
> -                                       pinctrl_flx0: flx0_default {
> -                                               atmel,pins =
> -                                                       <AT91_PIOA 0 AT91_PERIPH_A AT91_PINCTRL_NONE
> -                                                        AT91_PIOA 1 AT91_PERIPH_A AT91_PINCTRL_NONE>;
> -                                       };
> +                               pinctrl_flx0: flx0_default {
> +                                       atmel,pins =
> +                                               <AT91_PIOA 0 AT91_PERIPH_A AT91_PINCTRL_NONE
> +                                                AT91_PIOA 1 AT91_PERIPH_A AT91_PINCTRL_NONE>;
> +                               };
>
> -                                       pinctrl_onewire_tm_default: onewire_tm_default {
> -                                               atmel,pins =
> -                                                       <AT91_PIOD 14 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
> -                                       };
> +                               pinctrl_onewire_tm_default: onewire_tm_default {
> +                                       atmel,pins =
> +                                               <AT91_PIOD 14 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
> +                               };
>
>                         };
>                 };
> --
> 2.32.0
>
>

Reviewed-By: Michael Trimarchi <michael@amarulasolutions.com>


-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael@amarulasolutions.com
__________________________________

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info@amarulasolutions.com
www.amarulasolutions.com

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

* Re: [PATCH] ARM: dts: at91: sam9x60ek: fix indentation for pinctrl sub-nodes
  2022-09-20  8:43 ` Michael Nazzareno Trimarchi
@ 2022-10-21  6:10   ` Eugen.Hristev
  0 siblings, 0 replies; 3+ messages in thread
From: Eugen.Hristev @ 2022-10-21  6:10 UTC (permalink / raw)
  To: michael, dario.binacchi
  Cc: u-boot, andreas, hs, Sergiu.Moga, trini, Tudor.Ambarus, linux-amarula

On 9/20/22 11:43 AM, Michael Nazzareno Trimarchi wrote:

> Hi
> 
> On Tue, Sep 20, 2022 at 10:33 AM Dario Binacchi
> <dario.binacchi@amarulasolutions.com> wrote:
>>
>> The indentation went far on the right due to an extra tab for
>> each pinctrl sub-nodes.
>>
>> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
>> ---
>>
>>   arch/arm/dts/sam9x60ek.dts | 36 ++++++++++++++++++------------------
>>   1 file changed, 18 insertions(+), 18 deletions(-)
>>
>> diff --git a/arch/arm/dts/sam9x60ek.dts b/arch/arm/dts/sam9x60ek.dts
>> index 54c694bd7848..eb44868a3e3e 100644
>> --- a/arch/arm/dts/sam9x60ek.dts
>> +++ b/arch/arm/dts/sam9x60ek.dts
>> @@ -80,26 +80,26 @@
>>                          };
>>
>>                          pinctrl {
>> -                                       pinctrl_qspi: qspi {
>> -                                               atmel,pins =
>> -                                                       <AT91_PIOB 19 AT91_PERIPH_A AT91_PINCTRL_NONE
>> -                                                        AT91_PIOB 20 AT91_PERIPH_A AT91_PINCTRL_NONE
>> -                                                        AT91_PIOB 21 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
>> -                                                        AT91_PIOB 22 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
>> -                                                        AT91_PIOB 23 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
>> -                                                        AT91_PIOB 24 AT91_PERIPH_A AT91_PINCTRL_PULL_UP>;
>> -                                       };
>> +                               pinctrl_qspi: qspi {
>> +                                       atmel,pins =
>> +                                               <AT91_PIOB 19 AT91_PERIPH_A AT91_PINCTRL_NONE
>> +                                                AT91_PIOB 20 AT91_PERIPH_A AT91_PINCTRL_NONE
>> +                                                AT91_PIOB 21 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
>> +                                                AT91_PIOB 22 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
>> +                                                AT91_PIOB 23 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
>> +                                                AT91_PIOB 24 AT91_PERIPH_A AT91_PINCTRL_PULL_UP>;
>> +                               };
>>
>> -                                       pinctrl_flx0: flx0_default {
>> -                                               atmel,pins =
>> -                                                       <AT91_PIOA 0 AT91_PERIPH_A AT91_PINCTRL_NONE
>> -                                                        AT91_PIOA 1 AT91_PERIPH_A AT91_PINCTRL_NONE>;
>> -                                       };
>> +                               pinctrl_flx0: flx0_default {
>> +                                       atmel,pins =
>> +                                               <AT91_PIOA 0 AT91_PERIPH_A AT91_PINCTRL_NONE
>> +                                                AT91_PIOA 1 AT91_PERIPH_A AT91_PINCTRL_NONE>;
>> +                               };
>>
>> -                                       pinctrl_onewire_tm_default: onewire_tm_default {
>> -                                               atmel,pins =
>> -                                                       <AT91_PIOD 14 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
>> -                                       };
>> +                               pinctrl_onewire_tm_default: onewire_tm_default {
>> +                                       atmel,pins =
>> +                                               <AT91_PIOD 14 AT91_PERIPH_GPIO AT91_PINCTRL_PULL_UP>;
>> +                               };
>>
>>                          };
>>                  };
>> --
>> 2.32.0
>>
>>
> 
> Reviewed-By: Michael Trimarchi <michael@amarulasolutions.com>
> 
> 
Applied to u-boot-at91 / master , thanks !

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

end of thread, other threads:[~2022-10-21  6:10 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-20  8:32 [PATCH] ARM: dts: at91: sam9x60ek: fix indentation for pinctrl sub-nodes Dario Binacchi
2022-09-20  8:43 ` Michael Nazzareno Trimarchi
2022-10-21  6:10   ` Eugen.Hristev

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.