linux-kernel - Re: [PATCH] regulator: 88pm800: add regulator driver

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: <20130528100415.GE3660@sirena.org.uk>
Date:	Tue, 28 May 2013 06:04:15 -0400
From:	Mark Brown <broonie@...nel.org>
To:	yi zhang <yizhang.mrvl@...il.com>
Cc:	Liam Girdwood <lgirdwood@...il.com>, cxie4@...vell.com,
	jett.zhou@...vell.com, linux-kernel@...r.kernel.org,
	Yi Zhang <yizhang@...vell.com>
Subject: Re: [PATCH] regulator: 88pm800: add regulator driver

On Tue, May 28, 2013 at 10:32:09AM +0800, yi zhang wrote:

> HI, Brown:

In western culture it's either "Hi Mark" or "Hi Mr Brown" (though the
latter is pretty formal so not used much for things like mailing lists).

>           Sorry I'm a little confused:
>           You mean I should use a function for example
> "create_buck_table()" to create the BUCK voltage table?
>            such as:
>            void create_buck_table()
>            {
>                    if (voltage < 1587500) {
>                              /* initialize the BUCK_table */
>                    } else
>                              /* initialize the BUCK_table */
>            }

No, I mean you shouldn't be using a table at all but have mapping and
list functions which just do the calculation required to generate the
tables directly.  This saves scanning through the table as direct
calcuation is possible.

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