Re: [PATCH] staging: ks7010: Fix unnecessary long lines

[Greg KH <gregkh@linuxfoundation.org>]

On Sun, Oct 27, 2019 at 03:40:56AM +0000, Frank A. Cancio Bello wrote:
> checkpatch message: "WARNING:LONG_LINE: line over 80 characters"
> 
> Signed-off-by: Frank A. Cancio Bello <frank@generalsoftwareinc.com>
> ---
> 
> This is the first patch that I send for "Outreachy December 2019 to
> March 2020". My plan is to keep these first patches very simple.
> Once they get approved I will move to less trivial fixes ;)
> 
>  drivers/staging/ks7010/ks7010_sdio.c | 3 ++-
>  drivers/staging/ks7010/ks_hostif.c   | 9 ++++++---
>  drivers/staging/ks7010/ks_wlan_net.c | 3 ++-
>  3 files changed, 10 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/staging/ks7010/ks7010_sdio.c b/drivers/staging/ks7010/ks7010_sdio.c
> index 4b379542ecd5..c7ba658e234e 100644
> --- a/drivers/staging/ks7010/ks7010_sdio.c
> +++ b/drivers/staging/ks7010/ks7010_sdio.c
> @@ -446,7 +446,8 @@ static void ks_wlan_hw_rx(struct ks_wlan_private *priv, size_t size)
>  				     DUMP_PREFIX_OFFSET,
>  				     rx_buffer->data, 32);
>  #endif
> -		ret = ks7010_sdio_writeb(priv, READ_STATUS_REG, REG_STATUS_IDLE);
> +		ret = ks7010_sdio_writeb(priv,
> +					 READ_STATUS_REG, REG_STATUS_IDLE);

Why not just wrap the last parameter?  That's the normal way to do
something like this.

thanks,

greg k-h