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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAK=oMXepj7tmZJ3=b9ja+E1N9eAPmsPhBefAzoswVyGWBXwNOA@mail.gmail.com>
Date:	Fri, 24 May 2013 11:01:45 +0800
From:	yi zhang <yizhang.mrvl@...il.com>
To:	Mark Brown <broonie@...nel.org>
Cc:	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

Hi, Brown:

2013/5/23 Mark Brown <broonie@...nel.org>:
> On Wed, May 22, 2013 at 08:10:53PM +0800, yizhang.mrvl@...il.com wrote:
>
> You need a DT binding document for any new DT bindings like this one.
>
Thanks for your comments, I'll do it;

>> +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?

>> +static int pm800_list_voltage(struct regulator_dev *rdev, unsigned index)
>> +{
>> +     struct pm800_regulator_info *info = rdev_get_drvdata(rdev);
>> +     int ret = -EINVAL;
>> +
>> +     if (info->vol_table && (index < rdev->desc->n_voltages))
>> +             ret = info->vol_table[index];
>> +
>> +     return ret;
>> +}
>
> For things that are just table lookups use the framework helpers.
>
   OK, I'll change it; thanks for your comments;
>> +static int choose_voltage(struct regulator_dev *rdev, int min_uv, int max_uv)
>> +{
>
> Similarly here.
>
    OK, copy that; thanks;

>> +static int pm800_set_voltage(struct regulator_dev *rdev,
>> +                          int min_uv, int max_uv, unsigned *sel)
>> +{
>> +     *sel = choose_voltage(rdev, min_uv, max_uv);
>> +     if (*sel < 0)
>> +             return -EINVAL;
>> +     return regulator_set_voltage_sel_regmap(rdev, *sel);
>> +}
>
> Implement map_voltage.
>
>> +     for_each_child_of_node(nproot, np) {
>> +             if (!of_node_cmp(np->name, info->desc.name)) {
>> +                     config->init_data =
>> +                             of_get_regulator_init_data(&pdev->dev, np);
>> +                     config->of_node = np;
>> +                     break;
>> +             }
>> +     }
>
> Use of_regulator_match().

    Copy that, thanks for your comment; I'll change it;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/