Re: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and stmmac_dvr_probe

2016-12-28 Thread Florian Fainelli
On 12/28/2016 04:28 PM, Kweh, Hock Leong wrote: >> Although this is required, we can't be doing it in all circumstances, we >> need to >> mimic what stmmac_drv_remove() does. >> >> Let me submit an incremental fix which takes care of mdio bus unregistration. >> -- >> Florian > > Noted & Thanks. W

RE: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and stmmac_dvr_probe

2016-12-28 Thread Kweh, Hock Leong
..@gmail.com; > manab...@gmail.com; niklas.cas...@axis.com; jo...@kernel.org; > pa...@ucw.cz; Ong, Boon Leong ; Voon, Weifeng > ; lars.pers...@axis.com; net...@vger.kernel.org; > linux-kernel@vger.kernel.org > Subject: Re: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and > stmma

RE: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and stmmac_dvr_probe

2016-12-28 Thread Kweh, Hock Leong
st.com; alexandre.tor...@gmail.com; > manab...@gmail.com; niklas.cas...@axis.com; jo...@kernel.org; > pa...@ucw.cz; lars.pers...@axis.com; net...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: Re: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and > stmmac_dvr_pro

Re: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and stmmac_dvr_probe

2016-12-28 Thread Florian Fainelli
t; seraphin.bonna...@st.com; f.faine...@gmail.com; >> alexandre.tor...@gmail.com; manab...@gmail.com; niklas.cas...@axis.com; >> jo...@kernel.org; pa...@ucw.cz; Ong, Boon Leong >> ; Voon, Weifeng ; >> lars.pers...@axis.com; net...@vger.kernel.org; linux-kernel@vger.kernel.org &g

Re: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and stmmac_dvr_probe

2016-12-28 Thread Kishan Sandeep
m; >> seraphin.bonna...@st.com; f.faine...@gmail.com; >> alexandre.tor...@gmail.com; manab...@gmail.com; niklas.cas...@axis.com; >> jo...@kernel.org; pa...@ucw.cz; Ong, Boon Leong >> ; Voon, Weifeng ; >> lars.pers...@axis.com; net...@vger.kernel.org; linux-kernel@vger.k

RE: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and stmmac_dvr_probe

2016-12-27 Thread Kweh, Hock Leong
r...@gmail.com; manab...@gmail.com; niklas.cas...@axis.com; > jo...@kernel.org; pa...@ucw.cz; Ong, Boon Leong > ; Voon, Weifeng ; > lars.pers...@axis.com; net...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and > stm

RE: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and stmmac_dvr_probe

2016-12-27 Thread Kweh, Hock Leong
r...@gmail.com; manab...@gmail.com; niklas.cas...@axis.com; > jo...@kernel.org; pa...@ucw.cz; Ong, Boon Leong > ; Voon, Weifeng ; > lars.pers...@axis.com; net...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and > stm

Re: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and stmmac_dvr_probe

2016-12-27 Thread David Miller
From: "Kweh, Hock Leong" Date: Tue, 27 Dec 2016 22:42:36 +0800 > From: "Kweh, Hock Leong" You are not the author of this change, do not take credit for it. You have copied Florian's patch character by character, therefore he is the author. You also didn't CC: the netdev mailing list pr

Re: [PATCH v2] net: stmmac: bug fix to synchronize stmmac_open and stmmac_dvr_probe

2016-12-27 Thread Pavel Machek
On Tue 2016-12-27 22:42:36, Kweh, Hock Leong wrote: > From: "Kweh, Hock Leong" > > If kernel module stmmac driver being loaded after OS booted, there is a > race condition between stmmac_open() and stmmac_mdio_register(), which is > invoked inside stmmac_dvr_probe(), and the error is showed in dm