diff options
author | Ben Hutchings <bhutchings@solarflare.com> | 2008-09-01 07:48:46 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@redhat.com> | 2008-09-03 09:53:48 -0400 |
commit | bc3c90a2b70652b87cde8de65275d6f41d0f24c3 (patch) | |
tree | 3d9a7eebad85bc86f5a3fbd4fdf4cc2b7be950a4 /drivers/net/sfc/tx.c | |
parent | c1e5fcc980b7b2185b29e4f9f0d8266806ada9eb (diff) |
sfc: Remove some unreachable error paths
Some functions return an error code which is always 0. Change their
return types to void and simplify their callers accordingly.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers/net/sfc/tx.c')
-rw-r--r-- | drivers/net/sfc/tx.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/sfc/tx.c b/drivers/net/sfc/tx.c index a3a3eddd30b9..cdee7c200d63 100644 --- a/drivers/net/sfc/tx.c +++ b/drivers/net/sfc/tx.c | |||
@@ -443,7 +443,7 @@ int efx_probe_tx_queue(struct efx_tx_queue *tx_queue) | |||
443 | return rc; | 443 | return rc; |
444 | } | 444 | } |
445 | 445 | ||
446 | int efx_init_tx_queue(struct efx_tx_queue *tx_queue) | 446 | void efx_init_tx_queue(struct efx_tx_queue *tx_queue) |
447 | { | 447 | { |
448 | EFX_LOG(tx_queue->efx, "initialising TX queue %d\n", tx_queue->queue); | 448 | EFX_LOG(tx_queue->efx, "initialising TX queue %d\n", tx_queue->queue); |
449 | 449 | ||
@@ -454,7 +454,7 @@ int efx_init_tx_queue(struct efx_tx_queue *tx_queue) | |||
454 | BUG_ON(tx_queue->stopped); | 454 | BUG_ON(tx_queue->stopped); |
455 | 455 | ||
456 | /* Set up TX descriptor ring */ | 456 | /* Set up TX descriptor ring */ |
457 | return falcon_init_tx(tx_queue); | 457 | falcon_init_tx(tx_queue); |
458 | } | 458 | } |
459 | 459 | ||
460 | void efx_release_tx_buffers(struct efx_tx_queue *tx_queue) | 460 | void efx_release_tx_buffers(struct efx_tx_queue *tx_queue) |