[Bug 238037] [PATCH] Implement ig4 suspend/resume

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=238037

Vladimir Kondratyev <wulf at freebsd.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |wulf at freebsd.org

--- Comment #16 from Vladimir Kondratyev <wulf at freebsd.org> ---
The suspend/resume patch is obviously broken in a way it handles
IG4_REG_I2C_EN.
It should set IG4_REG_I2C_EN to 0 before restoring of other registers and than
set it to saved value only *after* all other registers have been restored. That
is due to writes to some registers can succeed only when IC_ENABLE=0. 
Note: setting IG4_REG_I2C_EN to 0 should be acknowledged by controller. See
set_controller() source for example.

-- 
You are receiving this mail because:
You are the assignee for the bug.