Remove unused code and cleanup devicetree bindings for DPU driver

From: Jordan Crouse <[email protected]>

Remove unused code from dpu_io_util.c. The functions are only used inside of the msm driver so remove the EXPORT_SYMBOL tags and move the header dpu_io_util.h from include/linux.

Signed-off-by Jordan Crouse <[email protected]>

[[email protected]: rebased and removed some extra unused code]

Signed-off-by Rajesh Yadav <[email protected]>

---
 drivers/gpu/drm/msm/dpu_io_util.c | 380 +-------------------------------------
 drivers/gpu/drm/msm/dpu_io_util.h |  61 ++++++
 drivers/gpu/drm/msm/msm_drv.h     |   1 -
 include/linux/dpu_io_util.h       | 115 ------------
 4 files changed, 66 insertions(+), 491 deletions(-)
 create mode 100644 drivers/gpu/drm/msm/dpu_io_util.h
 delete mode 100644 include/linux/dpu_io_util.h

drivers/gpu/drm/msm/dpu_io_util.c +5 -375

index ecc297c..f7caec3 100644
@@ -13,318 +13,9 @@
 
 #include <linux/clk.h>
 #include <linux/err.h>
-#include <linux/io.h>
-#include <linux/regulator/consumer.h>
 #include <linux/delay.h>
-#include <linux/dpu_io_util.h>
 
-#define MAX_I2C_CMDS  16
-void dss_reg_w(struct dss_io_data *io, u32 offset, u32 value, u32 debug)
-{
-       u32 in_val;
-
-       if (!io || !io->base) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return;
-       }
-
-       if (offset > io->len) {
-               DEV_ERR("%pS->%s: offset out of range\n",
-                       __builtin_return_address(0), __func__);
-               return;
-       }
-
-       writel_relaxed(value, io->base + offset);
-       if (debug) {
-               in_val = readl_relaxed(io->base + offset);
-               DEV_DBG("[%08x] => %08x [%08x]\n",
-                       (u32)(unsigned long)(io->base + offset),
-                       value, in_val);
-       }
-} /* dss_reg_w */
-EXPORT_SYMBOL(dss_reg_w);
-
-u32 dss_reg_r(struct dss_io_data *io, u32 offset, u32 debug)
-{
-       u32 value;
-
-       if (!io || !io->base) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return -EINVAL;
-       }
-
-       if (offset > io->len) {
-               DEV_ERR("%pS->%s: offset out of range\n",
-                       __builtin_return_address(0), __func__);
-               return -EINVAL;
-       }
-
-       value = readl_relaxed(io->base + offset);
-       if (debug)
-               DEV_DBG("[%08x] <= %08x\n",
-                       (u32)(unsigned long)(io->base + offset), value);
-
-       return value;
-} /* dss_reg_r */
-EXPORT_SYMBOL(dss_reg_r);
-
-void dss_reg_dump(void __iomem *base, u32 length, const char *prefix,
-       u32 debug)
-{
-       if (debug)
-               print_hex_dump(KERN_INFO, prefix, DUMP_PREFIX_OFFSET, 32, 4,
-                       (void *)base, length, false);
-} /* dss_reg_dump */
-EXPORT_SYMBOL(dss_reg_dump);
-
-static struct resource *msm_dss_get_res_byname(struct platform_device *pdev,
-       unsigned int type, const char *name)
-{
-       struct resource *res = NULL;
-
-       res = platform_get_resource_byname(pdev, type, name);
-       if (!res)
-               DEV_ERR("%s: '%s' resource not found\n", __func__, name);
-
-       return res;
-} /* msm_dss_get_res_byname */
-EXPORT_SYMBOL(msm_dss_get_res_byname);
-
-int msm_dss_ioremap_byname(struct platform_device *pdev,
-       struct dss_io_data *io_data, const char *name)
-{
-       struct resource *res = NULL;
-
-       if (!pdev || !io_data) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return -EINVAL;
-       }
-
-       res = msm_dss_get_res_byname(pdev, IORESOURCE_MEM, name);
-       if (!res) {
-               DEV_ERR("%pS->%s: '%s' msm_dss_get_res_byname failed\n",
-                       __builtin_return_address(0), __func__, name);
-               return -ENODEV;
-       }
-
-       io_data->len = (u32)resource_size(res);
-       io_data->base = ioremap(res->start, io_data->len);
-       if (!io_data->base) {
-               DEV_ERR("%pS->%s: '%s' ioremap failed\n",
-                       __builtin_return_address(0), __func__, name);
-               return -EIO;
-       }
-
-       return 0;
-} /* msm_dss_ioremap_byname */
-EXPORT_SYMBOL(msm_dss_ioremap_byname);
-
-void msm_dss_iounmap(struct dss_io_data *io_data)
-{
-       if (!io_data) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return;
-       }
-
-       if (io_data->base) {
-               iounmap(io_data->base);
-               io_data->base = NULL;
-       }
-       io_data->len = 0;
-} /* msm_dss_iounmap */
-EXPORT_SYMBOL(msm_dss_iounmap);
-
-int msm_dss_config_vreg(struct device *dev, struct dss_vreg *in_vreg,
-       int num_vreg, int config)
-{
-       int i = 0, rc = 0;
-       struct dss_vreg *curr_vreg = NULL;
-       enum dss_vreg_type type;
-
-       if (!in_vreg || !num_vreg)
-               return rc;
-
-       if (config) {
-               for (i = 0; i < num_vreg; i++) {
-                       curr_vreg = &in_vreg[i];
-                       curr_vreg->vreg = regulator_get(dev,
-                               curr_vreg->vreg_name);
-                       rc = PTR_RET(curr_vreg->vreg);
-                       if (rc) {
-                               DEV_ERR("%pS->%s: %s get failed. rc=%d\n",
-                                        __builtin_return_address(0), __func__,
-                                        curr_vreg->vreg_name, rc);
-                               curr_vreg->vreg = NULL;
-                               goto vreg_get_fail;
-                       }
-                       type = (regulator_count_voltages(curr_vreg->vreg) > 0)
-                                       ? DSS_REG_LDO : DSS_REG_VS;
-                       if (type == DSS_REG_LDO) {
-                               rc = regulator_set_voltage(
-                                       curr_vreg->vreg,
-                                       curr_vreg->min_voltage,
-                                       curr_vreg->max_voltage);
-                               if (rc < 0) {
-                                       DEV_ERR("%pS->%s: %s set vltg fail\n",
-                                               __builtin_return_address(0),
-                                               __func__,
-                                               curr_vreg->vreg_name);
-                                       goto vreg_set_voltage_fail;
-                               }
-                       }
-               }
-       } else {
-               for (i = num_vreg-1; i >= 0; i--) {
-                       curr_vreg = &in_vreg[i];
-                       if (curr_vreg->vreg) {
-                               type = (regulator_count_voltages(
-                                       curr_vreg->vreg) > 0)
-                                       ? DSS_REG_LDO : DSS_REG_VS;
-                               if (type == DSS_REG_LDO) {
-                                       regulator_set_voltage(curr_vreg->vreg,
-                                               0, curr_vreg->max_voltage);
-                               }
-                               regulator_put(curr_vreg->vreg);
-                               curr_vreg->vreg = NULL;
-                       }
-               }
-       }
-       return 0;
-
-vreg_unconfig:
-if (type == DSS_REG_LDO)
-       regulator_set_load(curr_vreg->vreg, 0);
-
-vreg_set_voltage_fail:
-       regulator_put(curr_vreg->vreg);
-       curr_vreg->vreg = NULL;
-
-vreg_get_fail:
-       for (i--; i >= 0; i--) {
-               curr_vreg = &in_vreg[i];
-               type = (regulator_count_voltages(curr_vreg->vreg) > 0)
-                       ? DSS_REG_LDO : DSS_REG_VS;
-               goto vreg_unconfig;
-       }
-       return rc;
-} /* msm_dss_config_vreg */
-EXPORT_SYMBOL(msm_dss_config_vreg);
-
-int msm_dss_enable_vreg(struct dss_vreg *in_vreg, int num_vreg, int enable)
-{
-       int i = 0, rc = 0;
-       bool need_sleep;
-
-       if (enable) {
-               for (i = 0; i < num_vreg; i++) {
-                       rc = PTR_RET(in_vreg[i].vreg);
-                       if (rc) {
-                               DEV_ERR("%pS->%s: %s regulator error. rc=%d\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_vreg[i].vreg_name, rc);
-                               goto vreg_set_opt_mode_fail;
-                       }
-                       need_sleep = !regulator_is_enabled(in_vreg[i].vreg);
-                       if (in_vreg[i].pre_on_sleep && need_sleep)
-                               usleep_range(in_vreg[i].pre_on_sleep * 1000,
-                                       in_vreg[i].pre_on_sleep * 1000);
-                       rc = regulator_set_load(in_vreg[i].vreg,
-                               in_vreg[i].enable_load);
-                       if (rc < 0) {
-                               DEV_ERR("%pS->%s: %s set opt m fail\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_vreg[i].vreg_name);
-                               goto vreg_set_opt_mode_fail;
-                       }
-                       rc = regulator_enable(in_vreg[i].vreg);
-                       if (in_vreg[i].post_on_sleep && need_sleep)
-                               usleep_range(in_vreg[i].post_on_sleep * 1000,
-                                       in_vreg[i].post_on_sleep * 1000);
-                       if (rc < 0) {
-                               DEV_ERR("%pS->%s: %s enable failed\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_vreg[i].vreg_name);
-                               goto disable_vreg;
-                       }
-               }
-       } else {
-               for (i = num_vreg-1; i >= 0; i--) {
-                       if (in_vreg[i].pre_off_sleep)
-                               usleep_range(in_vreg[i].pre_off_sleep * 1000,
-                                       in_vreg[i].pre_off_sleep * 1000);
-                       regulator_set_load(in_vreg[i].vreg,
-                               in_vreg[i].disable_load);
-                       regulator_disable(in_vreg[i].vreg);
-                       if (in_vreg[i].post_off_sleep)
-                               usleep_range(in_vreg[i].post_off_sleep * 1000,
-                                       in_vreg[i].post_off_sleep * 1000);
-               }
-       }
-       return rc;
-
-disable_vreg:
-       regulator_set_load(in_vreg[i].vreg, in_vreg[i].disable_load);
-
-vreg_set_opt_mode_fail:
-       for (i--; i >= 0; i--) {
-               if (in_vreg[i].pre_off_sleep)
-                       usleep_range(in_vreg[i].pre_off_sleep * 1000,
-                               in_vreg[i].pre_off_sleep * 1000);
-               regulator_set_load(in_vreg[i].vreg,
-                       in_vreg[i].disable_load);
-               regulator_disable(in_vreg[i].vreg);
-               if (in_vreg[i].post_off_sleep)
-                       usleep_range(in_vreg[i].post_off_sleep * 1000,
-                               in_vreg[i].post_off_sleep * 1000);
-       }
-
-       return rc;
-} /* msm_dss_enable_vreg */
-EXPORT_SYMBOL(msm_dss_enable_vreg);
-
-int msm_dss_enable_gpio(struct dss_gpio *in_gpio, int num_gpio, int enable)
-{
-       int i = 0, rc = 0;
-
-       if (enable) {
-               for (i = 0; i < num_gpio; i++) {
-                       DEV_DBG("%pS->%s: %s enable\n",
-                               __builtin_return_address(0), __func__,
-                               in_gpio[i].gpio_name);
-
-                       rc = gpio_request(in_gpio[i].gpio,
-                               in_gpio[i].gpio_name);
-                       if (rc < 0) {
-                               DEV_ERR("%pS->%s: %s enable failed\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_gpio[i].gpio_name);
-                               goto disable_gpio;
-                       }
-                       gpio_set_value(in_gpio[i].gpio, in_gpio[i].value);
-               }
-       } else {
-               for (i = num_gpio-1; i >= 0; i--) {
-                       DEV_DBG("%pS->%s: %s disable\n",
-                               __builtin_return_address(0), __func__,
-                               in_gpio[i].gpio_name);
-                       if (in_gpio[i].gpio)
-                               gpio_free(in_gpio[i].gpio);
-               }
-       }
-       return rc;
-
-disable_gpio:
-       for (i--; i >= 0; i--)
-               if (in_gpio[i].gpio)
-                       gpio_free(in_gpio[i].gpio);
-
-       return rc;
-} /* msm_dss_enable_gpio */
-EXPORT_SYMBOL(msm_dss_enable_gpio);
+#include "dpu_io_util.h"
 
 void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk)
 {

@@ -335,8 +26,7 @@ void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk)
                        clk_put(clk_arry[i].clk);
                clk_arry[i].clk = NULL;
        }
-} /* msm_dss_put_clk */
-EXPORT_SYMBOL(msm_dss_put_clk);
+}
 
 int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk)
 {

@@ -363,8 +53,7 @@ int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk)
        }
 
        return rc;
