[PATCH v3] [Outreachy kernel] staging: vt6655: remove line over 80 characters

[PATCH v3] [Outreachy kernel] staging: vt6655: remove line over 80 characters

[Payal Kshirsagar]

Reformatting code to not have lines over 80 characters.

Signed-off-by: Payal Kshirsagar <payalskshirsagar1234@gmail.com>
---
v3: Change in patch description.

 drivers/staging/vt6655/mac.h | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/vt6655/mac.h b/drivers/staging/vt6655/mac.h
index c7888c4e96f2..eb56e16637f6 100644
--- a/drivers/staging/vt6655/mac.h
+++ b/drivers/staging/vt6655/mac.h
@@ -884,9 +884,11 @@ bool MACbIsIntDisable(struct vnt_private *priv);
 void MACvSetShortRetryLimit(struct vnt_private *priv,
 			    unsigned char byRetryLimit);
 
-void MACvSetLongRetryLimit(struct vnt_private *priv, unsigned char byRetryLimit);
+void MACvSetLongRetryLimit(struct vnt_private *priv,
+			   unsigned char byRetryLimit);
 
-void MACvSetLoopbackMode(struct vnt_private *priv, unsigned char byLoopbackMode);
+void MACvSetLoopbackMode(struct vnt_private *priv,
+			 unsigned char byLoopbackMode);
 
 void MACvSaveContext(struct vnt_private *priv, unsigned char *pbyCxtBuf);
 void MACvRestoreContext(struct vnt_private *priv, unsigned char *pbyCxtBuf);
@@ -913,7 +915,8 @@ void MACvSetCurrSyncDescAddrEx(struct vnt_private *priv,
 void MACvSetCurrATIMDescAddrEx(struct vnt_private *priv,
 			       u32 curr_desc_addr);
 void MACvTimer0MicroSDelay(struct vnt_private *priv, unsigned int uDelay);
-void MACvOneShotTimer1MicroSec(struct vnt_private *priv, unsigned int uDelayTime);
+void MACvOneShotTimer1MicroSec(struct vnt_private *priv,
+			       unsigned int uDelayTime);
 
 void MACvSetMISCFifo(struct vnt_private *priv, unsigned short wOffset,
 		     u32 dwData);
-- 
2.17.1

Re: [PATCH v3] [Outreachy kernel] staging: vt6655: remove line over 80 characters

[Stefano Brivio]

Payal,

On Tue, 10 Mar 2020 18:02:28 +0530
Payal Kshirsagar <payalskshirsagar1234@gmail.com> wrote:

> Reformatting code to not have lines over 80 characters.

The change looks good, but the subject (that is, the commit title)
still doesn't match the contents of the change. You're not removing,
and it's not a single line.

Also,

> Signed-off-by: Payal Kshirsagar <payalskshirsagar1234@gmail.com>
> ---
> v3: Change in patch description.

This should be cumulative, that is:

v4: < you know what :) >
v3: Change in patch 
v2: < changes you listed for v2 >

Otherwise, reviewers and maintainers will have a hard time tracking
your submissions. Thanks,

-- 
Stefano