mbox series

[net-next,0/3] net: dsa: mv88e6xxx: Tiny fixes/improvements

Message ID 20210421120454.1541240-1-tobias@waldekranz.com
Headers show
Series net: dsa: mv88e6xxx: Tiny fixes/improvements | expand

Message

Tobias Waldekranz April 21, 2021, 12:04 p.m. UTC 
Just some small things I have noticed that do not fit in any other
series.

Tobias Waldekranz (3):
  net: dsa: mv88e6xxx: Correct spelling of define "ADRR" -> "ADDR"
  net: dsa: mv88e6xxx: Fix off-by-one in VTU devlink region size
  net: dsa: mv88e6xxx: Export cross-chip PVT as devlink region

 drivers/net/dsa/mv88e6xxx/chip.c    |  2 +-
 drivers/net/dsa/mv88e6xxx/chip.h    |  3 ++
 drivers/net/dsa/mv88e6xxx/devlink.c | 58 ++++++++++++++++++++++++++++-
 drivers/net/dsa/mv88e6xxx/global2.c | 17 +++++++++
 drivers/net/dsa/mv88e6xxx/global2.h |  4 +-
 5 files changed, 81 insertions(+), 3 deletions(-)

Comments

Andrew Lunn April 21, 2021, 12:17 p.m. UTC | #1 
On Wed, Apr 21, 2021 at 02:04:52PM +0200, Tobias Waldekranz wrote:
> Because ADRR is not a thing.
> 
> Signed-off-by: Tobias Waldekranz <tobias@waldekranz.com>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew
Andrew Lunn April 21, 2021, 12:28 p.m. UTC | #2 
On Wed, Apr 21, 2021 at 02:04:54PM +0200, Tobias Waldekranz wrote:
> Export the raw PVT data in a devlink region so that it can be
> inspected from userspace and compared to the current bridge
> configuration.
> 
> Signed-off-by: Tobias Waldekranz <tobias@waldekranz.com>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew