[i2c] i2c kernel-doc warnings

Randy Dunlap rdunlap at xenotime.net
Mon Aug 18 23:08:13 CEST 2008


On Mon, 18 Aug 2008 22:36:09 +0200 Jean Delvare wrote:

> Hi Randy,
> 
> On Mon, 21 Jul 2008 10:24:54 -0700, Randy Dunlap wrote:
> > Hi Jean,
> > 
> > Can you try to have someone fix these, please?
> > 
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'id'
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'attach_adapter'
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'detach_adapter'
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'detach_client'
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'probe'
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'remove'
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'shutdown'
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'suspend'
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'resume'
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'command'
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'driver'
> > Warning(/local/linsrc/repo/linhead//include/linux/i2c.h:169): No description found for parameter 'id_table'
> 
> Something like this?

Yes, that's good.  Thanks.

Acked-by: Randy Dunlap <rdunlap at xenotime.net>


> Add missing kernel descriptions of struct i2c_driver members.
> 
> Signed-off-by: Jean Delvare <khali at linux-fr.org>
> Cc: Randy Dunlap <rdunlap at xenotime.net>
> Cc: David Brownell <david-b at pacbell.net>
> ---
>  include/linux/i2c.h |   12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> --- linux-2.6.27-rc3.orig/include/linux/i2c.h	2008-08-05 18:18:01.000000000 +0200
> +++ linux-2.6.27-rc3/include/linux/i2c.h	2008-08-18 22:33:51.000000000 +0200
> @@ -97,7 +97,19 @@ extern s32 i2c_smbus_write_i2c_block_dat
>  
>  /**
>   * struct i2c_driver - represent an I2C device driver
> + * @id: Unique driver ID (optional)
>   * @class: What kind of i2c device we instantiate (for detect)
> + * @attach_adapter: Callback for bus addition (for legacy drivers)
> + * @detach_adapter: Callback for bus removal (for legacy drivers)
> + * @detach_client: Callback for device removal (for legacy drivers)
> + * @probe: Callback for device binding (new-style drivers)
> + * @remove: Callback for device unbinding (new-style drivers)
> + * @shutdown: Callback for device shutdown
> + * @suspend: Callback for device suspend
> + * @resume: Callback for device resume
> + * @command: Callback for bus-wide signaling (optional)
> + * @driver: Device driver model driver
> + * @id_table: List of I2C devices supported by this driver
>   * @detect: Callback for device detection
>   * @address_data: The I2C addresses to probe, ignore or force (for detect)
>   * @clients: List of detected clients we created (for i2c-core use only)


---
~Randy
Linux Plumbers Conference, 17-19 September 2008, Portland, Oregon USA
http://linuxplumbersconf.org/



More information about the i2c mailing list