[U-Boot] [PATCH] mvgbe: enable the reception of packets with an odd number of preamble nibbles

With the current hardware initialisation of the driver all packets with an odd number of preamble nibbles are dropped. Some switches seem to send all packets with such an preamble.
According to the functional specifications of the marvell 88F6180, 6190, 6192 and 6281 the reception of such packets can be enabled by setting bit 22 in register PSC1.
Signed-off-by: Klaus Flittner klaus@flittner.org --- drivers/net/mvgbe.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/drivers/net/mvgbe.c b/drivers/net/mvgbe.c index c701f43..f1081a2 100644 --- a/drivers/net/mvgbe.c +++ b/drivers/net/mvgbe.c @@ -446,6 +446,9 @@ static int mvgbe_init(struct eth_device *dev) MVGBE_REG_WR(regs->psc0, MVGBE_MAX_RX_PACKET_9700BYTE | (MVGBE_REG_RD(regs->psc0) & MRU_MASK));
+ /* Receive packets with odd number of preamble nibbles */ + MVGBE_REG_BITS_SET(regs->psc1, 1 << 22); + /* Enable port initially */ MVGBE_REG_BITS_SET(regs->psc0, MVGBE_SERIAL_PORT_EN);

Dear Klaus Flittner,
In message 20110131204025.1ffb0391@earth.lan you wrote:
With the current hardware initialisation of the driver all packets with an odd number of preamble nibbles are dropped. Some switches seem to send all packets with such an preamble.
According to the functional specifications of the marvell 88F6180, 6190, 6192 and 6281 the reception of such packets can be enabled by setting bit 22 in register PSC1.
Signed-off-by: Klaus Flittner klaus@flittner.org
drivers/net/mvgbe.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/drivers/net/mvgbe.c b/drivers/net/mvgbe.c index c701f43..f1081a2 100644 --- a/drivers/net/mvgbe.c +++ b/drivers/net/mvgbe.c @@ -446,6 +446,9 @@ static int mvgbe_init(struct eth_device *dev) MVGBE_REG_WR(regs->psc0, MVGBE_MAX_RX_PACKET_9700BYTE | (MVGBE_REG_RD(regs->psc0) & MRU_MASK));
/* Receive packets with odd number of preamble nibbles */
MVGBE_REG_BITS_SET(regs->psc1, 1 << 22);
/* Enable port initially */ MVGBE_REG_BITS_SET(regs->psc0, MVGBE_SERIAL_PORT_EN);
Your patch is white space corrupted and does not apply:
Applying: mvgbe: enable the reception of packets with an odd number of preamble nibbles Using index info to reconstruct a base tree... error: patch failed: drivers/net/mvgbe.c:446 error: drivers/net/mvgbe.c: patch does not apply Did you hand edit your patch? It does not apply to blobs recorded in its index. Cannot fall back to three-way merge. Patch failed at 0001 mvgbe: enable the reception of packets with an odd number of preamble nibbles
Please resubmit after fixing your mailer problems.
Best regards,
Wolfgang Denk

-----Original Message----- From: u-boot-bounces@lists.denx.de [mailto:u-boot-bounces@lists.denx.de] On Behalf Of Wolfgang Denk Sent: Wednesday, April 13, 2011 12:20 AM To: Klaus Flittner Cc: u-boot@lists.denx.de Subject: Re: [U-Boot] [PATCH] mvgbe: enable the reception of packets with an odd number of preamble nibbles
Dear Klaus Flittner,
In message 20110131204025.1ffb0391@earth.lan you wrote:
With the current hardware initialisation of the driver all packets
with
an odd number of preamble nibbles are dropped. Some switches seem to send all packets with such an preamble.
According to the functional specifications of the marvell 88F6180, 6190, 6192 and 6281 the reception of such packets can be enabled by setting bit 22 in register PSC1.
Klaus Flittner
Do you thing setting this by default will not disturb functionality on other hardware? If so, how about encapsulating it in #ifdef with SYS_CONFIG_xx ?
Regards.. Prafulla . .
participants (3)
-
Klaus Flittner
-
Prafulla Wadaskar
-
Wolfgang Denk