net: ipa: skip suspend/resume activities if not set up
authorAlex Elder <elder@linaro.org>
Fri, 9 Oct 2020 20:28:48 +0000 (15:28 -0500)
committerJakub Kicinski <kuba@kernel.org>
Fri, 9 Oct 2020 23:04:07 +0000 (16:04 -0700)
commitd1704382821032fede445b816f4296fd379baacf
tree0b0ae32ee9022bc277c3f47be3a95362dbf872c5
parentd3b2dc9472e1b76661bc381df4e26eaec3c9f2a8
net: ipa: skip suspend/resume activities if not set up

When processing a system suspend request we suspend modem endpoints
if they are enabled, and call ipa_cmd_tag_process() (which issues
IPA commands) to ensure the IPA pipeline is cleared.  It is an error
to attempt to issue an IPA command before setup is complete, so this
is clearly a bug.  But we also shouldn't suspend or resume any
endpoints that have not been set up.

Have ipa_endpoint_suspend() and ipa_endpoint_resume() immediately
return if setup hasn't completed, to avoid any attempt to configure
endpoints or issue IPA commands in that case.

Fixes: 84f9bd12d46d ("soc: qcom: ipa: IPA endpoints")
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ipa/ipa_endpoint.c