linux-kernel - Re: [PATCH] regulator: core: Define regulator_set_voltage

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]

Date:	Fri, 14 Aug 2015 17:25:06 +0100
From:	Mark Brown <broonie@...nel.org>
To:	Viresh Kumar <viresh.kumar@...aro.org>
Cc:	linaro-kernel@...ts.linaro.org, sboyd@...eaurora.org, nm@...com,
	Liam Girdwood <lgirdwood@...il.com>,
	"open list:VOLTAGE AND CURRENT REGULATOR FRAMEWORK" 
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] regulator: core: Define regulator_set_voltage_triplet()

On Fri, Aug 14, 2015 at 05:30:55PM +0530, Viresh Kumar wrote:

> The OPP (Operating performance points) v2 bindings allows regulator
> voltage to be supplied as a triplet of <target min max> voltages.

Looking at this I really don't understand why you tried to get me to
review this via an IRC pastebin instead of following the normal patch
submission process.  Please don't do that, it's not at all helpful.  IRC
can be useful for interactive discussions or things that are really
urgent, not for normal upstream code review.

> +static inline int regulator_set_voltage_triplet(struct regulator *regulator,
> +						int target_uV, int min_uV,
> +						int max_uV)

This seems awkward, these things are normally written as min <= target
<= max but this is target, min, max.

>  static inline int regulator_set_voltage_tol(struct regulator *regulator,
>  					    int new_uV, int tol_uV)
>  {
> -- 
> 2.4.0

You've not added a stub for the new function.

Download attachment "signature.asc" of type "application/pgp-signature" (474 bytes)