The driver was originally written for Kernel 4.9. It needs to
be ported to upstream:

        - Got rid of timeval;
        - Removed a bogus dependency;
        - Did cleanups at the header file.

Signed-off-by: Mauro Carvalho Chehab <mchehab+hua...@kernel.org>
---
 drivers/regulator/hisi_regulator_spmi.c | 34 ++++---------------------
 include/linux/mfd/hisi_pmic.h           | 26 -------------------
 2 files changed, 5 insertions(+), 55 deletions(-)

diff --git a/drivers/regulator/hisi_regulator_spmi.c 
b/drivers/regulator/hisi_regulator_spmi.c
index 7bc0ae27b110..904cb64b1dcd 100644
--- a/drivers/regulator/hisi_regulator_spmi.c
+++ b/drivers/regulator/hisi_regulator_spmi.c
@@ -49,7 +49,6 @@ struct hisi_regulator_register_info {
 struct hisi_regulator {
        const char *name;
        struct hisi_regulator_register_info register_info;
-       struct timeval last_off_time;
        u32 off_on_delay;
        u32 eco_uA;
        struct regulator_desc rdesc;
@@ -57,8 +56,6 @@ struct hisi_regulator {
 };
 
 static DEFINE_MUTEX(enable_mutex);
-struct timeval last_enabled;
-
 
 static inline struct hisi_pmic *rdev_to_pmic(struct regulator_dev *dev)
 {
@@ -72,27 +69,6 @@ static inline struct hisi_pmic *rdev_to_pmic(struct 
regulator_dev *dev)
 /* helper function to ensure when it returns it is at least 'delay_us'
  * microseconds after 'since'.
  */
-static void ensured_time_after(struct timeval since, u32 delay_us)
-{
-       struct timeval now;
-       u64 elapsed_ns64, delay_ns64;
-       u32 actual_us32;
-
-       delay_ns64 = delay_us * NSEC_PER_USEC;
-       do_gettimeofday(&now);
-       elapsed_ns64 = timeval_to_ns(&now) - timeval_to_ns(&since);
-       if (delay_ns64 > elapsed_ns64) {
-               actual_us32 = ((u32)(delay_ns64 - elapsed_ns64) /
-                                                       NSEC_PER_USEC);
-               if (actual_us32 >= 1000) {
-                       mdelay(actual_us32 / 1000); /*lint !e647 */
-                       udelay(actual_us32 % 1000);
-               } else if (actual_us32 > 0) {
-                       udelay(actual_us32);
-               }
-       }
-       return;
-}
 
 static int hisi_regulator_is_enabled(struct regulator_dev *dev)
 {
@@ -113,13 +89,16 @@ static int hisi_regulator_enable(struct regulator_dev *dev)
        struct hisi_pmic *pmic = rdev_to_pmic(dev);
 
        /* keep a distance of off_on_delay from last time disabled */
-       ensured_time_after(sreg->last_off_time, sreg->off_on_delay);
+       usleep_range(sreg->off_on_delay, sreg->off_on_delay + 1000);
 
        pr_debug("<[%s]: off_on_delay=%dus>\n", __func__, sreg->off_on_delay);
 
        /* cannot enable more than one regulator at one time */
        mutex_lock(&enable_mutex);
-       ensured_time_after(last_enabled, HISI_REGS_ENA_PROTECT_TIME);
+       usleep_range(HISI_REGS_ENA_PROTECT_TIME,
+                    HISI_REGS_ENA_PROTECT_TIME + 1000);
+
+
 
        /* set enable register */
        hisi_pmic_rmw(pmic, sreg->register_info.ctrl_reg,
@@ -128,7 +107,6 @@ static int hisi_regulator_enable(struct regulator_dev *dev)
        pr_debug("<[%s]: ctrl_reg=0x%x,enable_mask=0x%x>\n", __func__, 
sreg->register_info.ctrl_reg,\
                        sreg->register_info.enable_mask);
 
-       do_gettimeofday(&last_enabled);
        mutex_unlock(&enable_mutex);
 
        return 0;
@@ -143,8 +121,6 @@ static int hisi_regulator_disable(struct regulator_dev *dev)
        hisi_pmic_rmw(pmic, sreg->register_info.ctrl_reg,
                                sreg->register_info.enable_mask, 0);
 
-       do_gettimeofday(&sreg->last_off_time);
-
        return 0;
 }
 
diff --git a/include/linux/mfd/hisi_pmic.h b/include/linux/mfd/hisi_pmic.h
index 5be9b4d3f207..c3f6e59e2b7d 100644
--- a/include/linux/mfd/hisi_pmic.h
+++ b/include/linux/mfd/hisi_pmic.h
@@ -73,34 +73,9 @@ struct hisi_pmic {
        struct write_lock debug_lock;
 };
 
-/* 0:disable; 1:enable */
-unsigned int get_uv_mntn_status(void);
-void clear_uv_mntn_resered_reg_bit(void);
-void set_uv_mntn_resered_reg_bit(void);
-
-/* Register Access Helpers */
 u32 hisi_pmic_read(struct hisi_pmic *pmic, int reg);
 void hisi_pmic_write(struct hisi_pmic *pmic, int reg, u32 val);
 void hisi_pmic_rmw(struct hisi_pmic *pmic, int reg, u32 mask, u32 bits);
-unsigned int hisi_pmic_reg_read(int addr);
-void hisi_pmic_reg_write(int addr, int val);
-void hisi_pmic_reg_write_lock(int addr, int val);
-int hisi_pmic_array_read(int addr, char *buff, unsigned int len);
-int hisi_pmic_array_write(int addr, char *buff, unsigned int len);
-extern int hisi_get_pmic_irq_byname(unsigned int pmic_irq_list);
-extern int hisi_pmic_get_vbus_status(void);
-static inline u32 hisi_pmic_read(struct hisi_pmic *pmic, int reg) { return 0; }
-static inline void hisi_pmic_write(struct hisi_pmic *pmic, int reg, u32 val) {}
-static inline void hisi_pmic_rmw(struct hisi_pmic *pmic, int reg, u32 mask, 
u32 bits) {}
-static inline unsigned int hisi_pmic_reg_read(int addr) { return 0; }
-static inline void hisi_pmic_reg_write(int addr, int val) {}
-static inline void hisi_pmic_reg_write_lock(int addr, int val) {}
-static inline int hisi_pmic_array_read(int addr, char *buff, unsigned int len) 
{ return 0; }
-static inline int hisi_pmic_array_write(int addr, char *buff, unsigned int 
len) { return 0; }
-static inline int hisi_get_pmic_irq_byname(unsigned int pmic_irq_list) { 
return -1; }
-static inline int hisi_pmic_get_vbus_status(void) { return 1; }
-static inline u32 hisi_pmic_read_sub_pmu(u8 sid ,int reg) { return 0; }
-static inline void hisi_pmic_write_sub_pmu(u8 sid ,int reg, u32 val) {}
 
 enum pmic_irq_list {
        OTMP = 0,
@@ -120,4 +95,3 @@ enum pmic_irq_list {
        PMIC_IRQ_LIST_MAX,
 };
 #endif         /* __HISI_PMIC_H */
-
-- 
2.26.2

Reply via email to