Re: [PATCH] regulator: 88pm800: add regulator driver

[Mark Brown]

On Fri, May 24, 2013 at 11:01:45AM +0800, yi zhang wrote:
> 2013/5/23 Mark Brown <broonie@xxxxxxxxxx>:
> > On Wed, May 22, 2013 at 08:10:53PM +0800, yizhang.mrvl@xxxxxxxxx wrote:

> >> +static const unsigned int BUCK1_table[] = {
> >> +     /* 0x00-0x4F: from 0.6 to 1.5875V with step 0.0125V */
> >> +     /* 0x50-0x7F: from 1.6 to 1.8V with step 0.05V */

> > Write this out as code, don't use a big table for large sets of
> > voltages.

>      The voltage table is "const", if we write it out as code, seems
> it's hard for us
>      to keep this attribute;
>      And the voltage of this BUCK is not linear(it's separated as two parts),
>      seems I shouldn't use the framework helpers to implement it; Right?
>       what do you think?

You should open code the mapping functions but you should still do this
as calcuations, it's faster and produces smaller code.  You'll need a
small function with an if statement.

Attachment: signature.asc
Description: Digital signature