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

["Pillai, Manikandan" <mani.pillai@xxxxxx>]

> -----Original Message-----
> From: Mark Brown [mailto:broonie@xxxxxxxxxxxxx]
> Sent: Wednesday, January 14, 2009 4:32 PM
> To: Pillai, Manikandan
> Cc: linux-omap@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 1/2] Include TPS6235x based Power regulator support
> 
> On Wed, Jan 14, 2009 at 03:32:17PM +0530, Pillai, Manikandan wrote:
> 
> > > From: Mark Brown [mailto:broonie@xxxxxxxxxxxxx]
> 
> > > 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.
> 
> > The issue is that passing the initial value for regulators is done by
> 
> > struct regulator_init_data *init_data = dev->platform_data;
> 
> > in regulator_register() function. Do we have a way to initialize the
> > platform data for the i2c client struct.
> 
> The platform data is part of the struct device so you with your current
> code your driver can just assign directly to it.
[Pillai, Manikandan]  The platform_data is now initialized to clkrate.
(arch/arm/plat-omap2/i2c.c


--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html