Re: [PATCH net-next] net: dwc-xlgmac: add the initial ethtool support

From: David Miller
Date: Tue Apr 11 2017 - 10:47:57 EST


From: Jie Deng <Jie.Deng1@xxxxxxxxxxxx>
Date: Tue, 11 Apr 2017 16:14:04 +0800

> It is necessary to provide ethtool support for displaying and
> modifying parameters of dwc-xlgmac.
>
> Signed-off-by: Jie Deng <jiedeng@xxxxxxxxxxxx>

I don't see why you cannot allow the statistics to be obtained when
the device is not running. Similarly for other ethtool calls that
just fetch information stored in the software state of the driver.

I think your ->begin() method is way too restrictive.