LE CPU:
* set RSB_SWAP0 (both SYSTEMPORT and SYSTEMPORT Lite)
* clear RSB_SWAP1 (SYSTEMPORT Lite only)

BE CPU:
* clear RSB_SWAP0 (both SYSTEMPORT and SYSTEMPORTE lite)
* set RSB_SWAP1 (SYSTEMPORT Lite only)

With these settings, we have the Receive Status Block always match the
host endian and we do not need to perform any conversion.

Signed-off-by: Florian Fainelli <f.faine...@gmail.com>
---
 drivers/net/ethernet/broadcom/bcmsysport.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/broadcom/bcmsysport.c 
b/drivers/net/ethernet/broadcom/bcmsysport.c
index a7e84292af50..7c7558a6b720 100644
--- a/drivers/net/ethernet/broadcom/bcmsysport.c
+++ b/drivers/net/ethernet/broadcom/bcmsysport.c
@@ -1762,9 +1762,14 @@ static void rbuf_init(struct bcm_sysport_priv *priv)
        reg = rbuf_readl(priv, RBUF_CONTROL);
        reg |= RBUF_4B_ALGN | RBUF_RSB_EN;
        /* Set a correct RSB format on SYSTEMPORT Lite */
-       if (priv->is_lite) {
-               reg &= ~RBUF_RSB_SWAP1;
+       if (!IS_ENABLED(CONFIG_CPU_BIG_ENDIAN)) {
+               if (priv->is_lite)
+                       reg &= ~RBUF_RSB_SWAP1;
                reg |= RBUF_RSB_SWAP0;
+       } else {
+               if (priv->is_lite)
+                       reg |= RBUF_RSB_SWAP1;
+               reg &= ~RBUF_RSB_SWAP0;
        }
        rbuf_writel(priv, reg, RBUF_CONTROL);
 }
-- 
1.9.1

Reply via email to