linux-kernel - Re: [PATCH v3 3/3] pmbus: ltc2978: add regulator support

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: <20140924223322.GL16977@sirena.org.uk>
Date:	Wed, 24 Sep 2014 23:33:22 +0100
From:	Mark Brown <broonie@...nel.org>
To:	Guenter Roeck <linux@...ck-us.net>
Cc:	atull <atull@...nsource.altera.com>, jdelvare@...e.de,
	lm-sensors@...sensors.org, lgirdwood@...il.com,
	linux-kernel@...r.kernel.org, delicious.quinoa@...il.com,
	dinguyen@...nsource.altera.com, yvanderv@...nsource.altera.com
Subject: Re: [PATCH v3 3/3] pmbus: ltc2978: add regulator support

On Wed, Sep 24, 2014 at 02:13:32PM -0700, Guenter Roeck wrote:
> On Wed, Sep 24, 2014 at 03:48:40PM -0500, atull wrote:

> > > regulator.47/name:vout_en0

> > These are just default names, but I think I could make the name better.
> > How about <part #>-<i2c address>-vout<#> such as "ltc2978-5c-vout0"

> Problem with that is that i2c bus numbers are not static, which would
> make it quite difficult to describe the regulator in devicetree data.

> Any idea what other regulators do in such situations ?

What's there is correct (modulo the _en), the names are local to the
chip.  If the user actually cares they will be providing the name of the
supply on the board anyway.

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