Commit add64526 authored by Peter Senna Tschudin's avatar Peter Senna Tschudin Committed by Greg Kroah-Hartman

staging: ft1000: Remove useless return variables

This patch remove variables that are initialized with a constant,
are never updated, and are only used as parameter of return.
Return the constant instead of using a variable.

Verified by compilation only.

The coccinelle script that find and fixes this issue is:
// <smpl>
@@
type T;
constant C;
identifier ret;
@@
- T ret = C;
... when != ret
- return ret;
+ return C;
// </smpl>
Signed-off-by: default avatarPeter Senna Tschudin <peter.senna@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent bf526957
...@@ -522,7 +522,6 @@ static void usb_dnld_complete(struct urb *urb) ...@@ -522,7 +522,6 @@ static void usb_dnld_complete(struct urb *urb)
static int write_blk_fifo(struct ft1000_usb *ft1000dev, u16 **pUsFile, static int write_blk_fifo(struct ft1000_usb *ft1000dev, u16 **pUsFile,
u8 **pUcFile, long word_length) u8 **pUcFile, long word_length)
{ {
int Status = 0;
int byte_length; int byte_length;
byte_length = word_length * 4; byte_length = word_length * 4;
...@@ -547,7 +546,7 @@ static int write_blk_fifo(struct ft1000_usb *ft1000dev, u16 **pUsFile, ...@@ -547,7 +546,7 @@ static int write_blk_fifo(struct ft1000_usb *ft1000dev, u16 **pUsFile,
*pUsFile = *pUsFile + (word_length << 1); *pUsFile = *pUsFile + (word_length << 1);
*pUcFile = *pUcFile + (word_length << 2); *pUcFile = *pUcFile + (word_length << 2);
return Status; return 0;
} }
static int scram_start_dwnld(struct ft1000_usb *ft1000dev, u16 *hshake, static int scram_start_dwnld(struct ft1000_usb *ft1000dev, u16 *hshake,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment