Re: [PATCH 2/2] i2c: imx: Remove unneeded comments

[Wolfram Sang]

On Sun, Nov 01, 2015 at 02:22:52PM -0200, Fabio Estevam wrote:
> From: Fabio Estevam 
> 
> These multi-lines comments do not follow the standard kernel coding
> style. In fact, they are not useful comments, so get rid of them.
> 
> Signed-off-by: Fabio Estevam 

There have been some other changes to the imx driver. Could you rebase
against my for-next or for-4.5? Thanks!



signature.asc
Description: Digital signature

[PATCH 2/2] i2c: imx: Remove unneeded comments

[Fabio Estevam]

From: Fabio Estevam 

These multi-lines comments do not follow the standard kernel coding
style. In fact, they are not useful comments, so get rid of them.

Signed-off-by: Fabio Estevam 
---
 drivers/i2c/busses/i2c-imx.c | 12 
 1 file changed, 12 deletions(-)

diff --git a/drivers/i2c/busses/i2c-imx.c b/drivers/i2c/busses/i2c-imx.c
index 1e522c3..696c920 100644
--- a/drivers/i2c/busses/i2c-imx.c
+++ b/drivers/i2c/busses/i2c-imx.c
@@ -29,9 +29,6 @@
  *
  */
 
-/** Includes 
***
-***/
-
 #include 
 #include 
 #include 
@@ -54,9 +51,6 @@
 #include 
 #include 
 
-/** Defines 

-***/
-
 /* This will be the driver name the kernel reports */
 #define DRIVER_NAME "imx-i2c"
 
@@ -118,9 +112,6 @@
 #define I2CR_IEN_OPCODE_0  0x0
 #define I2CR_IEN_OPCODE_1  I2CR_IEN
 
-/** Variables 
**
-***/
-
 /*
  * sorted list of clock divider, register value pairs
  * taken from table 26-5, p.26-9, Freescale i.MX
@@ -409,9 +400,6 @@ static void i2c_imx_dma_free(struct imx_i2c_struct *i2c_imx)
dma->chan_using = NULL;
 }
 
-/** Functions for IMX I2C adapter driver 
***
-***/
-
 static int i2c_imx_bus_busy(struct imx_i2c_struct *i2c_imx, int for_busy)
 {
unsigned long orig_jiffies = jiffies;
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html