From: Hongwei Zhang <hongweiz@ami.com>
To: <linux-aspeed@lists.ozlabs.org>, <linux-kernel@vger.kernel.org>,
<openbmc@lists.ozlabs.org>, Jakub Kicinski <kuba@kernel.org>,
David S Miller <davem@davemloft.net>
Cc: Hongwei Zhang <hongweiz@ami.com>, netdev <netdev@vger.kernel.org>,
Joel Stanley <joel@jms.id.au>, Andrew Jeffery <andrew@aj.id.au>
Subject: [Aspeed,ncsi-rx, v2 0/1] net: ftgmac100: Fix AST2600EVB NCSI RX issue
Date: Mon, 21 Dec 2020 14:40:25 -0500 [thread overview]
Message-ID: <20201221194026.30715-1-hongweiz@ami.com> (raw)
In-Reply-To: <20201215192323.24359-1-hongweiz@ami.com>
Dear Reviewer,
When FTGMAC100 driver is used on other NCSI Ethernet controllers, few
controllers have compatible issue. One example is Intel I210 Ethernet
controller on AST2600 BMC, with FTGMAC100 driver, it always trigger
RXDES0_RX_ERR error, cause NCSI initialization failure, removing
FTGMAC100_RXDES0_RX_ERR bit from RXDES0_ANY_ERROR fix the issue.
Here are part of the debug logs:
......
[ 35.075552] ftgmac100_hard_start_xmit TXDESO=b000003c
[ 35.080843] ftgmac100 1e660000.ethernet eth0: tx_complete_packet 55
[ 35.087141] ftgmac100 1e660000.ethernet eth0: rx_packet_error RXDES0=0xb0070040
[ 35.094448] ftgmac100_rx_packet RXDES0=b0070040 RXDES1=f0800000 RXDES2=88f8
[ 35.101498] ftgmac100 1e660000.ethernet eth0: rx_packet_error 0xb0070040
[ 35.108205] ftgmac100 1e660000.ethernet eth0: [ISR] = 0xb0070040: RX_ERR
[ 35.287808] i2c i2c-1: new_device: Instantiated device slave-mqueue at 0x12
[ 35.428379] ftgmac100_hard_start_xmit TXDESO=b000003c
[ 35.433624] ftgmac100 1e660000.ethernet eth0: tx_complete_packet 56
[ 35.439915] ftgmac100 1e660000.ethernet eth0: rx_packet_error RXDES0=0xb0070040
[ 35.447225] ftgmac100_rx_packet RXDES0=b0070040 RXDES1=f0800000 RXDES2=88f8
[ 35.454273] ftgmac100 1e660000.ethernet eth0: rx_packet_error 0xb0070040
[ 35.460972] ftgmac100 1e660000.ethernet eth0: [ISR] = 0xb0070040: RX_ERR
[ 35.797825] ftgmac100_hard_start_xmit TXDESO=b000003c
[ 35.803241] ftgmac100 1e660000.ethernet eth0: tx_complete_packet 57
[ 35.809541] ftgmac100 1e660000.ethernet eth0: rx_packet_error RXDES0=0xb0070040
[ 35.816848] ftgmac100_rx_packet RXDES0=b0070040 RXDES1=f0800000 RXDES2=88f8
[ 35.823899] ftgmac100 1e660000.ethernet eth0: rx_packet_error 0xb0070040
[ 35.830597] ftgmac100 1e660000.ethernet eth0: [ISR] = 0xb0070040: RX_ERR
[ 36.179914] ftgmac100_hard_start_xmit TXDESO=b000003c
[ 36.185160] ftgmac100 1e660000.ethernet eth0: tx_complete_packet 58
[ 36.191454] ftgmac100 1e660000.ethernet eth0: rx_packet_error RXDES0=0xb0070040
[ 36.198761] ftgmac100_rx_packet RXDES0=b0070040 RXDES1=f0800000 RXDES2=88f8
[ 36.205813] ftgmac100 1e660000.ethernet eth0: rx_packet_error 0xb0070040
[ 36.212513] ftgmac100 1e660000.ethernet eth0: [ISR] = 0xb0070040: RX_ERR
[ 36.593688] ftgmac100_hard_start_xmit TXDESO=b000003c
[ 36.602937] ftgmac100 1e660000.ethernet eth0: tx_complete_packet 59
[ 36.609244] ftgmac100 1e660000.ethernet eth0: rx_packet_error RXDES0=0xb0070040
[ 36.616558] ftgmac100_rx_packet RXDES0=b0070040 RXDES1=f0800000 RXDES2=88f8
[ 36.623608] ftgmac100 1e660000.ethernet eth0: rx_packet_error 0xb0070040
[ 36.630315] ftgmac100 1e660000.ethernet eth0: [ISR] = 0xb0070040: RX_ERR
[ 37.031524] IPv6: ADDRCONF(NETDEV_UP): eth0: link is not ready
[ 37.067831] IPv6: ADDRCONF(NETDEV_UP): eth1: link is not ready
............
This patch add a configurable flag, FTGMAC100_RXDES0_RX_ERR_CHK, in FTGMAC100
driver, it is YES by default, so keep the orignal define of
RXDES0_ANY_ERROR. If it is needed, user can set the flag to NO to remove
the RXDES0_RX_ERR bit, to fix the issue.
Hongwei Zhang (1):
net: ftgmac100: Fix AST2600 EVB NCSI RX issue
drivers/net/ethernet/faraday/Kconfig | 9 +++++++++
drivers/net/ethernet/faraday/ftgmac100.h | 8 ++++++++
2 files changed, 17 insertions(+)
--
2.17.1
next prev parent reply other threads:[~2020-12-21 19:41 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-15 19:23 [Aspeed,ncsi-rx, v1 " Hongwei Zhang
2020-12-15 19:23 ` [Aspeed,ncsi-rx, v1 1/1] net: ftgmac100: Fix AST2600 EVB " Hongwei Zhang
2020-12-15 20:31 ` Jakub Kicinski
2020-12-15 22:47 ` Andrew Jeffery
2020-12-21 17:00 ` [Aspeed,ncsi-rx, v1 0/1] net: ftgmac100: Fix AST2600EVB " Hongwei Zhang
2020-12-22 2:26 ` Joel Stanley
2020-12-22 2:43 ` [Aspeed, ncsi-rx, " Dylan Hung
2020-12-21 17:00 ` [Aspeed,ncsi-rx, v1 1/1] net: ftgmac100: Fix AST2600 EVB " Hongwei Zhang
2020-12-21 17:00 ` [Aspeed,ncsi-rx, v1 0/1] net: ftgmac100: Fix AST2600EVB " Hongwei Zhang
2020-12-21 17:00 ` [Aspeed,ncsi-rx, v1] Answer to initial submission Hongwei Zhang
2020-12-21 19:40 ` Hongwei Zhang [this message]
2020-12-21 19:40 ` [Aspeed,ncsi-rx, v2 1/1] net: ftgmac100: Fix AST2600 EVB NCSI RX issue Hongwei Zhang
2020-12-21 22:09 ` Jakub Kicinski
2021-01-11 20:58 ` [Aspeed,ncsi-rx, v1 0/1] net: ftgmac100: Fix AST2600EVB " Hongwei Zhang
2021-01-11 20:59 ` [Aspeed,ncsi-rx, v2 1/1] " Hongwei Zhang
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=20201221194026.30715-1-hongweiz@ami.com \
--to=hongweiz@ami.com \
--cc=andrew@aj.id.au \
--cc=davem@davemloft.net \
--cc=joel@jms.id.au \
--cc=kuba@kernel.org \
--cc=linux-aspeed@lists.ozlabs.org \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=openbmc@lists.ozlabs.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
all inboxes | Powered by JetHome®