| Message ID | 20230626221010.3946263-1-robh@kernel.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | ARM: dts: at91-vinco: Fix "status" values | expand |
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 >
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 >
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!
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 {
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(-)