Merge branch 'net-ipa-don-t-abort-system-suspend'
authorPaolo Abeni <pabeni@redhat.com>
Tue, 27 Feb 2024 10:24:06 +0000 (11:24 +0100)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 27 Feb 2024 10:24:07 +0000 (11:24 +0100)
commit58cc8640b627558ffd1e6c5c415ff034688e3569
tree87dc5b4f1a34169664f596671e5b8f3e7f1bba86
parentb38061fe9cfa90a781e9e59fc761191fc8b469a1
parentf9345952e74a77ba905b5a23252bffde48162ef3
Merge branch 'net-ipa-don-t-abort-system-suspend'

Alex Elder says:

====================
net: ipa: don't abort system suspend

Currently the IPA code aborts an in-progress system suspend if an
IPA interrupt arrives before the suspend completes.  There is no
need to do that though, because the IPA driver handles a forced
suspend correctly, quiescing any hardware activity before finally
turning off clocks and interconnects.

This series drops the call to pm_wakeup_dev_event() if an IPA
SUSPEND interrupt arrives during system suspend.  Doing this
makes the two remaining IPA power flags unnecessary, and allows
some additional code to be cleaned up--and best of all, removed.
The result is much simpler (and I'm really glad not to be using
these flags any more).

The first patch implements the main change.  The second and
third remove the flags that were used to determine whether to
call pm_wakeup_dev_event().  The next two remove a function that
becomes a trivial wrapper, and the last one just avoids writing
a register unnecessarily.

Note that the first two patches will have checkpatch warnings,
because checkpatch disagrees with my compiler on what to do when
a block contains only a semicolon.  I went with what the compiler
recommends.

clang says: warning: suggest braces around empty body
checkpatch: WARNING: braces {} are not necessary for single statement blocks

====================

Link: https://lore.kernel.org/r/20240223133930.582041-1-elder@linaro.org
Signed-off-by: Paolo Abeni <pabeni@redhat.com>