Armadilloフォーラム

中村です。

すみません。訂正です。

> ioctl(fd, TIOCMBIS, TIOCM_RTS);

は、

int modem = TIOCM_RTS;
ioctl(fd, TIOCMBIS, &modem);

でした。

--
なかむら

中村です。

> ソースを見てませんが、たぶん、CRTSCTSがセットされていないときに
> RTSを真にする(activeのlowにする)という処理が抜けているのでしょう。

気になったので、ソースを見てみました。
どちらもimx_set_termios()の中です。

Armadillo-440のinux-3.14-at11/drivers/tty/serial/imx.c

        if (termios->c_cflag & CRTSCTS) {
                ...
        } else {
                if (port->rs485.flags & SER_RS485_ENABLED) {
                        /* disable transmitter */
                        if (!(port->rs485.flags & SER_RS485_RTS_AFTER_SEND))
                                ucr2 |= UCR2_CTS;
                } else {
                        ucr2 |= UCR2_CTS;
                }
        }

// UCR2_CTSがRS-232CのRTS信号になります

Armadillo-640のlinux-v4.14-at6/drivers/tty/serial/imx.c

        if (termios->c_cflag & CRTSCTS) {
                ...
        } else if (port->rs485.flags & SER_RS485_ENABLED) {
                /* disable transmitter */
                if (port->rs485.flags & SER_RS485_RTS_AFTER_SEND)
                        imx_port_rts_active(sport, &ucr2);
                else
                        imx_port_rts_inactive(sport, &ucr2);
        }

--
なかむら

中村です。

ソースのコピペミスの訂正です。

> imx.c全体を通してimx_port_rts_active()とimx_port_rts_inactive()が
> コールされているところを探すと、
> port->rs485.flags & SER_RS485_RTS_AFTER_SEND
> なときだけでした。(at_mizoさんのパッチ以外)

誤:
port->rs485.flags & SER_RS485_RTS_AFTER_SEND
正:
port->rs485.flags & SER_RS485_ENABLED
です。

port->rs485.flags は rs485conf->flags のときもあります。

--
なかむら

中村です。

RS-485の話なので、このスレッドの質問とは関係はないのですが、
先ほどの
> この部分が修正された経緯を調べました。
> これ(↓)のようです。
> [serial: imx: Fix the CTS_B polarity in RS485 mode]
> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit…
>
> 説明文を引用しておきます。
>
> When userspace passes the SER_RS485_RTS_ON_SEND flag it means that the
> CTS_B pin should go to logic level high before the transmission begins.
>
> CTS_B goes to logic level high when both CTSC and CTS bits are cleared.
>
> When userspace passes the SER_RS485_RTS_AFTER_SEND flag it means that the
> CTS_B pin should go to logic level low after the transmission finishes.
>
> CTS_B goes to logic level low when CTSC bit is cleared and CTS bit is set.
> So fix the CTS_B polarity logic.

の説明が何かおかしいような気がして、調べてみました。

> When userspace passes the SER_RS485_RTS_AFTER_SEND flag it means that the
> CTS_B pin should go to logic level low after the transmission finishes.
は、間違っているようです。

Documentation/serial/serial-rs485.txtのその部分：
| /* Set logical level for RTS pin equal to 1 when sending: */
| rs485conf.flags |= SER_RS485_RTS_ON_SEND;
| /* or, set logical level for RTS pin equal to 0 when sending: */
| rs485conf.flags &= ~(SER_RS485_RTS_ON_SEND);
|
| /* Set logical level for RTS pin equal to 1 after sending: */
| rs485conf.flags |= SER_RS485_RTS_AFTER_SEND;
| /* or, set logical level for RTS pin equal to 0 after sending: */
| rs485conf.flags &= ~(SER_RS485_RTS_AFTER_SEND);

で、"RTS pin equal to 1" (パッチの説明での"logic level high")といのが、
imx_port_rts_active()となるような実装に修正されたのが、
このパッチのようです。