-} /* msm_dss_get_clk */
-EXPORT_SYMBOL(msm_dss_get_clk);
+}
 
 int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk)
 {

@@ -397,8 +86,7 @@ int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk)
        }
 
        return rc;
-} /* msm_dss_clk_set_rate */
-EXPORT_SYMBOL(msm_dss_clk_set_rate);
+}
 
 int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable)
 {

@@ -445,8 +133,7 @@ int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable)
        }
 
        return rc;
-} /* msm_dss_enable_clk */
-EXPORT_SYMBOL(msm_dss_enable_clk);
+}
 
 int msm_dss_parse_clock(struct platform_device *pdev,
                struct dss_module_power *mp)

@@ -497,60 +184,3 @@ int msm_dss_parse_clock(struct platform_device *pdev,
 
        return rc;
 }
-
-int dpu_i2c_byte_read(struct i2c_client *client, uint8_t slave_addr,
-                       uint8_t reg_offset, uint8_t *read_buf)
-{
-       struct i2c_msg msgs[2];
-       int ret = -1;
-
-       pr_debug("%s: reading from slave_addr=[%x] and offset=[%x]\n",
-                __func__, slave_addr, reg_offset);
-
-       msgs[0].addr = slave_addr >> 1;
-       msgs[0].flags = 0;
-       msgs[0].buf = &reg_offset;
-       msgs[0].len = 1;
-
-       msgs[1].addr = slave_addr >> 1;
-       msgs[1].flags = I2C_M_RD;
-       msgs[1].buf = read_buf;
-       msgs[1].len = 1;
-
-       ret = i2c_transfer(client->adapter, msgs, 2);
-       if (ret < 1) {
-               pr_err("%s: I2C READ FAILED=[%d]\n", __func__, ret);
-               return -EACCES;
-       }
-       pr_debug("%s: i2c buf is [%x]\n", __func__, *read_buf);
-       return 0;
-}
-EXPORT_SYMBOL(dpu_i2c_byte_read);
-
-int dpu_i2c_byte_write(struct i2c_client *client, uint8_t slave_addr,
-                       uint8_t reg_offset, uint8_t *value)
-{
-       struct i2c_msg msgs[1];
-       uint8_t data[2];
-       int status = -EACCES;
-
-       pr_debug("%s: writing from slave_addr=[%x] and offset=[%x]\n",
-                __func__, slave_addr, reg_offset);
-
-       data[0] = reg_offset;
-       data[1] = *value;
-
-       msgs[0].addr = slave_addr >> 1;
-       msgs[0].flags = 0;
-       msgs[0].len = 2;
-       msgs[0].buf = data;
-
-       status = i2c_transfer(client->adapter, msgs, 1);
-       if (status < 1) {
-               pr_err("I2C WRITE FAILED=[%d]\n", status);
-               return -EACCES;
-       }
-       pr_debug("%s: I2C write status=%x\n", __func__, status);
-       return status;
-}
-EXPORT_SYMBOL(dpu_i2c_byte_write);

drivers/gpu/drm/msm/dpu_io_util.h new file +61 -0

new file mode 100644
index 0000000..773b52e
@@ -0,0 +1,61 @@
+/* Copyright (c) 2012, 2017-2018, The Linux Foundation. All rights reserved.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef __DPU_IO_UTIL_H__
+#define __DPU_IO_UTIL_H__
+
+#include <linux/gpio.h>
+#include <linux/platform_device.h>
+#include <linux/types.h>
+
+#ifdef DEBUG
+#define DEV_DBG(fmt, args...)   pr_err(fmt, ##args)
+#else
+#define DEV_DBG(fmt, args...)   pr_debug(fmt, ##args)
+#endif
+#define DEV_INFO(fmt, args...)  pr_info(fmt, ##args)
+#define DEV_WARN(fmt, args...)  pr_warn(fmt, ##args)
+#define DEV_ERR(fmt, args...)   pr_err(fmt, ##args)
+
+struct dss_gpio {
+       unsigned int gpio;
+       unsigned int value;
+       char gpio_name[32];
+};
+
+enum dss_clk_type {
+       DSS_CLK_AHB, /* no set rate. rate controlled through rpm */
+       DSS_CLK_PCLK,
+};
+
+struct dss_clk {
+       struct clk *clk; /* clk handle */
+       char clk_name[32];
+       enum dss_clk_type type;
+       unsigned long rate;
+       unsigned long max_rate;
+};
+
+struct dss_module_power {
+       unsigned int num_gpio;
+       struct dss_gpio *gpio_config;
+       unsigned int num_clk;
+       struct dss_clk *clk_config;
+};
+
+int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk);
+void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk);
+int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk);
+int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable);
+int msm_dss_parse_clock(struct platform_device *pdev,
+               struct dss_module_power *mp);
+#endif /* __DPU_IO_UTIL_H__ */

