Re: [PATCH 2/2] TWL4030: convert old-style function documentation to current kerneldoc

2008-08-04 Thread Paul Walmsley
On Mon, 4 Aug 2008, Tony Lindgren wrote:

> Are you planning to repost this patch? I can't seem to find any other
> version..

Reposting now.



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


Re: [PATCH 2/2] TWL4030: convert old-style function documentation to current kerneldoc

2008-08-04 Thread Tony Lindgren
* Paul Walmsley <[EMAIL PROTECTED]> [080722 07:19]:
> On Fri, 18 Jul 2008, Felipe Balbi wrote:
> 
> > On Thu, 17 Jul 2008 20:15:51 -0600, Paul Walmsley <[EMAIL PROTECTED]> wrote:
> 
> > > + * Results the result of operation - 0 is success
> 
> > I suppose you mean Returns here.
> 
> thanks Felipe, fixed.

Are you planning to repost this patch? I can't seem to find any other
version..

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


Re: [PATCH 2/2] TWL4030: convert old-style function documentation to current kerneldoc

2008-07-21 Thread Paul Walmsley
On Fri, 18 Jul 2008, Felipe Balbi wrote:

> On Thu, 17 Jul 2008 20:15:51 -0600, Paul Walmsley <[EMAIL PROTECTED]> wrote:

> > + * Results the result of operation - 0 is success

> I suppose you mean Returns here.

thanks Felipe, fixed.

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


Re: [PATCH 2/2] TWL4030: convert old-style function documentation to current kerneldoc

2008-07-18 Thread Felipe Balbi


On Thu, 17 Jul 2008 20:15:51 -0600, Paul Walmsley <[EMAIL PROTECTED]> wrote:
> Several functions use old-style function documentation; convert those to
> use the current kerneldoc style.
> 
> Signed-off-by: Paul Walmsley <[EMAIL PROTECTED]>
> ---
> 
>  drivers/i2c/chips/twl4030-core.c |   54
> ++
>  1 files changed, 26 insertions(+), 28 deletions(-)
> 
> diff --git a/drivers/i2c/chips/twl4030-core.c
> b/drivers/i2c/chips/twl4030-core.c
> index 54e392b..4104fd2 100644
> --- a/drivers/i2c/chips/twl4030-core.c
> +++ b/drivers/i2c/chips/twl4030-core.c
> @@ -389,17 +389,18 @@ static struct irq_chip twl4030_irq_chip = {
>  };
> 
>  /* Global Functions */
> -/*
> - * @brief twl4030_i2c_write - Writes a n bit register in TWL4030
> +
> +/**
> + * twl4030_i2c_write - Writes a n bit register in TWL4030
> + * @mod_no: module number
> + * @value: an array of num_bytes+1 containing data to write
> + * @reg: register address (just offset will do)
> + * @num_bytes: number of bytes to transfer
>   *
> - * @param mod_no - module number
> - * @param *value - an array of num_bytes+1 containing data to write
> - * IMPORTANT - Allocate value num_bytes+1 and valid data starts at
> - *Offset 1.
> - * @param reg - register address (just offset will do)
> - * @param num_bytes - number of bytes to transfer
> + * IMPORTANT: for 'value' parameter: Allocate value num_bytes+1 and
> + * valid data starts at Offset 1.
>   *
> - * @return result of operation - 0 is success
> + * Results the result of operation - 0 is success

I suppose you mean Returns here.

-- 
Best Regards,

Felipe Balbi
http://blog.felipebalbi.com
[EMAIL PROTECTED]

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


[PATCH 2/2] TWL4030: convert old-style function documentation to current kerneldoc

2008-07-17 Thread Paul Walmsley
Several functions use old-style function documentation; convert those to
use the current kerneldoc style.

Signed-off-by: Paul Walmsley <[EMAIL PROTECTED]>
---

 drivers/i2c/chips/twl4030-core.c |   54 ++
 1 files changed, 26 insertions(+), 28 deletions(-)

diff --git a/drivers/i2c/chips/twl4030-core.c b/drivers/i2c/chips/twl4030-core.c
index 54e392b..4104fd2 100644
--- a/drivers/i2c/chips/twl4030-core.c
+++ b/drivers/i2c/chips/twl4030-core.c
@@ -389,17 +389,18 @@ static struct irq_chip twl4030_irq_chip = {
 };
 
 /* Global Functions */
