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/rx.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/rx.c')
-rw-r--r-- | drivers/net/sfc/rx.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/sfc/rx.c b/drivers/net/sfc/rx.c index 7d2dc20d0ca3..0f805da4ce55 100644 --- a/drivers/net/sfc/rx.c +++ b/drivers/net/sfc/rx.c | |||
@@ -800,7 +800,7 @@ int efx_probe_rx_queue(struct efx_rx_queue *rx_queue) | |||
800 | return rc; | 800 | return rc; |
801 | } | 801 | } |
802 | 802 | ||
803 | int efx_init_rx_queue(struct efx_rx_queue *rx_queue) | 803 | void efx_init_rx_queue(struct efx_rx_queue *rx_queue) |
804 | { | 804 | { |
805 | struct efx_nic *efx = rx_queue->efx; | 805 | struct efx_nic *efx = rx_queue->efx; |
806 | unsigned int max_fill, trigger, limit; | 806 | unsigned int max_fill, trigger, limit; |
@@ -824,7 +824,7 @@ int efx_init_rx_queue(struct efx_rx_queue *rx_queue) | |||
824 | rx_queue->fast_fill_limit = limit; | 824 | rx_queue->fast_fill_limit = limit; |
825 | 825 | ||
826 | /* Set up RX descriptor ring */ | 826 | /* Set up RX descriptor ring */ |
827 | return falcon_init_rx(rx_queue); | 827 | falcon_init_rx(rx_queue); |
828 | } | 828 | } |
829 | 829 | ||
830 | void efx_fini_rx_queue(struct efx_rx_queue *rx_queue) | 830 | void efx_fini_rx_queue(struct efx_rx_queue *rx_queue) |