From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DFADE2AE90 for ; Thu, 23 Nov 2023 14:38:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="OMG5vuNG" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 648DCC433C8; Thu, 23 Nov 2023 14:38:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1700750291; bh=UCTz5cBRlznHaYwHngoEfU+e2S0kC2Om+8hE86GPxCU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=OMG5vuNG3fI8ZYPkS/1ZVNb4IDtP4nMDj0LgE9lEjzW9xrzmCcMvndo/U0o1MavQr fjviKd7zqTSExBIWrJ4blRTnhFGR0hcM81Pj6BiGcH/Cw/Iy0mWS/mxtyihQuhlkIS n4gpHYJEl/y7CN7C5B730uZWys5BrcN5IpMXs3rzRi6eN/wk4YjwoyhW42jhLe+fVx k5iaQsP7cgaXQvxnP3j1lquUXSYXAxi6snM41OEPbiqjCjwUlj2K1ZphrTDKRaSiLV wqUjrDVonvndLW8ZZlwkLikrtHDN+FhfRCOTufw5Iq/YVXT21jhIxhg+Qf+L+uwFi4 De/cmvImrjEyg== Date: Thu, 23 Nov 2023 14:38:06 +0000 From: Simon Horman To: Petr Machata Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , netdev@vger.kernel.org, Ido Schimmel , mlxsw@nvidia.com, Coverity Scan , Jiri Pirko Subject: Re: [PATCH net-next] mlxsw: pci: Fix missing error checking Message-ID: <20231123143806.GI6339@kernel.org> References: Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Thu, Nov 23, 2023 at 01:01:35PM +0100, Petr Machata wrote: > From: Ido Schimmel > > I accidentally removed the error checking after issuing the reset. > Restore it. > > Fixes: f257c73e5356 ("mlxsw: pci: Add support for new reset flow") > Reported-by: Coverity Scan > Signed-off-by: Ido Schimmel > Reviewed-by: Jiri Pirko > Signed-off-by: Petr Machata Reviewed-by: Simon Horman