diff mbox

[RESEND] tty/serial: atmel_serial: Fix device tree documentation

Message ID 1397835904-866-1-git-send-email-richard.genoud@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Richard Genoud April 18, 2014, 3:45 p.m. UTC
RTS pin is an active low pin.
For now, this doesn't change anything as the ACTIVE_LOW (1) flag is not
handled in atmel_serial, but it will be in 3.16.

Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
 Documentation/devicetree/bindings/serial/atmel-usart.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Richard Genoud April 22, 2014, 1:45 p.m. UTC | #1
2014-04-18 17:45 GMT+02:00 Richard Genoud <richard.genoud@gmail.com>:
> RTS pin is an active low pin.
> For now, this doesn't change anything as the ACTIVE_LOW (1) flag is not
> handled in atmel_serial, but it will be in 3.16.
>
> Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
> ---
>  Documentation/devicetree/bindings/serial/atmel-usart.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/serial/atmel-usart.txt b/Documentation/devicetree/bindings/serial/atmel-usart.txt
> index 17c1042b2df8..0dd0a5493a54 100644
> --- a/Documentation/devicetree/bindings/serial/atmel-usart.txt
> +++ b/Documentation/devicetree/bindings/serial/atmel-usart.txt
> @@ -35,7 +35,7 @@ Example:
>                 clock-names = "usart";
>                 atmel,use-dma-rx;
>                 atmel,use-dma-tx;
> -               rts-gpios = <&pioD 15 0>;
> +               rts-gpios = <&pioD 15 1>;
>         };
>
>  - use DMA:

Rob,
Does this has a chance to get to 3.15-rc3 or it's not worth it ?
If it's not programmed for the 3.15-rc cycle, please drop it : I've
got a patchset depending on it, and it will be in another tree.

Richard.
Alexandre Belloni April 22, 2014, 2:37 p.m. UTC | #2
Hi,

On 22/04/2014 at 15:45:26 +0200, Richard Genoud wrote :
> 2014-04-18 17:45 GMT+02:00 Richard Genoud <richard.genoud@gmail.com>:
> > RTS pin is an active low pin.
> > For now, this doesn't change anything as the ACTIVE_LOW (1) flag is not
> > handled in atmel_serial, but it will be in 3.16.
> >
> > Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
> > ---
> >  Documentation/devicetree/bindings/serial/atmel-usart.txt | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/serial/atmel-usart.txt b/Documentation/devicetree/bindings/serial/atmel-usart.txt
> > index 17c1042b2df8..0dd0a5493a54 100644
> > --- a/Documentation/devicetree/bindings/serial/atmel-usart.txt
> > +++ b/Documentation/devicetree/bindings/serial/atmel-usart.txt
> > @@ -35,7 +35,7 @@ Example:
> >                 clock-names = "usart";
> >                 atmel,use-dma-rx;
> >                 atmel,use-dma-tx;
> > -               rts-gpios = <&pioD 15 0>;
> > +               rts-gpios = <&pioD 15 1>;
> >         };
> >
> >  - use DMA:
> 
> Rob,
> Does this has a chance to get to 3.15-rc3 or it's not worth it ?
> If it's not programmed for the 3.15-rc cycle, please drop it : I've
> got a patchset depending on it, and it will be in another tree.
> 

I would say that kind of change should go through Nicolas' tree or
gregkh tty tree as it is directly related to the driver.
Richard Genoud April 22, 2014, 2:40 p.m. UTC | #3
2014-04-22 16:37 GMT+02:00 Alexandre Belloni
<alexandre.belloni@free-electrons.com>:
> Hi,
>
> On 22/04/2014 at 15:45:26 +0200, Richard Genoud wrote :
>> 2014-04-18 17:45 GMT+02:00 Richard Genoud <richard.genoud@gmail.com>:
>> > RTS pin is an active low pin.
>> > For now, this doesn't change anything as the ACTIVE_LOW (1) flag is not
>> > handled in atmel_serial, but it will be in 3.16.
>> >
>> > Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
>> > ---
>> >  Documentation/devicetree/bindings/serial/atmel-usart.txt | 2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> > diff --git a/Documentation/devicetree/bindings/serial/atmel-usart.txt b/Documentation/devicetree/bindings/serial/atmel-usart.txt
>> > index 17c1042b2df8..0dd0a5493a54 100644
>> > --- a/Documentation/devicetree/bindings/serial/atmel-usart.txt
>> > +++ b/Documentation/devicetree/bindings/serial/atmel-usart.txt
>> > @@ -35,7 +35,7 @@ Example:
>> >                 clock-names = "usart";
>> >                 atmel,use-dma-rx;
>> >                 atmel,use-dma-tx;
>> > -               rts-gpios = <&pioD 15 0>;
>> > +               rts-gpios = <&pioD 15 1>;
>> >         };
>> >
>> >  - use DMA:
>>
>> Rob,
>> Does this has a chance to get to 3.15-rc3 or it's not worth it ?
>> If it's not programmed for the 3.15-rc cycle, please drop it : I've
>> got a patchset depending on it, and it will be in another tree.
>>
>
> I would say that kind of change should go through Nicolas' tree or
> gregkh tty tree as it is directly related to the driver.

Ok, I'll send it to greg, since my patchset is on the tty tree.

thanks !
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/serial/atmel-usart.txt b/Documentation/devicetree/bindings/serial/atmel-usart.txt
index 17c1042b2df8..0dd0a5493a54 100644
--- a/Documentation/devicetree/bindings/serial/atmel-usart.txt
+++ b/Documentation/devicetree/bindings/serial/atmel-usart.txt
@@ -35,7 +35,7 @@  Example:
 		clock-names = "usart";
 		atmel,use-dma-rx;
 		atmel,use-dma-tx;
-		rts-gpios = <&pioD 15 0>;
+		rts-gpios = <&pioD 15 1>;
 	};
 
 - use DMA: