Use clrsetbits_le32() to make this complicated construct simpler.
No functional change.

Signed-off-by: Marek Vasut <[email protected]>
---
Cc: Joe Hershberger <[email protected]>
Cc: Nobuhiro Iwamatsu <[email protected]>
Cc: Ramon Fried <[email protected]>
Cc: Tom Rini <[email protected]>
Cc: [email protected]
---
 drivers/net/rswitch.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/net/rswitch.c b/drivers/net/rswitch.c
index 259d584ac98..9ebe1e97bd1 100644
--- a/drivers/net/rswitch.c
+++ b/drivers/net/rswitch.c
@@ -426,9 +426,9 @@ static int rswitch_mii_read_c45(struct mii_dev *miidev, int 
phyad, int devad, in
        rswitch_etha_change_mode(priv, EAMC_OPC_CONFIG);
 
        /* Enable Station Management clock */
-       reg = readl(etha->addr + MPIC);
-       reg &= ~MPIC_PSMCS_MASK & ~MPIC_PSMHT_MASK;
-       writel(reg | MPIC_MDC_CLK_SET, etha->addr + MPIC);
+       clrsetbits_le32(etha->addr + MPIC,
+                       MPIC_PSMCS_MASK | MPIC_PSMHT_MASK,
+                       MPIC_MDC_CLK_SET);
 
        /* Set Station Management Mode : Clause 45 */
        setbits_le32(etha->addr + MPSM, MPSM_MFF_C45);
@@ -458,9 +458,9 @@ int rswitch_mii_write_c45(struct mii_dev *miidev, int 
phyad, int devad, int rega
        rswitch_etha_change_mode(priv, EAMC_OPC_CONFIG);
 
        /* Enable Station Management clock */
-       reg = readl(etha->addr + MPIC);
-       reg &= ~MPIC_PSMCS_MASK & ~MPIC_PSMHT_MASK;
-       writel(reg | MPIC_MDC_CLK_SET, etha->addr + MPIC);
+       clrsetbits_le32(etha->addr + MPIC,
+                       MPIC_PSMCS_MASK | MPIC_PSMHT_MASK,
+                       MPIC_MDC_CLK_SET);
 
        /* Set Station Management Mode : Clause 45 */
        setbits_le32(etha->addr + MPSM, MPSM_MFF_C45);
-- 
2.45.2

Reply via email to