@@ -210,25 +210,18 @@ int devfreq_event_reset_event(struct devfreq_event_dev *edev)
EXPORT_SYMBOL_GPL(devfreq_event_reset_event);
/**
- * devfreq_event_get_edev_by_phandle() - Get the devfreq-event dev from
- * devicetree.
- * @dev : the pointer to the given device
- * @index : the index into list of devfreq-event device
+ * devfreq_event_get_edev_by_node() - Get the devfreq-event dev from devicetree
+ * @node : the pointer to device_node
*
* Note that this function return the pointer of devfreq-event device.
*/
-struct devfreq_event_dev *devfreq_event_get_edev_by_phandle(struct device *dev,
- int index)
+struct devfreq_event_dev *devfreq_event_get_edev_by_node(
+ struct device_node *node)
{
- struct device_node *node;
struct devfreq_event_dev *edev;
- if (!dev->of_node)
- return ERR_PTR(-EINVAL);
-
- node = of_parse_phandle(dev->of_node, "devfreq-events", index);
if (!node)
- return ERR_PTR(-ENODEV);
+ return ERR_PTR(-EINVAL);
mutex_lock(&devfreq_event_list_lock);
list_for_each_entry(edev, &devfreq_event_list, node) {
@@ -244,44 +237,12 @@ struct devfreq_event_dev *devfreq_event_get_edev_by_phandle(struct device *dev,
out:
mutex_unlock(&devfreq_event_list_lock);
- if (!edev) {
- of_node_put(node);
+ if (!edev)
return ERR_PTR(-ENODEV);
- }
-
- of_node_put(node);
return edev;
}
-EXPORT_SYMBOL_GPL(devfreq_event_get_edev_by_phandle);
-
-/**
- * devfreq_event_get_edev_count() - Get the count of devfreq-event dev
- * @dev : the pointer to the given device
- *
- * Note that this function return the count of devfreq-event devices.
- */
-int devfreq_event_get_edev_count(struct device *dev)
-{
- int count;
-
- if (!dev->of_node) {
- dev_err(dev, "device does not have a device node entry\n");
- return -EINVAL;
- }
-
- count = of_property_count_elems_of_size(dev->of_node, "devfreq-events",
- sizeof(u32));
- if (count < 0) {
- dev_err(dev,
- "failed to get the count of devfreq-event in %pOF node\n",
- dev->of_node);
- return count;
- }
-
- return count;
-}
-EXPORT_SYMBOL_GPL(devfreq_event_get_edev_count);
+EXPORT_SYMBOL_GPL(devfreq_event_get_edev_by_node);
static void devfreq_event_release_edev(struct device *dev)
{
@@ -96,6 +96,17 @@ static struct devfreq *exynos_bus_get_parent_devfreq(struct device_node *np)
return devfreq_get_devfreq_by_node(node);
}
+static struct devfreq_event_dev *exynos_bus_get_edev(struct device_node *np,
+ int index)
+{
+ struct device_node *node = of_parse_phandle(np, "devfreq-events",
+ index);
+
+ if (!node)
+ return ERR_PTR(-ENODEV);
+ return devfreq_event_get_edev_by_node(node);
+}
+
/*
* devfreq function for both simple-ondemand and passive governor
*/
@@ -202,7 +213,8 @@ static int exynos_bus_parent_parse_of(struct device_node *np,
* Get the devfreq-event devices to get the current utilization of
* buses. This raw data will be used in devfreq ondemand governor.
*/
- count = devfreq_event_get_edev_count(dev);
+ count = of_property_count_elems_of_size(dev->of_node,
+ "devfreq-events", sizeof(u32));
if (count < 0) {
dev_err(dev, "failed to get the count of devfreq-event dev\n");
ret = count;
@@ -218,7 +230,7 @@ static int exynos_bus_parent_parse_of(struct device_node *np,
}
for (i = 0; i < count; i++) {
- bus->edev[i] = devfreq_event_get_edev_by_phandle(dev, i);
+ bus->edev[i] = exynos_bus_get_edev(dev->of_node, i);
if (IS_ERR(bus->edev[i])) {
ret = -EPROBE_DEFER;
goto err_regulator;
@@ -303,6 +303,17 @@ static int of_get_ddr_timings(struct dram_timing *timing,
return ret;
}
+static struct devfreq_event_dev *of_get_edev(struct device_node *np,
+ int index)
+{
+ struct device_node *node = of_parse_phandle(np, "devfreq-events",
+ index);
+
+ if (!node)
+ return ERR_PTR(-ENODEV);
+ return devfreq_event_get_edev_by_node(node);
+}
+
static int rk3399_dmcfreq_probe(struct platform_device *pdev)
{
struct arm_smccc_res res;
@@ -339,7 +350,7 @@ static int rk3399_dmcfreq_probe(struct platform_device *pdev)
return PTR_ERR(data->dmc_clk);
}
- data->edev = devfreq_event_get_edev_by_phandle(dev, 0);
+ data->edev = of_get_edev(dev->of_node, 0);
if (IS_ERR(data->edev))
return -EPROBE_DEFER;
@@ -1267,6 +1267,16 @@ static int exynos5_dmc_init_clks(struct exynos5_dmc *dmc)
return 0;
}
+static struct devfreq_event_dev *exynos5_dmc_get_edev(struct device_node *np,
+ int index)
+{
+ struct device_node *node = of_parse_phandle(np, "devfreq-events",
+ index);
+ if (!node)
+ return ERR_PTR(-ENODEV);
+ return devfreq_event_get_edev_by_node(node);
+}
+
/**
* exynos5_performance_counters_init() - Initializes performance DMC's counters
* @dmc: DMC for which it does the setup
@@ -1281,7 +1291,8 @@ static int exynos5_performance_counters_init(struct exynos5_dmc *dmc)
int counters_size;
int ret, i;
- dmc->num_counters = devfreq_event_get_edev_count(dmc->dev);
+ dmc->num_counters = of_property_count_elems_of_size(dmc->dev->of_node,
+ "devfreq-events", sizeof(u32));
if (dmc->num_counters < 0) {
dev_err(dmc->dev, "could not get devfreq-event counters\n");
return dmc->num_counters;
@@ -1293,8 +1304,7 @@ static int exynos5_performance_counters_init(struct exynos5_dmc *dmc)
return -ENOMEM;
for (i = 0; i < dmc->num_counters; i++) {
- dmc->counter[i] =
- devfreq_event_get_edev_by_phandle(dmc->dev, i);
+ dmc->counter[i] = exynos5_dmc_get_edev(dmc->dev->of_node, i);
if (IS_ERR_OR_NULL(dmc->counter[i]))
return -EPROBE_DEFER;
}
@@ -105,9 +105,8 @@ extern int devfreq_event_set_event(struct devfreq_event_dev *edev);
extern int devfreq_event_get_event(struct devfreq_event_dev *edev,
struct devfreq_event_data *edata);
extern int devfreq_event_reset_event(struct devfreq_event_dev *edev);
-extern struct devfreq_event_dev *devfreq_event_get_edev_by_phandle(
- struct device *dev, int index);
-extern int devfreq_event_get_edev_count(struct device *dev);
+extern struct devfreq_event_dev *devfreq_event_get_edev_by_node(
+ struct device_node *node);
extern struct devfreq_event_dev *devfreq_event_add_edev(struct device *dev,
struct devfreq_event_desc *desc);
extern int devfreq_event_remove_edev(struct devfreq_event_dev *edev);
@@ -151,17 +150,12 @@ static inline int devfreq_event_reset_event(struct devfreq_event_dev *edev)
return -EINVAL;
}
-static inline struct devfreq_event_dev *devfreq_event_get_edev_by_phandle(
- struct device *dev, int index)
+static inline struct devfreq_event_dev *devfreq_event_get_edev_by_node(
+ struct device_node *node)
{
return ERR_PTR(-EINVAL);
}
-static inline int devfreq_event_get_edev_count(struct device *dev)
-{
- return -EINVAL;
-}
-
static inline struct devfreq_event_dev *devfreq_event_add_edev(struct device *dev,
struct devfreq_event_desc *desc)
{
Previously, devfreq-event core supports 'devfreq-events' property in order to get the devfreq_event_dev device by phandle. But, 'devfreq-events' property name is not proper on devicetree binding because this name doesn't mean the any h/w attribute. The devfreq-event core hands over the rights of which decide the property name for getting the devfreq device on devicetree, to each devfreq device driver. Each devfreq driver will decide the property name on devicetree binding and then get the devfreq device by using devfreq_event_get_edev_by_node(). And Remove devfreq_event_get_edev_count() function used 'devfreq-events'. Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> --- drivers/devfreq/devfreq-event.c | 53 ++++--------------------- drivers/devfreq/exynos-bus.c | 16 +++++++- drivers/devfreq/rk3399_dmc.c | 13 +++++- drivers/memory/samsung/exynos5422-dmc.c | 16 ++++++-- include/linux/devfreq-event.h | 14 ++----- 5 files changed, 50 insertions(+), 62 deletions(-)