[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20190827080825.GA27271@kroah.com>
Date: Tue, 27 Aug 2019 10:08:25 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: Peikan Tsai <peikantsai@...il.com>
Cc: christian.gromm@...rochip.com, gustavo@...eddedor.com,
suzuki.poulose@....com, colin.king@...onical.com,
devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging: most-core: Fix checkpatch warnings
On Mon, Aug 26, 2019 at 01:58:49AM +0800, Peikan Tsai wrote:
> Hi,
>
> This patch solves the following checkpatch.pl's messages in drivers/staging/most/core.c.
>
> WARNING: line over 80 characters
> + return snprintf(buf, PAGE_SIZE, "%s", ch_data_type[i].name);
>
> CHECK: Please use a blank line after function/struct/union/enum declarations
> +}
> +/**
>
> Signed-off-by: Peikan Tsai <peikantsai@...il.com>
> ---
> drivers/staging/most/core.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
Normally we want only one "type" of cleanup per patch, but this is so
tiny I'll just take it as-is.
thanks,
greg k-h
Powered by blists - more mailing lists