Re: [PATCH v3] staging: vt6655: Macro with braces issue change to inline function

From: Greg Kroah-Hartman
Date: Sun Feb 05 2023 - 08:16:17 EST


On Sun, Feb 05, 2023 at 02:08:02PM +0100, Guru Mehar Rachaputi wrote:
> This patch is to fix checkpatch warning: "Macro argument 'iobase' may be better
> as '(iobase)' to avoid precedence issues"
>
> ---
> Changes in v3:
> - Whitespace error from checkpatch fixed
>
> Changes in v2:
> - Macros with one statement that is to call 'iowrite8' function changed
> to inline function as reviewed by gregkh@xxxxxxxxxxxxxxxxxxx.
> In relation to this, names of the callers of macro are also modified
> to call this function.
>
> Signed-off-by: Guru Mehar Rachaputi <gurumeharrachaputi@xxxxxxxxx>

Try to take this patch and apply it to a tree, and see that everything
below the --- line is thrown away, including your signed-off-by: line :(