Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1599714

[PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

From Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Newsgroups linux.kernel
Subject [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time
Date 2017-03-13 20:30 +0100
Message-ID <tkDPk-58M-15@gated-at.bofh.it> (permalink)
Organization linux.* mail to news gateway

Show all headers | View raw


The ATU ageing time value programmed in the switch is rounded up to the
nearest multiple of its coefficient (variable depending on the model.)

Add a debug message to inform the user about the exact programmed value.

On 6352, "brctl setageing br0 18" gives "AgeTime set to 0x01 (15000 ms)"
while on 6390 we get "AgeTime set to 0x05 (18750 ms)".

Signed-off-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
---
 drivers/net/dsa/mv88e6xxx/global1_atu.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/drivers/net/dsa/mv88e6xxx/global1_atu.c b/drivers/net/dsa/mv88e6xxx/global1_atu.c
index f6cd3c939da4..bac34737b096 100644
--- a/drivers/net/dsa/mv88e6xxx/global1_atu.c
+++ b/drivers/net/dsa/mv88e6xxx/global1_atu.c
@@ -65,7 +65,14 @@ int mv88e6xxx_g1_atu_set_age_time(struct mv88e6xxx_chip *chip,
 	val &= ~0xff0;
 	val |= age_time << 4;
 
-	return mv88e6xxx_g1_write(chip, GLOBAL_ATU_CONTROL, val);
+	err = mv88e6xxx_g1_write(chip, GLOBAL_ATU_CONTROL, val);
+	if (err)
+		return err;
+
+	dev_dbg(chip->dev, "AgeTime set to 0x%02x (%d ms)\n", age_time,
+		age_time * coeff);
+
+	return 0;
 }
 
 /* Offset 0x0B: ATU Operation Register */
-- 
2.12.0

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

[PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-03-13 20:30 +0100
  Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time Andrew Lunn <andrew@lunn.ch> - 2017-03-13 23:40 +0100
    Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time Florian Fainelli <f.fainelli@gmail.com> - 2017-03-13 23:50 +0100
      Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time Andrew Lunn <andrew@lunn.ch> - 2017-03-14 00:00 +0100
        Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time Matthias May <matthias.may@neratec.com> - 2017-03-14 12:20 +0100
          Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time Andrew Lunn <andrew@lunn.ch> - 2017-03-14 13:10 +0100
            Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-03-14 15:10 +0100
              Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time Andrew Lunn <andrew@lunn.ch> - 2017-03-14 15:20 +0100

csiph-web