drivers/gpu/drm/msm/msm_drv.h +0 -1

index d5937ba..811f7d4 100644
@@ -33,7 +33,6 @@
 #include <linux/types.h>
 #include <linux/of_graph.h>
 #include <linux/of_device.h>
-#include <linux/dpu_io_util.h>
 #include <asm/sizes.h>
 #include <linux/kthread.h>
 

include/linux/dpu_io_util.h deleted +0 -116

deleted file mode 100644
index 45e606f..0000000
@@ -1,115 +0,0 @@
-/* Copyright (c) 2012, 2017-2018, The Linux Foundation. All rights reserved.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 and
- * only version 2 as published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- */
-
-#ifndef __DPU_IO_UTIL_H__
-#define __DPU_IO_UTIL_H__
-
-#include <linux/gpio.h>
-#include <linux/platform_device.h>
-#include <linux/regulator/consumer.h>
-#include <linux/i2c.h>
-#include <linux/types.h>
-
-#ifdef DEBUG
-#define DEV_DBG(fmt, args...)   pr_err(fmt, ##args)
-#else
-#define DEV_DBG(fmt, args...)   pr_debug(fmt, ##args)
-#endif
-#define DEV_INFO(fmt, args...)  pr_info(fmt, ##args)
-#define DEV_WARN(fmt, args...)  pr_warn(fmt, ##args)
-#define DEV_ERR(fmt, args...)   pr_err(fmt, ##args)
-
-struct dss_io_data {
-       u32 len;
-       void __iomem *base;
-};
-
-void dss_reg_w(struct dss_io_data *io, u32 offset, u32 value, u32 debug);
-u32 dss_reg_r(struct dss_io_data *io, u32 offset, u32 debug);
-void dss_reg_dump(void __iomem *base, u32 len, const char *prefix, u32 debug);
-
-#define DSS_REG_W_ND(io, offset, val)  dss_reg_w(io, offset, val, false)
-#define DSS_REG_W(io, offset, val)     dss_reg_w(io, offset, val, true)
-#define DSS_REG_R_ND(io, offset)       dss_reg_r(io, offset, false)
-#define DSS_REG_R(io, offset)          dss_reg_r(io, offset, true)
-
-enum dss_vreg_type {
-       DSS_REG_LDO,
-       DSS_REG_VS,
-};
-
-struct dss_vreg {
-       struct regulator *vreg; /* vreg handle */
-       char vreg_name[32];
-       int min_voltage;
-       int max_voltage;
-       int enable_load;
-       int disable_load;
-       int pre_on_sleep;
-       int post_on_sleep;
-       int pre_off_sleep;
-       int post_off_sleep;
-};
-
-struct dss_gpio {
-       unsigned int gpio;
-       unsigned int value;
-       char gpio_name[32];
-};
-
-enum dss_clk_type {
-       DSS_CLK_AHB, /* no set rate. rate controlled through rpm */
-       DSS_CLK_PCLK,
-       DSS_CLK_OTHER,
-};
-
-struct dss_clk {
-       struct clk *clk; /* clk handle */
-       char clk_name[32];
-       enum dss_clk_type type;
-       unsigned long rate;
-       unsigned long max_rate;
-};
-
-struct dss_module_power {
-       unsigned int num_vreg;
-       struct dss_vreg *vreg_config;
-       unsigned int num_gpio;
-       struct dss_gpio *gpio_config;
-       unsigned int num_clk;
-       struct dss_clk *clk_config;
-};
-
-int msm_dss_ioremap_byname(struct platform_device *pdev,
-       struct dss_io_data *io_data, const char *name);
-void msm_dss_iounmap(struct dss_io_data *io_data);
-
-int msm_dss_enable_gpio(struct dss_gpio *in_gpio, int num_gpio, int enable);
-int msm_dss_gpio_enable(struct dss_gpio *in_gpio, int num_gpio, int enable);
-
-int msm_dss_config_vreg(struct device *dev, struct dss_vreg *in_vreg,
-       int num_vreg, int config);
-int msm_dss_enable_vreg(struct dss_vreg *in_vreg, int num_vreg,        int enable);
-
-int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk);
-void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk);
-int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk);
-int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable);
-int msm_dss_parse_clock(struct platform_device *pdev,
-               struct dss_module_power *mp);
-
-int dpu_i2c_byte_read(struct i2c_client *client, uint8_t slave_addr,
-                      uint8_t reg_offset, uint8_t *read_buf);
-int dpu_i2c_byte_write(struct i2c_client *client, uint8_t slave_addr,
-                       uint8_t reg_offset, uint8_t *value);
-
-#endif /* __DPU_IO_UTIL_H__ */
--

The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project

Quoted 10 lines · Rajesh Yadav

From: Jordan Crouse <[email protected]>

Remove unused code from dpu_io_util.c. The functions are only used inside of the msm driver so remove the EXPORT_SYMBOL tags and move the header dpu_io_util.h from include/linux.

Signed-off-by Jordan Crouse <[email protected]>

[[email protected]: rebased and removed some extra unused code]

Signed-off-by Rajesh Yadav <[email protected]>

Hi Rajesh and Jordan,
I'm backporting this series for testing, and have found that the mdss-pll driver uses both dpu_io_util and a bunch of functions/struct members removed in this patchset. Do we anticipate having to add those back for mdss-pll?

Quoted 638 lines

---
 drivers/gpu/drm/msm/dpu_io_util.c | 380 +-------------------------------------
 drivers/gpu/drm/msm/dpu_io_util.h |  61 ++++++
 drivers/gpu/drm/msm/msm_drv.h     |   1 -
 include/linux/dpu_io_util.h       | 115 ------------
 4 files changed, 66 insertions(+), 491 deletions(-)
 create mode 100644 drivers/gpu/drm/msm/dpu_io_util.h
 delete mode 100644 include/linux/dpu_io_util.h

drivers/gpu/drm/msm/dpu_io_util.c +5 -375

index ecc297c..f7caec3 100644
@@ -13,318 +13,9 @@
 
 #include <linux/clk.h>
 #include <linux/err.h>
-#include <linux/io.h>
-#include <linux/regulator/consumer.h>
 #include <linux/delay.h>
-#include <linux/dpu_io_util.h>
 
