Commit b67030b1 authored by Colin Ian King's avatar Colin Ian King Committed by David S. Miller

fjes: use currently unused variable my_epid and max_epid

Variables my_epid and max_epid are currently assigned and not being
used - however, I suspect they were intended to be used in the for-loops
to reduce the dereferencing of hw.  Replace hw->my_epid and hw->max_epid
with these variables.

Cleans up clang warnings:
warning: variable 'my_epid' set but not used [-Wunused-but-set-variable]
variable 'max_epid' set but not used [-Wunused-but-set-variable]
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 2f9be181
...@@ -1395,8 +1395,8 @@ static void fjes_watch_unshare_task(struct work_struct *work) ...@@ -1395,8 +1395,8 @@ static void fjes_watch_unshare_task(struct work_struct *work)
while ((unshare_watch_bitmask || hw->txrx_stop_req_bit) && while ((unshare_watch_bitmask || hw->txrx_stop_req_bit) &&
(wait_time < 3000)) { (wait_time < 3000)) {
for (epidx = 0; epidx < hw->max_epid; epidx++) { for (epidx = 0; epidx < max_epid; epidx++) {
if (epidx == hw->my_epid) if (epidx == my_epid)
continue; continue;
is_shared = fjes_hw_epid_is_shared(hw->hw_info.share, is_shared = fjes_hw_epid_is_shared(hw->hw_info.share,
...@@ -1453,8 +1453,8 @@ static void fjes_watch_unshare_task(struct work_struct *work) ...@@ -1453,8 +1453,8 @@ static void fjes_watch_unshare_task(struct work_struct *work)
} }
if (hw->hw_info.buffer_unshare_reserve_bit) { if (hw->hw_info.buffer_unshare_reserve_bit) {
for (epidx = 0; epidx < hw->max_epid; epidx++) { for (epidx = 0; epidx < max_epid; epidx++) {
if (epidx == hw->my_epid) if (epidx == my_epid)
continue; continue;
if (test_bit(epidx, if (test_bit(epidx,
......
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