Re: [PATCH v2] drivers/fbtft: Fix indentation

From: Tobin C. Harding
Date: Wed Jan 10 2018 - 20:11:31 EST


On Wed, Jan 10, 2018 at 06:30:35PM +0100, Jonny Schaefer wrote:
> From: Luis Gerhorst <luis.gerhorst@xxxxxx>
>
> This fixes the checkpatch message:
>
> CHECK: Alignment should match open parenthesis
> #1380: FILE: drivers/staging/fbtft/fbtft-core.c:1380:
> + dev_warn(dev,
> + "no default functions for regwidth=%d and buswidth=%d\n",
>

Perhaps:

Checkpatch emits CHECK: Alignment should match open parenthesis

Align code to open parenthesis.

Reasoning:

Patch description should describe the problem then describe what the
patch does (in imperative mood). Better advice is given in
Documentation/process/submitting-patches.rst section 2

Hope this helps,
Tobin.