linux-kernel - Re: [PATCH v2] mfd: max8997: Replace 8998 with 8997

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]

Date:   Sun, 04 Apr 2021 14:22:55 +0000
From:   Timon Baetz <timon.baetz@...tonmail.com>
To:     Lee Jones <lee.jones@...aro.org>
Cc:     Timon Baetz <timon.baetz@...tonmail.com>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] mfd: max8997: Replace 8998 with 8997

On Wed, 27 Jan 2021 07:32:42 +0000, Timon Baetz wrote:
> The max8997 header is using "max8998" in some identifiers.
> Fix it by replacing 8998 with 8997 in enum and macro.
>
> Signed-off-by: Timon Baetz <timon.baetz@...tonmail.com>
> ---
> v2: Fix commit message.
>
>  include/linux/mfd/max8997.h | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/include/linux/mfd/max8997.h b/include/linux/mfd/max8997.h
> index e955e2f0a2cc..6c98edcf4b0b 100644
> --- a/include/linux/mfd/max8997.h
> +++ b/include/linux/mfd/max8997.h
> @@ -14,13 +14,13 @@
>   * others and b) it can be enabled simply by using MAX17042 driver.
>   */
>
> -#ifndef __LINUX_MFD_MAX8998_H
> -#define __LINUX_MFD_MAX8998_H
> +#ifndef __LINUX_MFD_MAX8997_H
> +#define __LINUX_MFD_MAX8997_H
>
>  #include <linux/regulator/consumer.h>
>
>  /* MAX8997/8966 regulator IDs */
> -enum max8998_regulators {
> +enum max8997_regulators {
>  	MAX8997_LDO1 = 0,
>  	MAX8997_LDO2,
>  	MAX8997_LDO3,
> @@ -207,4 +207,4 @@ struct max8997_platform_data {
>  	struct max8997_led_platform_data *led_pdata;
>  };
>
> -#endif /* __LINUX_MFD_MAX8998_H */
> +#endif /* __LINUX_MFD_MAX8997_H */

Hi, is there an issues with this patch?