-#define MAX_I2C_CMDS  16
-void dss_reg_w(struct dss_io_data *io, u32 offset, u32 value, u32 debug)
-{
-       u32 in_val;
-
-       if (!io || !io->base) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return;
-       }
-
-       if (offset > io->len) {
-               DEV_ERR("%pS->%s: offset out of range\n",
-                       __builtin_return_address(0), __func__);
-               return;
-       }
-
-       writel_relaxed(value, io->base + offset);
-       if (debug) {
-               in_val = readl_relaxed(io->base + offset);
-               DEV_DBG("[%08x] => %08x [%08x]\n",
-                       (u32)(unsigned long)(io->base + offset),
-                       value, in_val);
-       }
-} /* dss_reg_w */
-EXPORT_SYMBOL(dss_reg_w);
-
-u32 dss_reg_r(struct dss_io_data *io, u32 offset, u32 debug)
-{
-       u32 value;
-
-       if (!io || !io->base) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return -EINVAL;
-       }
-
-       if (offset > io->len) {
-               DEV_ERR("%pS->%s: offset out of range\n",
-                       __builtin_return_address(0), __func__);
-               return -EINVAL;
-       }
-
-       value = readl_relaxed(io->base + offset);
-       if (debug)
-               DEV_DBG("[%08x] <= %08x\n",
-                       (u32)(unsigned long)(io->base + offset), value);
-
-       return value;
-} /* dss_reg_r */
-EXPORT_SYMBOL(dss_reg_r);
-
-void dss_reg_dump(void __iomem *base, u32 length, const char *prefix,
-       u32 debug)
-{
-       if (debug)
-               print_hex_dump(KERN_INFO, prefix, DUMP_PREFIX_OFFSET, 32, 4,
-                       (void *)base, length, false);
-} /* dss_reg_dump */
-EXPORT_SYMBOL(dss_reg_dump);
-
-static struct resource *msm_dss_get_res_byname(struct platform_device *pdev,
-       unsigned int type, const char *name)
-{
-       struct resource *res = NULL;
-
-       res = platform_get_resource_byname(pdev, type, name);
-       if (!res)
-               DEV_ERR("%s: '%s' resource not found\n", __func__, name);
-
-       return res;
-} /* msm_dss_get_res_byname */
-EXPORT_SYMBOL(msm_dss_get_res_byname);
-
-int msm_dss_ioremap_byname(struct platform_device *pdev,
-       struct dss_io_data *io_data, const char *name)
-{
-       struct resource *res = NULL;
-
-       if (!pdev || !io_data) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return -EINVAL;
-       }
-
-       res = msm_dss_get_res_byname(pdev, IORESOURCE_MEM, name);
-       if (!res) {
-               DEV_ERR("%pS->%s: '%s' msm_dss_get_res_byname failed\n",
-                       __builtin_return_address(0), __func__, name);
-               return -ENODEV;
-       }
-
-       io_data->len = (u32)resource_size(res);
-       io_data->base = ioremap(res->start, io_data->len);
-       if (!io_data->base) {
-               DEV_ERR("%pS->%s: '%s' ioremap failed\n",
-                       __builtin_return_address(0), __func__, name);
-               return -EIO;
-       }
-
-       return 0;
-} /* msm_dss_ioremap_byname */
-EXPORT_SYMBOL(msm_dss_ioremap_byname);
-
-void msm_dss_iounmap(struct dss_io_data *io_data)
-{
-       if (!io_data) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return;
-       }
-
-       if (io_data->base) {
-               iounmap(io_data->base);
-               io_data->base = NULL;
-       }
-       io_data->len = 0;
-} /* msm_dss_iounmap */
-EXPORT_SYMBOL(msm_dss_iounmap);
-
-int msm_dss_config_vreg(struct device *dev, struct dss_vreg *in_vreg,
-       int num_vreg, int config)
-{
-       int i = 0, rc = 0;
-       struct dss_vreg *curr_vreg = NULL;
-       enum dss_vreg_type type;
-
-       if (!in_vreg || !num_vreg)
-               return rc;
-
-       if (config) {
-               for (i = 0; i < num_vreg; i++) {
-                       curr_vreg = &in_vreg[i];
-                       curr_vreg->vreg = regulator_get(dev,
-                               curr_vreg->vreg_name);
-                       rc = PTR_RET(curr_vreg->vreg);
-                       if (rc) {
-                               DEV_ERR("%pS->%s: %s get failed. rc=%d\n",
-                                        __builtin_return_address(0), __func__,
-                                        curr_vreg->vreg_name, rc);
-                               curr_vreg->vreg = NULL;
-                               goto vreg_get_fail;
-                       }
-                       type = (regulator_count_voltages(curr_vreg->vreg) > 0)
-                                       ? DSS_REG_LDO : DSS_REG_VS;
-                       if (type == DSS_REG_LDO) {
-                               rc = regulator_set_voltage(
-                                       curr_vreg->vreg,
-                                       curr_vreg->min_voltage,
-                                       curr_vreg->max_voltage);
-                               if (rc < 0) {
-                                       DEV_ERR("%pS->%s: %s set vltg fail\n",
-                                               __builtin_return_address(0),
-                                               __func__,
-                                               curr_vreg->vreg_name);
-                                       goto vreg_set_voltage_fail;
-                               }
-                       }
-               }
-       } else {
-               for (i = num_vreg-1; i >= 0; i--) {
-                       curr_vreg = &in_vreg[i];
-                       if (curr_vreg->vreg) {
-                               type = (regulator_count_voltages(
-                                       curr_vreg->vreg) > 0)
-                                       ? DSS_REG_LDO : DSS_REG_VS;
-                               if (type == DSS_REG_LDO) {
-                                       regulator_set_voltage(curr_vreg->vreg,
-                                               0, curr_vreg->max_voltage);
-                               }
-                               regulator_put(curr_vreg->vreg);
-                               curr_vreg->vreg = NULL;
-                       }
-               }
-       }
-       return 0;
-
-vreg_unconfig:
-if (type == DSS_REG_LDO)
-       regulator_set_load(curr_vreg->vreg, 0);
-
-vreg_set_voltage_fail:
-       regulator_put(curr_vreg->vreg);
-       curr_vreg->vreg = NULL;
-
-vreg_get_fail:
-       for (i--; i >= 0; i--) {
-               curr_vreg = &in_vreg[i];
-               type = (regulator_count_voltages(curr_vreg->vreg) > 0)
-                       ? DSS_REG_LDO : DSS_REG_VS;
-               goto vreg_unconfig;
-       }
-       return rc;
-} /* msm_dss_config_vreg */
-EXPORT_SYMBOL(msm_dss_config_vreg);
-
-int msm_dss_enable_vreg(struct dss_vreg *in_vreg, int num_vreg, int enable)
-{
-       int i = 0, rc = 0;
-       bool need_sleep;
-
-       if (enable) {
-               for (i = 0; i < num_vreg; i++) {
-                       rc = PTR_RET(in_vreg[i].vreg);
-                       if (rc) {
-                               DEV_ERR("%pS->%s: %s regulator error. rc=%d\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_vreg[i].vreg_name, rc);
-                               goto vreg_set_opt_mode_fail;
-                       }
-                       need_sleep = !regulator_is_enabled(in_vreg[i].vreg);
-                       if (in_vreg[i].pre_on_sleep && need_sleep)
-                               usleep_range(in_vreg[i].pre_on_sleep * 1000,
-                                       in_vreg[i].pre_on_sleep * 1000);
-                       rc = regulator_set_load(in_vreg[i].vreg,
-                               in_vreg[i].enable_load);
-                       if (rc < 0) {
-                               DEV_ERR("%pS->%s: %s set opt m fail\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_vreg[i].vreg_name);
-                               goto vreg_set_opt_mode_fail;
-                       }
-                       rc = regulator_enable(in_vreg[i].vreg);
-                       if (in_vreg[i].post_on_sleep && need_sleep)
-                               usleep_range(in_vreg[i].post_on_sleep * 1000,
-                                       in_vreg[i].post_on_sleep * 1000);
-                       if (rc < 0) {
-                               DEV_ERR("%pS->%s: %s enable failed\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_vreg[i].vreg_name);
-                               goto disable_vreg;
-                       }
-               }
-       } else {
-               for (i = num_vreg-1; i >= 0; i--) {
-                       if (in_vreg[i].pre_off_sleep)
-                               usleep_range(in_vreg[i].pre_off_sleep * 1000,
-                                       in_vreg[i].pre_off_sleep * 1000);
-                       regulator_set_load(in_vreg[i].vreg,
-                               in_vreg[i].disable_load);
-                       regulator_disable(in_vreg[i].vreg);
-                       if (in_vreg[i].post_off_sleep)
-                               usleep_range(in_vreg[i].post_off_sleep * 1000,
-                                       in_vreg[i].post_off_sleep * 1000);
-               }
-       }
-       return rc;
-
-disable_vreg:
-       regulator_set_load(in_vreg[i].vreg, in_vreg[i].disable_load);
-
-vreg_set_opt_mode_fail:
-       for (i--; i >= 0; i--) {
-               if (in_vreg[i].pre_off_sleep)
-                       usleep_range(in_vreg[i].pre_off_sleep * 1000,
-                               in_vreg[i].pre_off_sleep * 1000);
-               regulator_set_load(in_vreg[i].vreg,
-                       in_vreg[i].disable_load);
-               regulator_disable(in_vreg[i].vreg);
-               if (in_vreg[i].post_off_sleep)
-                       usleep_range(in_vreg[i].post_off_sleep * 1000,
-                               in_vreg[i].post_off_sleep * 1000);
-       }
-
-       return rc;
-} /* msm_dss_enable_vreg */
-EXPORT_SYMBOL(msm_dss_enable_vreg);
-
-int msm_dss_enable_gpio(struct dss_gpio *in_gpio, int num_gpio, int enable)
-{
-       int i = 0, rc = 0;
-
-       if (enable) {
-               for (i = 0; i < num_gpio; i++) {
-                       DEV_DBG("%pS->%s: %s enable\n",
-                               __builtin_return_address(0), __func__,
-                               in_gpio[i].gpio_name);
-
-                       rc = gpio_request(in_gpio[i].gpio,
-                               in_gpio[i].gpio_name);
-                       if (rc < 0) {
-                               DEV_ERR("%pS->%s: %s enable failed\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_gpio[i].gpio_name);
-                               goto disable_gpio;
-                       }
-                       gpio_set_value(in_gpio[i].gpio, in_gpio[i].value);
-               }
-       } else {
-               for (i = num_gpio-1; i >= 0; i--) {
-                       DEV_DBG("%pS->%s: %s disable\n",
-                               __builtin_return_address(0), __func__,
-                               in_gpio[i].gpio_name);
-                       if (in_gpio[i].gpio)
-                               gpio_free(in_gpio[i].gpio);
-               }
-       }
-       return rc;
-
-disable_gpio:
-       for (i--; i >= 0; i--)
-               if (in_gpio[i].gpio)
-                       gpio_free(in_gpio[i].gpio);
-
-       return rc;
-} /* msm_dss_enable_gpio */
-EXPORT_SYMBOL(msm_dss_enable_gpio);
+#include "dpu_io_util.h"
 
 void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk)
 {

@@ -335,8 +26,7 @@ void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk)
                        clk_put(clk_arry[i].clk);
                clk_arry[i].clk = NULL;
        }
-} /* msm_dss_put_clk */
-EXPORT_SYMBOL(msm_dss_put_clk);
+}
 
 int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk)
 {

@@ -363,8 +53,7 @@ int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk)
        }
 
        return rc;
