On Wed, 3 May 2017 14:44:35 +1000 Gavin Shan <[email protected]> wrote:
> +void ncsi_ethtool_register_dev(struct net_device *dev)
> +{
> + struct ethtool_ops *ops;
> +
> + ops = (struct ethtool_ops *)(dev->ethtool_ops);
> + if (!ops)
> + return;
> +
> + ops->get_ncsi_channels = ncsi_get_channels;
> +}
> +
Instead of casting away const which opens up potential security
issues. Have two ethtool_ops structures.
