Thread (12 messages) 12 messages, 3 authors, 2015-03-31

RE: [PATCH v2 7/8] net/macb: fix the peripheral version test

From: Punnaiah Choudary Kalluri <hidden>
Date: 2015-03-31 15:23:34
Also in: linux-arm-kernel, lkml, stable

-----Original Message-----
From: Nicolas Ferre [mailto:nicolas.ferre@atmel.com]
Sent: Tuesday, March 31, 2015 6:32 PM
To: linux-arm-kernel@lists.infradead.org; netdev@vger.kernel.org; David S.
Miller
Cc: linux-kernel@vger.kernel.org; Boris BREZILLON; Cyrille Pitchen; Alexandre
Belloni; Michal Simek; Punnaiah Choudary Kalluri; Nicolas Ferre;
stable@vger.kernel.org
Subject: [PATCH v2 7/8] net/macb: fix the peripheral version test

We currently need two checks of the peripheral version in MACB_MID
register.
One of them got out of sync after modification by 8a013a9c71b2 (net: macb:
Include multi queue support for xilinx ZynqMP ethernet version).
Fix this in macb_configure_caps() so that xilinx ZynqMP will be considered
as a GEM flavor.

Fixes: 8a013a9c71b2 ("net: macb: Include multi queue support for xilinx
ZynqMP
ethernet version")

Signed-off-by: Nicolas Ferre <redacted>
   Reviewed-by: Punnaiah Choudary Kalluri [off-list ref]

Thanks for the patch. We have this change in our tree but somehow missed while sending
this patch to mainline.

Regards,
Punnaiah
quoted hunk ↗ jump to hunk
Cc: Michal Simek <redacted>
Cc: Punnaiah Choudary Kalluri <redacted>
Cc: <redacted> #4.0 (if it doesn't make it for -final)
---

Changes in v2:
- fix peripheral version test

 drivers/net/ethernet/cadence/macb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/cadence/macb.c
b/drivers/net/ethernet/cadence/macb.c
index babe972a7c32..4412895cf4a8 100644
--- a/drivers/net/ethernet/cadence/macb.c
+++ b/drivers/net/ethernet/cadence/macb.c
@@ -2139,7 +2139,7 @@ static void macb_configure_caps(struct macb *bp,
const struct macb_config *dt_co
      if (dt_conf)
              bp->caps = dt_conf->caps;

-     if (MACB_BFEXT(IDNUM, macb_readl(bp, MID)) == 0x2)
+     if (MACB_BFEXT(IDNUM, macb_readl(bp, MID)) >= 0x2)
              bp->caps |= MACB_CAPS_MACB_IS_GEM;

      if (macb_is_gem(bp)) {
--
2.1.3


This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help