Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1468329 > unrolled thread

Re: [PATCH 1/2] pinctrl: sunxi: fix uart1 CTS/RTS pins at PG on A23/A33

Started byMaxime Ripard <maxime.ripard@free-electrons.com>
First post2016-08-23 09:00 +0200
Last post2016-08-23 12:00 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH 1/2] pinctrl: sunxi: fix uart1 CTS/RTS pins at PG on  A23/A33 Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-08-23 09:00 +0200
    Re: [PATCH 1/2] pinctrl: sunxi: fix uart1 CTS/RTS pins at PG on A23/A33 Linus Walleij <linus.walleij@linaro.org> - 2016-08-23 11:40 +0200
      Re: [PATCH 1/2] pinctrl: sunxi: fix uart1 CTS/RTS pins at PG on  A23/A33 Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-08-23 12:00 +0200

#1468329 — Re: [PATCH 1/2] pinctrl: sunxi: fix uart1 CTS/RTS pins at PG on A23/A33

FromMaxime Ripard <maxime.ripard@free-electrons.com>
Date2016-08-23 09:00 +0200
SubjectRe: [PATCH 1/2] pinctrl: sunxi: fix uart1 CTS/RTS pins at PG on A23/A33
Message-ID<s9dQJ-lX-9@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

Hi,

On Tue, Aug 23, 2016 at 01:58:25PM +0800, Icenowy Zheng wrote:
> PG8, PG9 is said to be the CTS/RTS pins for UART1 according to the A23/33
> datasheets. However, the function is wrongly named "uart2" in the pinctrl
> driver. This patch fixes this by modifying them to be named "uart1".
> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>

This looks good, but could you send it to stable?

Thanks,
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[toc] | [next] | [standalone]


#1468440 — Re: [PATCH 1/2] pinctrl: sunxi: fix uart1 CTS/RTS pins at PG on A23/A33

FromLinus Walleij <linus.walleij@linaro.org>
Date2016-08-23 11:40 +0200
SubjectRe: [PATCH 1/2] pinctrl: sunxi: fix uart1 CTS/RTS pins at PG on A23/A33
Message-ID<s9glA-23O-19@gated-at.bofh.it>
In reply to#1468329
On Tue, Aug 23, 2016 at 8:47 AM, Maxime Ripard
<maxime.ripard@free-electrons.com> wrote:
> On Tue, Aug 23, 2016 at 01:58:25PM +0800, Icenowy Zheng wrote:
>> PG8, PG9 is said to be the CTS/RTS pins for UART1 according to the A23/33
>> datasheets. However, the function is wrongly named "uart2" in the pinctrl
>> driver. This patch fixes this by modifying them to be named "uart1".
>>
>> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
>
> This looks good, but could you send it to stable?

If that is an ACK I can tag it for stable when applying.

Yours,
Linus Walleij

[toc] | [prev] | [next] | [standalone]


#1468456

FromMaxime Ripard <maxime.ripard@free-electrons.com>
Date2016-08-23 12:00 +0200
Message-ID<s9gEW-2aI-29@gated-at.bofh.it>
In reply to#1468440

[Multipart message — attachments visible in raw view] — view raw

On Tue, Aug 23, 2016 at 11:27:10AM +0200, Linus Walleij wrote:
> On Tue, Aug 23, 2016 at 8:47 AM, Maxime Ripard
> <maxime.ripard@free-electrons.com> wrote:
> > On Tue, Aug 23, 2016 at 01:58:25PM +0800, Icenowy Zheng wrote:
> >> PG8, PG9 is said to be the CTS/RTS pins for UART1 according to the A23/33
> >> datasheets. However, the function is wrongly named "uart2" in the pinctrl
> >> driver. This patch fixes this by modifying them to be named "uart1".
> >>
> >> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> >
> > This looks good, but could you send it to stable?
> 
> If that is an ACK I can tag it for stable when applying.

Yes, it's an ACK. Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web