ARM: dts: at91: sam9x60ek: fix indentation for pinctrl sub-nodes

Message ID 20220920083248.370604-1-dario.binacchi@amarulasolutions.com
State New
Headers show
Series
  • ARM: dts: at91: sam9x60ek: fix indentation for pinctrl sub-nodes
Related show

Commit Message

Dario Binacchi Sept. 20, 2022, 8:32 a.m. UTC
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(-)

Comments

Michael Trimarchi Sept. 20, 2022, 8:43 a.m. UTC | #1
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>
'Krzysztof Kozlowski' via Amarula Linux Oct. 21, 2022, 6:10 a.m. UTC | #2
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 !

Patch

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>;
+				};
 
 			};
 		};