Thread (36 messages) 36 messages, 4 authors, 2022-06-30

Re: [PATCH 2/8] serial: core, 8250: set RS485 termination gpio in serial core

From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date: 2022-06-22 17:09:32
Also in: linux-devicetree, linux-serial, lkml

On Wed, Jun 22, 2022 at 05:46:53PM +0200, Lino Sanfilippo wrote:
From: Lino Sanfilippo <redacted>

In serial8250_em485_config() the termination GPIO is set with the uart_port
spinlock held. This is an issue if setting the GPIO line can sleep (e.g.
since the concerning GPIO expander is connected via SPI or I2C).

Fix this by setting the termination line outside of the uart_port spinlock
in the serial core.
This doesn't describe that this patch is actually changing GPIO to support
sleep mode. So, it doesn't fix anything. Please rephrase the commit message
accordingly.
This also makes setting the termination GPIO generic for all uart drivers.
UART

-- 
With Best Regards,
Andy Shevchenko



_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help