All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] staging: vt6655: Fixed two lines over 80 characters
@ 2015-09-29 17:07 Arjun Krishna Babu
  2015-09-30  2:57 ` Greg KH
  0 siblings, 1 reply; 2+ messages in thread
From: Arjun Krishna Babu @ 2015-09-29 17:07 UTC (permalink / raw)
  To: forest
  Cc: gregkh, tvboxspy, himani93, dilekuzulmez, sakshi.april5, devel,
	linux-kernel, Arjun Krishna Babu

Two lines of code that were over 80 characters long is fixed by
splitting them into multiple lines.

This way, the lines of code are now easier to read.

Issue found by checkpatch.

Signed-off-by: Arjun Krishna Babu <arjunkrishnababu96@gmail.com>
---
 drivers/staging/vt6655/baseband.c | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/vt6655/baseband.c b/drivers/staging/vt6655/baseband.c
index 9e61f2d..3639f9d 100644
--- a/drivers/staging/vt6655/baseband.c
+++ b/drivers/staging/vt6655/baseband.c
@@ -2089,9 +2089,18 @@ bool BBbVT3253Init(struct vnt_private *priv)
 				byVT3253B0_UW2451[ii][0],
 				byVT3253B0_UW2451[ii][1]);
 
-		/* Init ANT B select,TX Config CR09 = 0x61->0x45, 0x45->0x41(VC1/VC2 define, make the ANT_A, ANT_B inverted) */
+		/* Init ANT B select,
+		 * TX Config CR09 = 0x61->0x45, 
+		 * 0x45->0x41(VC1/VC2 define, make the ANT_A, ANT_B inverted)
+		 */
+
 		/*bResult &= BBbWriteEmbedded(dwIoBase,0x09,0x41);*/
-		/* Init ANT B select,RX Config CR10 = 0x28->0x2A, 0x2A->0x28(VC1/VC2 define, make the ANT_A, ANT_B inverted) */
+
+		/* Init ANT B select,
+		 * RX Config CR10 = 0x28->0x2A, 
+		 * 0x2A->0x28(VC1/VC2 define, make the ANT_A, ANT_B inverted) 
+		 */
+
 		/*bResult &= BBbWriteEmbedded(dwIoBase,0x0a,0x28);*/
 		/* Select VC1/VC2, CR215 = 0x02->0x06 */
 		bResult &= BBbWriteEmbedded(priv, 0xd7, 0x06);
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] staging: vt6655: Fixed two lines over 80 characters
  2015-09-29 17:07 [PATCH] staging: vt6655: Fixed two lines over 80 characters Arjun Krishna Babu
@ 2015-09-30  2:57 ` Greg KH
  0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2015-09-30  2:57 UTC (permalink / raw)
  To: Arjun Krishna Babu
  Cc: forest, tvboxspy, himani93, dilekuzulmez, sakshi.april5, devel,
	linux-kernel

On Tue, Sep 29, 2015 at 10:37:24PM +0530, Arjun Krishna Babu wrote:
> Two lines of code that were over 80 characters long is fixed by
> splitting them into multiple lines.
> 
> This way, the lines of code are now easier to read.
> 
> Issue found by checkpatch.
> 
> Signed-off-by: Arjun Krishna Babu <arjunkrishnababu96@gmail.com>
> ---
>  drivers/staging/vt6655/baseband.c | 13 +++++++++++--
>  1 file changed, 11 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/vt6655/baseband.c b/drivers/staging/vt6655/baseband.c
> index 9e61f2d..3639f9d 100644
> --- a/drivers/staging/vt6655/baseband.c
> +++ b/drivers/staging/vt6655/baseband.c
> @@ -2089,9 +2089,18 @@ bool BBbVT3253Init(struct vnt_private *priv)
>  				byVT3253B0_UW2451[ii][0],
>  				byVT3253B0_UW2451[ii][1]);
>  
> -		/* Init ANT B select,TX Config CR09 = 0x61->0x45, 0x45->0x41(VC1/VC2 define, make the ANT_A, ANT_B inverted) */
> +		/* Init ANT B select,
> +		 * TX Config CR09 = 0x61->0x45, 
> +		 * 0x45->0x41(VC1/VC2 define, make the ANT_A, ANT_B inverted)
> +		 */
> +
>  		/*bResult &= BBbWriteEmbedded(dwIoBase,0x09,0x41);*/
> -		/* Init ANT B select,RX Config CR10 = 0x28->0x2A, 0x2A->0x28(VC1/VC2 define, make the ANT_A, ANT_B inverted) */
> +
> +		/* Init ANT B select,
> +		 * RX Config CR10 = 0x28->0x2A, 
> +		 * 0x2A->0x28(VC1/VC2 define, make the ANT_A, ANT_B inverted) 
> +		 */
> +

Please always run your patches through checkpatch.pl, so that you don't
get an email from the maintainers saying that you need to run your patch
through checkpatch.pl.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-09-30  2:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-29 17:07 [PATCH] staging: vt6655: Fixed two lines over 80 characters Arjun Krishna Babu
2015-09-30  2:57 ` Greg KH

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.