[Bug 109241] [CI][BAT] all suspend tests - dmesg-warn - called from state HALTED
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Mon Jan 7 23:13:02 UTC 2019
https://bugs.freedesktop.org/show_bug.cgi?id=109241
--- Comment #3 from Chris Wilson <chris at chris-wilson.co.uk> ---
Fwiw,
commit 2b3e88ea65287ba738a798622405b15344871085
Author: Heiner Kallweit <hkallweit1 at gmail.com>
Date: Sun Dec 16 18:30:14 2018 +0100
net: phy: improve phy state checking
Add helpers phy_is_started() and __phy_is_started() to avoid open-coded
checks whether PHY has been started. To make the check easier move
PHY_HALTED before PHY_UP in enum phy_state. Further improvements:
phy_start_aneg():
Return -EBUSY and print warning if function is called from a non-started
state (DOWN, READY, HALTED). Better check because function is exported
and drivers may use it incorrectly.
phy_interrupt():
Return IRQ_NONE also if state is DOWN or READY. We should never receive
an interrupt in one of these states, but better play safe.
phy_stop():
Just return and print a warning if PHY is in a non-started state.
This warning should help to identify drivers with unbalanced calls to
phy_start() / phy_stop().
phy_state_machine():
Schedule state machine run only if PHY is in a started state.
E.g. if state is READY we don't need the state machine, it will be
started by phy_start().
v2:
- don't use __func__ within phy_warn_state
v3:
- use WARN() instead of printing error message to facilitate debugging
Signed-off-by: Heiner Kallweit <hkallweit1 at gmail.com>
Reviewed-by: Florian Fainelli <f.fainelli at gmail.com>
Signed-off-by: David S. Miller <davem at davemloft.net>
deemed to turn phy_start_aneg() while HALTED into an error (previously it still
did the phy_config_aneg()).
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-gfx-bugs/attachments/20190107/a15d3512/attachment.html>
More information about the intel-gfx-bugs
mailing list