Bug 197177 - Wrong registers used for linkup status and tx fifo empty status in mge driver on ramada 370 and armada xp socs
Summary: Wrong registers used for linkup status and tx fifo empty status in mge driver...
Status: Closed Overcome By Events
Alias: None
Product: Base System
Classification: Unclassified
Component: arm (show other bugs)
Version: CURRENT
Hardware: arm Any
: --- Affects Only Me
Assignee: freebsd-arm (Nobody)
URL:
Keywords: patch
: 197165 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-01-29 16:23 UTC by Alexander Zhigunov
Modified: 2025-02-05 02:02 UTC (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Zhigunov 2015-01-29 16:23:39 UTC
Registers used in mge driver are valid for marvell kirkwood family soc, but they are not valid for armada xp and armada 370 socs.
As you can see in marvell documentation - registers are changed on armada 370 and armada xp.
kirkwood:
http://www.marvell.com/embedded-processors/kirkwood/assets/FS_88F6180_9x_6281_OpenSource.pdf
armada xp (same for armada 370):
http://www.marvell.com/embedded-processors/armada-xp/assets/ARMADA-XP-Functional-SpecDatasheet.pdf

So mge driver doesn't work in current distribution on armada xp and armada 370 SoCs.

So here is my diff.

diff --git a/sys/dev/mge/if_mge.c b/sys/dev/mge/if_mge.c
index c35dd04..e2305b8 100644
--- a/sys/dev/mge/if_mge.c
+++ b/sys/dev/mge/if_mge.c
@@ -1654,11 +1654,11 @@ mge_stop(struct mge_softc *sc)
        }
 
        /* Wait for end of transmission */
-       count = 0x100000;
-       while (count--) {
-               reg_val = MGE_READ(sc, MGE_PORT_STATUS);
-               if ( !(reg_val & MGE_STATUS_TX_IN_PROG) &&
-                   (reg_val & MGE_STATUS_TX_FIFO_EMPTY))
+       count = 0x100001;
+       while (--count) {
+               reg_val = MGE_READ(sc, MGE_ETH_PORT_STATUS);
+               if ( !(reg_val & MGE_ETH_STATUS_TX_IN_PROG) && 
+                    (reg_val & MGE_ETH_STATUS_TX_FIFO_EMPTY))
                        break;
                DELAY(100);
        }
diff --git a/sys/dev/mge/if_mgevar.h b/sys/dev/mge/if_mgevar.h
index 0ab28bf..7cfd20e 100644
--- a/sys/dev/mge/if_mgevar.h
+++ b/sys/dev/mge/if_mgevar.h
@@ -228,14 +228,21 @@ struct mge_softc {
 #define PORT_SERIAL_GMII_SPEED_1000    (1 << 23)
 #define PORT_SERIAL_MII_SPEED_100      (1 << 24)
 
-#define MGE_PORT_STATUS                        0x444
-#define MGE_STATUS_LINKUP              (1 << 1)
-#define MGE_STATUS_FULL_DUPLEX         (1 << 2)
-#define MGE_STATUS_FLOW_CONTROL                (1 << 3)
-#define MGE_STATUS_1000MB              (1 << 4)
-#define MGE_STATUS_100MB               (1 << 5)
-#define MGE_STATUS_TX_IN_PROG          (1 << 7)
-#define MGE_STATUS_TX_FIFO_EMPTY       (1 << 10)
+#define MGE_ETH_PORT_STATUS            0x444
+#define MGE_ETH_STATUS_RX_FIFO_EMPTY    (1 << 16)
+#define MGE_ETH_STATUS_TX_IN_PROG      (1 << 0)
+#define MGE_ETH_STATUS_TX_FIFO_EMPTY    (1 << 8)
+
+#define MGE_PORT_STATUS                        0xc10
+#define MGE_STATUS_LINKUP              (1 << 0)
+#define MGE_STATUS_GMII_SPEED          (1 << 1) /* 0 = 10/100 Mbps: Port speed is 10 Mbps or 100 Mbps according to <MIISpeed>. 1 = 1000 Mbps: Port speed is 1000 Mbps. */
+#define MGE_STATUS_MII_SPEED           (1 << 2) /* 0 = 10: Port speed is 10 Mbps. 1 = 100: Port speed is 100 Mbps. */
+#define MGE_STATUS_FULL_DUPLEX         (1 << 3) /* 0 = half-duplex: Port is in half-duplex mode. 1 = full-duplex: Port is in full-duplex mode. */
+#define MGE_STATUS_RX_FLOW_CONTROL     (1 << 4)
+#define MGE_STATUS_TX_FLOW_CONTROL     (1 << 5)
+#define MGE_STATUS_RX_PAUSE            (1 << 6)
+#define MGE_STATUS_TX_PAUSE            (1 << 7)
+#define MGE_STATUS_AUTONEG_DONE                (1 << 11)
 
 #define MGE_TX_QUEUE_CMD       0x448
 #define MGE_ENABLE_TXQ         (1 << 0)


Maybe we need to check soc id in header file and alter definition for registers? 
Can you add my changes to make possible usage of onchip ethernet controller on armada xp soc?
Comment 1 Alexander Zhigunov 2015-01-29 16:29:33 UTC
*** Bug 197165 has been marked as a duplicate of this bug. ***
Comment 2 Mark Linimon freebsd_committer freebsd_triage 2025-01-25 02:07:42 UTC
^Triage: note that there is an inline patch.

To submitter: is this board even supported any more?
Comment 3 Alexander Zhigunov 2025-01-27 12:31:14 UTC
(In reply to Mark Linimon from comment #2)
No, this board is no longer supported :)
Comment 4 Mark Linimon freebsd_committer freebsd_triage 2025-02-05 02:02:23 UTC
^Triage: overcome by events.