Re: [PATCH 1/7] PM / OPP: Add dev_pm_opp_get_of_node()
On Sat, Apr 07, 2018 at 06:00:29AM -0700, Stephen Boyd wrote: > Quoting Rajendra Nayak (2018-03-15 21:08:18) > > From: Viresh Kumar > > > > 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: Viresh Kumar > > Signed-off-by: Rajendra Nayak > > --- > > Are there two versions of this patch floating around? The other one was mine, but it is identical to this. Jordan -- The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
Re: [PATCH 1/7] PM / OPP: Add dev_pm_opp_get_of_node()
On 07-04-18, 06:00, Stephen Boyd wrote: > Quoting Rajendra Nayak (2018-03-15 21:08:18) > > From: Viresh Kumar > > > > 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: Viresh Kumar > > Signed-off-by: Rajendra Nayak > > --- > > Are there two versions of this patch floating around? Yeah, I will carry this patch in my series now going forward. > > 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 21265af55117..b17715bc3c0a 100644 > > --- a/drivers/opp/of.c > > +++ b/drivers/opp/of.c > > @@ -736,3 +736,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 of_node_put() after using it. > > put the node with of_node_put() Fixed it up, thanks. -- viresh
Re: [PATCH 1/7] PM / OPP: Add dev_pm_opp_get_of_node()
Quoting Rajendra Nayak (2018-03-15 21:08:18) > From: Viresh Kumar > > 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: Viresh Kumar > Signed-off-by: Rajendra Nayak > --- Are there two versions of this patch floating around? > 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 21265af55117..b17715bc3c0a 100644 > --- a/drivers/opp/of.c > +++ b/drivers/opp/of.c > @@ -736,3 +736,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 of_node_put() after using it. put the node with of_node_put()