* Re: [PATCH V2] staging: rts5208: fix Lines should not end with a '('.
[not found] <20220221214344.5895-1-info@samuelsjoberg.se>
@ 2022-02-25 9:01 ` Greg Kroah-Hartman
0 siblings, 0 replies; only message in thread
From: Greg Kroah-Hartman @ 2022-02-25 9:01 UTC (permalink / raw)
To: Samuel Sjöberg; +Cc: linux-staging, linux-kernel
On Mon, Feb 21, 2022 at 10:43:44PM +0100, Samuel Sjöberg wrote:
> Fix the following error reported by checkpatch.pl:
>
> CHECK: Lines should not end with a '('
> + retval = ms_transfer_tpc(
>
> Signed-off-by: Samuel Sjöberg <info@samuelsjoberg.se>
> ---
> Changes in v2:
> - Fixed subject line and commit message.
>
> drivers/staging/rts5208/ms.c | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/staging/rts5208/ms.c b/drivers/staging/rts5208/ms.c
> index 2a6fab5c117a..14449f8afad5 100644
> --- a/drivers/staging/rts5208/ms.c
> +++ b/drivers/staging/rts5208/ms.c
> @@ -1749,11 +1749,10 @@ static int ms_copy_page(struct rtsx_chip *chip, u16 old_blk, u16 new_blk,
>
> for (rty_cnt = 0; rty_cnt < MS_MAX_RETRY_COUNT;
> rty_cnt++) {
> - retval = ms_transfer_tpc(
> - chip,
> - MS_TM_NORMAL_WRITE,
> - WRITE_PAGE_DATA,
> - 0, NO_WAIT_INT);
> + retval = ms_transfer_tpc(chip,
> + MS_TM_NORMAL_WRITE,
> + WRITE_PAGE_DATA,
> + 0, NO_WAIT_INT);
> if (retval == STATUS_SUCCESS)
> break;
> }
> --
> 2.35.1
>
This change never ended up on the mailing list for some reason :(
Please fix up and resend.
thanks,
greg k-h
^ permalink raw reply [flat|nested] only message in thread