-} /* msm_dss_get_clk */
-EXPORT_SYMBOL(msm_dss_get_clk);
+}
 
 int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk)
 {

@@ -397,8 +86,7 @@ int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk)
        }
 
        return rc;
-} /* msm_dss_clk_set_rate */
-EXPORT_SYMBOL(msm_dss_clk_set_rate);
+}
 
 int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable)
 {

@@ -445,8 +133,7 @@ int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable)
        }
 
        return rc;
-} /* msm_dss_enable_clk */
-EXPORT_SYMBOL(msm_dss_enable_clk);
+}
 
 int msm_dss_parse_clock(struct platform_device *pdev,
                struct dss_module_power *mp)

@@ -497,60 +184,3 @@ int msm_dss_parse_clock(struct platform_device *pdev,
 
        return rc;
 }
-
-int dpu_i2c_byte_read(struct i2c_client *client, uint8_t slave_addr,
-                       uint8_t reg_offset, uint8_t *read_buf)
-{
-       struct i2c_msg msgs[2];
-       int ret = -1;
-
-       pr_debug("%s: reading from slave_addr=[%x] and offset=[%x]\n",
-                __func__, slave_addr, reg_offset);
-
-       msgs[0].addr = slave_addr >> 1;
-       msgs[0].flags = 0;
-       msgs[0].buf = &reg_offset;
-       msgs[0].len = 1;
-
-       msgs[1].addr = slave_addr >> 1;
-       msgs[1].flags = I2C_M_RD;
-       msgs[1].buf = read_buf;
-       msgs[1].len = 1;
-
-       ret = i2c_transfer(client->adapter, msgs, 2);
-       if (ret < 1) {
-               pr_err("%s: I2C READ FAILED=[%d]\n", __func__, ret);
-               return -EACCES;
-       }
-       pr_debug("%s: i2c buf is [%x]\n", __func__, *read_buf);
-       return 0;
-}
-EXPORT_SYMBOL(dpu_i2c_byte_read);
-
-int dpu_i2c_byte_write(struct i2c_client *client, uint8_t slave_addr,
-                       uint8_t reg_offset, uint8_t *value)
-{
-       struct i2c_msg msgs[1];
-       uint8_t data[2];
-       int status = -EACCES;
-
-       pr_debug("%s: writing from slave_addr=[%x] and offset=[%x]\n",
-                __func__, slave_addr, reg_offset);
-
-       data[0] = reg_offset;
-       data[1] = *value;
-
-       msgs[0].addr = slave_addr >> 1;
-       msgs[0].flags = 0;
-       msgs[0].len = 2;
-       msgs[0].buf = data;
-
-       status = i2c_transfer(client->adapter, msgs, 1);
-       if (status < 1) {
-               pr_err("I2C WRITE FAILED=[%d]\n", status);
-               return -EACCES;
-       }
-       pr_debug("%s: I2C write status=%x\n", __func__, status);
-       return status;
-}
-EXPORT_SYMBOL(dpu_i2c_byte_write);

drivers/gpu/drm/msm/dpu_io_util.h new file +61 -0

new file mode 100644
index 0000000..773b52e
@@ -0,0 +1,61 @@
+/* Copyright (c) 2012, 2017-2018, The Linux Foundation. All rights reserved.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef __DPU_IO_UTIL_H__
+#define __DPU_IO_UTIL_H__
+
+#include <linux/gpio.h>
+#include <linux/platform_device.h>
+#include <linux/types.h>
+
+#ifdef DEBUG
+#define DEV_DBG(fmt, args...)   pr_err(fmt, ##args)
+#else
+#define DEV_DBG(fmt, args...)   pr_debug(fmt, ##args)
+#endif
+#define DEV_INFO(fmt, args...)  pr_info(fmt, ##args)
+#define DEV_WARN(fmt, args...)  pr_warn(fmt, ##args)
+#define DEV_ERR(fmt, args...)   pr_err(fmt, ##args)
+
+struct dss_gpio {
+       unsigned int gpio;
+       unsigned int value;
+       char gpio_name[32];
+};
+
+enum dss_clk_type {
+       DSS_CLK_AHB, /* no set rate. rate controlled through rpm */
+       DSS_CLK_PCLK,
+};
+
+struct dss_clk {
+       struct clk *clk; /* clk handle */
+       char clk_name[32];
+       enum dss_clk_type type;
+       unsigned long rate;
+       unsigned long max_rate;
+};
+
+struct dss_module_power {
+       unsigned int num_gpio;
+       struct dss_gpio *gpio_config;
+       unsigned int num_clk;
+       struct dss_clk *clk_config;
+};
+
+int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk);
+void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk);
+int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk);
+int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable);
+int msm_dss_parse_clock(struct platform_device *pdev,
+               struct dss_module_power *mp);
+#endif /* __DPU_IO_UTIL_H__ */

drivers/gpu/drm/msm/msm_drv.h +0 -1

index d5937ba..811f7d4 100644
@@ -33,7 +33,6 @@
 #include <linux/types.h>
 #include <linux/of_graph.h>
 #include <linux/of_device.h>
-#include <linux/dpu_io_util.h>
 #include <asm/sizes.h>
 #include <linux/kthread.h>
 

include/linux/dpu_io_util.h deleted +0 -116

deleted file mode 100644
index 45e606f..0000000
@@ -1,115 +0,0 @@
-/* Copyright (c) 2012, 2017-2018, The Linux Foundation. All rights reserved.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 and
- * only version 2 as published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- */
-
-#ifndef __DPU_IO_UTIL_H__
-#define __DPU_IO_UTIL_H__
-
-#include <linux/gpio.h>
-#include <linux/platform_device.h>
-#include <linux/regulator/consumer.h>
-#include <linux/i2c.h>
-#include <linux/types.h>
-
-#ifdef DEBUG
-#define DEV_DBG(fmt, args...)   pr_err(fmt, ##args)
-#else
-#define DEV_DBG(fmt, args...)   pr_debug(fmt, ##args)
-#endif
-#define DEV_INFO(fmt, args...)  pr_info(fmt, ##args)
-#define DEV_WARN(fmt, args...)  pr_warn(fmt, ##args)
-#define DEV_ERR(fmt, args...)   pr_err(fmt, ##args)
-
-struct dss_io_data {
-       u32 len;
-       void __iomem *base;
-};
-
-void dss_reg_w(struct dss_io_data *io, u32 offset, u32 value, u32 debug);
-u32 dss_reg_r(struct dss_io_data *io, u32 offset, u32 debug);
-void dss_reg_dump(void __iomem *base, u32 len, const char *prefix, u32 debug);
-
-#define DSS_REG_W_ND(io, offset, val)  dss_reg_w(io, offset, val, false)
-#define DSS_REG_W(io, offset, val)     dss_reg_w(io, offset, val, true)
-#define DSS_REG_R_ND(io, offset)       dss_reg_r(io, offset, false)
-#define DSS_REG_R(io, offset)          dss_reg_r(io, offset, true)
-
-enum dss_vreg_type {
-       DSS_REG_LDO,
-       DSS_REG_VS,
-};
-
-struct dss_vreg {
-       struct regulator *vreg; /* vreg handle */
-       char vreg_name[32];
-       int min_voltage;
-       int max_voltage;
-       int enable_load;
-       int disable_load;
-       int pre_on_sleep;
-       int post_on_sleep;
-       int pre_off_sleep;
-       int post_off_sleep;
-};
-
-struct dss_gpio {
-       unsigned int gpio;
-       unsigned int value;
-       char gpio_name[32];
-};
-
-enum dss_clk_type {
-       DSS_CLK_AHB, /* no set rate. rate controlled through rpm */
-       DSS_CLK_PCLK,
-       DSS_CLK_OTHER,
-};
-
-struct dss_clk {
-       struct clk *clk; /* clk handle */
-       char clk_name[32];
-       enum dss_clk_type type;
-       unsigned long rate;
-       unsigned long max_rate;
-};
-
-struct dss_module_power {
-       unsigned int num_vreg;
-       struct dss_vreg *vreg_config;
-       unsigned int num_gpio;
-       struct dss_gpio *gpio_config;
-       unsigned int num_clk;
-       struct dss_clk *clk_config;
-};
-
-int msm_dss_ioremap_byname(struct platform_device *pdev,
-       struct dss_io_data *io_data, const char *name);
-void msm_dss_iounmap(struct dss_io_data *io_data);
-
-int msm_dss_enable_gpio(struct dss_gpio *in_gpio, int num_gpio, int enable);
-int msm_dss_gpio_enable(struct dss_gpio *in_gpio, int num_gpio, int enable);
-
-int msm_dss_config_vreg(struct device *dev, struct dss_vreg *in_vreg,
-       int num_vreg, int config);
-int msm_dss_enable_vreg(struct dss_vreg *in_vreg, int num_vreg,        int enable);
-
-int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk);
-void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk);
-int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk);
-int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable);
-int msm_dss_parse_clock(struct platform_device *pdev,
-               struct dss_module_power *mp);
-
-int dpu_i2c_byte_read(struct i2c_client *client, uint8_t slave_addr,
-                      uint8_t reg_offset, uint8_t *read_buf);
-int dpu_i2c_byte_write(struct i2c_client *client, uint8_t slave_addr,
-                       uint8_t reg_offset, uint8_t *value);
-
-#endif /* __DPU_IO_UTIL_H__ */
--

