From: Dmitri Epshtein <d...@marvell.com>

Some literal values are actually already defined by macros, so let's use
them.

[gregory.clem...@free-electrons.com: split intial commit in two
individual changes]
Signed-off-by: Dmitri Epshtein <d...@marvell.com>
Signed-off-by: Gregory CLEMENT <gregory.clem...@free-electrons.com>
---
 drivers/net/ethernet/marvell/mvneta.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/marvell/mvneta.c 
b/drivers/net/ethernet/marvell/mvneta.c
index 58ad36876e8b..225d933259c0 100644
--- a/drivers/net/ethernet/marvell/mvneta.c
+++ b/drivers/net/ethernet/marvell/mvneta.c
@@ -880,7 +880,7 @@ static void mvneta_port_down(struct mvneta_port *pp)
                mdelay(1);
 
                val = mvreg_read(pp, MVNETA_RXQ_CMD);
-       } while (val & 0xff);
+       } while (val & MVNETA_RXQ_ENABLE_MASK);
 
        /* Stop Tx port activity. Check port Tx activity. Issue stop
         * command for active channels only
@@ -905,7 +905,7 @@ static void mvneta_port_down(struct mvneta_port *pp)
                /* Check TX Command reg that all Txqs are stopped */
                val = mvreg_read(pp, MVNETA_TXQ_CMD);
 
-       } while (val & 0xff);
+       } while (val & MVNETA_TXQ_ENABLE_MASK);
 
        /* Double check to verify that TX FIFO is empty */
        count = 0;
-- 
2.5.0

Reply via email to