2023-10-12 12:51:48

by Soumya Negi

Subject: [PATCH 0/2] staging: rts5208: Fix checkpatch macro warnings

In the driver staging/rts5208, checkpatch.pl warns of possible precedence
issues in the header rtsx.h because macro arguments are not parenthesized.
This patch set handles the respective macros differently(as needed) in
each patch to fix the warnings. Patches can be applied in any order.

Soumya Negi (2):
staging: rts5208: Refactor macros to static inline functions
staging: rts5208: Remove macros scsi_lock(), scsi_unlock()

drivers/staging/rts5208/rtsx.c | 24 ++++++-------
drivers/staging/rts5208/rtsx.h | 66 ++++++++++++++++++++--------------
2 files changed, 52 insertions(+), 38 deletions(-)

--
2.17.1

2023-10-13 07:47:32

by Soumya Negi

Subject: Re: [PATCH 0/2] staging: rts5208: Fix checkpatch macro warnings

On Thu, Oct 12, 2023 at 05:50:21AM -0700, Soumya Negi wrote:
> In the driver staging/rts5208, checkpatch.pl warns of possible precedence
> issues in the header rtsx.h because macro arguments are not parenthesized.
> This patch set handles the respective macros differently(as needed) in
> each patch to fix the warnings. Patches can be applied in any order.
>
> Soumya Negi (2):
> staging: rts5208: Refactor macros to static inline functions
> staging: rts5208: Remove macros scsi_lock(), scsi_unlock()
>
> drivers/staging/rts5208/rtsx.c | 24 ++++++-------
> drivers/staging/rts5208/rtsx.h | 66 ++++++++++++++++++++--------------
> 2 files changed, 52 insertions(+), 38 deletions(-)

Please ignore this patch set. Its incomplete & the rest of the patches
were not sent successfully. I have successfuly resent the complete
patchset(unchanged) as v2 with an explanation after --- in the cover patch.


> --
> 2.17.1
>