Re: [PATCH v2 4/4] staging: vt6655: device_main: Break up multiple assignments

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

On Sun, Sep 18, 2016 at 02:35:53PM +0530, Namrata A Shettar wrote:
> Break up multiple assignments on the same line to resolve checkpatch
> issue.
> 
> Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
> ---
>  drivers/staging/vt6655/device_main.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/vt6655/device_main.c b/drivers/staging/vt6655/device_main.c
> index f109eea..86532ca 100644
> --- a/drivers/staging/vt6655/device_main.c
> +++ b/drivers/staging/vt6655/device_main.c
> @@ -669,7 +669,8 @@ static void device_init_td1_ring(struct vnt_private *priv)
>  
>  	if (i > 0)
>  		priv->apTD1Rings[i-1].next_desc = cpu_to_le32(priv->td1_pool_dma);
> -	priv->apTailTD[1] = priv->apCurrTD[1] = &priv->apTD1Rings[0];
> +	 priv->apCurrTD[1] = &priv->apTD1Rings[0];
> +	 priv->apTailTD[1] = priv->apCurrTD[1];

Same problem here, please be careful with your indentation, always run
your patches through checkpatch.pl before sending them out so that
people don't have to remind you to run your patches through
checkpatch.pl before sending them out.

thanks,

greg k-h