Re: [PATCH 1/2] Include TPS6235x based Power regulator support

[Mark Brown]

On Tue, Jan 13, 2009 at 01:11:08PM +0530, Manikandan Pillai wrote:

> +static
> +int tps_6235x_probe(struct i2c_client *client, const struct i2c_device_id 
> *id)
> +{
> +     struct  regulator_dev           *rdev = NULL;
> +     unsigned char reg_val;
> +     struct device *dev_child = NULL;

...

> +     /* Register the regulators */
> +     dev_child = device_find_child(client->adapter->dev.parent,
> +                     (void *)regulator_consumer_name[id->driver_data],
> +                     omap_i2c_match_child);
> +     rdev = regulator_register(&regulators[id->driver_data],
> +                     dev_child, client);

With current mainline this should be rewritten as:

        rdev = regulator_register(&regulators[id->driver_data],
                                  &client->dev, client);

though I'm not sure what's currently merged up into the OMAP trees.

As discussed in the followup to your previous patch the register I/O
functions should also be moved into this driver.
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html