Re: [PATCH v2] staging: rts5208: Lines should not end with a '('

[Greg KH <gregkh@linuxfoundation.org>]

On Thu, Mar 14, 2019 at 11:28:28AM -0400, Branden Bonaby wrote:
> Start function arguments after initial bracket so that the
> line does not end with a opening bracket.
> CHECK: Lines should not end with a '('
> 
> Signed-off-by: Branden Bonaby <brandonbonaby94@gmail.com>
> ---
> Changes in v2:
> 	Moved function arguments that do not go over the
> 	80 character a line limit, to the same line as the
> 	function name.
> 
>  drivers/staging/rts5208/rtsx_transport.c | 14 ++++++++------
>  1 file changed, 8 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/staging/rts5208/rtsx_transport.c b/drivers/staging/rts5208/rtsx_transport.c
> index 8277d7895608..a476e22767e9 100644
> --- a/drivers/staging/rts5208/rtsx_transport.c
> +++ b/drivers/staging/rts5208/rtsx_transport.c
> @@ -257,8 +257,9 @@ int rtsx_send_cmd(struct rtsx_chip *chip, u8 card, int timeout)
>  	spin_unlock_irq(&rtsx->reg_lock);
>  
>  	/* Wait for TRANS_OK_INT */
> -	timeleft = wait_for_completion_interruptible_timeout(
> -		&trans_done, msecs_to_jiffies(timeout));
> +	timeleft =
> +	   wait_for_completion_interruptible_timeout(&trans_done,

Indenting 4 spaces is not acceptable coding style, sorry.

greg k-h