linux-kernel - Re: [PATCH] regulator: mcp16502: add enum MCP16502_REG

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: <c8a2c56c-b372-6888-404b-7f548e52275e@microchip.com>
Date:   Tue, 17 Jan 2023 08:32:27 +0000
From:   <Claudiu.Beznea@...rochip.com>
To:     <rdunlap@...radead.org>, <linux-kernel@...r.kernel.org>
CC:     <andrei.stefanescu@...rochip.com>, <broonie@...nel.org>,
        <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH] regulator: mcp16502: add enum MCP16502_REG_HPM
 description

On 17.01.2023 09:01, Randy Dunlap wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Add description for MCP16502_REG_HPM to prevent a kernel-doc warning:
> 
> drivers/regulator/mcp16502.c:90: warning: Enum value 'MCP16502_REG_HPM' not described in enum 'mcp16502_reg'
> 
> Fixes: 3e5532a011b0 ("regulator: mcp16502: adapt for get/set on other registers")

Fixes tag could be removed as this isn't a functional fix.

With that addressed:
Reviewed-by: Claudiu Beznea <claudiu.beznea@...rochip.com>

> Signed-off-by: Randy Dunlap <rdunlap@...radead.org>
> Cc: Andrei Stefanescu <andrei.stefanescu@...rochip.com>
> Cc: Claudiu Beznea <claudiu.beznea@...rochip.com>
> Cc: Mark Brown <broonie@...nel.org>
> Cc: linux-arm-kernel@...ts.infradead.org
> ---
>  drivers/regulator/mcp16502.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff -- a/drivers/regulator/mcp16502.c b/drivers/regulator/mcp16502.c
> --- a/drivers/regulator/mcp16502.c
> +++ b/drivers/regulator/mcp16502.c
> @@ -77,6 +77,7 @@
>   * @MCP16502_REG_A: active state register
>   * @MCP16502_REG_LPM: low power mode state register
>   * @MCP16502_REG_HIB: hibernate state register
> + * @MCP16502_REG_HPM: high-performance mode register
>   * @MCP16502_REG_SEQ: startup sequence register
>   * @MCP16502_REG_CFG: configuration register
>   */