The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project

Quoted 12 lines · Sean Paul

Quoted 10 lines · Rajesh Yadav

From: Jordan Crouse <[email protected]>

Remove unused code from dpu_io_util.c. The functions are only used inside of the msm driver so remove the EXPORT_SYMBOL tags and move the header dpu_io_util.h from include/linux.

Signed-off-by Jordan Crouse <[email protected]>

[[email protected]: rebased and removed some extra unused code]

Signed-off-by Rajesh Yadav <[email protected]>

Quoted 5 lines

Hi Rajesh and Jordan,
I'm backporting this series for testing, and have found that the mdss-pll driver uses both dpu_io_util and a bunch of functions/struct members removed in this patchset. Do we anticipate having to add those back for mdss-pll?

At some point in the distant past some of the downstream folks discovered that they were using relatively similar functions so they decided to share the code. The code originally lived in the display driver so it happened that the exported code continued to be hosted there.

I'm not sure what the upstream status of the mdss-pll code is but I can't imagine that sharing this code would be in the long term plan and even if it was basic module guidelines would dictate that it would live with mdss-pll which would be more likely to be either be built in or act as a dependency for DPU.

Jordan

Quoted 631 lines

Quoted 631 lines

---
 drivers/gpu/drm/msm/dpu_io_util.c | 380 +-------------------------------------
 drivers/gpu/drm/msm/dpu_io_util.h |  61 ++++++
 drivers/gpu/drm/msm/msm_drv.h     |   1 -
 include/linux/dpu_io_util.h       | 115 ------------
 4 files changed, 66 insertions(+), 491 deletions(-)
 create mode 100644 drivers/gpu/drm/msm/dpu_io_util.h
 delete mode 100644 include/linux/dpu_io_util.h

drivers/gpu/drm/msm/dpu_io_util.c +5 -375

index ecc297c..f7caec3 100644
@@ -13,318 +13,9 @@
 
 #include <linux/clk.h>
 #include <linux/err.h>
-#include <linux/io.h>
-#include <linux/regulator/consumer.h>
 #include <linux/delay.h>
-#include <linux/dpu_io_util.h>
 
-#define MAX_I2C_CMDS  16
-void dss_reg_w(struct dss_io_data *io, u32 offset, u32 value, u32 debug)
-{
-       u32 in_val;
-
-       if (!io || !io->base) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return;
-       }
-
-       if (offset > io->len) {
-               DEV_ERR("%pS->%s: offset out of range\n",
-                       __builtin_return_address(0), __func__);
-               return;
-       }
-
-       writel_relaxed(value, io->base + offset);
-       if (debug) {
-               in_val = readl_relaxed(io->base + offset);
-               DEV_DBG("[%08x] => %08x [%08x]\n",
-                       (u32)(unsigned long)(io->base + offset),
-                       value, in_val);
-       }
-} /* dss_reg_w */
-EXPORT_SYMBOL(dss_reg_w);
-
-u32 dss_reg_r(struct dss_io_data *io, u32 offset, u32 debug)
-{
-       u32 value;
-
-       if (!io || !io->base) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return -EINVAL;
-       }
-
-       if (offset > io->len) {
-               DEV_ERR("%pS->%s: offset out of range\n",
-                       __builtin_return_address(0), __func__);
-               return -EINVAL;
-       }
-
-       value = readl_relaxed(io->base + offset);
-       if (debug)
-               DEV_DBG("[%08x] <= %08x\n",
-                       (u32)(unsigned long)(io->base + offset), value);
-
-       return value;
-} /* dss_reg_r */
-EXPORT_SYMBOL(dss_reg_r);
-
-void dss_reg_dump(void __iomem *base, u32 length, const char *prefix,
-       u32 debug)
-{
-       if (debug)
-               print_hex_dump(KERN_INFO, prefix, DUMP_PREFIX_OFFSET, 32, 4,
-                       (void *)base, length, false);
-} /* dss_reg_dump */
-EXPORT_SYMBOL(dss_reg_dump);
-
-static struct resource *msm_dss_get_res_byname(struct platform_device *pdev,
-       unsigned int type, const char *name)
-{
-       struct resource *res = NULL;
-
-       res = platform_get_resource_byname(pdev, type, name);
-       if (!res)
-               DEV_ERR("%s: '%s' resource not found\n", __func__, name);
-
-       return res;
-} /* msm_dss_get_res_byname */
-EXPORT_SYMBOL(msm_dss_get_res_byname);
-
-int msm_dss_ioremap_byname(struct platform_device *pdev,
-       struct dss_io_data *io_data, const char *name)
-{
-       struct resource *res = NULL;
-
-       if (!pdev || !io_data) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return -EINVAL;
-       }
-
-       res = msm_dss_get_res_byname(pdev, IORESOURCE_MEM, name);
-       if (!res) {
-               DEV_ERR("%pS->%s: '%s' msm_dss_get_res_byname failed\n",
-                       __builtin_return_address(0), __func__, name);
-               return -ENODEV;
-       }
-
-       io_data->len = (u32)resource_size(res);
-       io_data->base = ioremap(res->start, io_data->len);
-       if (!io_data->base) {
-               DEV_ERR("%pS->%s: '%s' ioremap failed\n",
-                       __builtin_return_address(0), __func__, name);
-               return -EIO;
-       }
-
-       return 0;
-} /* msm_dss_ioremap_byname */
-EXPORT_SYMBOL(msm_dss_ioremap_byname);
-
-void msm_dss_iounmap(struct dss_io_data *io_data)
-{
-       if (!io_data) {
-               DEV_ERR("%pS->%s: invalid input\n",
-                       __builtin_return_address(0), __func__);
-               return;
-       }
-
-       if (io_data->base) {
-               iounmap(io_data->base);
-               io_data->base = NULL;
-       }
-       io_data->len = 0;
-} /* msm_dss_iounmap */
-EXPORT_SYMBOL(msm_dss_iounmap);
-
-int msm_dss_config_vreg(struct device *dev, struct dss_vreg *in_vreg,
-       int num_vreg, int config)
-{
-       int i = 0, rc = 0;
-       struct dss_vreg *curr_vreg = NULL;
-       enum dss_vreg_type type;
-
-       if (!in_vreg || !num_vreg)
-               return rc;
-
-       if (config) {
-               for (i = 0; i < num_vreg; i++) {
-                       curr_vreg = &in_vreg[i];
-                       curr_vreg->vreg = regulator_get(dev,
-                               curr_vreg->vreg_name);
-                       rc = PTR_RET(curr_vreg->vreg);
-                       if (rc) {
-                               DEV_ERR("%pS->%s: %s get failed. rc=%d\n",
-                                        __builtin_return_address(0), __func__,
-                                        curr_vreg->vreg_name, rc);
-                               curr_vreg->vreg = NULL;
-                               goto vreg_get_fail;
-                       }
-                       type = (regulator_count_voltages(curr_vreg->vreg) > 0)
-                                       ? DSS_REG_LDO : DSS_REG_VS;
-                       if (type == DSS_REG_LDO) {
-                               rc = regulator_set_voltage(
-                                       curr_vreg->vreg,
-                                       curr_vreg->min_voltage,
-                                       curr_vreg->max_voltage);
-                               if (rc < 0) {
-                                       DEV_ERR("%pS->%s: %s set vltg fail\n",
-                                               __builtin_return_address(0),
-                                               __func__,
-                                               curr_vreg->vreg_name);
-                                       goto vreg_set_voltage_fail;
-                               }
-                       }
-               }
-       } else {
-               for (i = num_vreg-1; i >= 0; i--) {
-                       curr_vreg = &in_vreg[i];
-                       if (curr_vreg->vreg) {
-                               type = (regulator_count_voltages(
-                                       curr_vreg->vreg) > 0)
-                                       ? DSS_REG_LDO : DSS_REG_VS;
-                               if (type == DSS_REG_LDO) {
-                                       regulator_set_voltage(curr_vreg->vreg,
-                                               0, curr_vreg->max_voltage);
-                               }
-                               regulator_put(curr_vreg->vreg);
-                               curr_vreg->vreg = NULL;
-                       }
-               }
-       }
-       return 0;
-
-vreg_unconfig:
-if (type == DSS_REG_LDO)
-       regulator_set_load(curr_vreg->vreg, 0);
-
-vreg_set_voltage_fail:
-       regulator_put(curr_vreg->vreg);
-       curr_vreg->vreg = NULL;
-
-vreg_get_fail:
-       for (i--; i >= 0; i--) {
-               curr_vreg = &in_vreg[i];
-               type = (regulator_count_voltages(curr_vreg->vreg) > 0)
-                       ? DSS_REG_LDO : DSS_REG_VS;
-               goto vreg_unconfig;
-       }
-       return rc;
-} /* msm_dss_config_vreg */
-EXPORT_SYMBOL(msm_dss_config_vreg);
-
-int msm_dss_enable_vreg(struct dss_vreg *in_vreg, int num_vreg, int enable)
-{
-       int i = 0, rc = 0;
-       bool need_sleep;
-
-       if (enable) {
-               for (i = 0; i < num_vreg; i++) {
-                       rc = PTR_RET(in_vreg[i].vreg);
-                       if (rc) {
-                               DEV_ERR("%pS->%s: %s regulator error. rc=%d\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_vreg[i].vreg_name, rc);
-                               goto vreg_set_opt_mode_fail;
-                       }
-                       need_sleep = !regulator_is_enabled(in_vreg[i].vreg);
-                       if (in_vreg[i].pre_on_sleep && need_sleep)
-                               usleep_range(in_vreg[i].pre_on_sleep * 1000,
-                                       in_vreg[i].pre_on_sleep * 1000);
-                       rc = regulator_set_load(in_vreg[i].vreg,
-                               in_vreg[i].enable_load);
-                       if (rc < 0) {
-                               DEV_ERR("%pS->%s: %s set opt m fail\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_vreg[i].vreg_name);
-                               goto vreg_set_opt_mode_fail;
-                       }
-                       rc = regulator_enable(in_vreg[i].vreg);
-                       if (in_vreg[i].post_on_sleep && need_sleep)
-                               usleep_range(in_vreg[i].post_on_sleep * 1000,
-                                       in_vreg[i].post_on_sleep * 1000);
-                       if (rc < 0) {
-                               DEV_ERR("%pS->%s: %s enable failed\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_vreg[i].vreg_name);
-                               goto disable_vreg;
-                       }
-               }
-       } else {
-               for (i = num_vreg-1; i >= 0; i--) {
-                       if (in_vreg[i].pre_off_sleep)
-                               usleep_range(in_vreg[i].pre_off_sleep * 1000,
-                                       in_vreg[i].pre_off_sleep * 1000);
-                       regulator_set_load(in_vreg[i].vreg,
-                               in_vreg[i].disable_load);
-                       regulator_disable(in_vreg[i].vreg);
-                       if (in_vreg[i].post_off_sleep)
-                               usleep_range(in_vreg[i].post_off_sleep * 1000,
-                                       in_vreg[i].post_off_sleep * 1000);
-               }
-       }
-       return rc;
-
-disable_vreg:
-       regulator_set_load(in_vreg[i].vreg, in_vreg[i].disable_load);
-
-vreg_set_opt_mode_fail:
-       for (i--; i >= 0; i--) {
-               if (in_vreg[i].pre_off_sleep)
-                       usleep_range(in_vreg[i].pre_off_sleep * 1000,
-                               in_vreg[i].pre_off_sleep * 1000);
-               regulator_set_load(in_vreg[i].vreg,
-                       in_vreg[i].disable_load);
-               regulator_disable(in_vreg[i].vreg);
-               if (in_vreg[i].post_off_sleep)
-                       usleep_range(in_vreg[i].post_off_sleep * 1000,
-                               in_vreg[i].post_off_sleep * 1000);
-       }
-
-       return rc;
-} /* msm_dss_enable_vreg */
-EXPORT_SYMBOL(msm_dss_enable_vreg);
-
-int msm_dss_enable_gpio(struct dss_gpio *in_gpio, int num_gpio, int enable)
-{
-       int i = 0, rc = 0;
-
-       if (enable) {
-               for (i = 0; i < num_gpio; i++) {
-                       DEV_DBG("%pS->%s: %s enable\n",
-                               __builtin_return_address(0), __func__,
-                               in_gpio[i].gpio_name);
-
-                       rc = gpio_request(in_gpio[i].gpio,
-                               in_gpio[i].gpio_name);
-                       if (rc < 0) {
-                               DEV_ERR("%pS->%s: %s enable failed\n",
-                                       __builtin_return_address(0), __func__,
-                                       in_gpio[i].gpio_name);
-                               goto disable_gpio;
-                       }
-                       gpio_set_value(in_gpio[i].gpio, in_gpio[i].value);
-               }
-       } else {
-               for (i = num_gpio-1; i >= 0; i--) {
-                       DEV_DBG("%pS->%s: %s disable\n",
-                               __builtin_return_address(0), __func__,
-                               in_gpio[i].gpio_name);
-                       if (in_gpio[i].gpio)
-                               gpio_free(in_gpio[i].gpio);
-               }
-       }
-       return rc;
-
-disable_gpio:
-       for (i--; i >= 0; i--)
-               if (in_gpio[i].gpio)
-                       gpio_free(in_gpio[i].gpio);
-
-       return rc;
-} /* msm_dss_enable_gpio */
-EXPORT_SYMBOL(msm_dss_enable_gpio);
+#include "dpu_io_util.h"
 
 void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk)
 {

@@ -335,8 +26,7 @@ void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk)
                        clk_put(clk_arry[i].clk);
                clk_arry[i].clk = NULL;
        }
-} /* msm_dss_put_clk */
-EXPORT_SYMBOL(msm_dss_put_clk);
+}
 
 int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk)
 {

@@ -363,8 +53,7 @@ int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk)
        }
 
        return rc;
