Message-ID: <1425079588.2690.20.camel@perches.com>
Subject: Re: [PATCH 1/3] remoteproc/ste: add blank lines after declarations
From: Joe Perches <joe@perches.com>
To: Suman Anna <s-anna@ti.com>
Cc: Ohad Ben-Cohen <ohad@wizery.com>, linux-kernel@vger.kernel.org
Date: Fri, 27 Feb 2015 15:26:28 -0800
In-Reply-To: <1425079103-41695-2-git-send-email-s-anna@ti.com>
References: <1425079103-41695-1-git-send-email-s-anna@ti.com>
	 <1425079103-41695-2-git-send-email-s-anna@ti.com>
Content-Type: text/plain; charset="ISO-8859-1"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 792
Lines: 22

On Fri, 2015-02-27 at 17:18 -0600, Suman Anna wrote:
> Fix couple of checkpatch warnings of the type,
> "WARNING: Missing a blank line after declarations"
[]
> diff --git a/drivers/remoteproc/ste_modem_rproc.c b/drivers/remoteproc/ste_modem_rproc.c
[]
> @@ -230,6 +229,7 @@ static int sproc_start(struct rproc *rproc)
>  static int sproc_stop(struct rproc *rproc)
>  {
>  	struct sproc *sproc = rproc->priv;
> +
>  	sproc_dbg(sproc, "stop ste-modem\n");

You could probably delete this sproc_dbg
statement too as it duplicates function tracing.


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