Re: [PATCH] serdev: BREAK/FRAME/PARITY/OVERRUN notification prototype V2

On Sun, Dec 12, 2021 at 10:21:28PM +0900, Magnus Damm wrote:

From: Magnus Damm <redacted>

Allow serdev device drivers get notified by hardware errors such as BREAK,
FRAME, PARITY and OVERRUN.

With this patch, in the event of an error detected in the UART device driver
the serdev_device_driver will get the newly introduced ->error() callback
invoked if serdev_device_set_error_mask() has previously been used to enable
the type of error. The errors are taken straight from the TTY layer and fed
into the serdev_device_driver after filtering out only enabled errors.

Without this patch the hardware errors never reach the serdev_device_driver.

Signed-off-by: Magnus Damm <redacted>
---

 Applies to linux-5.16-rc4

 Change since V1:
 - Use __set_bit() instead of set_bit() in ttyport_receive_buf()
 - Switch to assign_bit() in ttyport_set_error_mask()

 Thanks to Geert for feedback!

 The following prototype patch is using serdev error notifications:
 [PATCH] r8a77995 Draak SCIF0 LED and KEY Serdev prototype V2

Looks good, now applied to my tty tree.

thanks,

greg k-h