Re: [PATCH V2] staging: rts5208: fix Lines should not end with a '('.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxxx>
> ---
> 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




[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux