Thread (12 messages) flat view 12 messages, 4 authors, 2018-12-17

Re: [PATCH net-next 0/2] net: phy: don't stop state machine in case of MDIO error

From: David Miller <davem@davemloft.net>
Date: 2018-12-16 18:43:46

From: Heiner Kallweit <hkallweit1@gmail.com>
Date: Sun, 16 Dec 2018 09:29:16 +0100
On 15.12.2018 17:17, Heiner Kallweit wrote:
quoted
If we detect a MDIO error, it seems to be a little bit too aggressive
to stop the state machine and bring down the PHY completely.
E.g. when polling and we miss one update, then this has no relevant
impact.

Heiner Kallweit (2):
  net: phy: don't stop state machine in case of MDIO error
  net: fec: remove workaround to restart state machine on MDIO error

 drivers/net/ethernet/freescale/fec.h      |  1 -
 drivers/net/ethernet/freescale/fec_main.c | 12 -------
 drivers/net/phy/phy.c                     | 42 +++++------------------
 include/linux/phy.h                       |  2 +-
 4 files changed, 10 insertions(+), 47 deletions(-)
David, could you please wait with applying this? I should have marked
it RFC (did so in patchwork meanwhile), need to discuss it with
Andrew and Florian.
Ok.

I'd like to kindly ask that you not modify patchwork state even for your
own changes, and let me take care of it.

When the patch state changes behind my back it's confusing and I start
wondering if I made a mistake changing the state of patches in my queue.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help