MIME-Version: 1.0
References: <2633024.mvXUDI8C0e@kreacher>
In-Reply-To: <2633024.mvXUDI8C0e@kreacher>
From:   Ulf Hansson <ulf.hansson@linaro.org>
Date:   Mon, 6 Dec 2021 12:26:52 +0100
Message-ID: <CAPDyKFq4KPbrQM7B79NsE1MjiOJ-3rRNt8ZEN+JvW=_pS-1JbQ@mail.gmail.com>
Subject: Re: [PATCH] PM: runtime: Fix pm_runtime_active() kerneldoc comment
To:     "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc:     Linux PM <linux-pm@vger.kernel.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Alan Stern <stern@rowland.harvard.edu>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Fri, 3 Dec 2021 at 17:19, Rafael J. Wysocki <rjw@rjwysocki.net> wrote:
>
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
>
> The kerneldoc comment of pm_runtime_active() does not reflect the
> behavior of the function, so update it accordingly.
>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

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

Kind regards
Uffe

> ---
>  include/linux/pm_runtime.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> Index: linux-pm/include/linux/pm_runtime.h
> ===================================================================
> --- linux-pm.orig/include/linux/pm_runtime.h
> +++ linux-pm/include/linux/pm_runtime.h
> @@ -129,7 +129,7 @@ static inline bool pm_runtime_suspended(
>   * pm_runtime_active - Check whether or not a device is runtime-active.
>   * @dev: Target device.
>   *
> - * Return %true if runtime PM is enabled for @dev and its runtime PM status is
> + * Return %true if runtime PM is disabled for @dev or its runtime PM status is
>   * %RPM_ACTIVE, or %false otherwise.
>   *
>   * Note that the return value of this function can only be trusted if it is
>
>
>