> -----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. Best Regards, Joakim Zhang
