lkml.org 
[lkml]   [2009]   [Sep]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/2] U300 AB3100 boardinfo v3
On Tue, Sep 01, 2009 at 12:05:24AM +0200, Linus Walleij wrote:
> This defines regulator platform data and board power
> regulator hogs for the ST-Ericsson U300 platform.

This is pretty much OK, just...

> +static struct regulator_consumer_supply supply_ldo_a[] = {
> + {
> + .dev = NULL,
> + .supply = "vrad", /* Powers the radio */
> + },
> +};

Now you're using the regulator init stuff to set the voltages you should
be able to drop most of the deviceless supplies. I'd expect that you'd
want to keep the CPU core supplies for cpufreq support but the others
shouldn't need to be defined. The names can be set in the constraints
- if set there they will be displayed to user space which normally makes
the diagnostics easier to follow for board users.


\
 
 \ /
  Last update: 2009-09-02 20:45    [W:0.030 / U:0.068 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site