staging: hikey9xx: hi6421v600-regulator: cleanup debug msgs

While those were useful during port time from downstream
version, let's get rid of them for good, as it is possible to
get about the same things by enabling regulator debugging code.

Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Link: https://lore.kernel.org/r/be1b704fcba9f1dd0559174835f1e5390df1cf94.1611212783.git.mchehab+huawei@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/hikey9xx/hi6421v600-regulator.c b/drivers/staging/hikey9xx/hi6421v600-regulator.c
index 72e30159673534b8eacd7b5bceffb8849dc4c514..54ad07418ec28a96832644595c3e7f9fc2c76ae0 100644 (file)
--- a/drivers/staging/hikey9xx/hi6421v600-regulator.c
+++ b/drivers/staging/hikey9xx/hi6421v600-regulator.c
@@ -23,9 +23,6 @@
 #include <linux/regulator/driver.h>
 #include <linux/spmi.h>
 
-#define rdev_dbg(rdev, fmt, arg...)    \
-                pr_debug("%s: %s: " fmt, (rdev)->desc->name, __func__, ##arg)
-
 struct hi6421_spmi_reg_info {
        struct regulator_desc   desc;
        struct hi6421_spmi_pmic *pmic;
@@ -112,11 +109,6 @@ static int hi6421_spmi_regulator_is_enabled(struct regulator_dev *rdev)
 
        reg_val = hi6421_spmi_pmic_read(pmic, rdev->desc->enable_reg);
 
-       rdev_dbg(rdev,
-                "enable_reg=0x%x, val= 0x%x, enable_state=%d\n",
-                rdev->desc->enable_reg,
-                reg_val, (reg_val & rdev->desc->enable_mask));
-
        return ((reg_val & rdev->desc->enable_mask) != 0);
 }
 
@@ -131,11 +123,6 @@ static int hi6421_spmi_regulator_enable(struct regulator_dev *rdev)
                     HISI_REGS_ENA_PROTECT_TIME + 1000);
 
        /* set enable register */
-       rdev_dbg(rdev,
-                "off_on_delay=%d us, enable_reg=0x%x, enable_mask=0x%x\n",
-                rdev->desc->off_on_delay, rdev->desc->enable_reg,
-                rdev->desc->enable_mask);
-
        hi6421_spmi_pmic_rmw(pmic, rdev->desc->enable_reg,
                             rdev->desc->enable_mask,
                             rdev->desc->enable_mask);
@@ -151,9 +138,6 @@ static int hi6421_spmi_regulator_disable(struct regulator_dev *rdev)
        struct hi6421_spmi_pmic *pmic = sreg->pmic;
 
        /* set enable register to 0 */
-       rdev_dbg(rdev, "enable_reg=0x%x, enable_mask=0x%x\n",
-                rdev->desc->enable_reg, rdev->desc->enable_mask);
-
        hi6421_spmi_pmic_rmw(pmic, rdev->desc->enable_reg,
                             rdev->desc->enable_mask, 0);
 
@@ -164,19 +148,12 @@ static int hi6421_spmi_regulator_get_voltage_sel(struct regulator_dev *rdev)
 {
        struct hi6421_spmi_reg_info *sreg = rdev_get_drvdata(rdev);
        struct hi6421_spmi_pmic *pmic = sreg->pmic;
-       u32 reg_val, selector;
+       u32 reg_val;
 
        /* get voltage selector */
        reg_val = hi6421_spmi_pmic_read(pmic, rdev->desc->vsel_reg);
 
-       selector = (reg_val & rdev->desc->vsel_mask) >> (ffs(rdev->desc->vsel_mask) - 1);
-
-       rdev_dbg(rdev,
-                "vsel_reg=0x%x, value=0x%x, entry=0x%x, voltage=%d mV\n",
-                rdev->desc->vsel_reg, reg_val, selector,
-               rdev->desc->ops->list_voltage(rdev, selector) / 1000);
-
-       return selector;
+       return (reg_val & rdev->desc->vsel_mask) >>     (ffs(rdev->desc->vsel_mask) - 1);
 }
 
 static int hi6421_spmi_regulator_set_voltage_sel(struct regulator_dev *rdev,
@@ -192,11 +169,6 @@ static int hi6421_spmi_regulator_set_voltage_sel(struct regulator_dev *rdev,
        reg_val = selector << (ffs(rdev->desc->vsel_mask) - 1);
 
        /* set voltage selector */
-       rdev_dbg(rdev,
-                "vsel_reg=0x%x, mask=0x%x, value=0x%x, voltage=%d mV\n",
-                rdev->desc->vsel_reg, rdev->desc->vsel_mask, reg_val,
-                rdev->desc->ops->list_voltage(rdev, selector) / 1000);
-
        hi6421_spmi_pmic_rmw(pmic, rdev->desc->vsel_reg,
                             rdev->desc->vsel_mask, reg_val);
 
@@ -207,22 +179,14 @@ static unsigned int hi6421_spmi_regulator_get_mode(struct regulator_dev *rdev)
 {
        struct hi6421_spmi_reg_info *sreg = rdev_get_drvdata(rdev);
        struct hi6421_spmi_pmic *pmic = sreg->pmic;
-       unsigned int mode;
        u32 reg_val;
 
        reg_val = hi6421_spmi_pmic_read(pmic, rdev->desc->enable_reg);
 
        if (reg_val & sreg->eco_mode_mask)
-               mode = REGULATOR_MODE_IDLE;
-       else
-               mode = REGULATOR_MODE_NORMAL;
+               return REGULATOR_MODE_IDLE;
 
-       rdev_dbg(rdev,
-                "enable_reg=0x%x, eco_mode_mask=0x%x, reg_val=0x%x, %s mode\n",
-                rdev->desc->enable_reg, sreg->eco_mode_mask, reg_val,
-                mode == REGULATOR_MODE_IDLE ? "idle" : "normal");
-
-       return mode;
+       return REGULATOR_MODE_NORMAL;
 }
 
 static int hi6421_spmi_regulator_set_mode(struct regulator_dev *rdev,
@@ -244,9 +208,6 @@ static int hi6421_spmi_regulator_set_mode(struct regulator_dev *rdev,
        }
 
        /* set mode */
-       rdev_dbg(rdev, "enable_reg=0x%x, eco_mode_mask=0x%x, value=0x%x\n",
-                rdev->desc->enable_reg, sreg->eco_mode_mask, val);
-
        hi6421_spmi_pmic_rmw(pmic, rdev->desc->enable_reg,
                             sreg->eco_mode_mask, val);