crypto: marvell - Don't copy hash operation twice into the SRAM
authorRomain Perier <romain.perier@free-electrons.com>
Mon, 5 Dec 2016 08:56:38 +0000 (09:56 +0100)
committerHerbert Xu <herbert@gondor.apana.org.au>
Wed, 7 Dec 2016 11:55:16 +0000 (19:55 +0800)
No need to copy the template of an hash operation twice into the SRAM
from the step function.

Fixes: commit 85030c5168f1 ("crypto: marvell - Add support for chai...")
Signed-off-by: Romain Perier <romain.perier@free-electrons.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/marvell/hash.c

index 9f284682c091aa1b675ea5a1fdda806a5345935c..daff38801d0be012c9236663181b970033c9e14c 100644 (file)
@@ -172,9 +172,6 @@ static void mv_cesa_ahash_std_step(struct ahash_request *req)
        for (i = 0; i < digsize / 4; i++)
                writel_relaxed(creq->state[i], engine->regs + CESA_IVDIG(i));
 
-       mv_cesa_adjust_op(engine, &creq->op_tmpl);
-       memcpy_toio(engine->sram, &creq->op_tmpl, sizeof(creq->op_tmpl));
-
        if (creq->cache_ptr)
                memcpy_toio(engine->sram + CESA_SA_DATA_SRAM_OFFSET,
                            creq->cache, creq->cache_ptr);