On 2/6/23 07:01, Herbert Xu wrote:
In crypto4xx_cipher_done, we should be unmapping the dst page, not
mapping it.
This was flagged by a sparse warning about the unused addr variable.
While we're at it, also fix a sparse warning regarding the unused
ctx variable in crypto4xx_ahash_done (by actually using it).
Fixes: 049359d65527 ("crypto: amcc - Add crypt4xx driver")
Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Tested this on the MyBook Live. Internal cryptmgr_test pass with:
[ 2.888784] alg: No test for stdrng (crypto4xx_rng)
[ 7.519740] "cryptomgr_test" (102) uses obsolete ecb(arc4) skcipher
(but all crypto4xx entries in /proc/crypto say that the selftest pass)
as well as libkcapi kcapi-enc-test.sh passes.
The ahash portion is a bit "underused". Currently the driver doesn't
register any hashes (this is because in testing I found the getting
the crypto-hardware to do those is so much slower than letting the
CPU do these).
Anyway:
Tested-by: Christian Lamparter <chunkeey@xxxxxxxxx>
Thanks!
