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;
From:   Colin Foster <colin.foster@in-advantage.com>
To:     colin.foster@in-advantage.com,
        Vladimir Oltean <vladimir.oltean@nxp.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        UNGLinuxDriver@microchip.com,
        "David S. Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>
Cc:     netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH v1 net] net: mscc: ocelot: remove buggy and useless write to ANA_PFC_PFC_CFG
Date:   Wed, 15 Sep 2021 18:09:37 -0700
Message-Id: <20210916010938.517698-1-colin.foster@in-advantage.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?cxX4uMECOx75Bueu1Gw9x+dL1Jp8/uARtAJB/zmUUMyFXoeLKS2LmiHwIXDs?=
 =?us-ascii?Q?5CFO29MJyYWGEog23PLqdUUWNre1UdGxBs4rM+HaQ4Odurd5HLv2XNquLzoN?=
 =?us-ascii?Q?9juKxe+qu3kJfdyrHWt1YP+dbIynxqOkqjizsxf6mh0X1j7ZxrF4OITsC4DK?=
 =?us-ascii?Q?wpgAKPBieNRSJF7/gtgQWFIkWjcFYXPon93fkM5HJ/ZrqYPjKN03qYULRndu?=
 =?us-ascii?Q?5jYBg6JGcC/adxmfuaPCJZcFKPDrR3v4MTvyJpWEOyldnb6h3TdYdP3Qq2cR?=
 =?us-ascii?Q?DNGWVkdIPxjmgVs1h4dgAEM8mxHEGmfhwTML4PRHIrCPQwrm0vaF9BVuEewg?=
 =?us-ascii?Q?GxpTl0CM5prYeD2R3fM1/PvhOH8919xqOrwMLK4vVjTgsTwizlqWnvxgtFwT?=
 =?us-ascii?Q?Xn3dntOLpx+2LY1I2+B0lSLbHl+FCI3vPx7sSgkJTggfS/4mCCWP0/KKZ0gx?=
 =?us-ascii?Q?q+bhbnAQaEry4q1obOHr+1LmWcdrUfIb1m2WbNjJdQdUNwSUY7TijyewuYb6?=
 =?us-ascii?Q?GHc1epGpWaKlgeAsB/hsPoFwZWiSMZHywozgoW8AHXbtGxizMQPt81cagqi9?=
 =?us-ascii?Q?kwLOo4u1QdpRjwspS6hSzTYYThb7HXEmYgzAryWYhlgSoztkdnMoYYQ1tldH?=
 =?us-ascii?Q?zglVcl29tZO2afHi0r2nBGL1WdI+fpqfCxRxPUamXBKenK3Z4vUT3ARpXk33?=
 =?us-ascii?Q?RlWDQD7IzHRshE5SG20P7EU3u4rjAs+eHABJMY6SSZ6/d30ZhEtHBH5sFZ/L?=
 =?us-ascii?Q?Zet+gyKu36otJaOdEF3LwRgHIixiWIuLWThYiSfwEeK/7VBakD8q9ywvj4xI?=
 =?us-ascii?Q?CmNjShSM6FKnMm37raFVO7CXDPH8HRqZ9dhr6frm0DBpNxAKLKVvElk1db+8?=
 =?us-ascii?Q?j/zZO/NMkw5yvCwE1EeXRYS+aAa7gUdqQ6JeZXJHQn/52ndrIJcrE7Usp8me?=
 =?us-ascii?Q?PAaPHaCrI1QKUyR0rofznSRWFDivusPMbWfSVBQk++TQvhWORoS3i8wg+aIr?=
 =?us-ascii?Q?/rPlSJG3mYrrB9zYHFHDxrW8k0O1I2ENYfbF/LCXxoyeekSBBrIMTqq2U4H1?=
 =?us-ascii?Q?/q4TKkGxVhs/FrJIImtPHlDAMRqm9ig4kOkKYd3+mIT1ZFol+UltVyUQ+bbJ?=
 =?us-ascii?Q?MsPUZnmMpxiBvFfRjYupgYkerJkTflLyzfKhXV6jyi8Brh4mRZuE5lZHFAxy?=
 =?us-ascii?Q?/+VBzfqlAMFPAAWx7du5PKO47Li26o4BshebISSCqhyJd5jiRyDn3230vOEU?=
 =?us-ascii?Q?rJJpnDRUt7rhCRhZrupSA9g6wUQpMwYeShylyaJeCcImodgwlc1QdE0ULGnQ?=
 =?us-ascii?Q?7Pa+J74TAqo8zquWbN5FhbtN?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9b4cf925-daf5-4323-4818-08d978aebfd7
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Sep 2021 01:10:19.0853
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 48e842ca-fbd8-4633-a79d-0c955a7d3aae
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: lFGvRMjuRejP8rPPUct1janUqJk47rf9qx6bHGQg9v3pIHJvMwwR6SSWaa4WprvpWKFgEcmZPwrnMDxWxiZDs7SrihALL6VP7t1wuXuPKA4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR10MB1598
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

A useless write to ANA_PFC_PFC_CFG was left in while refactoring ocelot to
phylink. Since priority flow control is disabled, writing the speed has no
effect.

Further, it was using ethtool.h SPEED_ instead of OCELOT_SPEED_ macros,
which are incorrectly offset for GENMASK.

Lastly, for priority flow control to properly function, some scenarios
would rely on the rate adaptation from the PCS while the MAC speed would
be fixed. So it isn't used, and even if it was, neither "speed" nor
"mac_speed" are necessarily the correct values to be used.

Fixes: e6e12df625f2 ("net: mscc: ocelot: convert to phylink")
Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
---
 drivers/net/ethernet/mscc/ocelot.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/net/ethernet/mscc/ocelot.c b/drivers/net/ethernet/mscc/ocelot.c
index c581b955efb3..08be0440af28 100644
--- a/drivers/net/ethernet/mscc/ocelot.c
+++ b/drivers/net/ethernet/mscc/ocelot.c
@@ -569,10 +569,6 @@ void ocelot_phylink_mac_link_up(struct ocelot *ocelot, int port,
 	ocelot_port_writel(ocelot_port, DEV_CLOCK_CFG_LINK_SPEED(speed),
 			   DEV_CLOCK_CFG);
 
-	/* No PFC */
-	ocelot_write_gix(ocelot, ANA_PFC_PFC_CFG_FC_LINK_SPEED(speed),
-			 ANA_PFC_PFC_CFG, port);
-
 	/* Core: Enable port for frame transfer */
 	ocelot_fields_write(ocelot, port,
 			    QSYS_SWITCH_PORT_MODE_PORT_ENA, 1);
-- 
2.25.1