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.