Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp3405552pxu; Mon, 30 Nov 2020 02:19:21 -0800 (PST) X-Google-Smtp-Source: ABdhPJznU+6LsPoDeOvgWLheyyplwwj4BckDkKR0WFlSL4WtfrLLN7m8aMsxsNNWCX/VGqG7C61o X-Received: by 2002:a17:906:2741:: with SMTP id a1mr19832114ejd.85.1606731561560; Mon, 30 Nov 2020 02:19:21 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1606731561; cv=none; d=google.com; s=arc-20160816; b=S4evOdwF2Q4/xVWXesoDjaAOOeYpoHo0PNUmH4bZi4plFD98Krrxwv1fvvyWIVJcp1 2ghWfzl/0kIjCRtqKshEpCkgZf4uIVMjS4doec2Ez/24O9AKCjkLS0cEaeLrUJ9DeDcQ I0J8wqPFYd7hxvpmNRgJYtbSNMnS6x37prffK7pICfsCNA40/nVO7XYPWcc4/HioViQw j8qoOS6aXnYsHuxq9RU+O1aruJjgipl1vw0Vitqsht2b57yQm0AJgeuRR9PvyFxdBsfq NGOcjFGFeC/U21+p7ZpXnxG1N0Eapt6QMQHHM+u/90D9Uc/f58lBd/md5kMfXDK87vPZ VNvA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:message-id:date:subject:cc:to:from :dkim-signature; bh=3xMjSgPQz6CGAoleFFXqRYTZJcd26r+56+BnmttwkJA=; b=ipCVRc8EXYHxkn7hMo9bYoCKO537YyTYNzx7mP7TqV/8yRF9T53YV+66kKw2NVuvEM lLbR6NLu1pPXoBW++TcYNrzOOdC2VKoqI3nylcQT58c9wHC0A8I/MN1VNk6TLfBnexKk bgJBxZXt75/i5tioyQdN//BVuFvDsE73lthiyTJbSP+IP0SS1EZZR8AylKFhs4xn49mO +EwvR6lmE7PM+UWUr81ZDDeFOtIdki7lXmsM11JS6C6V/ziOnLN6PCNoX/Em04mZ5ue4 zAmHbCD/zeGzjnoHYfsDkCcYDKyDtbEESZL0XU8do5qPMSqh4omG/HunfVRKEtApjCYN YL9Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=bY586ArA; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id j10si10622411ejn.26.2020.11.30.02.18.58; Mon, 30 Nov 2020 02:19:21 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=bY586ArA; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726532AbgK3KRo (ORCPT + 99 others); Mon, 30 Nov 2020 05:17:44 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36398 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725976AbgK3KRn (ORCPT ); Mon, 30 Nov 2020 05:17:43 -0500 Received: from mail-pf1-x441.google.com (mail-pf1-x441.google.com [IPv6:2607:f8b0:4864:20::441]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BF1C4C0613CF; Mon, 30 Nov 2020 02:16:57 -0800 (PST) Received: by mail-pf1-x441.google.com with SMTP id e8so10105081pfh.2; Mon, 30 Nov 2020 02:16:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id; bh=3xMjSgPQz6CGAoleFFXqRYTZJcd26r+56+BnmttwkJA=; b=bY586ArA+mA+U0ZmXqHCEx4MyovGLeW12+APm0ujqisZFufAhK4J/6/PvmtbJmVVfB It0IpTt4bRn/qPH4ggKS2ZVBEJxsjXLQXGbvoTY4pZCmTTAkPg1tPOwu0onWhhwpkpnU hnhJse8B3nVGV252/ECWxS7x+pDUbwSnNcOt5048NzdJCxx/QzobHM538S2g7AcX9YW/ hoAX/UgYGdPUHWj4w3rUpCXBT4JutaDjiTFbir1RTcEsrWtbcFTjfVFnH/JjWGnQZpnV GJ0Yy0vkENHEJ5rTomR8aOHZhyL5l6uq3nsAG+6/w/LFWvCNDtPGLWp5UpppsY7o+MTf YULA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=3xMjSgPQz6CGAoleFFXqRYTZJcd26r+56+BnmttwkJA=; b=Hk5lQQSJUHT7cWmIM09Kqp09xq1z+OKemWTfyrFVs+iiruFO/z0jZF6b/VSMgvzgG4 U1pGcZK048Jt1bv2stmJtQlzXcSZkjPa6HDxYu8TGNzeCzRfHQjQUjrFDQii2+Oh2Wiq 5x37een2DXCUZp7b9VSwdLaqf7NDdU6WpOtclbctqW5y5xks81XMxUgtGr7LGRZM/wkV 426Hv1depxs3R9G2cxG7Hz0kz449ZfsQ2pPRdtNAtQh/YHUolH7wWbrAjCLdE+zv5keO 4zPpVQPEqEW5aeurFVcUAS+Hl5b9orjyOehPOJetb32TuiP3crXYmEHour1lb5sMUJMp K7Vw== X-Gm-Message-State: AOAM531fIXGmofdY/tl20PLhnOfH0ocAUBhIpXK7HEPi7Y5WzO+c/PiB yOFdp4qo2RXG03O5TbYqko2q9QIfXKA= X-Received: by 2002:a65:6404:: with SMTP id a4mr5874447pgv.2.1606731417209; Mon, 30 Nov 2020 02:16:57 -0800 (PST) Received: from localhost.localdomain ([8.210.202.142]) by smtp.gmail.com with ESMTPSA id 10sm11964739pjt.35.2020.11.30.02.16.52 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Mon, 30 Nov 2020 02:16:56 -0800 (PST) From: Yejune Deng To: andrew@lunn.ch, hkallweit1@gmail.com, linux@armlinux.org.uk, davem@davemloft.net, kuba@kernel.org Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, yejune.deng@gmail.com Subject: [PATCH] net: phy: marvell: replace __phy_modify() Date: Mon, 30 Nov 2020 18:16:39 +0800 Message-Id: <1606731399-8772-1-git-send-email-yejune.deng@gmail.com> X-Mailer: git-send-email 1.9.1 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org a set of phy_set_bits() looks more neater Signed-off-by: Yejune Deng --- drivers/net/phy/marvell.c | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c index 587930a..f402e7f 100644 --- a/drivers/net/phy/marvell.c +++ b/drivers/net/phy/marvell.c @@ -1132,8 +1132,8 @@ static int m88e1510_config_init(struct phy_device *phydev) return err; /* PHY reset is necessary after changing MODE[2:0] */ - err = phy_modify(phydev, MII_88E1510_GEN_CTRL_REG_1, 0, - MII_88E1510_GEN_CTRL_REG_1_RESET); + err = phy_set_bits(phydev, MII_88E1510_GEN_CTRL_REG_1, + MII_88E1510_GEN_CTRL_REG_1_RESET); if (err < 0) return err; @@ -1725,7 +1725,7 @@ static int m88e1318_set_wol(struct phy_device *phydev, __phy_read(phydev, MII_M1011_IEVENT); /* Enable the WOL interrupt */ - err = __phy_modify(phydev, MII_88E1318S_PHY_CSIER, 0, + err = phy_set_bits(phydev, MII_88E1318S_PHY_CSIER, MII_88E1318S_PHY_CSIER_WOL_EIE); if (err < 0) goto error; @@ -1735,10 +1735,10 @@ static int m88e1318_set_wol(struct phy_device *phydev, goto error; /* Setup LED[2] as interrupt pin (active low) */ - err = __phy_modify(phydev, MII_88E1318S_PHY_LED_TCR, - MII_88E1318S_PHY_LED_TCR_FORCE_INT, - MII_88E1318S_PHY_LED_TCR_INTn_ENABLE | - MII_88E1318S_PHY_LED_TCR_INT_ACTIVE_LOW); + err = phy_modify(phydev, MII_88E1318S_PHY_LED_TCR, + MII_88E1318S_PHY_LED_TCR_FORCE_INT, + MII_88E1318S_PHY_LED_TCR_INTn_ENABLE | + MII_88E1318S_PHY_LED_TCR_INT_ACTIVE_LOW); if (err < 0) goto error; @@ -1764,7 +1764,7 @@ static int m88e1318_set_wol(struct phy_device *phydev, goto error; /* Clear WOL status and enable magic packet matching */ - err = __phy_modify(phydev, MII_88E1318S_PHY_WOL_CTRL, 0, + err = phy_set_bits(phydev, MII_88E1318S_PHY_WOL_CTRL, MII_88E1318S_PHY_WOL_CTRL_CLEAR_WOL_STATUS | MII_88E1318S_PHY_WOL_CTRL_MAGIC_PACKET_MATCH_ENABLE); if (err < 0) @@ -1775,9 +1775,9 @@ static int m88e1318_set_wol(struct phy_device *phydev, goto error; /* Clear WOL status and disable magic packet matching */ - err = __phy_modify(phydev, MII_88E1318S_PHY_WOL_CTRL, - MII_88E1318S_PHY_WOL_CTRL_MAGIC_PACKET_MATCH_ENABLE, - MII_88E1318S_PHY_WOL_CTRL_CLEAR_WOL_STATUS); + err = phy_modify(phydev, MII_88E1318S_PHY_WOL_CTRL, + MII_88E1318S_PHY_WOL_CTRL_MAGIC_PACKET_MATCH_ENABLE, + MII_88E1318S_PHY_WOL_CTRL_CLEAR_WOL_STATUS); if (err < 0) goto error; } @@ -1995,7 +1995,7 @@ static int marvell_cable_test_start_common(struct phy_device *phydev) return bmsr; if (bmcr & BMCR_ANENABLE) { - ret = phy_modify(phydev, MII_BMCR, BMCR_ANENABLE, 0); + ret = phy_clear_bits(phydev, MII_BMCR, BMCR_ANENABLE); if (ret < 0) return ret; ret = genphy_soft_reset(phydev); -- 1.9.1