Le 06/12/2023 à 08:36, Jiri Slaby (SUSE) a écrit :
> Switch character types to u8 and sizes to size_t. To conform to
> characters/sizes in the rest of the tty layer.
> 
> Signed-off-by: Jiri Slaby (SUSE) <jirisl...@kernel.org>
> Cc: Michael Ellerman <m...@ellerman.id.au>
> Cc: Nicholas Piggin <npig...@gmail.com>
> Cc: Christophe Leroy <christophe.le...@csgroup.eu>
> Cc: Amit Shah <a...@kernel.org>
> Cc: Arnd Bergmann <a...@arndb.de>
> Cc: Paul Walmsley <paul.walms...@sifive.com>
> Cc: Palmer Dabbelt <pal...@dabbelt.com>
> Cc: Albert Ou <a...@eecs.berkeley.edu>
> Cc: linuxppc-...@lists.ozlabs.org
> Cc: virtualizat...@lists.linux.dev
> Cc: linux-ri...@lists.infradead.org
> ---
>   arch/powerpc/include/asm/hvconsole.h       |  4 ++--
>   arch/powerpc/include/asm/hvsi.h            | 18 ++++++++--------
>   arch/powerpc/include/asm/opal.h            |  8 +++++---
>   arch/powerpc/platforms/powernv/opal.c      | 14 +++++++------
>   arch/powerpc/platforms/pseries/hvconsole.c |  4 ++--
>   drivers/char/virtio_console.c              | 10 ++++-----
>   drivers/tty/hvc/hvc_console.h              |  4 ++--
>   drivers/tty/hvc/hvc_dcc.c                  | 24 +++++++++++-----------
>   drivers/tty/hvc/hvc_iucv.c                 | 18 ++++++++--------
>   drivers/tty/hvc/hvc_opal.c                 |  5 +++--
>   drivers/tty/hvc/hvc_riscv_sbi.c            |  9 ++++----
>   drivers/tty/hvc/hvc_rtas.c                 | 11 +++++-----
>   drivers/tty/hvc/hvc_udbg.c                 |  9 ++++----
>   drivers/tty/hvc/hvc_vio.c                  | 18 ++++++++--------
>   drivers/tty/hvc/hvc_xen.c                  | 23 +++++++++++----------
>   drivers/tty/hvc/hvsi_lib.c                 | 20 ++++++++++--------
>   16 files changed, 107 insertions(+), 92 deletions(-)
> 
> diff --git a/arch/powerpc/include/asm/hvconsole.h 
> b/arch/powerpc/include/asm/hvconsole.h
> index ccb2034506f0..d841a97010a0 100644
> --- a/arch/powerpc/include/asm/hvconsole.h
> +++ b/arch/powerpc/include/asm/hvconsole.h
> @@ -21,8 +21,8 @@
>    * Vio firmware always attempts to fetch MAX_VIO_GET_CHARS chars.  The 
> 'count'
>    * parm is included to conform to put_chars() function pointer template
>    */
> -extern int hvc_get_chars(uint32_t vtermno, char *buf, int count);
> -extern int hvc_put_chars(uint32_t vtermno, const char *buf, int count);
> +extern ssize_t hvc_get_chars(uint32_t vtermno, u8 *buf, size_t count);
> +extern ssize_t hvc_put_chars(uint32_t vtermno, const u8 *buf, size_t count);

Would be a good opportunity to drop this pointless deprecated 'extern' 
keyword on all function prototypes you are changing.

Christophe

Reply via email to