From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH v2 net-next] net: stmmac: Populate missing callbacks in HWIF initialization Date: Fri, 18 May 2018 13:56:57 -0400 (EDT) Message-ID: <20180518.135657.533651743320003936.davem@davemloft.net> References: <6f26505abcb91747df73f09019c7e2bcb0c3835a.1526658592.git.joabreu@synopsys.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, clabbe.montjoie@gmail.com, Joao.Pinto@synopsys.com, peppe.cavallaro@st.com, alexandre.torgue@st.com To: Jose.Abreu@synopsys.com Return-path: Received: from shards.monkeyblade.net ([184.105.139.130]:48248 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750957AbeERR46 (ORCPT ); Fri, 18 May 2018 13:56:58 -0400 In-Reply-To: <6f26505abcb91747df73f09019c7e2bcb0c3835a.1526658592.git.joabreu@synopsys.com> Sender: netdev-owner@vger.kernel.org List-ID: From: Jose Abreu Date: Fri, 18 May 2018 16:54:38 +0100 > Some HW specific setups, like sun8i, do not populate all the necessary > callbacks, which is what HWIF helpers were expecting. > > Fix this by always trying to get the generic helpers and populate them > if they were not previously populated by HW specific setup. > > Signed-off-by: Jose Abreu > Fixes: 5f0456b43140 ("net: stmmac: Implement logic to automatically > select HW Interface") Please don't split up Fixes: tag lines like this in the future. No matter how long it is, keep it a single line. > Reported-by: Corentin Labbe > Tested-by: Corentin Labbe Applied, thank you.