Re: In kernel power management domain_pm created for async schedules

["gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>]

On Tue, Dec 05, 2017 at 09:24:18AM +0000, Vikas Bansal wrote:
> >From 8e948f8c697d661f38c2c3a85787e4c48b165adc Mon Sep 17 00:00:00 2001
> From: Vikas Bansal <vikas.bansal@samsung.com>
> Date: Tue, 5 Dec 2017 01:14:41 -0800

Why is all of this here in the patch itself?

> Subject: [PATCH] Put all the async schdules in PM core in domain with name
>  domain_pm. This avoids pm core to wait for async schedules
>  in system other than PM core.

Something went wrong with your commit, you have a very long subject and
no change log text :(

> Signed-off-by: Vikas Bansal <vikas.bansal@samsung.com>
> ---
>  drivers/base/power/main.c |   25 +++++++++++++------------
>  1 file changed, 13 insertions(+), 12 deletions(-)
> 
> diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
> index db2f044..2d19dfb 100644
> --- a/drivers/base/power/main.c
> +++ b/drivers/base/power/main.c
> @@ -39,6 +39,7 @@
>  #include "power.h"
>  
>  typedef int (*pm_callback_t)(struct device *);
> +ASYNC_DOMAIN(domain_pm);

Why is this a global symbol?

I don't understand why this is needed, please explain the problem you
are having with the existing code.

thanks,

greg k-h