nvmem: qfprom: Improve the comment about regulator setting
authorDouglas Anderson <dianders@chromium.org>
Fri, 11 Jun 2021 08:33:48 +0000 (09:33 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 11 Jun 2021 08:57:58 +0000 (10:57 +0200)
In review feedback Joe Perches found the existing comment
confusing. Let's use something based on the wording proposed by Joe.

Suggested-by: Joe Perches <joe@perches.com>
Signed-off-by: Douglas Anderson <dianders@chromium.org>
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Link: https://lore.kernel.org/r/20210611083348.20170-10-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/nvmem/qfprom.c

index 1ba666b..81fbad5 100644 (file)
@@ -196,9 +196,9 @@ static int qfprom_enable_fuse_blowing(const struct qfprom_priv *priv,
        }
 
        /*
-        * Hardware requires a min voltage for fuse blowing; this may be
-        * a rail shared do don't specify a max--regulator constraints
-        * will handle.
+        * Hardware requires a minimum voltage for fuse blowing.
+        * This may be a shared rail so don't specify a maximum.
+        * Regulator constraints will cap to the actual maximum.
         */
        ret = regulator_set_voltage(priv->vcc, qfprom_blow_uV, INT_MAX);
        if (ret) {