lkml.org 
[lkml]   [2018]   [Oct]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH V3 10/10] OPP: Remove of_dev_pm_opp_find_required_opp()
On 25 October 2018 at 07:52, Viresh Kumar <viresh.kumar@linaro.org> wrote:
> This isn't used anymore, remove it.
>
> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

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

Kind regards
Uffe

> ---
> drivers/opp/of.c | 54 ------------------------------------------
> include/linux/pm_opp.h | 5 ----
> 2 files changed, 59 deletions(-)
>
> diff --git a/drivers/opp/of.c b/drivers/opp/of.c
> index 0755ee307b1a..da31874c7fe9 100644
> --- a/drivers/opp/of.c
> +++ b/drivers/opp/of.c
> @@ -1013,60 +1013,6 @@ unsigned int of_get_required_opp_performance_state(struct device_node *np,
> }
> EXPORT_SYMBOL_GPL(of_get_required_opp_performance_state);
>
> -/**
> - * of_dev_pm_opp_find_required_opp() - Search for required OPP.
> - * @dev: The device whose OPP node is referenced by the 'np' DT node.
> - * @np: Node that contains the "required-opps" property.
> - *
> - * Returns the OPP of the device 'dev', whose phandle is present in the "np"
> - * node. Although the "required-opps" property supports having multiple
> - * phandles, this helper routine only parses the very first phandle in the list.
> - *
> - * Return: Matching opp, else returns ERR_PTR in case of error and should be
> - * handled using IS_ERR.
> - *
> - * The callers are required to call dev_pm_opp_put() for the returned OPP after
> - * use.
> - */
> -struct dev_pm_opp *of_dev_pm_opp_find_required_opp(struct device *dev,
> - struct device_node *np)
> -{
> - struct dev_pm_opp *temp_opp, *opp = ERR_PTR(-ENODEV);
> - struct device_node *required_np;
> - struct opp_table *opp_table;
> -
> - opp_table = _find_opp_table(dev);
> - if (IS_ERR(opp_table))
> - return ERR_CAST(opp_table);
> -
> - required_np = of_parse_phandle(np, "required-opps", 0);
> - if (unlikely(!required_np)) {
> - dev_err(dev, "Unable to parse required-opps\n");
> - goto put_opp_table;
> - }
> -
> - mutex_lock(&opp_table->lock);
> -
> - list_for_each_entry(temp_opp, &opp_table->opp_list, node) {
> - if (temp_opp->available && temp_opp->np == required_np) {
> - opp = temp_opp;
> -
> - /* Increment the reference count of OPP */
> - dev_pm_opp_get(opp);
> - break;
> - }
> - }
> -
> - mutex_unlock(&opp_table->lock);
> -
> - of_node_put(required_np);
> -put_opp_table:
> - dev_pm_opp_put_opp_table(opp_table);
> -
> - 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
> diff --git a/include/linux/pm_opp.h b/include/linux/pm_opp.h
> index 889bb347fbd9..2b2c3fd985ab 100644
> --- a/include/linux/pm_opp.h
> +++ b/include/linux/pm_opp.h
> @@ -313,7 +313,6 @@ int dev_pm_opp_of_cpumask_add_table(const struct cpumask *cpumask);
> 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);
> unsigned int of_get_required_opp_performance_state(struct device_node *np, int index);
> #else
> @@ -350,10 +349,6 @@ static inline struct device_node *dev_pm_opp_of_get_opp_desc_node(struct device
> return NULL;
> }
>
> -static inline struct dev_pm_opp *of_dev_pm_opp_find_required_opp(struct device *dev, struct device_node *np)
> -{
> - return NULL;
> -}
> static inline struct device_node *dev_pm_opp_get_of_node(struct dev_pm_opp *opp)
> {
> return NULL;
> --
> 2.19.1.568.g152ad8e3369a
>

\
 
 \ /
  Last update: 2018-10-25 12:56    [W:0.099 / U:5.040 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site