lkml.org 
[lkml]   [2023]   [Feb]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
Subject[PATCH v2 0/4] staging: vt6655: Macro with braces issue changed to inline function

This patch is to fix checkpatch warning: "Macro argument 'iobase' may be better
as '(iobase)' to avoid precedence issues" changed to inline function. In
relation to this, names of the callers of macro are also modified to call
this function.


Changes in v4:
- Modified commit message to be clear

Changes in v3:
1. Whitespace error from checkpatch fixed
2. Should be applied together with changes in v2 for testing

Changes in v2:
- Macros with one statement that is to call 'iowrite8' function changed
to inline function as reviewed by gregkh@linuxfoundation.org.
In relation to this, names of the callers of macro are also modified
to call this function.


Guru Mehar Rachaputi (4):
staging: vt6655: Add braces to macro parameter 'iobase'
staging: vt6655: Macro with braces issue change to inline function
staging: vt6655: Macro with braces issue change to inline function
staging: vt6655: Macro with braces issue changed to inline function

drivers/staging/vt6655/card.c | 4 ++--
drivers/staging/vt6655/channel.c | 4 ++--
drivers/staging/vt6655/device_main.c | 12 ++++++------
drivers/staging/vt6655/mac.c | 8 ++++----
drivers/staging/vt6655/mac.h | 12 +++++++++---
5 files changed, 23 insertions(+), 17 deletions(-)

--
2.34.1


--
Thanks & Regards,
Guru

\
 
 \ /
  Last update: 2023-03-27 00:12    [W:0.071 / U:1.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site