On Thu, Feb 4, 2021 at 8:18 PM Joakim Zhang <qiangqing.zh...@nxp.com> wrote: > > > > -----Original Message----- > > From: Willem de Bruijn <willemdebruijn.ker...@gmail.com> > > Sent: 2021年2月4日 21:20 > > To: Joakim Zhang <qiangqing.zh...@nxp.com> > > Cc: Giuseppe Cavallaro <peppe.cavall...@st.com>; Alexandre Torgue > > <alexandre.tor...@st.com>; Jose Abreu <joab...@synopsys.com>; David > > Miller <da...@davemloft.net>; Jakub Kicinski <k...@kernel.org>; Network > > Development <netdev@vger.kernel.org>; Andrew Lunn <and...@lunn.ch>; > > Florian Fainelli <f.faine...@gmail.com>; Willem de Bruijn > > <willemdebruijn.ker...@gmail.com> > > Subject: Re: [PATCH net-next 2/2] net: stmmac: slightly adjust the order of > > the > > codes in stmmac_resume() > > > > On Thu, Feb 4, 2021 at 5:18 AM Joakim Zhang <qiangqing.zh...@nxp.com> > > wrote: > > > > > > Slightly adjust the order of the codes in stmmac_resume(), remove the > > > check "if (!device_may_wakeup(priv->device) || !priv->plat->pmt)". > > > > > > Signed-off-by: Joakim Zhang <qiangqing.zh...@nxp.com> > > > > This commit message says what the code does, but not why or why it's > > correct. > > This just slight changes the code order, there is no function change, I will > improve the commit message if needed.
And it is correct to move this phylink initialization before serdes_powerup? That is not immediately obvious to me. A comment would definitely be good. More relevant than just stating that the code is moved, which I see from the patch easily.
