On 02/27/2015 05:26 PM, Joe Perches wrote: > 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.
Thanks for the comment, I will leave that to be done in a different cleanup patch. regards Suman -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/