Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Colin Foster <colin.foster@in-advantage.com>
To:     linux-kernel@vger.kernel.org, netdev@vger.kernel.org
Cc:     Paolo Abeni <pabeni@redhat.com>, Jakub Kicinski <kuba@kernel.org>,
        Eric Dumazet <edumazet@google.com>,
        "David S. Miller" <davem@davemloft.net>,
        UNGLinuxDriver@microchip.com,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Vladimir Oltean <vladimir.oltean@nxp.com>
Subject: [PATCH v1 net-next 2/2] net: mscc: ocelot: check return values of writes during reset
Date:   Fri, 16 Sep 2022 12:13:49 -0700
Message-Id: <20220916191349.1659269-3-colin.foster@in-advantage.com>
In-Reply-To: <20220916191349.1659269-1-colin.foster@in-advantage.com>
References: <20220916191349.1659269-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?9Yx6qox14nM3SHaY4zGUxs/CdvGeTuO0APyxAvIdx+o3Hv8hVOJl8EVFSGr8?=
 =?us-ascii?Q?Qdm2JMK/pex7QGUhTEyuqmPpU9V1iciC4C6KFduj0DFU2ai//97igx/yCEhp?=
 =?us-ascii?Q?QpczCeQk3OKQlqW3BViJgDZcH97f0kkHnPiiMR4/lmFVd0JqvNbhIV7EYmJU?=
 =?us-ascii?Q?cp2/5ZbU7CWMlDvLQJt7pqRmBgcNSmUFUibQJdKgo3bjiZCp0Di4o9tTl4zj?=
 =?us-ascii?Q?kSdVuo1H//LKpMpzjA6Ja50PlYm07tB9YFjkYQwk1NKFosTQkLJrDEAkdcA6?=
 =?us-ascii?Q?L8Jr92vwn8cZ2/mq2E3jlx+iW5afWdb0NaCODAAkJHFAdseqoDRpqmS+7sPL?=
 =?us-ascii?Q?CrXFQUH6hsyWBB50bwq9wps/gA/YIlIgj7l/B4Xml5f0QbrMn+SXKyLoyLnZ?=
 =?us-ascii?Q?4DGVPctU67gTaWpYDoKuuoWrSKcl9E8EFCnRdJac49DvK9wp+kwxvB3FZUNf?=
 =?us-ascii?Q?4J8N3SuDA19YF0I4RQ8jxX5kmiqqaPcafYiTd2PAM6PFwAYndINEnk4KTfKa?=
 =?us-ascii?Q?SOTdSZXAvVN+iHHXXswJZtfB3Ks3uSthsbrEtL74ERJim6D95BH/Ae0LaHVi?=
 =?us-ascii?Q?gpUafM+3mVmQSLOHZoIkaJV0SqeIGrIy0FliLvDpdEA3x9qgLlJedSjGxKsI?=
 =?us-ascii?Q?QkHJvthJftqMSx8osu39kjL3D00IEk6QCjbNysCxm6UPSnurceOCWw8PLgRN?=
 =?us-ascii?Q?CEIr1izFnhrxmvXCEbijCDeJLO8eRoRAFw+KqP7GJEeTnMNEYDi4Ht0nfLGe?=
 =?us-ascii?Q?BBKlreeiGx+7qnsv0XPoxSOG5SD0IoVBHuqpYS4fpmlQP+GqLM0ROPW4T1L3?=
 =?us-ascii?Q?sL3ZuCBElSdcIG9c2NoCyoEYIeooPg5JbW/8T8VyqBv2VEJzKPfUW7SSLp6j?=
 =?us-ascii?Q?tNDAsSknYbdDEKsjbUefHtLa/NQobEWZ52EMs8Vu++JLac+E+55oV8APHppV?=
 =?us-ascii?Q?Jl+P3F2rNz7ksli8dhboIBUOwQZoLtcgrjsark/RMbsTCk41q1yy8Iu3bGs1?=
 =?us-ascii?Q?pLaAPOH883TBgkkyDKELJ9AMxoS5Hv0A1eViFnt2dIuUclQ4GgOS0vYqBWq5?=
 =?us-ascii?Q?ao9crRopVQhGRlVZPNUvugPp+lH3+osnBcUVgtZwfKVaJr0Y+Vd5nUXSTGnl?=
 =?us-ascii?Q?LzhD5LtU5f5+DjOklFXLFFFxfTCsl3GhugPpCJTyLZ6/G1Hd2RB16NJ8FJCp?=
 =?us-ascii?Q?mw5fAUvCulosaa0SqWVlnBBUSvhYsMj0BkAcZG3vB4I1zsJXAnLY+DZTu7kj?=
 =?us-ascii?Q?2oKF+l9nM/c1Au/jE8bfsxruMVBqCUH+H6WCx8oCgZjbPaA4lrIV01EJ/fRh?=
 =?us-ascii?Q?NzK9mDl3qAGj6n+7GtLS9anB4hBHDf1zxGbFjDzOWk1kxf7S7fpkRQYu4caA?=
 =?us-ascii?Q?RYFE2uUlCQxxObLynIC+Lp/jNSiLtnZ/fscno+zOWJR8QIUYDcI9XPJ0ovJM?=
 =?us-ascii?Q?ccTJDmzIdEME4zf/XkzYcy3yrU1H6qf8UyFUi3PG+5MSR3te1O1uezA1yMl0?=
 =?us-ascii?Q?PInsXuNQ1NYJ8KYL0X/mshJmQRBbG55HyvvvbZ7sKtgBsUJMXnICkpe+HUJk?=
 =?us-ascii?Q?DXqmWKTZ20biV1xbguoWqufshCHmFhMb6m0spA6sihyIb3rlFKbBh+VyebY+?=
 =?us-ascii?Q?AiqYv3wSicu71/KeankyzV8=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 43b3784a-0cb2-4339-d24a-08da98179d92
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Sep 2022 19:14:02.3067
 (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: BqRCQtjYr3cNJ4xQJKvTwcV57aeQtQAWbc9sujK044fOEO7KFykAmtQXKm5AWkeUx7I3wpCnMsmpz5muuxjaDqcJy0s6jpCB0TXsA2hCc2I=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR10MB4492
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

The ocelot_reset() function utilizes regmap_field_write() but wasn't
checking return values. While this won't cause issues for the current MMIO
regmaps, it could be an issue for externally controlled interfaces.

Add checks for these return values.

Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
---
 drivers/net/ethernet/mscc/ocelot_vsc7514.c | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

diff --git a/drivers/net/ethernet/mscc/ocelot_vsc7514.c b/drivers/net/ethernet/mscc/ocelot_vsc7514.c
index 79b7af36b4f4..415b7f4c7277 100644
--- a/drivers/net/ethernet/mscc/ocelot_vsc7514.c
+++ b/drivers/net/ethernet/mscc/ocelot_vsc7514.c
@@ -211,8 +211,13 @@ static int ocelot_reset(struct ocelot *ocelot)
 	int err;
 	u32 val;
 
-	regmap_field_write(ocelot->regfields[SYS_RESET_CFG_MEM_INIT], 1);
-	regmap_field_write(ocelot->regfields[SYS_RESET_CFG_MEM_ENA], 1);
+	err = regmap_field_write(ocelot->regfields[SYS_RESET_CFG_MEM_INIT], 1);
+	if (err)
+		return err;
+
+	err = regmap_field_write(ocelot->regfields[SYS_RESET_CFG_MEM_ENA], 1);
+	if (err)
+		return err;
 
 	/* MEM_INIT is a self-clearing bit. Wait for it to be cleared (should be
 	 * 100us) before enabling the switch core.
@@ -222,10 +227,13 @@ static int ocelot_reset(struct ocelot *ocelot)
 	if (IS_ERR_VALUE(err))
 		return err;
 
-	regmap_field_write(ocelot->regfields[SYS_RESET_CFG_MEM_ENA], 1);
-	regmap_field_write(ocelot->regfields[SYS_RESET_CFG_CORE_ENA], 1);
+	err = regmap_field_write(ocelot->regfields[SYS_RESET_CFG_MEM_ENA], 1);
+	if (err)
+		return err;
 
-	return 0;
+	err = regmap_field_write(ocelot->regfields[SYS_RESET_CFG_CORE_ENA], 1);
+
+	return err;
 }
 
 /* Watermark encode
-- 
2.25.1