Re: [PATCH] Staging: mt7621-dma: Add blank line after declarations

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Tue, Mar 19, 2019 at 09:18:54PM -0700, Wentao Cai wrote:
> Add a blank line after declaration to mute checkpatch.pl warning:
> WARNING: Missing a blank line after declarations
> 
> Signed-off-by: Wentao Cai <etsai042@gmail.com>
> ---
>  drivers/staging/mt7621-dma/mtk-hsdma.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/staging/mt7621-dma/mtk-hsdma.c b/drivers/staging/mt7621-dma/mtk-hsdma.c
> index 97571f1d697b..beeff4258b19 100644
> --- a/drivers/staging/mt7621-dma/mtk-hsdma.c
> +++ b/drivers/staging/mt7621-dma/mtk-hsdma.c
> @@ -157,6 +157,7 @@ struct mtk_hsdam_engine {
>  	struct device_dma_parameters dma_parms;
>  	void __iomem *base;
>  	struct tasklet_struct task;
> +
>  	volatile unsigned long chan_issued;

Please look at what you did here and compare it to what the checkpatch
warning said it was doing.  Hint, checkpatch is wrong here...

Always use your brain when dealing with checkpatch as it is just a dumb
perl script :)

thanks,

greg k-h