-} /* msm_dss_get_clk */
-EXPORT_SYMBOL(msm_dss_get_clk);
+}
 
 int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk)
 {

@@ -397,8 +86,7 @@ int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk)
        }
 
        return rc;
-} /* msm_dss_clk_set_rate */
-EXPORT_SYMBOL(msm_dss_clk_set_rate);
+}
 
 int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable)
 {

@@ -445,8 +133,7 @@ int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable)
        }
 
        return rc;
-} /* msm_dss_enable_clk */
-EXPORT_SYMBOL(msm_dss_enable_clk);
+}
 
 int msm_dss_parse_clock(struct platform_device *pdev,
                struct dss_module_power *mp)

@@ -497,60 +184,3 @@ int msm_dss_parse_clock(struct platform_device *pdev,
 
        return rc;
 }
-
-int dpu_i2c_byte_read(struct i2c_client *client, uint8_t slave_addr,
-                       uint8_t reg_offset, uint8_t *read_buf)
-{
-       struct i2c_msg msgs[2];
-       int ret = -1;
-
-       pr_debug("%s: reading from slave_addr=[%x] and offset=[%x]\n",
-                __func__, slave_addr, reg_offset);
-
-       msgs[0].addr = slave_addr >> 1;
-       msgs[0].flags = 0;
-       msgs[0].buf = &reg_offset;
-       msgs[0].len = 1;
-
-       msgs[1].addr = slave_addr >> 1;
-       msgs[1].flags = I2C_M_RD;
-       msgs[1].buf = read_buf;
-       msgs[1].len = 1;
-
-       ret = i2c_transfer(client->adapter, msgs, 2);
-       if (ret < 1) {
-               pr_err("%s: I2C READ FAILED=[%d]\n", __func__, ret);
-               return -EACCES;
-       }
-       pr_debug("%s: i2c buf is [%x]\n", __func__, *read_buf);
-       return 0;
-}
-EXPORT_SYMBOL(dpu_i2c_byte_read);
-
-int dpu_i2c_byte_write(struct i2c_client *client, uint8_t slave_addr,
-                       uint8_t reg_offset, uint8_t *value)
-{
-       struct i2c_msg msgs[1];
-       uint8_t data[2];
-       int status = -EACCES;
-
-       pr_debug("%s: writing from slave_addr=[%x] and offset=[%x]\n",
-                __func__, slave_addr, reg_offset);
-
-       data[0] = reg_offset;
-       data[1] = *value;
-
-       msgs[0].addr = slave_addr >> 1;
-       msgs[0].flags = 0;
-       msgs[0].len = 2;
-       msgs[0].buf = data;
-
-       status = i2c_transfer(client->adapter, msgs, 1);
-       if (status < 1) {
-               pr_err("I2C WRITE FAILED=[%d]\n", status);
-               return -EACCES;
-       }
-       pr_debug("%s: I2C write status=%x\n", __func__, status);
-       return status;
-}
-EXPORT_SYMBOL(dpu_i2c_byte_write);

drivers/gpu/drm/msm/dpu_io_util.h new file +61 -0

