From: Oleksij Rempel <o.rempel@pengutronix.de>
To: barebox@lists.infradead.org
Cc: Oleksij Rempel <o.rempel@pengutronix.de>
Subject: [PATCH v1 1/2] net: macb: init multiple dummy TX queues
Date: Wed, 14 Aug 2019 12:27:28 +0200 [thread overview]
Message-ID: <20190814102729.23857-1-o.rempel@pengutronix.de> (raw)
Microchip SAMA5D27 has more then one TX queue. So it will
go in to TX timeout if only one was initialized.
Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
---
drivers/net/macb.c | 32 ++++++++++++++++++++++++++------
drivers/net/macb.h | 7 +++++++
2 files changed, 33 insertions(+), 6 deletions(-)
diff --git a/drivers/net/macb.c b/drivers/net/macb.c
index a0411d6e4b..0b3c909433 100644
--- a/drivers/net/macb.c
+++ b/drivers/net/macb.c
@@ -322,6 +322,29 @@ static void macb_configure_dma(struct macb_device *bp)
}
}
+static int gmac_init_dummy_tx_queues(struct macb_device *macb)
+{
+ int i, num_queues = 1;
+ u32 queue_mask;
+
+ /* bit 0 is never set but queue 0 always exists */
+ queue_mask = gem_readl(macb, DCFG6) & 0xff;
+ queue_mask |= 0x1;
+
+ for (i = 1; i < MACB_MAX_QUEUES; i++)
+ if (queue_mask & (1 << i))
+ num_queues++;
+
+ macb->gem_q1_descs[0].addr = 0;
+ macb->gem_q1_descs[0].ctrl = MACB_BIT(TX_WRAP) |
+ MACB_BIT(TX_LAST) | MACB_BIT(TX_USED);
+
+ for (i = 1; i < num_queues; i++)
+ gem_writel_queue_TBQP(macb, &macb->gem_q1_descs[0], i - 1);
+
+ return 0;
+}
+
static void macb_init(struct macb_device *macb)
{
unsigned long paddr, val = 0;
@@ -357,16 +380,13 @@ static void macb_init(struct macb_device *macb)
macb_writel(macb, TBQP, (ulong)macb->tx_ring);
if (macb->is_gem && macb->gem_q1_descs) {
- /* Disable the second priority queue */
- macb->gem_q1_descs[0].addr = 0;
- macb->gem_q1_descs[0].ctrl = MACB_BIT(TX_WRAP) |
- MACB_BIT(TX_LAST) |
- MACB_BIT(TX_USED);
+ gmac_init_dummy_tx_queues(macb);
+
+ /* Disable the second priority rx queue */
macb->gem_q1_descs[1].addr = MACB_BIT(RX_USED) |
MACB_BIT(RX_WRAP);
macb->gem_q1_descs[1].ctrl = 0;
- gem_writel(macb, TQ1, (ulong)&macb->gem_q1_descs[0]);
gem_writel(macb, RQ1, (ulong)&macb->gem_q1_descs[1]);
}
diff --git a/drivers/net/macb.h b/drivers/net/macb.h
index fda4d08663..2e5b64e5e9 100644
--- a/drivers/net/macb.h
+++ b/drivers/net/macb.h
@@ -5,6 +5,8 @@
#ifndef __DRIVERS_MACB_H__
#define __DRIVERS_MACB_H__
+#define MACB_MAX_QUEUES 8
+
/* MACB register offsets */
#define MACB_NCR 0x0000
#define MACB_NCFGR 0x0004
@@ -75,6 +77,8 @@
#define GEM_TQ1 0x0440
#define GEM_RQ1 0x0480
+#define GEM_TBQP(hw_q) (0x0440 + ((hw_q) << 2))
+
/* Bitfields in NCR */
#define MACB_LB_OFFSET 0
#define MACB_LB_SIZE 1
@@ -436,4 +440,7 @@ struct macb_dma_desc {
#define MACB_TX_USED_OFFSET 31
#define MACB_TX_USED_SIZE 1
+#define gem_writel_queue_TBQP(port, value, queue_num) \
+ writel((value), (port)->regs + GEM_TBQP(queue_num))
+
#endif /* __DRIVERS_MACB_H__ */
--
2.20.1
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next reply other threads:[~2019-08-14 10:27 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-14 10:27 Oleksij Rempel [this message]
2019-08-14 10:27 ` [PATCH v1 2/2] net: macb: extend support to Microchip SAMA5D2 Oleksij Rempel
2019-08-14 16:39 ` [PATCH v1 1/2] net: macb: init multiple dummy TX queues Ahmad Fatoum
2019-08-16 11:42 ` Sascha Hauer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190814102729.23857-1-o.rempel@pengutronix.de \
--to=o.rempel@pengutronix.de \
--cc=barebox@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox