linux-kernel - Re: [PATCH 1/4] opp: add dev_pm_opp_is

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150728142713.GB17745@amd>
Date:	Tue, 28 Jul 2015 16:27:13 +0200
From:	Pavel Machek <pavel@....cz>
To:	Viresh Kumar <viresh.kumar@...aro.org>
Cc:	Rafael Wysocki <rjw@...ysocki.net>, linaro-kernel@...ts.linaro.org,
	linux-pm@...r.kernel.org, b.zolnierkie@...sung.com, nm@...com,
	sboyd@...eaurora.org, Tomasz Figa <tomasz.figa@...il.com>,
	Michael Turquette <mturquette@...libre.com>,
	Javier Martinez Canillas <javier@...hile0.org>,
	Thomas Abraham <thomas.ab@...sung.com>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Len Brown <len.brown@...el.com>,
	open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/4] opp: add dev_pm_opp_is_turbo() helper

On Tue 2015-07-28 16:50:00, Viresh Kumar wrote:
> From: Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>
> 
> Add dev_pm_opp_is_turbo() helper to verify if an opp is to be used only
> for turbo mode or not.
> 
> Cc: Tomasz Figa <tomasz.figa@...il.com>
> Cc: Michael Turquette <mturquette@...libre.com>
> Cc: Javier Martinez Canillas <javier@...hile0.org>
> Cc: Thomas Abraham <thomas.ab@...sung.com>
> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>
> Signed-off-by: Viresh Kumar <viresh.kumar@...aro.org>
> ---
>  drivers/base/power/opp.c | 30 ++++++++++++++++++++++++++++++
>  include/linux/pm_opp.h   |  7 +++++++
>  2 files changed, 37 insertions(+)
> 
> diff --git a/drivers/base/power/opp.c b/drivers/base/power/opp.c
> index b38f2600d797..a0c54058c4dd 100644
> --- a/drivers/base/power/opp.c
> +++ b/drivers/base/power/opp.c
> @@ -281,6 +281,36 @@ unsigned long dev_pm_opp_get_freq(struct dev_pm_opp *opp)
>  EXPORT_SYMBOL_GPL(dev_pm_opp_get_freq);
>  
>  /**
> + * dev_pm_opp_is_turbo() - Returns true if opp was trubo OPP

Typo.

And this is not exactly most helpful comment. opp is turbo... returns
if opp was turbo. Explanation what "turbo" means in this context would
be nice. I'm pretty sure it is not about air compression :-).

> + * Return: true if opp is turbo opp, else false.

Duplicate from above.

Best regards,
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/