Re: [PATCH] linux-user/sh4/termbits: Silence warning about TIOCSER_TEMT double definition

[Yoshinori Sato]

On Wed, 30 Mar 2022 22:43:02 +0900,
Thomas Huth wrote:
> 
> Seen while compiling on Alpine:
> 
>  In file included from ../linux-user/strace.c:17:
>  In file included from ../linux-user/qemu.h:11:
>  In file included from ../linux-user/syscall_defs.h:1247:
>  ../linux-user/sh4/termbits.h:276:10: warning: 'TIOCSER_TEMT' macro redefined
>   [-Wmacro-redefined]
>  # define TIOCSER_TEMT    0x01   /* Transmitter physically empty */
>           ^
>  /usr/include/sys/ioctl.h:50:9: note: previous definition is here
>  #define TIOCSER_TEMT 1
>          ^
>  1 warning generated.
> 
> Add the TARGET_ prefix here, too, like we do it on the other architectures.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  linux-user/sh4/termbits.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/linux-user/sh4/termbits.h b/linux-user/sh4/termbits.h
> index f91b5c51cf..eeabd2d7a9 100644
> --- a/linux-user/sh4/termbits.h
> +++ b/linux-user/sh4/termbits.h
> @@ -273,7 +273,7 @@ ebugging only */
>  #define TARGET_TIOCSERGETLSR   TARGET_IOR('T', 89, unsigned int) /* 0x5459 
> */ /* Get line sta
>  tus register */
>    /* ioctl (fd, TIOCSERGETLSR, &result) where result may be as below */
> -# define TIOCSER_TEMT    0x01   /* Transmitter physically empty */
> +# define TARGET_TIOCSER_TEMT   0x01   /* Transmitter physically empty */
>  #define TARGET_TIOCSERGETMULTI TARGET_IOR('T', 90, int) /* 0x545A
>  */ /* Get multiport config  */
>  #define TARGET_TIOCSERSETMULTI TARGET_IOW('T', 91, int) /* 0x545B
> -- 
> 2.27.0
> 

Reviewed-by: Yoshinori Sato <ysato@users.sourceforge.jp>

-- 
Yosinori Sato