Received-SPF: pass (google.com: domain of linux-kernel+bounces-40432-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: Farouk Bouabid <farouk.bouabid@theobroma-systems.com>
Subject: [PATCH v5 0/6] serial: 8250: Add support for rs485 half/full
 duplex on puma/ringneck-haikou
Date: Fri, 26 Jan 2024 18:27:41 +0100
Message-Id: <20240126-dev-rx-enable-v5-0-5d934eda05ca@theobroma-systems.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, 
 Jiri Slaby <jirislaby@kernel.org>, Rob Herring <robh+dt@kernel.org>, 
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, 
 Conor Dooley <conor+dt@kernel.org>, Heiko Stuebner <heiko@sntech.de>
Cc: Rob Herring <robh@kernel.org>, linux-kernel@vger.kernel.org, 
 linux-serial@vger.kernel.org, devicetree@vger.kernel.org, 
 linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, 
 quentin.schulz@theobroma-systems.com, 
 Heiko Stuebner <heiko.stuebner@cherry.de>, 
 Farouk Bouabid <farouk.bouabid@theobroma-systems.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?U1lHNlEyMTk5MUdhK2VTdzFzOVpMdGQ1a2hDazM2RFl3WkJ4YjZqQzEzM2x1?=
 =?utf-8?B?aWo0N2RCVkFUMGxuVTZqbUJEd2pXU2JFVDI5NXBRaWZXVkVFNi9Dc2c5NUxX?=
 =?utf-8?B?cFlCSWpXNkRBUStzUytkc2t4cnJrUWlZekJHVUZoME1CK2hMeXBSUU1pV3dq?=
 =?utf-8?B?bnNhbFlHRmw5QUJQUlhTejNmUHRXZUl3MmNJUnNsNHFKOGh5U0hRMStkcHJ2?=
 =?utf-8?B?N3VlK3JCemM2R0RXbHFRTVFWbkVIUGNna3pmRkErRU1qNisvUEJqbVZLd2Vm?=
 =?utf-8?B?cHJZZ3NiU0JudjN5KzB0MUlCMU9qVW9JS1hBdGxnN0hQSjFKeWpseEJGNVIy?=
 =?utf-8?B?c054c25KZGRncENpUVJPWHAwYWFWVlYvNGRMY1h1endqYWNiTXg3Q1U5dkZq?=
 =?utf-8?B?Tjhxam5SalNKalFBSnJTSjZYZExpczNyRUo2aWMvd0x1ZGdJWHlFQWdwTEpV?=
 =?utf-8?B?UW5ZdFRPQUltc1JuTXFSNENKcmxQUDlDRkFjeEsrSnR4azNlcHF4Sk1KbnB4?=
 =?utf-8?B?SVJ0WCtMSHFFQmV5UXU0cGswRytFTmZFdjJic244RGg1ZzAxcDN6ZXNmZExD?=
 =?utf-8?B?VGJsb3hvM3YvOHAyLzhQM0MwcHpkZzF5dk10SmNncXpqSU5yZzNadkc5amFG?=
 =?utf-8?B?YWh5U05yM0lzL1VpSkZ1QUNrQ2JWZzNSNFJOZEozdEppWnJzUHNiK3ByYnNt?=
 =?utf-8?B?eEQvV2R3MTVnMHBVem9xdWtFb0FKek1MWFVPcmRPUHN4b1Z1Z0x6UGllbWJz?=
 =?utf-8?B?Vk42a0FrSzhkT21SUE9QTTFKZGQ1NzBuQm1NWmFSbzhucDBtd01lRDVYVHZx?=
 =?utf-8?B?OW1YVUl5cXMyRy8xaTFWbDFHTjVkYlVOL1FPbysway9KUHBZWmpUaW4zeDA2?=
 =?utf-8?B?UTZjbFhQcTB0bXBRczFyWnhuTEFtbjBBSHpPc1VLN0RXeTdtZ2YyNHdvRW50?=
 =?utf-8?B?ZHI4Q0M0T2dWd0o3U2xFQndtSTZ0eWgzQ3FhTjlRWUtXZHNiT3hUUGx3OWpm?=
 =?utf-8?B?Wm1pRFhoYlJqZWp5SzJEd1lZRk93T2VPTTBMMHpuQ3NWc0JKZklsYlZtTlpu?=
 =?utf-8?B?Nlh3R2JDMFNJWTNCR2hmL0xuVXVreE5QL0pnbTN2bVlieFpRRE1COTVIbGpU?=
 =?utf-8?B?YUdFT0w0U0RxdzdyaGN4YWdLeHd4b1JhMFByVnlTK0tyTFpEbEtWWHUxblo1?=
 =?utf-8?B?cVkzV3BaaFZENlBoMFFqZFdlU09zRml1aHo4QUt2VUNhWmRTc1FnTTRnTG83?=
 =?utf-8?B?YWdGTFVnMXo0cy9LdGtQSGVJQjM3eERiMER2dW9NdWw2cHVHaDZUbVdHdmZI?=
 =?utf-8?B?SXV5UDNoM29MYmV2cHhoTmRHOENzYUQwWWIrY2tZbUJ5bFZqM2MzZWNQTTht?=
 =?utf-8?B?NzZIZWNRRi8wb0JoU1ZwOGQ2N0NyRVNWdXA0YXV6eUxac2ptUkxHWUp2Slp5?=
 =?utf-8?B?ZDZnVUc1bjRYT0tOdS9vd08yYm5wZ3ZSZXl5NU1wVjJIMmdWd3pKSWRiN0hk?=
 =?utf-8?B?T0xQWnpRRDQwUGFPYjNnOHdodk9sS1lxenpubGxQaWcwRWVCUnVya2RVbVlZ?=
 =?utf-8?B?YWVUZHRrekZmc3hYc3F6MVY5NUJXR0N0TjFLczdWR2dIQ1hzK0FoU1o4R3g0?=
 =?utf-8?B?R3JmN2tLcGp2aDZIV0JsaXRIbkZ3Z2RONVFVQjBTZlkySTk5MUdibjh5YTJF?=
 =?utf-8?B?ODZmK3dvT3d0TUx3MVZGdVVLcDdHWUcrWkZubUtkRyttK21CdU03N1JVanhk?=
 =?utf-8?B?UkcxQlQzbVhORVU4TFdXRlBVcUlnSGdDUDdCOVpzVld3OTB6MzJsQzZrMWM1?=
 =?utf-8?B?RHo0Z2NaQ0tKbXlTR1lUMHUrMVNNbGZqQ1Bva0dxMDQxSStxN2lKUUpaN2RN?=
 =?utf-8?B?WDh2cVRNcVpLelJUc2xFUGtNcU03eGVGRzZCMFNPL09TdHlKb2oyODBGUDJP?=
 =?utf-8?B?bWJ6MHpBY3dkRHNRY2JTcWtqUEtpWS93L09sWjVWeEl0cWVpTGVmaWdRM2xm?=
 =?utf-8?B?empOMnFCZ29LY080UnBPQ2ZFa01xVDI1N2xxaElFNnpMNFJ6cUZUQXFmL3hY?=
 =?utf-8?B?QWE5TnRVeUh4azVkcWNRd3czeU4ycnhjakJja0pTVVF6cGNFMGlkNmFBYzhu?=
 =?utf-8?B?bFoyQmFZQkVYUlExd3pBdDJyWmgzQWtPMVR4QzBZUm85ZDVKNDNNZC9aNHQw?=
 =?utf-8?Q?GpXgn5MSPh37VL9X2xDZH8k=3D?=
X-OriginatorOrg: theobroma-systems.com
X-MS-Exchange-CrossTenant-Network-Message-Id: acdd3803-7402-4637-e53f-08dc1e943910
X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6382.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jan 2024 17:28:34.2700
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 5e0e1b52-21b5-4e7b-83bb-514ec460677e
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: WSe4Lkdte9WQccOXkUjtutcWYxVu0ilDiyWffXLVNnCaV3c/vug6YeLY0ZWb4VKLM3AW3v6UCnJbQD+cnHWTWJl2zAvcf4/Q9UPn9xe7Nkxf2tixY6J55WkxZc3fbZYQ
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB9820

This series tries to revive the work of Heiko Stuebner from 2020

On the boards that we are using (ringneck/puma-haikou) a hardware switch
can set the rs485 transceiver into half or full duplex mode.

In half-duplex mode the DE/RE signal of the rs485 transceiver is not
connected to an RTS signal whose control is already handled in the rs485
emulation (start/stop callbacks), but rather to a gpio. And since enabling
the receiver requires setting this gpio active we need to do that in em485
while receiving and disable it while sending to enable the driver mode.

In full-duplex mode RE is grounded and separated  from DE. Meanwhile the
rx-enable gpio remains connected to the DE pin. In this case the
receiver-enable gpio should be disabled to enable driver mode in parallel
to the enabled receiver.

This patch-series adds support for controlling the receiver mode using a
gpio in em485 for half-duplex mode while allowing users to keep using the
full-duplex feature if em485 is disabled.

Changes in v5:
- set port->rs485_re_gpio in "serial: 8250: Support separate rs485 rx-enable GPIO"
- Link to v4: https://lore.kernel.org/r/20240126-dev-rx-enable-v4-0-45aaf4d96328@theobroma-systems.com

Changes in v4:
- define the state of rx-enable gpio when em485 is disabled
- add rs485 half/full duplex support to ringneck/puma-haikou
- use dev_err_probe instead of dev_err if error is -EPROBE_DEFER

Changes from the 2020 submission include:
- external gpio for optional receiver-enable handling
- Link to v3: https://lore.kernel.org/all/20200517215610.2131618-1-heiko@sntech.de/

---
Farouk Bouabid (4):
      dt-bindings: serial: add binding for rs485 rx-enable state when rs485 is disabled
      serial: 8250: set rx-enable gpio state when rs485 is disabled
      arm64: dts: rockchip: rk3399-puma-haikou: add rs485 support on uart2
      arm64: dts: rockchip: px30-ringneck-haikou: add rs485 support on uart5

Heiko Stuebner (2):
      dt-bindings: serial: Add binding for rs485 receiver enable GPIO
      serial: 8250: Support separate rs485 rx-enable GPIO

 Documentation/devicetree/bindings/serial/rs485.yaml   |  9 +++++++++
 arch/arm64/boot/dts/rockchip/px30-ringneck-haikou.dts |  2 ++
 arch/arm64/boot/dts/rockchip/rk3399-puma-haikou.dts   |  4 +++-
 drivers/tty/serial/8250/8250_port.c                   | 11 ++++++++++-
 drivers/tty/serial/serial_core.c                      | 11 +++++++++++
 include/linux/serial_core.h                           |  2 ++
 6 files changed, 37 insertions(+), 2 deletions(-)
---
base-commit: 5ebe731c2a586b379103f736cd498bcca3cf1ea9
change-id: 20240125-dev-rx-enable-d8818dbf7c28

Best regards,
-- 
Farouk Bouabid <farouk.bouabid@theobroma-systems.com>