Re: [patch 2.6.29-rc] regulator: add get_status()

[Liam Girdwood]

On Thu, 2009-01-15 at 12:40 +0000, Mark Brown wrote:

> but...
> 
> > +enum regulator_status {
> > +	REGULATOR_STATUS_OFF,
> > +	REGULATOR_STATUS_ON,
> > +	REGULATOR_STATUS_ERROR,
> > +	/* fast/normal/idle/standby are flavors of "on" */
> > +	REGULATOR_STATUS_FAST,
> > +	REGULATOR_STATUS_NORMAL,
> > +	REGULATOR_STATUS_IDLE,
> > +	REGULATOR_STATUS_STANDBY,
> > +};
> > +
> 
> > +	/* report regulator status ... most other accessors report
> > +	 * control inputs, this reports results of combining inputs
> > +	 * from Linux (and other sources) with the actual load.
> > +	 */
> > +	int (*get_status)(struct regulator_dev *);
> > +
> 
> ...this needs kerneldoc adding.

Please send a separate patch for the kerneldoc as I've already applied.

Thanks

Liam

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