mirror of
https://github.com/torvalds/linux.git
synced 2026-05-31 02:24:24 +02:00
sfc: remove 'log-tc-errors' ethtool private flag
It no longer does anything now that we're using formatted extacks instead. So we can remove the driver's whole get/set priv_flags implementation. Signed-off-by: Edward Cree <ecree.xilinx@gmail.com> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
parent
ad1c80d5f7
commit
b799f052a9
|
|
@ -43,8 +43,6 @@ const struct ethtool_ops ef100_ethtool_ops = {
|
|||
.get_pauseparam = efx_ethtool_get_pauseparam,
|
||||
.set_pauseparam = efx_ethtool_set_pauseparam,
|
||||
.get_sset_count = efx_ethtool_get_sset_count,
|
||||
.get_priv_flags = efx_ethtool_get_priv_flags,
|
||||
.set_priv_flags = efx_ethtool_set_priv_flags,
|
||||
.self_test = efx_ethtool_self_test,
|
||||
.get_strings = efx_ethtool_get_strings,
|
||||
.get_link_ksettings = efx_ethtool_get_link_ksettings,
|
||||
|
|
|
|||
|
|
@ -101,14 +101,6 @@ static const struct efx_sw_stat_desc efx_sw_stat_desc[] = {
|
|||
|
||||
#define EFX_ETHTOOL_SW_STAT_COUNT ARRAY_SIZE(efx_sw_stat_desc)
|
||||
|
||||
static const char efx_ethtool_priv_flags_strings[][ETH_GSTRING_LEN] = {
|
||||
"log-tc-errors",
|
||||
};
|
||||
|
||||
#define EFX_ETHTOOL_PRIV_FLAGS_LOG_TC_ERRS BIT(0)
|
||||
|
||||
#define EFX_ETHTOOL_PRIV_FLAGS_COUNT ARRAY_SIZE(efx_ethtool_priv_flags_strings)
|
||||
|
||||
void efx_ethtool_get_drvinfo(struct net_device *net_dev,
|
||||
struct ethtool_drvinfo *info)
|
||||
{
|
||||
|
|
@ -460,8 +452,6 @@ int efx_ethtool_get_sset_count(struct net_device *net_dev, int string_set)
|
|||
efx_ptp_describe_stats(efx, NULL);
|
||||
case ETH_SS_TEST:
|
||||
return efx_ethtool_fill_self_tests(efx, NULL, NULL, NULL);
|
||||
case ETH_SS_PRIV_FLAGS:
|
||||
return EFX_ETHTOOL_PRIV_FLAGS_COUNT;
|
||||
default:
|
||||
return -EINVAL;
|
||||
}
|
||||
|
|
@ -488,39 +478,12 @@ void efx_ethtool_get_strings(struct net_device *net_dev,
|
|||
case ETH_SS_TEST:
|
||||
efx_ethtool_fill_self_tests(efx, NULL, strings, NULL);
|
||||
break;
|
||||
case ETH_SS_PRIV_FLAGS:
|
||||
for (i = 0; i < EFX_ETHTOOL_PRIV_FLAGS_COUNT; i++)
|
||||
strscpy(strings + i * ETH_GSTRING_LEN,
|
||||
efx_ethtool_priv_flags_strings[i],
|
||||
ETH_GSTRING_LEN);
|
||||
break;
|
||||
default:
|
||||
/* No other string sets */
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
u32 efx_ethtool_get_priv_flags(struct net_device *net_dev)
|
||||
{
|
||||
struct efx_nic *efx = efx_netdev_priv(net_dev);
|
||||
u32 ret_flags = 0;
|
||||
|
||||
if (efx->log_tc_errs)
|
||||
ret_flags |= EFX_ETHTOOL_PRIV_FLAGS_LOG_TC_ERRS;
|
||||
|
||||
return ret_flags;
|
||||
}
|
||||
|
||||
int efx_ethtool_set_priv_flags(struct net_device *net_dev, u32 flags)
|
||||
{
|
||||
struct efx_nic *efx = efx_netdev_priv(net_dev);
|
||||
|
||||
efx->log_tc_errs =
|
||||
!!(flags & EFX_ETHTOOL_PRIV_FLAGS_LOG_TC_ERRS);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
void efx_ethtool_get_stats(struct net_device *net_dev,
|
||||
struct ethtool_stats *stats,
|
||||
u64 *data)
|
||||
|
|
|
|||
|
|
@ -27,8 +27,6 @@ int efx_ethtool_fill_self_tests(struct efx_nic *efx,
|
|||
int efx_ethtool_get_sset_count(struct net_device *net_dev, int string_set);
|
||||
void efx_ethtool_get_strings(struct net_device *net_dev, u32 string_set,
|
||||
u8 *strings);
|
||||
u32 efx_ethtool_get_priv_flags(struct net_device *net_dev);
|
||||
int efx_ethtool_set_priv_flags(struct net_device *net_dev, u32 flags);
|
||||
void efx_ethtool_get_stats(struct net_device *net_dev,
|
||||
struct ethtool_stats *stats __attribute__ ((unused)),
|
||||
u64 *data);
|
||||
|
|
|
|||
|
|
@ -855,7 +855,6 @@ enum efx_xdp_tx_queues_mode {
|
|||
* @timer_max_ns: Interrupt timer maximum value, in nanoseconds
|
||||
* @irq_rx_adaptive: Adaptive IRQ moderation enabled for RX event queues
|
||||
* @irqs_hooked: Channel interrupts are hooked
|
||||
* @log_tc_errs: Error logging for TC filter insertion is enabled
|
||||
* @irq_rx_mod_step_us: Step size for IRQ moderation for RX event queues
|
||||
* @irq_rx_moderation_us: IRQ moderation time for RX event queues
|
||||
* @msg_enable: Log message enable flags
|
||||
|
|
@ -1018,7 +1017,6 @@ struct efx_nic {
|
|||
unsigned int timer_max_ns;
|
||||
bool irq_rx_adaptive;
|
||||
bool irqs_hooked;
|
||||
bool log_tc_errs;
|
||||
unsigned int irq_mod_step_us;
|
||||
unsigned int irq_rx_moderation_us;
|
||||
u32 msg_enable;
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user