PM suspend returning a none zero value is not an error. It simply indicates a suspend is not advised right now so don't treat it as an error. Signed-off-by: Gilad Ben-Yossef <gilad@xxxxxxxxxxxxx> --- drivers/staging/ccree/ssi_request_mgr.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/staging/ccree/ssi_request_mgr.c b/drivers/staging/ccree/ssi_request_mgr.c index db3b29f..e99c148 100644 --- a/drivers/staging/ccree/ssi_request_mgr.c +++ b/drivers/staging/ccree/ssi_request_mgr.c @@ -598,9 +598,6 @@ static void proc_completions(struct cc_drvdata *drvdata) drvdata->request_mgr_handle; unsigned int *tail = &request_mgr_handle->req_queue_tail; unsigned int *head = &request_mgr_handle->req_queue_head; -#if defined(CONFIG_PM) - int rc = 0; -#endif while (request_mgr_handle->axi_completed) { request_mgr_handle->axi_completed--; @@ -625,10 +622,7 @@ static void proc_completions(struct cc_drvdata *drvdata) dev_dbg(dev, "Request completed. axi_completed=%d\n", request_mgr_handle->axi_completed); #if defined(CONFIG_PM) - rc = cc_pm_put_suspend(dev); - if (rc) - dev_err(dev, "Failed to set runtime suspension %d\n", - rc); + cc_pm_put_suspend(dev); #endif } } -- 2.7.4