diff mbox series

[V2,06/11] PM / OPP: Implement dev_pm_opp_get_of_node()

Message ID a7617cdbfa02db20572d1d06af40132d475649bd.1523273291.git.viresh.kumar@linaro.org
State Accepted
Commit e2f4b5f8dc59c28605a320ea923905e519fd2ca7
Headers show
Series PM / genpd & OPP: Parse performance state from DT | expand

Commit Message

Viresh Kumar April 9, 2018, 11:43 a.m. UTC
This adds a new helper to let the power domain drivers to access
opp->np, so that they can read platform specific properties from the
node.

Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org>

Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org>

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

---
 drivers/opp/of.c       | 19 +++++++++++++++++++
 include/linux/pm_opp.h |  5 +++++
 2 files changed, 24 insertions(+)

-- 
2.15.0.194.g9af6a3dea062

Comments

Ulf Hansson April 9, 2018, 3:22 p.m. UTC | #1
On 9 April 2018 at 13:43, Viresh Kumar <viresh.kumar@linaro.org> wrote:
> This adds a new helper to let the power domain drivers to access

> opp->np, so that they can read platform specific properties from the

> node.

>

> Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org>

> Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org>

> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>


Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>


Kind regards
Uffe

> ---

>  drivers/opp/of.c       | 19 +++++++++++++++++++

>  include/linux/pm_opp.h |  5 +++++

>  2 files changed, 24 insertions(+)

>

> diff --git a/drivers/opp/of.c b/drivers/opp/of.c

> index 6380ec3d695b..de41e68b780f 100644

> --- a/drivers/opp/of.c

> +++ b/drivers/opp/of.c

> @@ -727,3 +727,22 @@ struct dev_pm_opp *of_dev_pm_opp_find_required_opp(struct device *dev,

>         return opp;

>  }

>  EXPORT_SYMBOL_GPL(of_dev_pm_opp_find_required_opp);

> +

> +/**

> + * dev_pm_opp_get_of_node() - Gets the DT node corresponding to an opp

> + * @opp:       opp for which DT node has to be returned for

> + *

> + * Return: DT node corresponding to the opp, else 0 on success.

> + *

> + * The caller needs to put the node with of_node_put() after using it.

> + */

> +struct device_node *dev_pm_opp_get_of_node(struct dev_pm_opp *opp)

> +{

> +       if (IS_ERR_OR_NULL(opp)) {

> +               pr_err("%s: Invalid parameters\n", __func__);

> +               return NULL;

> +       }

> +

> +       return of_node_get(opp->np);

> +}

> +EXPORT_SYMBOL_GPL(dev_pm_opp_get_of_node);

> diff --git a/include/linux/pm_opp.h b/include/linux/pm_opp.h

> index 70686f434c13..8fd34c4398b2 100644

> --- a/include/linux/pm_opp.h

> +++ b/include/linux/pm_opp.h

> @@ -310,6 +310,7 @@ void dev_pm_opp_of_cpumask_remove_table(const struct cpumask *cpumask);

>  int dev_pm_opp_of_get_sharing_cpus(struct device *cpu_dev, struct cpumask *cpumask);

>  struct device_node *dev_pm_opp_of_get_opp_desc_node(struct device *dev);

>  struct dev_pm_opp *of_dev_pm_opp_find_required_opp(struct device *dev, struct device_node *np);

> +struct device_node *dev_pm_opp_get_of_node(struct dev_pm_opp *opp);

>  #else

>  static inline int dev_pm_opp_of_add_table(struct device *dev)

>  {

> @@ -348,6 +349,10 @@ static inline struct dev_pm_opp *of_dev_pm_opp_find_required_opp(struct device *

>  {

>         return NULL;

>  }

> +static inline struct device_node *dev_pm_opp_get_of_node(struct dev_pm_opp *opp)

> +{

> +       return NULL;

> +}

>  #endif

>

>  #endif         /* __LINUX_OPP_H__ */

> --

> 2.15.0.194.g9af6a3dea062

>
diff mbox series

Patch

diff --git a/drivers/opp/of.c b/drivers/opp/of.c
index 6380ec3d695b..de41e68b780f 100644
--- a/drivers/opp/of.c
+++ b/drivers/opp/of.c
@@ -727,3 +727,22 @@  struct dev_pm_opp *of_dev_pm_opp_find_required_opp(struct device *dev,
 	return opp;
 }
 EXPORT_SYMBOL_GPL(of_dev_pm_opp_find_required_opp);
+
+/**
+ * dev_pm_opp_get_of_node() - Gets the DT node corresponding to an opp
+ * @opp:	opp for which DT node has to be returned for
+ *
+ * Return: DT node corresponding to the opp, else 0 on success.
+ *
+ * The caller needs to put the node with of_node_put() after using it.
+ */
+struct device_node *dev_pm_opp_get_of_node(struct dev_pm_opp *opp)
+{
+	if (IS_ERR_OR_NULL(opp)) {
+		pr_err("%s: Invalid parameters\n", __func__);
+		return NULL;
+	}
+
+	return of_node_get(opp->np);
+}
+EXPORT_SYMBOL_GPL(dev_pm_opp_get_of_node);
diff --git a/include/linux/pm_opp.h b/include/linux/pm_opp.h
index 70686f434c13..8fd34c4398b2 100644
--- a/include/linux/pm_opp.h
+++ b/include/linux/pm_opp.h
@@ -310,6 +310,7 @@  void dev_pm_opp_of_cpumask_remove_table(const struct cpumask *cpumask);
 int dev_pm_opp_of_get_sharing_cpus(struct device *cpu_dev, struct cpumask *cpumask);
 struct device_node *dev_pm_opp_of_get_opp_desc_node(struct device *dev);
 struct dev_pm_opp *of_dev_pm_opp_find_required_opp(struct device *dev, struct device_node *np);
+struct device_node *dev_pm_opp_get_of_node(struct dev_pm_opp *opp);
 #else
 static inline int dev_pm_opp_of_add_table(struct device *dev)
 {
@@ -348,6 +349,10 @@  static inline struct dev_pm_opp *of_dev_pm_opp_find_required_opp(struct device *
 {
 	return NULL;
 }
+static inline struct device_node *dev_pm_opp_get_of_node(struct dev_pm_opp *opp)
+{
+	return NULL;
+}
 #endif
 
 #endif		/* __LINUX_OPP_H__ */