[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20140729173631.GX17528@sirena.org.uk>
Date: Tue, 29 Jul 2014 18:36:31 +0100
From: Mark Brown <broonie@...nel.org>
To: Javier Martinez Canillas <javier.martinez@...labora.co.uk>
Cc: Kukjin Kim <kgene.kim@...sung.com>,
Doug Anderson <dianders@...omium.org>,
Olof Johansson <olof@...om.net>,
Yuvaraj Kumar C D <yuvaraj.cd@...il.com>,
linux-samsung-soc@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [RFC 4/5] regulator: tps65090: Set voltage for fixed regulators
On Tue, Jul 29, 2014 at 06:28:58PM +0200, Javier Martinez Canillas wrote:
> +#define tps65090_REG_VAR(_id, _sname, en_reg, _en_bits, _ops) \
> + tps65090_REG_DESC(_id, _sname, en_reg, _en_bits, 0, 0, _ops)
I'd expect this to describe a variable regulator when in fact it
seems it describes a switch with no regulation. Better name please.
Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)
Powered by blists - more mailing lists