Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <9d5cff18-5493-f6dd-4bd6-9bafa2a503a7@camlingroup.com>
Date:   Tue, 15 Feb 2022 07:03:18 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.5.0
Subject: Re: [PATCH] serial: imx: leave IRTS disabled if using modem-control
 CTS
To:     Tim Harvey <tharvey@gateworks.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Jiri Slaby <jirislaby@kernel.org>,
        Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        NXP Linux Team <linux-imx@nxp.com>,
        linux-serial@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org
Cc:     Richard Genoud <richard.genoud@gmail.com>,
        Sergey Organov <sorganov@gmail.com>
References: <20220214213020.685-1-tharvey@gateworks.com>
From:   =?UTF-8?Q?Tomasz_Mo=c5=84?= <tomasz.mon@camlingroup.com>
In-Reply-To: <20220214213020.685-1-tharvey@gateworks.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?UVBXcjJVYm1mYWtHR1dtNDZseCtSeUFCNlBYelp2dlVYazdISHJ2SENrK2tm?=
 =?utf-8?B?Yy9LZC9DUTZiSy9aYnppc3RubTF2M3VHYzV5S1FJSlpTT2g1aG1aczR5Tk5H?=
 =?utf-8?B?YWc0R3Jqa3kwRkM4TzZUSk9zVC9lTUIyeUhrZmZqeEx3ak5qV2cvUGpZeVR1?=
 =?utf-8?B?VnFabEN1aWxEVEx0R2RmK1p6d3hiOGhlQVpDQ3F3RjRiRFFkUmMrQzc2K0U1?=
 =?utf-8?B?UWFXNDlETVU2WW9LQ2R5cWJhUGZGVUsybGdUdHIrd000QWV2OXlyaGt1Z25n?=
 =?utf-8?B?Vk5kQ2lYb3NiaU9DM0pORXI3MDJmVUhTbTFlS1BpdkFZWTdHYU5zVXE4d2Na?=
 =?utf-8?B?cy8xM0dwYUd2NUQwd1Y5VHl6VGpCWERxcGk5eml2ckxTR3Z0dHJPUXNOZlk2?=
 =?utf-8?B?cVJNMXpBVDRITy9UMytGZWQwbzRxSVM3cnVzMTZRdmFCdTBOcmxJY0lOb0xh?=
 =?utf-8?B?eVpoOGo1SW5rT2xRMmdRYlRwSXNHYmlKYm5PN0FiOHRJQmRtR3hhU2JsWXht?=
 =?utf-8?B?cHpMd2c0Q21CTEwwTDdtQTRYRDJ2eG52YmVtb2cvam9IaVpna3pLSmFmSlNN?=
 =?utf-8?B?UmpyN3F5ZFRuTWNmU3p6UGhsNmJDUGtFUXpUM2pKaDBXS2Z4emc4M0t3VWpM?=
 =?utf-8?B?RGdobk9rS3NnNm8wSjBrTFlvZnFpOEVMUWxkdTlwOFJYQlRNY0JZUlRXQzVZ?=
 =?utf-8?B?ZUNHaWJueXlIcmpkWitQNXE5bXdUWTB2cFk4cW9iVXBaT3Vmck5VYXNjaDVq?=
 =?utf-8?B?R3VMb3UrM2pIM3o4R1UvWjZXNFdNZjJ1V0hCQjlYN3IycmY5cVZ2V29DTTMy?=
 =?utf-8?B?RUYvaURuSE5xMncyakU0Q05uVUpNcUUwb25WWlpsc296RHNLeWY5OHhWaWZ0?=
 =?utf-8?B?ZHNjUk9XMXpsSFQxZk8xRVVLUHpvQnV6QXNCcCtFeit6SkU3WGNnUGNPWXJU?=
 =?utf-8?B?LzlYV3AzYlVJaXc5eEwvQ3VhWGVXQUMrS21WekQvdXhTNmU3RzYwYnlXSkpv?=
 =?utf-8?B?b215U0o0YnlhZ2ttdmFlVm5DbFpaZ2M1SkwzZVZLUE1XVlNiUldKWm93cHNN?=
 =?utf-8?B?ZGJUZVcwempIb09lWTdsTFVOMFZtVlVBVWgrNXloR0pYQXpoRTZhN3NzL084?=
 =?utf-8?B?L1dFbGJFeXkyQmlhNkdEenpZS28zcHJVRTJOMTMzZHI0dWRFL2d5UU5wSXZ3?=
 =?utf-8?B?TG9vdTFSdWI2WVNuYW1YQTBEVUc4WGQyUXd2ait0dWZ1SmZWR0N3cjZRaHQw?=
 =?utf-8?B?NEM0bEpZY2hZaWkzQ0FpaE5TZmkwRGpWS1FnMkM1NVpYZTVDZVBLbmIvUVhE?=
 =?utf-8?B?d1FWSnFaeEtWZmFCLysxeDN3b3Npc3BQeVJ0WUYrU01FVk5EZ0xNZ1ErbzIy?=
 =?utf-8?B?MDRreWRQeitUNmRoSFE5Z0ErcU5YeGhSTkNJWnFlanNZd3BBV2g2WnpEWGEz?=
 =?utf-8?B?bnk1cGlrRk5ZaHhqd25reFNnaGg3M2ZSNWZXTFU3MTJGOFVPeDhGY0JNZzhz?=
 =?utf-8?B?cFYraEpDdG0wVUpyeWpFaURPQkliOW14enhqSUdDQnYwVUJReE9NRWxYbVZt?=
 =?utf-8?B?ZGtVMFA5dUpKSnc1b0RBN3RUYmwrKzVRSldRdWZEMFY4MDYrNjBmbzFZN1gy?=
 =?utf-8?B?OHFWTmhuK2lmT2hZL0JRRElHQnJ2Qmdidm5QWDV3WUdYNklJdXJCNWFSUUZ3?=
 =?utf-8?B?UCs1alVsWjR1b2E5b05Pd2ZCUytKWmFMemNxajFveUVFcFVTdlNFUndSV1lv?=
 =?utf-8?B?aXNZbEtmOHEvWi9uUThKSVNXdnkxS1lrbHlodjNEUFZQVnNXZ3dES2lSNmtV?=
 =?utf-8?B?VTRvTHh5VXNGMWlLU3duT1p1S2R3elc3M1JZSTVmeTVDOEtiUnFSUE43SE5h?=
 =?utf-8?B?Y3J3dXFwdGNCQ3dxb3pldEluRHBqVDQ2N3RLSm5jNDdybXk4dFNzR3RoekJl?=
 =?utf-8?B?TmxkUlpTWkxFaFNKekVSSzdJejU1M0xJaWoyc0ttN09sbXEwSXBFNFJ4dTFR?=
 =?utf-8?B?VjBpRStuZlZlQndUWnd4eis4MHg5aG5mSjJibk9EeE5IUzBqUmNDekU0NTFs?=
 =?utf-8?B?ZmxDRkJ6QnQ2YUdyVm5pcDFxcTlZQk5uQmhmeWw2YUsyM0J0UHEzcGxHSjVJ?=
 =?utf-8?B?bDZ0VFFFVkJqNWhlRVVwbzZIL29BdGhQK0EyL2R4NUxhSjladWUxWUNVang3?=
 =?utf-8?B?cXl2NE16K0lPaFpYNDIxSHZGWHRaT0NqT2JxMzNQZiswK2VBN0l4dEl1OW1V?=
 =?utf-8?Q?q4YfRTxIHXkg2OCbJfm4gqybhnEjQj1P5dgxXFh8Xk=3D?=
