Re: [PATCH v3 2/2] tty: serial: samsung_tty: cast the interrupt's void *id just once

From: Jiri Slaby
Date: Thu Aug 08 2024 - 04:35:40 EST


On 08. 08. 24, 10:11, André Draszik wrote:
The interrupt handler routines and helpers are casting the 'void *'
pointer to 'struct exynos_uart_port *' all over the place.

There is no need for that, we can do the casting once and keep passing
the 'struct exynos_uart_port *', simplifying the code and saving a few
lines of code.

No functional changes.

Reviewed-by: Tudor Ambarus <tudor.ambarus@xxxxxxxxxx>
Signed-off-by: André Draszik <andre.draszik@xxxxxxxxxx>
---
v3: undo too eager removal of 'const' where unnecessary (Jiri)

LGTM now

Reviewed-by: Jiri Slaby <jirislaby@xxxxxxxxxx>

v2: fix -Wdiscarded-qualifiers, sorry

thanks,
--
js
suse labs