Re: [PATCH net-next] net: dsa: mv88e6xxx: Add serdes read/write dynamic debug

From: Andrew Lunn
Date: Thu Jul 09 2020 - 16:09:55 EST


On Thu, Jul 09, 2020 at 11:43:18AM -0700, Chris Healy wrote:
> Add deb_dbg print statements in both serdes_read and serdes_write
> functions.

Hi Chris

Why is SERDES access special? Why not all accesses? global1, global2,
global3, port, etc.

As David said, tracepoints are better for this. Take a look at
include/trace/events/mdio.h and drivers/net/phy/mdio_bus.c as an
example which traces all mdio accesses.

Andrew