X-OriginatorOrg: camlingroup.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0e9d4781-2ac2-43fb-39cc-08d9f048de74
X-MS-Exchange-CrossTenant-AuthSource: CWLP123MB5572.GBRP123.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Feb 2022 06:03:21.1502
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: fd4b1729-b18d-46d2-9ba0-2717b852b252
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: EBWdemYB3qtU+78GG6Ela4BpYLCBR5SQahhSeFgdH9nIybDCHSCA5jcwuef90nwKiASaU3Hsv0g9if4cwbR0On3ENN/TUzwWSw7g1EYbM/Y=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: LO2P123MB3774
Authentication-Results: relay.mimecast.com;
        auth=pass smtp.auth=CUK97A341 smtp.mailfrom=tomasz.mon@camlingroup.com
X-Mimecast-Spam-Score: 0
X-Mimecast-Originator: camlingroup.com
Content-Language: en-US
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_LOW,
        SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 14.02.2022 22:30, Tim Harvey wrote:
> If using modem-control gpios for CTS we must leave IRTS disabled
> as otherwise the hardware will only transmit based on the internal RTS
> pin routed to it.
>=20
> This allows hardware flow control to be used with cts-gpios.

This hardware flow control sounds quite limited. Once CTS becomes
inactive, the transmitter will still output all characters from TxFIFO.
Transmitting whole TxFIFO already sounds quite bad, but that's the best
case scenario where gpio interrupt is handled right away without any
delay (so more than TxFIFO characters can actually be transmitted).

Does the internal RTS default to inactive when it's not pinmuxed to the
actual pin? If so, then controlling UCR2_IRTS based on CTS gpio could
halt the transmission when the TxFIFO is not empty.

Best Regards,
Tomasz Mon