[PATCH] Staging: cxt1e1: ossiRelease.c: fixed a coding style issue.

[PATCH] Staging: cxt1e1: ossiRelease.c: fixed a coding style issue.

[Aldo Iljazi]

Fixed a coding style issue, specifically checkpatch.pl complain:
ossiRelease.c:27: WARNING: line over 80 characters

Signed-off-by: Aldo Iljazi <neonsync1@gmail.com>
---
 drivers/staging/cxt1e1/ossiRelease.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/cxt1e1/ossiRelease.c b/drivers/staging/cxt1e1/ossiRelease.c
index f17a902..35575bf 100644
--- a/drivers/staging/cxt1e1/ossiRelease.c
+++ b/drivers/staging/cxt1e1/ossiRelease.c
@@ -24,6 +24,7 @@
  *-----------------------------------------------------------------------------
  */
 
-char pmcc4_OSSI_release[] = "$Release: PMCC4_3_1B,  Copyright (c) 2008 One Stop Systems$";
+char pmcc4_OSSI_release[] = "$Release: PMCC4_3_1B,  "
+"Copyright (c) 2008 One Stop Systems$";
 
 /***  End-of-File  ***/
-- 
1.7.9.5

Re: [PATCH] Staging: cxt1e1: ossiRelease.c: fixed a coding style issue.

[Greg KH]

On Mon, Jul 08, 2013 at 07:40:17PM +0300, Aldo Iljazi wrote:
> Fixed a coding style issue, specifically checkpatch.pl complain:
> ossiRelease.c:27: WARNING: line over 80 characters

And it's wrong, this fix is not the correct way to do this.

I'd recommend just removing it entirely, as it doesn't make much sense
anymore now that the driver is in the kernel tree.

thanks,

greg k-h

Re: [PATCH] Staging: cxt1e1: ossiRelease.c: fixed a coding style issue.

[Greg KH]

On Mon, Jul 08, 2013 at 07:33:48PM +0300, Aldo Iljazi wrote:
> Fixed a coding style issue, specifically checkpatch.pl complain:
> ossiRelease.c:27: WARNING: line over 80 characters
> 
> Signed-off-by: Aldo Iljazi <neonsync1@gmail.com>
> ---
>  drivers/staging/cxt1e1/ossiRelease.c |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/cxt1e1/ossiRelease.c b/drivers/staging/cxt1e1/ossiRelease.c
> index f17a902..35575bf 100644
> --- a/drivers/staging/cxt1e1/ossiRelease.c
> +++ b/drivers/staging/cxt1e1/ossiRelease.c
> @@ -24,6 +24,7 @@
>   *-----------------------------------------------------------------------------
>   */
>  
> -char pmcc4_OSSI_release[] = "$Release: PMCC4_3_1B,  Copyright (c) 2008 One Stop Systems$";
> +char pmcc4_OSSI_release[] = "$Release: PMCC4_3_1B,  "
> +"Copyright (c) 2008 One Stop Systems$";

No, please fix this correctly, by just removing it entirely, it's not
needed.

thanks,

greg k-h