[PATCH 1/4] ARM: shmobile: Ignore callbacks for subsys generic_pm_domain_data

Simon Horman horms at verge.net.au
Thu Apr 24 18:06:51 PDT 2014


On Thu, Apr 24, 2014 at 01:25:14PM +0200, Ulf Hansson wrote:
> On 14 April 2014 11:22, Ulf Hansson <ulf.hansson at linaro.org> wrote:
> > On 14 April 2014 03:20, Simon Horman <horms at verge.net.au> wrote:
> >> On Fri, Apr 11, 2014 at 05:26:41PM +0200, Ulf Hansson wrote:
> >>> There are no active users of these callbacks, thus there are no benefit
> >>> of trying to invoke them.
> >>>
> >>> Cc: Simon Horman <horms at verge.net.au>
> >>> Cc: Magnus Damm <magnus.damm at gmail.com>
> >>> Signed-off-by: Ulf Hansson <ulf.hansson at linaro.org>
> >>> ---
> >>>
> >>> I am seeking ack from the SOC maintainers since this needs to go with the other
> >>> patches in this patchset on the PM core.
> >>
> >> In order to reduce the possibility of conflicts I would prefer
> >> to take this patch through my tree.
> >
> > Okay, go ahead!
> >
> 
> Hi Simon,
> 
> Just wanted to know, if you decided to queue this patch in your tree.
> Or were there anything else you needed me to fix here?

Sorry for dropping the ball. I will queue-up this patch.

> 
> > Kind regards
> > Ulf Hansson
> >
> >>
> >>> ---
> >>>  arch/arm/mach-shmobile/pm-rmobile.c |   38 +++--------------------------------
> >>>  1 file changed, 3 insertions(+), 35 deletions(-)
> >>>
> >>> diff --git a/arch/arm/mach-shmobile/pm-rmobile.c b/arch/arm/mach-shmobile/pm-rmobile.c
> >>> index 1fc05d9..f710235 100644
> >>> --- a/arch/arm/mach-shmobile/pm-rmobile.c
> >>> +++ b/arch/arm/mach-shmobile/pm-rmobile.c
> >>> @@ -99,39 +99,7 @@ static int rmobile_pd_power_up(struct generic_pm_domain *genpd)
> >>>
> >>>  static bool rmobile_pd_active_wakeup(struct device *dev)
> >>>  {
> >>> -     bool (*active_wakeup)(struct device *dev);
> >>> -
> >>> -     active_wakeup = dev_gpd_data(dev)->ops.active_wakeup;
> >>> -     return active_wakeup ? active_wakeup(dev) : true;
> >>> -}
> >>> -
> >>> -static int rmobile_pd_stop_dev(struct device *dev)
> >>> -{
> >>> -     int (*stop)(struct device *dev);
> >>> -
> >>> -     stop = dev_gpd_data(dev)->ops.stop;
> >>> -     if (stop) {
> >>> -             int ret = stop(dev);
> >>> -             if (ret)
> >>> -                     return ret;
> >>> -     }
> >>> -     return pm_clk_suspend(dev);
> >>> -}
> >>> -
> >>> -static int rmobile_pd_start_dev(struct device *dev)
> >>> -{
> >>> -     int (*start)(struct device *dev);
> >>> -     int ret;
> >>> -
> >>> -     ret = pm_clk_resume(dev);
> >>> -     if (ret)
> >>> -             return ret;
> >>> -
> >>> -     start = dev_gpd_data(dev)->ops.start;
> >>> -     if (start)
> >>> -             ret = start(dev);
> >>> -
> >>> -     return ret;
> >>> +     return true;
> >>>  }
> >>>
> >>>  static void rmobile_init_pm_domain(struct rmobile_pm_domain *rmobile_pd)
> >>> @@ -140,8 +108,8 @@ static void rmobile_init_pm_domain(struct rmobile_pm_domain *rmobile_pd)
> >>>       struct dev_power_governor *gov = rmobile_pd->gov;
> >>>
> >>>       pm_genpd_init(genpd, gov ? : &simple_qos_governor, false);
> >>> -     genpd->dev_ops.stop             = rmobile_pd_stop_dev;
> >>> -     genpd->dev_ops.start            = rmobile_pd_start_dev;
> >>> +     genpd->dev_ops.stop             = pm_clk_suspend;
> >>> +     genpd->dev_ops.start            = pm_clk_resume;
> >>>       genpd->dev_ops.active_wakeup    = rmobile_pd_active_wakeup;
> >>>       genpd->dev_irq_safe             = true;
> >>>       genpd->power_off                = rmobile_pd_power_down;
> >>> --
> >>> 1.7.9.5
> >>>
> >>>
> >>> _______________________________________________
> >>> linux-arm-kernel mailing list
> >>> linux-arm-kernel at lists.infradead.org
> >>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> >>>
> 



More information about the linux-arm-kernel mailing list