On Wed, Jun 11, 2025 at 12:02:54PM +0200, Jiri Slaby (SUSE) wrote:
> uart_port::{serial_in,serial_out} (and plat_serial8250_port::* likewise)
> historically use:
> * 'unsigned int' for 32-bit register values in reads and writes, and
> * 'int' for offsets.
>
> Make them sane such that:
> * 'u32' is used for register values, and
> * 'unsigned int' is used for offsets.
>
> While at it, name hooks' parameters, so it is clear what is what.
At a glance this looks just mechanical change. Have you used coccinelle for
that?
As for the whole idea, I like it,
Reviewed-by: Andy Shevchenko <andriy.shevche...@linux.intel.com>
> Signed-off-by: Jiri Slaby (SUSE) <jirisl...@kernel.org>
...
> Cc: Madhavan Srinivasan <ma...@linux.ibm.com>
> Cc: Michael Ellerman <m...@ellerman.id.au>
> Cc: Nicholas Piggin <npig...@gmail.com>
> Cc: Christophe Leroy <christophe.le...@csgroup.eu>
> Cc: "Ilpo Järvinen" <ilpo.jarvi...@linux.intel.com>
> Cc: Andy Shevchenko <andriy.shevche...@linux.intel.com>
> Cc: Paul Cercueil <p...@crapouillou.net>
> Cc: Vladimir Zapolskiy <v...@mleia.com>
> Cc: Kunihiko Hayashi <hayashi.kunih...@socionext.com>
> Cc: Masami Hiramatsu <mhira...@kernel.org>
These also can be moved under '---' :-)
> ---
> Cc: linuxppc-dev@lists.ozlabs.org
> Cc: linux-m...@vger.kernel.org
> Cc: linux-arm-ker...@lists.infradead.org
> ---
--
With Best Regards,
Andy Shevchenko
