Re: [PATCH] regulator: mcp16502: add enum MCP16502_REG_HPM description

From: Claudiu.Beznea
Date: Tue Jan 17 2023 - 03:32:43 EST


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@xxxxxxxxxxxxx>

> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Cc: Andrei Stefanescu <andrei.stefanescu@xxxxxxxxxxxxx>
> Cc: Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx>
> Cc: Mark Brown <broonie@xxxxxxxxxx>
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> ---
> 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
> */