diff mbox

[U-Boot] net: altera_tse: Zap unused variable

Message ID 1441360228-4562-1-git-send-email-marex@denx.de
State Accepted
Delegated to: Marek Vasut
Headers show

Commit Message

Marek Vasut Sept. 4, 2015, 9:50 a.m. UTC
Zap variable which is set but never used.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
Cc: Joe Hershberger <joe.hershberger@ni.com>
---
 drivers/net/altera_tse.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Thomas Chou Sept. 4, 2015, 1:27 p.m. UTC | #1
Hi Marek,

On 09/04/2015 05:50 PM, Marek Vasut wrote:
> Zap variable which is set but never used.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
> Cc: Joe Hershberger <joe.hershberger@ni.com>
> ---
>   drivers/net/altera_tse.c | 2 --
>   1 file changed, 2 deletions(-)

Tested-by: Thomas Chou <thomas@wytron.com.tw>
Marek Vasut Sept. 4, 2015, 2:10 p.m. UTC | #2
On Friday, September 04, 2015 at 03:27:37 PM, Thomas Chou wrote:
> Hi Marek,

Hi,

> On 09/04/2015 05:50 PM, Marek Vasut wrote:
> > Zap variable which is set but never used.
> > 
> > Signed-off-by: Marek Vasut <marex@denx.de>
> > Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
> > Cc: Joe Hershberger <joe.hershberger@ni.com>
> > ---
> > 
> >   drivers/net/altera_tse.c | 2 --
> >   1 file changed, 2 deletions(-)
> 
> Tested-by: Thomas Chou <thomas@wytron.com.tw>

Applied to u-boot-socfpga/master , it's a cleanup afterall.

Best regards,
Marek Vasut
diff mbox

Patch

diff --git a/drivers/net/altera_tse.c b/drivers/net/altera_tse.c
index c4fd6ec..ad3bd1e 100644
--- a/drivers/net/altera_tse.c
+++ b/drivers/net/altera_tse.c
@@ -182,12 +182,10 @@  static int alt_sgdma_do_sync_transfer(volatile struct alt_sgdma_registers *dev,
 static int alt_sgdma_do_async_transfer(volatile struct alt_sgdma_registers *dev,
 				volatile struct alt_sgdma_descriptor *desc)
 {
-	unsigned int status;
 	int counter = 0;
 
 	/* Wait for any pending transfers to complete */
 	alt_sgdma_print_desc(desc);
-	status = dev->status;
 
 	counter = 0;
 	while (dev->status & ALT_SGDMA_STATUS_BUSY_MSK) {