ARM: dts: at91-vinco: Fix "status" values

[PATCH] ARM: dts: at91-vinco: Fix "status" values

Posted by Rob Herring 2 years, 7 months ago

The defined value for "status" is "disabled", not "disable".

Signed-off-by: Rob Herring <robh@kernel.org>
---
 arch/arm/boot/dts/microchip/at91-vinco.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/microchip/at91-vinco.dts b/arch/arm/boot/dts/microchip/at91-vinco.dts
index ebeaa6ab500e..ecbdacf48708 100644
--- a/arch/arm/boot/dts/microchip/at91-vinco.dts
+++ b/arch/arm/boot/dts/microchip/at91-vinco.dts
@@ -159,7 +159,7 @@ usb0: gadget@400000 {
 			atmel,vbus-gpio = <&pioE 31 GPIO_ACTIVE_HIGH>;
 			pinctrl-names = "default";
 			pinctrl-0 = <&pinctrl_usba_vbus>;
-			status = "disable";
+			status = "disabled";
 		};
 
 		usb1: ohci@500000 {
@@ -168,7 +168,7 @@ usb1: ohci@500000 {
 					   &pioE 11 GPIO_ACTIVE_LOW
 					   &pioE 12 GPIO_ACTIVE_LOW
 					  >;
-			status = "disable";
+			status = "disabled";
 		};
 
 		usb2: ehci@600000 {
-- 
2.40.1

Re: [PATCH] ARM: dts: at91-vinco: Fix "status" values

Posted by Rob Herring 2 years, 6 months ago

On Mon, Jun 26, 2023 at 4:10 PM Rob Herring <robh@kernel.org> wrote:
>
> The defined value for "status" is "disabled", not "disable".
>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  arch/arm/boot/dts/microchip/at91-vinco.dts | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Ping!

>
> diff --git a/arch/arm/boot/dts/microchip/at91-vinco.dts b/arch/arm/boot/dts/microchip/at91-vinco.dts
> index ebeaa6ab500e..ecbdacf48708 100644
> --- a/arch/arm/boot/dts/microchip/at91-vinco.dts
> +++ b/arch/arm/boot/dts/microchip/at91-vinco.dts
> @@ -159,7 +159,7 @@ usb0: gadget@400000 {
>                         atmel,vbus-gpio = <&pioE 31 GPIO_ACTIVE_HIGH>;
>                         pinctrl-names = "default";
>                         pinctrl-0 = <&pinctrl_usba_vbus>;
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 usb1: ohci@500000 {
> @@ -168,7 +168,7 @@ usb1: ohci@500000 {
>                                            &pioE 11 GPIO_ACTIVE_LOW
>                                            &pioE 12 GPIO_ACTIVE_LOW
>                                           >;
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 usb2: ehci@600000 {
> --
> 2.40.1
>

Re: [PATCH] ARM: dts: at91-vinco: Fix "status" values

Posted by Nicolas Ferre 2 years, 6 months ago

On 27/06/2023 at 00:10, Rob Herring wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> The defined value for "status" is "disabled", not "disable".
> 
> Signed-off-by: Rob Herring <robh@kernel.org>

Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
We'll queue the patch in our tree (and linux-next) soon (tm).
Best regards,
   Nicolas

> ---
>   arch/arm/boot/dts/microchip/at91-vinco.dts | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/microchip/at91-vinco.dts b/arch/arm/boot/dts/microchip/at91-vinco.dts
> index ebeaa6ab500e..ecbdacf48708 100644
> --- a/arch/arm/boot/dts/microchip/at91-vinco.dts
> +++ b/arch/arm/boot/dts/microchip/at91-vinco.dts
> @@ -159,7 +159,7 @@ usb0: gadget@400000 {
>                          atmel,vbus-gpio = <&pioE 31 GPIO_ACTIVE_HIGH>;
>                          pinctrl-names = "default";
>                          pinctrl-0 = <&pinctrl_usba_vbus>;
> -                       status = "disable";
> +                       status = "disabled";
>                  };
> 
>                  usb1: ohci@500000 {
> @@ -168,7 +168,7 @@ usb1: ohci@500000 {
>                                             &pioE 11 GPIO_ACTIVE_LOW
>                                             &pioE 12 GPIO_ACTIVE_LOW
>                                            >;
> -                       status = "disable";
> +                       status = "disabled";
>                  };
> 
>                  usb2: ehci@600000 {
> --
> 2.40.1
>

Re: [PATCH] ARM: dts: at91-vinco: Fix "status" values

Posted by claudiu beznea 2 years, 6 months ago


On 18.07.2023 19:37, Nicolas Ferre wrote:
> On 27/06/2023 at 00:10, Rob Herring wrote:
>> EXTERNAL EMAIL: Do not click links or open attachments unless you know
>> the content is safe
>>
>> The defined value for "status" is "disabled", not "disable".
>>
>> Signed-off-by: Rob Herring <robh@kernel.org>
> 
> Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
> We'll queue the patch in our tree (and linux-next) soon (tm).
> Best regards,
>   Nicolas

Applied to at91-dt, thanks!