<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body>
<div dir="ltr">
<div></div>
<div>
<div dir="ltr">Hi there,</div>
<div dir="ltr"><br>
</div>
<div dir="ltr">I would like to apol<span>ogize for this.</span><span></span></div>
<div dir="ltr"><span>Due to some mistake, I had resubmitted v4 at the same day before you review it.</span></div>
<div dir="ltr"><span>I am working on v5 based on your review.</span></div>
<div dir="ltr">Very sorry for the <span>confusion and appre<span>ciate your review.</span></span></div>
<div dir="ltr"><br>
</div>
<div dir="ltr">Thanks & Best Regards,</div>
<div dir="ltr">Wei Sheng</div>
<div id="ms-outlook-mobile-signature" dir="ltr">
<div></div>
</div>
<div id="mail-editor-reference-message-container" class="ms-outlook-mobile-reference-message">
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif"><b>From:</b> Lobakin, Alexandr <alexandr.lobakin@intel.com><br>
<b>Sent:</b> Saturday, November 26, 2022 00:02<br>
<b>To:</b> Goh, Wei Sheng <wei.sheng.goh@intel.com><br>
<b>Cc:</b> Lobakin, Alexandr <alexandr.lobakin@intel.com>; Giuseppe Cavallaro <peppe.cavallaro@st.com>; Alexandre Torgue <alexandre.torgue@st.com>; Jose Abreu <joabreu@synopsys.com>; David S . Miller <davem@davemloft.net>; Eric Dumazet <edumazet@google.com>;
 Jakub Kicinski <kuba@kernel.org>; Paolo Abeni <pabeni@redhat.com>; Maxime Coquelin <mcoquelin.stm32@gmail.com>; netdev@vger.kernel.org <netdev@vger.kernel.org>; linux-stm32@st-md-mailman.stormreply.com <linux-stm32@st-md-mailman.stormreply.com>; linux-arm-kernel@lists.infradead.org
 <linux-arm-kernel@lists.infradead.org>; linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>; Alexandre Torgue <alexandre.torgue@foss.st.com>; Ong, Boon Leong <boon.leong.ong@intel.com>; Voon, Weifeng <weifeng.voon@intel.com>; Tan, Tee Min <tee.min.tan@intel.com>;
 Ahmad Tarmizi, Noor Azura <noor.azura.ahmad.tarmizi@intel.com>; Looi, Hong Aun <hong.aun.looi@intel.com><br>
<b>Subject:</b> Re: [PATCH net v4] net: stmmac: Set MAC's flow control register to reflect current settings
<div> </div>
</font></div>
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><font size="2"><span style="font-size:11pt;">
<div class="PlainText">From: "Goh, Wei Sheng" <wei.sheng.goh@intel.com><br>
Date: Wed, 23 Nov 2022 18:51:10 +0800<br>
<br>
> Currently, pause frame register GMAC_RX_FLOW_CTRL_RFE is not updated<br>
> correctly when 'ethtool -A <IFACE> autoneg off rx off tx off' command<br>
> is issued. This fix ensures the flow control change is reflected directly<br>
> in the GMAC_RX_FLOW_CTRL_RFE register.<br>
<br>
Any particular reason why you completely ignored by review comments<br>
to the v3[0]? I'd like to see them fixed or at least replied.<br>
<br>
> <br>
> Fixes: 46f69ded988d ("net: stmmac: Use resolved link config in mac_link_up()")<br>
> Cc: <stable@vger.kernel.org> # 5.10.x<br>
> Signed-off-by: Goh, Wei Sheng <wei.sheng.goh@intel.com><br>
> Signed-off-by: Noor Azura Ahmad Tarmizi <noor.azura.ahmad.tarmizi@intel.com><br>
> ---<br>
> V3 -> V4: Fix commit message and incorrect insertions(+) value<br>
> V2 -> V3: Removed value assign for 'flow' in else statement based on review comments<br>
> V1 -> V2: Removed needless condition based on review comments<br>
> <br>
>  drivers/net/ethernet/stmicro/stmmac/dwmac4_core.c |  2 ++<br>
>  drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 12 ++++++++++--<br>
>  2 files changed, 12 insertions(+), 2 deletions(-)<br>
<br>
[...]<br>
<br>
> -- <br>
> 2.17.1<br>
<br>
[0] <a href="https://lore.kernel.org/netdev/20221123180947.488302-1-alexandr.lobakin@intel.com">
https://lore.kernel.org/netdev/20221123180947.488302-1-alexandr.lobakin@intel.com</a><br>
<br>
Thanks,<br>
Olek<br>
</div>
</span></font></div>
</div>
</div>
</body>
</html>