new file mode 100644
index 0000000..773b52e
@@ -0,0 +1,61 @@
+/* Copyright (c) 2012, 2017-2018, The Linux Foundation. All rights reserved.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef __DPU_IO_UTIL_H__
+#define __DPU_IO_UTIL_H__
+
+#include <linux/gpio.h>
+#include <linux/platform_device.h>
+#include <linux/types.h>
+
+#ifdef DEBUG
+#define DEV_DBG(fmt, args...)   pr_err(fmt, ##args)
+#else
+#define DEV_DBG(fmt, args...)   pr_debug(fmt, ##args)
+#endif
+#define DEV_INFO(fmt, args...)  pr_info(fmt, ##args)
+#define DEV_WARN(fmt, args...)  pr_warn(fmt, ##args)
+#define DEV_ERR(fmt, args...)   pr_err(fmt, ##args)
+
+struct dss_gpio {
+       unsigned int gpio;
+       unsigned int value;
+       char gpio_name[32];
+};
+
+enum dss_clk_type {
+       DSS_CLK_AHB, /* no set rate. rate controlled through rpm */
+       DSS_CLK_PCLK,
+};
+
+struct dss_clk {
+       struct clk *clk; /* clk handle */
+       char clk_name[32];
+       enum dss_clk_type type;
+       unsigned long rate;
+       unsigned long max_rate;
+};
+
+struct dss_module_power {
+       unsigned int num_gpio;
+       struct dss_gpio *gpio_config;
+       unsigned int num_clk;
+       struct dss_clk *clk_config;
+};
+
+int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk);
+void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk);
+int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk);
+int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable);
+int msm_dss_parse_clock(struct platform_device *pdev,
+               struct dss_module_power *mp);
+#endif /* __DPU_IO_UTIL_H__ */

drivers/gpu/drm/msm/msm_drv.h +0 -1

index d5937ba..811f7d4 100644
@@ -33,7 +33,6 @@
 #include <linux/types.h>
 #include <linux/of_graph.h>
 #include <linux/of_device.h>
-#include <linux/dpu_io_util.h>
 #include <asm/sizes.h>
 #include <linux/kthread.h>
 

include/linux/dpu_io_util.h deleted +0 -111

deleted file mode 100644
index 45e606f..0000000
@@ -1,115 +0,0 @@
-/* Copyright (c) 2012, 2017-2018, The Linux Foundation. All rights reserved.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 and
- * only version 2 as published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- */
-
-#ifndef __DPU_IO_UTIL_H__
-#define __DPU_IO_UTIL_H__
-
-#include <linux/gpio.h>
-#include <linux/platform_device.h>
-#include <linux/regulator/consumer.h>
-#include <linux/i2c.h>
-#include <linux/types.h>
-
-#ifdef DEBUG
-#define DEV_DBG(fmt, args...)   pr_err(fmt, ##args)
-#else
-#define DEV_DBG(fmt, args...)   pr_debug(fmt, ##args)
-#endif
-#define DEV_INFO(fmt, args...)  pr_info(fmt, ##args)
-#define DEV_WARN(fmt, args...)  pr_warn(fmt, ##args)
-#define DEV_ERR(fmt, args...)   pr_err(fmt, ##args)
-
-struct dss_io_data {
-       u32 len;
-       void __iomem *base;
-};
-
-void dss_reg_w(struct dss_io_data *io, u32 offset, u32 value, u32 debug);
-u32 dss_reg_r(struct dss_io_data *io, u32 offset, u32 debug);
-void dss_reg_dump(void __iomem *base, u32 len, const char *prefix, u32 debug);
-
-#define DSS_REG_W_ND(io, offset, val)  dss_reg_w(io, offset, val, false)
-#define DSS_REG_W(io, offset, val)     dss_reg_w(io, offset, val, true)
-#define DSS_REG_R_ND(io, offset)       dss_reg_r(io, offset, false)
-#define DSS_REG_R(io, offset)          dss_reg_r(io, offset, true)
-
-enum dss_vreg_type {
-       DSS_REG_LDO,
-       DSS_REG_VS,
-};
-
-struct dss_vreg {
-       struct regulator *vreg; /* vreg handle */
-       char vreg_name[32];
-       int min_voltage;
-       int max_voltage;
-       int enable_load;
-       int disable_load;
-       int pre_on_sleep;
-       int post_on_sleep;
-       int pre_off_sleep;
-       int post_off_sleep;
-};
-
-struct dss_gpio {
-       unsigned int gpio;
-       unsigned int value;
-       char gpio_name[32];
-};
-
-enum dss_clk_type {
-       DSS_CLK_AHB, /* no set rate. rate controlled through rpm */
-       DSS_CLK_PCLK,
-       DSS_CLK_OTHER,
-};
-
-struct dss_clk {
-       struct clk *clk; /* clk handle */
-       char clk_name[32];
-       enum dss_clk_type type;
-       unsigned long rate;
-       unsigned long max_rate;
-};
-
-struct dss_module_power {
-       unsigned int num_vreg;
-       struct dss_vreg *vreg_config;
-       unsigned int num_gpio;
-       struct dss_gpio *gpio_config;
-       unsigned int num_clk;
-       struct dss_clk *clk_config;
-};
-
-int msm_dss_ioremap_byname(struct platform_device *pdev,
-       struct dss_io_data *io_data, const char *name);
-void msm_dss_iounmap(struct dss_io_data *io_data);
-
-int msm_dss_enable_gpio(struct dss_gpio *in_gpio, int num_gpio, int enable);
-int msm_dss_gpio_enable(struct dss_gpio *in_gpio, int num_gpio, int enable);
-
-int msm_dss_config_vreg(struct device *dev, struct dss_vreg *in_vreg,
-       int num_vreg, int config);
-int msm_dss_enable_vreg(struct dss_vreg *in_vreg, int num_vreg,        int enable);
-
-int msm_dss_get_clk(struct device *dev, struct dss_clk *clk_arry, int num_clk);
-void msm_dss_put_clk(struct dss_clk *clk_arry, int num_clk);
-int msm_dss_clk_set_rate(struct dss_clk *clk_arry, int num_clk);
-int msm_dss_enable_clk(struct dss_clk *clk_arry, int num_clk, int enable);
-int msm_dss_parse_clock(struct platform_device *pdev,
-               struct dss_module_power *mp);
-
-int dpu_i2c_byte_read(struct i2c_client *client, uint8_t slave_addr,
-                      uint8_t reg_offset, uint8_t *read_buf);

Quoted 8 lines

Quoted 8 lines

include/linux/dpu_io_util.h deleted +0 -5

-int dpu_i2c_byte_write(struct i2c_client *client, uint8_t slave_addr,
-                       uint8_t reg_offset, uint8_t *value);
-
-#endif /* __DPU_IO_UTIL_H__ */
--

The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project

dpu_io_util.h is moved from standard include path to driver folder, correct the include path in code.

Signed-off-by Rajesh Yadav <[email protected]>

---
 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c | 1 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c +0 -1

index 24c3274..f997bd9 100644
@@ -20,7 +20,6 @@
 #include <linux/slab.h>
 #include <linux/mutex.h>
 #include <linux/of_platform.h>
-#include <linux/dpu_io_util.h>
 
 #include "dpu_power_handle.h"
 #include "dpu_trace.h"

drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h +1 -1

index 9a6d4b9..193f468 100644
@@ -21,7 +21,7 @@
 #define DPU_POWER_HANDLE_ENABLE_BUS_IB_QUOTA	1600000000
 #define DPU_POWER_HANDLE_DISABLE_BUS_IB_QUOTA	0
 
-#include <linux/dpu_io_util.h>
+#include "dpu_io_util.h"
 
 /* event will be triggered before power handler disable */
 #define DPU_POWER_EVENT_PRE_DISABLE	0x1

Quoted 4 lines · Rajesh Yadav

dpu_io_util.h is moved from standard include path to driver folder, correct the include path in code.

Signed-off-by Rajesh Yadav <[email protected]>

If the previous patch doesn't compile without this fix you should squash them.

Quoted 26 lines

---
 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c | 1 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c +0 -1

index 24c3274..f997bd9 100644
@@ -20,7 +20,6 @@
 #include <linux/slab.h>
 #include <linux/mutex.h>
 #include <linux/of_platform.h>
-#include <linux/dpu_io_util.h>
 
 #include "dpu_power_handle.h"
 #include "dpu_trace.h"

drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h +0 -0

index 9a6d4b9..193f468 100644
@@ -21,7 +21,7 @@
 #define DPU_POWER_HANDLE_ENABLE_BUS_IB_QUOTA	1600000000
 #define DPU_POWER_HANDLE_DISABLE_BUS_IB_QUOTA	0
 

drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h line 27

Quoted 5 lines

drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h +1 -1

-#include <linux/dpu_io_util.h>
+#include "dpu_io_util.h"
 
 /* event will be triggered before power handler disable */
 #define DPU_POWER_EVENT_PRE_DISABLE	0x1