public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] regulator: fix kernel-doc warnings
@ 2009-01-08 19:50 Randy Dunlap
  2009-01-08 20:07 ` Liam Girdwood
  0 siblings, 1 reply; 2+ messages in thread
From: Randy Dunlap @ 2009-01-08 19:50 UTC (permalink / raw)
  To: lkml; +Cc: akpm, Liam Girdwood, Mark Brown

From: Randy Dunlap <randy.dunlap@oracle.com>

Fix kernel-doc warnings in regulator/driver.h:

Warning(linux-next-20090108//include/linux/regulator/driver.h:95): Excess struct/union/enum/typedef member 'set_current' description in 'regulator_ops'
Warning(linux-next-20090108//include/linux/regulator/driver.h:95): Excess struct/union/enum/typedef member 'get_current' description in 'regulator_ops'
Warning(linux-next-20090108//include/linux/regulator/driver.h:124): No description found for parameter 'irq'

Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
cc: Liam Girdwood <lrg@slimlogic.co.uk>
cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
---
 include/linux/regulator/driver.h |    7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

--- linux-next-20090108.orig/include/linux/regulator/driver.h
+++ linux-next-20090108/include/linux/regulator/driver.h
@@ -29,16 +29,12 @@ struct regulator_init_data;
  *
  * @enable: Enable the regulator.
  * @disable: Disable the regulator.
- * @is_enabled: Return 1 if the reguator is enabled, 0 otherwise.
+ * @is_enabled: Return 1 if the regulator is enabled, 0 otherwise.
  *
  * @set_voltage: Set the voltage for the regulator within the range specified.
  *               The driver should select the voltage closest to min_uV.
  * @get_voltage: Return the currently configured voltage for the regulator.
  *
- * @set_current: Set the current for the regulator within the range specified.
- *               The driver should select the current closest to min_uA.
- * @get_current: Return the currently configured current for the regulator.
- *
  * @set_current_limit: Configure a limit for a current-limited regulator.
  * @get_current_limit: Get the limit for a current-limited regulator.
  *
@@ -111,6 +107,7 @@ enum regulator_type {
  * @name: Identifying name for the regulator.
  * @id: Numerical identifier for the regulator.
  * @ops: Regulator operations table.
+ * @irq: Interrupt number for the regulator.
  * @type: Indicates if the regulator is a voltage or current regulator.
  * @owner: Module providing the regulator, used for refcounting.
  */

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] regulator: fix kernel-doc warnings
  2009-01-08 19:50 [PATCH] regulator: fix kernel-doc warnings Randy Dunlap
@ 2009-01-08 20:07 ` Liam Girdwood
  0 siblings, 0 replies; 2+ messages in thread
From: Liam Girdwood @ 2009-01-08 20:07 UTC (permalink / raw)
  To: Randy Dunlap; +Cc: lkml, akpm, Mark Brown

On Thu, 2009-01-08 at 11:50 -0800, Randy Dunlap wrote:
> From: Randy Dunlap <randy.dunlap@oracle.com>
> 
> Fix kernel-doc warnings in regulator/driver.h:
> 
> Warning(linux-next-20090108//include/linux/regulator/driver.h:95): Excess struct/union/enum/typedef member 'set_current' description in 'regulator_ops'
> Warning(linux-next-20090108//include/linux/regulator/driver.h:95): Excess struct/union/enum/typedef member 'get_current' description in 'regulator_ops'
> Warning(linux-next-20090108//include/linux/regulator/driver.h:124): No description found for parameter 'irq'
> 
> Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
> cc: Liam Girdwood <lrg@slimlogic.co.uk>
> cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
> -

Applied.

Thanks

Liam


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-01-08 20:15 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-01-08 19:50 [PATCH] regulator: fix kernel-doc warnings Randy Dunlap
2009-01-08 20:07 ` Liam Girdwood

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox