Thread (32 messages) 32 messages, 7 authors, 2008-12-04

Re: [PATCH/RFC] 8250: Auto RS485 direction control

From: Laurent Pinchart <hidden>
Date: 2008-08-05 09:41:10

On Monday 04 August 2008, Grant Edwards wrote:
On 2008-08-04, Laurent Pinchart [off-list ref] wrote:
quoted
On Monday 04 August 2008, Tosoni wrote:
quoted
About the flags names -- CARTS, UART_FCTR_RS485

May I suggest CRTSTOGGLE since it is known by that kind of name in other
OS's :-)
I like Russell's proposal of sticking to CRTSCTS and adding options to
setserial:
quoted
Should CRTSCTS be a global "enable some kind of flow control" bit and
setserial be used to configure the actual flow control method
(conventional RTS/CTS, DTR/DSR, alternate RTS/CTS, RS485 on RTS,
RS485 on DTR) ?
Any opinion on that ?
Opposed.

 1) CRTSCTS means something quite specific.
That's right, and this is my main concern.
    Rather than overload that name for something unrelated, create a new
    mode bit for the new mode.
We already have 3 distinct modes for RTS/CTS (see my mail from today 11:23:36 in this thread). If we add DSR/DTR hardware support we can easily get 5 or 6 modes. We will run out of bits in c_cflags.
 2) Most of what's in setserial is useless and irrelevent to
    non-PC-motherboard-16550-uart serial ports (the drivers for
    which often don't support setserial).

Let's just suck it up and do it the right way.
As the number of bits in c_cflags is not infinite we will need an ioctl to configure the desired hardware handshake mode (as most modes don't control the data flow I'd rather talk about hardware handshake which is a more generic name). We can either use one of the struct serial_struct reserved fields with TIOCSSERIAL, or create another ioctl.
quoted
quoted
And further, it says was RTS will do, instead of why. Maybe
someone could use it for something other than RS485 ?
I agree with you here. The name should reflect that RTS is
used in 'envelope' mode (asserted during data transmission,
idle between frames).
I don't really care what the name is.  "RTS toggle" or
"half-duplex" is what everybody I know calls it.
-- 
Laurent Pinchart
CSE Semaphore Belgium

Chaussee de Bruxelles, 732A
B-1410 Waterloo
Belgium

T +32 (2) 387 42 59
F +32 (2) 387 42 75

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help