From: Peter Griffin <peter.griffin@linaro.org>
To: Lee Jones <lee.jones@linaro.org>
Cc: Patrice Chotard <patrice.chotard@st.com>,
gregkh@linuxfoundation.org, jslaby@suse.com,
linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, kernel@stlinux.com
Subject: Re: [STLinux Kernel] [PATCH 2/3] serial: st-asc: Provide RTS functionality
Date: Tue, 6 Dec 2016 13:03:52 +0000 [thread overview]
Message-ID: <20161206130352.GA17980@griffinp-ThinkPad-X1-Carbon-2nd> (raw)
In-Reply-To: <20161206124808.GD25385@dell.home>
Hi Lee,
On Tue, 06 Dec 2016, Lee Jones wrote:
> On Tue, 06 Dec 2016, Lee Jones wrote:
>
> > On Tue, 06 Dec 2016, Peter Griffin wrote:
> > > On Mon, 05 Dec 2016, Patrice Chotard wrote:
> > > > On 12/02/2016 03:11 PM, Lee Jones wrote:
> > > > > Until this point, it has not been possible for serial applications
> > > > > to toggle the UART RTS line. This can be useful with certain
> > > > > configurations. For example, when using a Mezzanine on a Linaro
> > > > > 96board, RTS line is used to take the the on-board microcontroller
> > > >
> > > > typo "the the on"
> > > >
> > > > > in and out of reset.
> > > > >
> > > > > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > > > > ---
> > > > > drivers/tty/serial/st-asc.c | 14 ++++++++++++++
> > > > > 1 file changed, 14 insertions(+)
> > > > >
> > > > > diff --git a/drivers/tty/serial/st-asc.c b/drivers/tty/serial/st-asc.c
> > > > > index 379e5bd..ce46898 100644
> > > > > --- a/drivers/tty/serial/st-asc.c
> > > > > +++ b/drivers/tty/serial/st-asc.c
> > > > > @@ -30,6 +30,7 @@
> > > > > #include <linux/of_platform.h>
> > > > > #include <linux/serial_core.h>
> > > > > #include <linux/clk.h>
> > > > > +#include <linux/gpio/consumer.h>
> > > > >
> > > > > #define DRIVER_NAME "st-asc"
> > > > > #define ASC_SERIAL_NAME "ttyAS"
> > > > > @@ -38,6 +39,7 @@
> > > > >
> > > > > struct asc_port {
> > > > > struct uart_port port;
> > > > > + struct gpio_desc *rts;
> > > > > struct clk *clk;
> > > > > unsigned int hw_flow_control:1;
> > > > > unsigned int force_m1:1;
> > > > > @@ -381,12 +383,16 @@ static unsigned int asc_tx_empty(struct uart_port *port)
> > > > >
> > > > > static void asc_set_mctrl(struct uart_port *port, unsigned int mctrl)
> > > > > {
> > > > > + struct asc_port *ascport = to_asc_port(port);
> > > > > +
> > > > > /*
> > > > > * This routine is used for seting signals of: DTR, DCD, CTS/RTS
> > > > > * We use ASC's hardware for CTS/RTS, so don't need any for that.
> > > > > * Some boards have DTR and DCD implemented using PIO pins,
> > > > > * code to do this should be hooked in here.
> > > > > */
> > > > > +
> > > > > + gpiod_set_value(ascport->rts, mctrl & TIOCM_RTS);
> > >
> > > This now puts the code and the comment above out of sync with each
> > > other.
> > >
> > > However just checking the stih410 datasheet and I don't think this patch
> > > to control RTS via a GPIO is required anyway.
> > >
> > > The correct way of handling this is to add UART10_RTS and UART10_CTS pins
> > > to the pinctrl_serial0 group so they are properly configured for their cts/rts
> > > alternate function. Once the pins are correctly configured, the IP block should
> > > control the signals correctly like the comment says.
> >
> > That was the first thing I tried. It didn't work.
>
> cts = <&pio17 2 ALT1 IN>;
> rts = <&pio17 3 ALT1 OUT>;
>
That implies that either some additional configuration of the IP is required in
st-asc to get the IP to control those signals or that there is a bug in the
ST pinctrl driver and the correct settings aren't being applied.
regards,
Peter.
next prev parent reply other threads:[~2016-12-06 13:04 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-02 14:11 [PATCH 1/3] ARM: dts: STiH410-b2260: Identify the UART RTS line Lee Jones
2016-12-02 14:11 ` [PATCH 2/3] serial: st-asc: Provide RTS functionality Lee Jones
2016-12-05 14:11 ` Patrice Chotard
2016-12-06 12:31 ` [STLinux Kernel] " Peter Griffin
2016-12-06 12:45 ` Lee Jones
2016-12-06 12:48 ` Lee Jones
2016-12-06 13:03 ` Peter Griffin [this message]
2016-12-02 14:11 ` [PATCH 3/3] serial: st-asc: Ignore the parity error bit if 8-bit mode is enabled Lee Jones
2016-12-06 8:39 ` [PATCH 1/3] ARM: dts: STiH410-b2260: Identify the UART RTS line Patrice Chotard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20161206130352.GA17980@griffinp-ThinkPad-X1-Carbon-2nd \
--to=peter.griffin@linaro.org \
--cc=gregkh@linuxfoundation.org \
--cc=jslaby@suse.com \
--cc=kernel@stlinux.com \
--cc=lee.jones@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=patrice.chotard@st.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).