Re: input: fix comments and blank lines in new ff code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 01 Jun 2006 22:07:16 +0300 Anssi Hannula wrote:

> Fix comments so that they conform to kernel-doc and add/remove some
> blank lines.
> 
> Signed-off-by: Anssi Hannula <[email protected]>

(attachments make review/comments more trying, so this is an
abbreviated reply)

 /**
- * Lock the mutex and check the device has not been deleted
+ * input_ff_safe_lock - lock the mutex and check for the ff_device struct
+ *
+ * Returns 0 if device still present, 1 otherwise.
  */
 static inline int input_ff_safe_lock(struct input_dev *dev)
 {

Functions that are commented with kernel-doc need all of the parameters
documented also, like:

 * @dev: the input_dev (or better description)

There were a few places where you had these and deleted them.  :(
~~~~~~~~~~~~~~

(repeat for several functions...)

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

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux