linux-kernel - Re: [PATCH] accessibility/speakup/speakup_acnt: Add header file macro definition

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20220705111621.smcxnfx45ai7vixy@begin>
Date:   Tue, 5 Jul 2022 13:16:21 +0200
From:   Samuel Thibault <samuel.thibault@...-lyon.org>
To:     Li zeming <zeming@...china.com>
Cc:     w.d.hubbs@...il.com, chris@...-brannons.com, kirk@...sers.ca,
        speakup@...ux-speakup.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] accessibility/speakup/speakup_acnt: Add header file
 macro definition

Hello,

Li zeming, le mar. 05 juil. 2022 18:19:55 +0800, a ecrit:
> I think the header file could avoid redefinition errors.
>  at compile time by adding macro definitions.

Redefining a macro to the same value is not a problem, but better have it
indeed.

> Signed-off-by: Li zeming <zeming@...china.com>

Reviewed-by: Samuel Thibault <samuel.thibault@...-lyon.org>

Could you also patch speakup_dtlk.h the same way? Thanks.

Samuel

> ---
>  drivers/accessibility/speakup/speakup_acnt.h | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/accessibility/speakup/speakup_acnt.h b/drivers/accessibility/speakup/speakup_acnt.h
> index cffa938ae580..cea05d770f6d 100644
> --- a/drivers/accessibility/speakup/speakup_acnt.h
> +++ b/drivers/accessibility/speakup/speakup_acnt.h
> @@ -1,5 +1,7 @@
>  /* SPDX-License-Identifier: GPL-2.0 */
>  /* speakup_acntpc.h - header file for speakups Accent-PC driver. */
> +#ifndef _SPEAKUP_ACNT_H
> +#define _SPEAKUP_ACNT_H
>  
>  #define SYNTH_IO_EXTENT	0x02
>  
> @@ -17,3 +19,4 @@
>  #define SYNTH_FULL	'F' /* synth is full. */
>  #define SYNTH_ALMOST_EMPTY 'M' /* synth has less than 2 seconds of text left */
>  #define SYNTH_SPEAKING	's' /* synth is speaking and has a fare way to go */
> +#endif
> -- 
> 2.18.2
>