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;
From:   Minas Harutyunyan <Minas.Harutyunyan@synopsys.com>
To:     Josh Abraham <j.abraham1776@gmail.com>,
        "Minas.Harutyunyan@synopsys.com" <Minas.Harutyunyan@synopsys.com>
CC:     "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
        "kernel-janitors@vger.kernel.org" <kernel-janitors@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] usb: dwc2: remove set but unused variable
Thread-Topic: [PATCH] usb: dwc2: remove set but unused variable
Thread-Index: AQHUSsS0IZp2wfB40U2wTqsBZDms6g==
Date:   Thu, 13 Sep 2018 06:36:35 +0000
Message-ID: <410670D7E743164D87FA6160E7907A56013A769BC4@am04wembxa.internal.synopsys.com>
References: <20180912181610.GA31006@josh-ThinkPad-T460s>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="Windows-1252"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 9/12/2018 10:16 PM, Josh Abraham wrote:=0A=
> This patch removes a set but unused variable in hcd.c.=0A=
> =0A=
> Fixes warning:=0A=
> variable =91data_fifo=92 set but not used [-Wunused-but-set-variable]=0A=
> =0A=
> Signed-off-by: Joshua Abraham <j.abraham1776@gmail.com>=0A=
> ---=0A=
>   drivers/usb/dwc2/hcd.c | 3 ---=0A=
>   1 file changed, 3 deletions(-)=0A=
> =0A=
> diff --git a/drivers/usb/dwc2/hcd.c b/drivers/usb/dwc2/hcd.c=0A=
> index 2bd6e6bfc241..5f23b933cafc 100644=0A=
> --- a/drivers/usb/dwc2/hcd.c=0A=
> +++ b/drivers/usb/dwc2/hcd.c=0A=
> @@ -1328,14 +1328,11 @@ static void dwc2_hc_write_packet(struct dwc2_hsot=
g *hsotg,=0A=
>   	u32 remaining_count;=0A=
>   	u32 byte_count;=0A=
>   	u32 dword_count;=0A=
> -	u32 __iomem *data_fifo;=0A=
>   	u32 *data_buf =3D (u32 *)chan->xfer_buf;=0A=
>   =0A=
>   	if (dbg_hc(chan))=0A=
>   		dev_vdbg(hsotg->dev, "%s()\n", __func__);=0A=
>   =0A=
> -	data_fifo =3D (u32 __iomem *)(hsotg->regs + HCFIFO(chan->hc_num));=0A=
> -=0A=
>   	remaining_count =3D chan->xfer_len - chan->xfer_count;=0A=
>   	if (remaining_count > chan->max_packet)=0A=
>   		byte_count =3D chan->max_packet;=0A=
> =0A=
Acked-by: Minas Harutyunyan <hminas@synopsys.com>=0A=