crypto: mxs-dcp - return errcode in mxs_dcp_aes_enqueue and dcp_sha_update_fx

YueHaibing · herbertx · commit dbbaffefd269 · 2019-04-08T14:36:16.000+08:00
'err' is set in err path, but it's not returned to callers.
Don't always return -EINPROGRESS, return err.

Signed-off-by: YueHaibing &lt;yuehaibing@huawei.com&gt;
Reviewed-by: Mukesh Ojha &lt;mojha@codeaurora.org&gt;
Signed-off-by: Herbert Xu &lt;herbert@gondor.apana.org.au&gt;
diff --git a/drivers/crypto/mxs-dcp.c b/drivers/crypto/mxs-dcp.c
@@ -471,7 +471,7 @@ static int mxs_dcp_aes_enqueue(struct ablkcipher_request *req, int enc, int ecb)
 
 	wake_up_process(sdcp->thread[actx->chan]);
 
-	return -EINPROGRESS;
+	return ret;
 }
 
 static int mxs_dcp_aes_ecb_decrypt(struct ablkcipher_request *req)
@@ -797,7 +797,7 @@ static int dcp_sha_update_fx(struct ahash_request *req, int fini)
 	wake_up_process(sdcp->thread[actx->chan]);
 	mutex_unlock(&actx->mutex);
 
-	return -EINPROGRESS;
+	return ret;
 }
 
 static int dcp_sha_update(struct ahash_request *req)

Original file line number	Diff line number	Diff line change
`@@ -471,7 +471,7 @@ static int mxs_dcp_aes_enqueue(struct ablkcipher_request *req, int enc, int ecb)`
`471`	`471`
`472`	`472`	`wake_up_process(sdcp->thread[actx->chan]);`
`473`	`473`
`474`		`- return -EINPROGRESS;`
	`474`	`+ return ret;`
`475`	`475`	`}`
`476`	`476`
`477`	`477`	`static int mxs_dcp_aes_ecb_decrypt(struct ablkcipher_request *req)`
`@@ -797,7 +797,7 @@ static int dcp_sha_update_fx(struct ahash_request *req, int fini)`
`797`	`797`	`wake_up_process(sdcp->thread[actx->chan]);`
`798`	`798`	`mutex_unlock(&actx->mutex);`
`799`	`799`
`800`		`- return -EINPROGRESS;`
	`800`	`+ return ret;`
`801`	`801`	`}`
`802`	`802`
`803`	`803`	`static int dcp_sha_update(struct ahash_request *req)`