Re: [PATCH] Fix checkpatch issue

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Fri, Feb 16, 2018 at 12:41:00PM -0800, Ishita Tripathi wrote:
> Deleted line break after dereference operator (->)

Your subject is a bit "odd" :)

Please look at other patches for this directory to see the proper prefix
to use.

> 
> Signed-off-by: Ishita Tripathi <sakurakhadag@gmail.com>
> 
> ---
>  drivers/staging/comedi/drivers/rtd520.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/staging/comedi/drivers/rtd520.c b/drivers/staging/comedi/drivers/rtd520.c
> index bb400e0..47b7093 100644
> --- a/drivers/staging/comedi/drivers/rtd520.c
> +++ b/drivers/staging/comedi/drivers/rtd520.c
> @@ -815,9 +815,8 @@ static int rtd_ai_cmdtest(struct comedi_device *dev,
>  
>  		if (cmd->scan_begin_src == TRIG_TIMER) {
>  			arg = cmd->convert_arg * cmd->scan_end_arg;
> -			err |= comedi_check_trigger_arg_min(&cmd->
> -							    scan_begin_arg,
> -							    arg);
> +			err |= comedi_check_trigger_arg_min(&cmd->scan_begin_arg,
> +					arg);


No need to move the last line here, right?  Please keep it as-is.

thanks,

greg k-h