On Thu, Dec 03, 2020 at 08:01:40PM +0200, Vladimir Oltean wrote: > On Thu, Dec 03, 2020 at 06:53:20PM +0100, Oleksij Rempel wrote: > > It is possible to poll it more frequently, but it make no reals sense > > on this low power devices. > > Frankly I thought you understood the implications of periodic polling > and you're ok with them, I added polling to read out small counters to avoid overflow. > just wanting to have _something_. Having something is good, but making it good is better :D > But fine, > welcome to my world, happy to have you onboard... > > > What kind of options do we have? > > https://www.spinics.net/lists/netdev/msg703774.html > https://www.spinics.net/lists/netdev/msg704370.html > > Unfortunately I've been absolutely snowed under with work lately. I hope > to be able to come back to that during the weekend or something like that. Ok, so the strategy is to fix the original issue. Sound good. For now I'll resend this patches without accessing mdio regs from the stats64 callback. It will give initial play ground, so we can see what else should be done for DSA specific use case. Regards, Oleksij -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |