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 4D3F329AB for ; Tue, 26 Sep 2023 18:44:45 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8C864C433C8; Tue, 26 Sep 2023 18:44:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1695753885; bh=KG2sRyrAO/Zemo0Lf4waWvy+trM3E7P3N8Bf2Td0z9E=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=AUvW9oG/V5HXkG38frkAq6WJzect8Sgy3xpgqbzNCDcmQcWeKBskGWSS86bxX22C6 pukD3Jz/wPvip4uK95+/ZcYWPuhGCFzsO0h5UQsaAE5LQ99akHuEfUuwky9s3lfPMl 6+NRLwrVT3y1vdc8iiQ0/utVKY+RZE0eYG6aQq//AYNcT2jri5C48JKLn8rIdauOox fHe1VgYBkx+umMrpAJ+C7miVJRd2ZO8Foosapd1Tel1Ic3xLvzWGt56/JXn9nyfRof tKaglTkf6d88T5y/wHNpIhY9BQV9ttPlzNx+KpnHhkVvwfkLbWQcqcju691qJ1m7Y6 MElvZMb99CogA== Message-ID: <9670bbbe-e429-4a17-97a9-4d18c3ce907d@kernel.org> Date: Tue, 26 Sep 2023 21:44:40 +0300 Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/3 net] net: ethernet: ti: am65-cpsw: Fix error code in am65_cpsw_nuss_init_tx_chns() Content-Language: en-US To: Dan Carpenter , Grygorii Strashko Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Siddharth Vadapalli , netdev@vger.kernel.org, kernel-janitors@vger.kernel.org References: <4c2073cc-e7ef-4f16-9655-1a46cfed9fe9@moroto.mountain> From: Roger Quadros In-Reply-To: <4c2073cc-e7ef-4f16-9655-1a46cfed9fe9@moroto.mountain> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 26/09/2023 17:04, Dan Carpenter wrote: > This accidentally returns success, but it should return a negative error > code. > > Fixes: 93a76530316a ("net: ethernet: ti: introduce am65x/j721e gigabit eth subsystem driver") > Signed-off-by: Dan Carpenter Reviewed-by: Roger Quadros