Commit cac7663c authored by David S. Miller's avatar David S. Miller

Merge branch 'sfc-misc-cleanups'

Edward Cree says:

====================
sfc: misc cleanups

Clean up a few nits I noticed while working on TXQ stuff.
====================
Reviewed-by: default avatarJakub Kicinski <kuba@kernel.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents 12913f74 4da24fa6
...@@ -1200,10 +1200,6 @@ static int ef100_probe_main(struct efx_nic *efx) ...@@ -1200,10 +1200,6 @@ static int ef100_probe_main(struct efx_nic *efx)
if (rc) if (rc)
goto fail; goto fail;
rc = efx_init_channels(efx);
if (rc)
goto fail;
down_write(&efx->filter_sem); down_write(&efx->filter_sem);
rc = ef100_filter_table_probe(efx); rc = ef100_filter_table_probe(efx);
up_write(&efx->filter_sem); up_write(&efx->filter_sem);
......
...@@ -27,7 +27,6 @@ int ef100_tx_probe(struct efx_tx_queue *tx_queue) ...@@ -27,7 +27,6 @@ int ef100_tx_probe(struct efx_tx_queue *tx_queue)
(tx_queue->ptr_mask + 2) * (tx_queue->ptr_mask + 2) *
sizeof(efx_oword_t), sizeof(efx_oword_t),
GFP_KERNEL); GFP_KERNEL);
return 0;
} }
void ef100_tx_init(struct efx_tx_queue *tx_queue) void ef100_tx_init(struct efx_tx_queue *tx_queue)
......
...@@ -505,8 +505,7 @@ static void efx_filter_rfs_expire(struct work_struct *data) ...@@ -505,8 +505,7 @@ static void efx_filter_rfs_expire(struct work_struct *data)
#endif #endif
/* Allocate and initialise a channel structure. */ /* Allocate and initialise a channel structure. */
struct efx_channel * static struct efx_channel *efx_alloc_channel(struct efx_nic *efx, int i)
efx_alloc_channel(struct efx_nic *efx, int i, struct efx_channel *old_channel)
{ {
struct efx_rx_queue *rx_queue; struct efx_rx_queue *rx_queue;
struct efx_tx_queue *tx_queue; struct efx_tx_queue *tx_queue;
...@@ -545,7 +544,7 @@ int efx_init_channels(struct efx_nic *efx) ...@@ -545,7 +544,7 @@ int efx_init_channels(struct efx_nic *efx)
unsigned int i; unsigned int i;
for (i = 0; i < EFX_MAX_CHANNELS; i++) { for (i = 0; i < EFX_MAX_CHANNELS; i++) {
efx->channel[i] = efx_alloc_channel(efx, i, NULL); efx->channel[i] = efx_alloc_channel(efx, i);
if (!efx->channel[i]) if (!efx->channel[i])
return -ENOMEM; return -ENOMEM;
efx->msi_context[i].efx = efx; efx->msi_context[i].efx = efx;
......
...@@ -31,8 +31,6 @@ void efx_stop_eventq(struct efx_channel *channel); ...@@ -31,8 +31,6 @@ void efx_stop_eventq(struct efx_channel *channel);
void efx_fini_eventq(struct efx_channel *channel); void efx_fini_eventq(struct efx_channel *channel);
void efx_remove_eventq(struct efx_channel *channel); void efx_remove_eventq(struct efx_channel *channel);
struct efx_channel *
efx_alloc_channel(struct efx_nic *efx, int i, struct efx_channel *old_channel);
int efx_realloc_channels(struct efx_nic *efx, u32 rxq_entries, u32 txq_entries); int efx_realloc_channels(struct efx_nic *efx, u32 rxq_entries, u32 txq_entries);
void efx_get_channel_name(struct efx_channel *channel, char *buf, size_t len); void efx_get_channel_name(struct efx_channel *channel, char *buf, size_t len);
void efx_set_channel_names(struct efx_nic *efx); void efx_set_channel_names(struct efx_nic *efx);
......
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