Hi Russell,

Russell King <rmk+ker...@arm.linux.org.uk> writes:

> When adding the software padding, this must be done using the first/mid
> fragment mode, and any subsequent operation needs to be a mid-fragment.
> Fix this.
>
> Signed-off-by: Russell King <rmk+ker...@arm.linux.org.uk>
> ---
>  drivers/crypto/marvell/hash.c | 8 --------
>  1 file changed, 8 deletions(-)
>
> diff --git a/drivers/crypto/marvell/hash.c b/drivers/crypto/marvell/hash.c
> index da541e59cc1d..34271e9eb3a5 100644
> --- a/drivers/crypto/marvell/hash.c
> +++ b/drivers/crypto/marvell/hash.c
> @@ -575,20 +575,12 @@ mv_cesa_ahash_dma_last_req(struct mv_cesa_tdma_chain 
> *chain,
>               if (IS_ERR(op))
>                       return op;
>  
> -             mv_cesa_update_op_cfg(op, CESA_SA_DESC_CFG_MID_FRAG,
> -                                   CESA_SA_DESC_CFG_FRAG_MSK);
> -
>               if (len == trailerlen)
>                       return op;
>  
>               padoff += len;
>       }
>  
> -     if (!mv_cesa_mac_op_is_first_frag(&creq->op_tmpl))
> -             mv_cesa_update_op_cfg(&creq->op_tmpl,
> -                                   CESA_SA_DESC_CFG_MID_FRAG,
> -                                   CESA_SA_DESC_CFG_FRAG_MSK);
> -
>       ret = mv_cesa_dma_add_data_transfer(chain,
>                                           CESA_SA_DATA_SRAM_OFFSET,
>                                           ahashdreq->padding_dma +

I'll consider it's just late here and I need some sleep but I fail to
align the commit message w/ the content of the patch, i.e. it's
unclear to me.

Cheers,

a+
--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to