-/*
- * @brief twl4030_i2c_write - Writes a n bit register in TWL4030
+
+/**
+ * twl4030_i2c_write - Writes a n bit register in TWL4030
+ * @mod_no: module number
+ * @value: an array of num_bytes+1 containing data to write
+ * @reg: register address (just offset will do)
+ * @num_bytes: number of bytes to transfer
  *
- * @param mod_no - module number
- * @param *value - an array of num_bytes+1 containing data to write
- * IMPORTANT - Allocate value num_bytes+1 and valid data starts at
- *  Offset 1.
- * @param reg - register address (just offset will do)
- * @param num_bytes - number of bytes to transfer
+ * IMPORTANT: for 'value' parameter: Allocate value num_bytes+1 and
+ * valid data starts at Offset 1.
  *
- * @return result of operation - 0 is success
+ * Results the result of operation - 0 is success
  */
 int twl4030_i2c_write(u8 mod_no, u8 *value, u8 reg, u8 num_bytes)
 {
@@ -443,14 +444,13 @@ int twl4030_i2c_write(u8 mod_no, u8 *value, u8 reg, u8 
num_bytes)
 EXPORT_SYMBOL(twl4030_i2c_write);
 
 /**
- * @brief twl4030_i2c_read - Reads a n bit register in TWL4030
+ * twl4030_i2c_read - Reads a n bit register in TWL4030
+ * @mod_no: module number
+ * @value: an array of num_bytes containing data to be read
+ * @reg: register address (just offset will do)
+ * @num_bytes: number of bytes to transfer
  *
- * @param mod_no - module number
- * @param *value - an array of num_bytes containing data to be read
- * @param reg - register address (just offset will do)
- * @param num_bytes - number of bytes to transfer
- *
- * @return result of operation - num_bytes is success else failure.
+ * Returns result of operation - num_bytes is success else failure.
  */
 int twl4030_i2c_read(u8 mod_no, u8 *value, u8 reg, u8 num_bytes)
 {
@@ -497,13 +497,12 @@ int twl4030_i2c_read(u8 mod_no, u8 *value, u8 reg, u8 
num_bytes)
 EXPORT_SYMBOL(twl4030_i2c_read);
 
 /**
- * @brief twl4030_i2c_write_u8 - Writes a 8 bit register in TWL4030
- *
- * @param mod_no - module number
- * @param value - the value to be written 8 bit
- * @param reg - register address (just offset will do)
+ * twl4030_i2c_write_u8 - Writes a 8 bit register in TWL4030
+ * @mod_no: module number
+ * @value: the value to be written 8 bit
+ * @reg: register address (just offset will do)
  *
- * @return result of operation - 0 is success
+ * Returns result of operation - 0 is success
  */
 int twl4030_i2c_write_u8(u8 mod_no, u8 value, u8 reg)
 {
@@ -517,13 +516,12 @@ int twl4030_i2c_write_u8(u8 mod_no, u8 value, u8 reg)
 EXPORT_SYMBOL(twl4030_i2c_write_u8);
 
 /**
- * @brief twl4030_i2c_read_u8 - Reads a 8 bit register from TWL4030
- *
- * @param mod_no - module number
- * @param *value - the value read 8 bit
- * @param reg - register address (just offset will do)
+ * twl4030_i2c_read_u8 - Reads a 8 bit register from TWL4030
+ * @mod_no: module number
+ * @value: the value read 8 bit
+ * @reg: register address (just offset will do)
  *
- * @return result of operation - 0 is success
+ * Returns result of operation - 0 is success
  */
 int twl4030_i2c_read_u8(u8 mod_no, u8 *value, u8 reg)
 {


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