linux-kernel - Re: [PATCH V2] Staging:tidspbridge Fix minor checkpatch.pl warning.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Wed, 18 Jun 2014 15:24:57 -0700
From:	Greg KH <gregkh@...uxfoundation.org>
To:	Adithya Krishnamurthy <linux.challenge1@...il.com>
Cc:	omar.ramirez@...itl.com, rashika.kheria@...il.com,
	freemangordon@....bg, dan.carpenter@...cle.com, ody.guru@...il.com,
	devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH V2] Staging:tidspbridge Fix minor checkpatch.pl warning.

On Sun, Jun 08, 2014 at 02:26:20PM +0530, Adithya Krishnamurthy wrote:
> From: Adithya Krishnamurthy <linux.challenge1@...il.com>
> 
> Fixed checkpatch "WARNING: Missing a blank line after declarations"
> 
> Signed-off-by: Adithya Krishnamurthy <linux.challenge1@...il.com>
> ---
>  drivers/staging/tidspbridge/core/chnl_sm.c       |    1 +
>  drivers/staging/tidspbridge/core/io_sm.c         |    1 +
>  drivers/staging/tidspbridge/core/tiomap3430.c    |    2 ++
>  drivers/staging/tidspbridge/core/tiomap_io.c     |    2 ++
>  drivers/staging/tidspbridge/core/wdt.c           |    1 +
>  drivers/staging/tidspbridge/dynload/cload.c      |    7 +++++++
>  drivers/staging/tidspbridge/dynload/reloc.c      |    2 ++
>  drivers/staging/tidspbridge/pmgr/chnl.c          |    1 +
>  drivers/staging/tidspbridge/rmgr/dbdcd.c         |    1 +
>  drivers/staging/tidspbridge/rmgr/drv_interface.c |    2 ++
>  drivers/staging/tidspbridge/rmgr/nldr.c          |    1 +
>  drivers/staging/tidspbridge/rmgr/node.c          |    2 ++
>  drivers/staging/tidspbridge/rmgr/proc.c          |    3 +++
>  13 files changed, 26 insertions(+)
> 
> diff --git a/drivers/staging/tidspbridge/core/chnl_sm.c b/drivers/staging/tidspbridge/core/chnl_sm.c
> index 16fa346..c855992 100644
> --- a/drivers/staging/tidspbridge/core/chnl_sm.c
> +++ b/drivers/staging/tidspbridge/core/chnl_sm.c
> @@ -486,6 +486,7 @@ int bridge_chnl_get_info(struct chnl_object *chnl_obj,
>  {
>  	int status = 0;
>  	struct chnl_object *pchnl = (struct chnl_object *)chnl_obj;
> +
>  	if (channel_info != NULL) {
>  		if (pchnl) {
>  			/* Return the requested information: */


Your email client put an extra space in front of every line, making this
patch not able to be applied :(

Care to fix that up and resend?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/