linux-kernel - Re: [PATCH v2] regulator: lp872x: support the device tree feature

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] [day] [month] [year] [list]

Message-ID: <20130516154627.GN24524@sirena.org.uk>
Date:	Thu, 16 May 2013 19:46:27 +0400
From:	Mark Brown <broonie@...nel.org>
To:	"Kim, Milo" <Milo.Kim@...com>
Cc:	Liam Girdwood <lgirdwood@...il.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2] regulator: lp872x: support the device tree feature

On Fri, May 10, 2013 at 07:17:48AM +0000, Kim, Milo wrote:

> Patch v2.
>   Move the opmode code of the device tree from 'of_regulator' to the lp872x
>   because its mode depends on the device.
>   Supported modes in LP8720/5 are 'fast' and 'normal'.

> Patch v1.
>   Initial patch

Please keep these changelogs after the --- so they don't get included in
the final changelog by git am.

> +Optional properties:
> +- general-config: the value of LP872X_GENERAL_CFG register (u8)
> +- update-config: define it when LP872X_GENERAL_CFG register should be updated

A bit more information on this one wouldn't hurt but the information
will be in the datasheet I guess.

> +- dvs-gpio: GPIO number for DVS control

If this is the AP side GPIO then it ought to be a standard GPIO
specifier rather than a GPIO number.  If this is the pin on the PMIC
then it probably ought to be spelled out a bit more clearly.

> +- dvs-vsel: DVS selector. 0 = SEL_V1, 1 = SEL_V2.
> +- dvs-state: initial DVS pin state. 0 = DVS_LOG, 1 = DVS_HIGH.

Any custom properties are supposed to be namespaced, for example
"ti,general-config".

> +Child node
> +- regulator-id: the value of enum lp872x_regulator_id. (u8)

These would need to be listed as part of the binding, though it'd be
much more friendly to use plain text names and of_regulator_match() to
look them up.  That makes the binding much more legible.

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