Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Sat, 29 Sep 2018 02:34:39 -0700
From:   Greg KH <gregkh@linuxfoundation.org>
To:     Shubham Singh <dhoni.singh094@gmail.com>
Cc:     forest@alittletooquite.net, devel@driver.osuosl.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Staging: vt6656: baseband.c: fixed unnecessary
 paranthesis
Message-ID: <20180929093439.GA30302@kroah.com>
References: <20180929085854.15880-1-dhoni.singh094@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20180929085854.15880-1-dhoni.singh094@gmail.com>
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Sat, Sep 29, 2018 at 02:28:54PM +0530, Shubham Singh wrote:
> From: shubhsherl <dhoni.singh094@gmail.com>
> 
> Fixed a coding line issue.
> 
> Signed-off-by: shubhsherl <dhoni.singh094@gmail.com>

We need a real name please.

> ---
>  drivers/staging/vt6656/baseband.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/staging/vt6656/baseband.c b/drivers/staging/vt6656/baseband.c
> index b29ba237fa29..540cc9eafd07 100644
> --- a/drivers/staging/vt6656/baseband.c
> +++ b/drivers/staging/vt6656/baseband.c
> @@ -382,8 +382,8 @@ int vnt_vt3184_init(struct vnt_private *priv)
>  
>  	dev_dbg(&priv->usb->dev, "RF Type %d\n", priv->rf_type);
>  
> -	if ((priv->rf_type == RF_AL2230) ||
> -	    (priv->rf_type == RF_AL2230S)) {
> +	if (priv->rf_type == RF_AL2230 ||
> +	    priv->rf_type == RF_AL2230S) {

Huh?  What coding style issue caused this?  The original code is fine,
don't you agree?  This type of change just makes it harder to work with
over time (hint, you have to go look up the order of operations all the
time...)

thanks

greg k-h