Hello.

On 6/26/2015 9:14 AM, Sunny Kumar wrote:

This patch fixes checkpatch.pl warning c99 style commenting.

Signed-off-by: Sunny Kumar <[email protected]>
---
  drivers/usb/storage/sddr55.c | 16 ++++++++--------
  1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/drivers/usb/storage/sddr55.c b/drivers/usb/storage/sddr55.c
index aacedef..54d0a59 100644
--- a/drivers/usb/storage/sddr55.c
+++ b/drivers/usb/storage/sddr55.c
@@ -209,10 +209,10 @@ static int sddr55_read_data(struct us_data *us,
        unsigned int len, offset;
        struct scatterlist *sg;

-       // Since we only read in one block at a time, we have to create
-       // a bounce buffer and move the data a piece at a time between the
-       // bounce buffer and the actual transfer buffer.
-
+       /* Since we only read in one block at a time, we have to create
+        * a bounce buffer and move the data a piece at a time between the
+        * bounce buffer and the actual transfer buffer.
+        */

This style of commenting is preferred only in the networking code. The other parts of the kernel prefer this style:

/*
 * bla
 * bla
 */

WBR, Sergei

--
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/

Reply via email to