２つ前の投稿に書いたようにG3系Armadilloのkernel-4.9での実装は
このパッチがあたる前のものなので、imx_port_rts_active()と
imx_port_rts_inactive()をコールする側が逆になってました。
たとえば、imx_start_tx()の中で、A-640の4.14では、

        if (port->rs485.flags & SER_RS485_ENABLED) {
                temp = readl(port->membase + UCR2);
                if (port->rs485.flags & SER_RS485_RTS_ON_SEND)
                        imx_port_rts_active(sport, &temp);
                else
                        imx_port_rts_inactive(sport, &temp);

となっているころがG3の4.9では

       if (port->rs485.flags & SER_RS485_ENABLED) {
                temp = readl(port->membase + UCR2);
                if (port->rs485.flags & SER_RS485_RTS_ON_SEND)
                        imx_port_rts_inactive(sport, &temp);
                else
                        imx_port_rts_active(sport, &temp);

となっています。

このactive()とinactive()が逆になったのは、
[serial: imx: Fix the CTS_B polarity in RS485 mode]
の１つ前に(同じ日に)コミットされている次(↓)の修正です。
[serial: imx: Fix the RTS GPIO polarity in RS485 mode]
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit…

この修正は、GPIOのRTSを使ったときに、
Documentation/serial/serial-rs485.txtに書いてあるのと逆になってるので
inactiveとactiveの呼び出しを逆にするように直しました、
ということみたいです。
この修正で、GPIOのRTS方はＯＫになったけれどもimxのCTS_Bピンを
使う方が逆になってしまったので、その次のコミットでCTS_Bピンの方も
直しました、とのことのようです。

それから、GPIOのRTSのときの処理で、A-640のimx_port_rts_active()が
sport->port.mctrl |= TIOCM_RTS;
となっていて、これはカーネルソースのgitものと一致しているのですが、
G3系のlinux-4.9-x1-at3のimx_port_rts_active()が
sport->port.mctrl &= ~TIOCM_RTS;
となっていて、この部分が逆になっています。
カーネルのgitの履歴にこの修正は見当たりませんでした。
上に書いたロジック反転の問題に対して、アットマークテクノさん独自の
修正がされたものなのかな？と思っています。

--
なかむら

松本です。

> ucr2のことですよね？ 変数"csr2"ではなくて。
> "CSR2レジスタ"は、UCR2レジスタ。
>
ごめんなさい、ご指摘のとおりUCR2です。

また、論理が逆ではとのご意見には私は賛同しません。
これは、いわゆる”（レガシーな）シリアル専用チップ”では制御線は負論理で入出力していました。
GPIOな入出力でであればソフト側で反転して読書き（読替え）をしてるのではないでしょうか？

さらに、
下記(添付）の修正を行うことで、残っていたRTSCTSの問題は解消したように動作しました。
なお、imx_set_mctrl()にてTIOCM_RTSをみてUCR2_CTSCも操作していましたがこれは誤りではと考えて修正しました。
今回TIMCM_DTRでRTS信号が影響を受けたのはこれが原因かと思います。TIOCM_DTRを０とするときだけimx_set_mctrl()が呼ばれ、偶然にも（imx_set_termios()のucs2変数でとり残された）UCR2のUCR2_CTSCが完了する方向に倒れたのだと思われます。

--- drivers/tty/serial/imx.c.save    2018-11-22 17:47:06.283371101 +0900
+++ drivers/tty/serial/imx.c    2018-11-26 10:39:12.261026518 +0900
@@ -899,10 +899,9 @@
     unsigned long temp;
 
     if (!(port->rs485.flags & SER_RS485_ENABLED)) {
-        temp = readl(sport->port.membase + UCR2);
-        temp &= ~(UCR2_CTS | UCR2_CTSC);
+        temp = readl(sport->port.membase + UCR2) & ~(UCR2_CTS);
         if (mctrl & TIOCM_RTS)
-            temp |= UCR2_CTS | UCR2_CTSC;
+            temp |= UCR2_CTS;
         writel(temp, sport->port.membase + UCR2);
     }
 
@@ -1564,7 +1563,7 @@
     old_ucr2 = readl(sport->port.membase + UCR2);
     writel(old_ucr2 & ~(UCR2_TXEN | UCR2_RXEN),
             sport->port.membase + UCR2);
-    old_ucr2 &= (UCR2_TXEN | UCR2_RXEN | UCR2_ATEN);
+    old_ucr2 &= (UCR2_TXEN | UCR2_RXEN | UCR2_ATEN | UCR2_CTSC);
 
     /* custom-baudrate handling */
     div = sport->port.uartclk / (baud * 16);