[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20120406085115.GD3172@opensource.wolfsonmicro.com>
Date: Fri, 6 Apr 2012 09:51:15 +0100
From: Mark Brown <broonie@...nsource.wolfsonmicro.com>
To: Laxman Dewangan <ldewangan@...dia.com>
Cc: Axel Lin <axel.lin@...il.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Liam Girdwood <lrg@...com>
Subject: Re: [PATCH] regulator: rc5t583: Simplify RC5T583_REG macro
On Fri, Apr 06, 2012 at 11:03:36AM +0530, Laxman Dewangan wrote:
> I thought of doing this on my first patch and then thought that
> table should have all configuration picture rather than deriving
> from few parameters.
> If this is a good way then I will take care for my next PMIC driver
> development.
This way is good - if you look at for example the wm831x driver where
there's a bunch of regulators with the same register layout replicated
in different places in the chip all it does is specify the base for the
block when defining a regulator.
Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)
Powered by blists - more mailing lists