2018-01-22 10:21:40

by Dan Carpenter

Subject: [PATCH] crypto: chelsio - Delete stray tabs in create_authenc_wr()

We removed some if statements but left these statements indented too
far.

Signed-off-by: Dan Carpenter <[email protected]>

diff --git a/drivers/crypto/chelsio/chcr_algo.c b/drivers/crypto/chelsio/chcr_algo.c
index a9c894bf9c01..34a02d690548 100644
--- a/drivers/crypto/chelsio/chcr_algo.c
+++ b/drivers/crypto/chelsio/chcr_algo.c
@@ -2112,11 +2112,11 @@ static struct sk_buff *create_authenc_wr(struct aead_request *req,
error = chcr_aead_common_init(req, op_type);
if (error)
return ERR_PTR(error);
- dnents = sg_nents_xlen(req->dst, assoclen, CHCR_DST_SG_SIZE, 0);
- dnents += sg_nents_xlen(req->dst, req->cryptlen +
- (op_type ? -authsize : authsize), CHCR_DST_SG_SIZE,
- req->assoclen);
- dnents += MIN_AUTH_SG; // For IV
+ dnents = sg_nents_xlen(req->dst, assoclen, CHCR_DST_SG_SIZE, 0);
+ dnents += sg_nents_xlen(req->dst, req->cryptlen +
+ (op_type ? -authsize : authsize), CHCR_DST_SG_SIZE,
+ req->assoclen);
+ dnents += MIN_AUTH_SG; // For IV

dst_size = get_space_for_phys_dsgl(dnents);
kctx_len = (ntohl(KEY_CONTEXT_CTX_LEN_V(aeadctx->key_ctx_hdr)) << 4)

2018-01-23 04:01:27

by Harsh Jain

Subject: Re: [PATCH] crypto: chelsio - Delete stray tabs in create_authenc_wr()

On 22-01-2018 15:51, Dan Carpenter wrote:
> We removed some if statements but left these statements indented too
> far.
HI Dan,

Change already applied to cryptodev tree.

https://www.mail-archive.com/[email protected]/msg30560.html

>
> Signed-off-by: Dan Carpenter <[email protected]>
>
> diff --git a/drivers/crypto/chelsio/chcr_algo.c b/drivers/crypto/chelsio/chcr_algo.c
> index a9c894bf9c01..34a02d690548 100644
> --- a/drivers/crypto/chelsio/chcr_algo.c
> +++ b/drivers/crypto/chelsio/chcr_algo.c
> @@ -2112,11 +2112,11 @@ static struct sk_buff *create_authenc_wr(struct aead_request *req,
> error = chcr_aead_common_init(req, op_type);
> if (error)
> return ERR_PTR(error);
> - dnents = sg_nents_xlen(req->dst, assoclen, CHCR_DST_SG_SIZE, 0);
> - dnents += sg_nents_xlen(req->dst, req->cryptlen +
> - (op_type ? -authsize : authsize), CHCR_DST_SG_SIZE,
> - req->assoclen);
> - dnents += MIN_AUTH_SG; // For IV
> + dnents = sg_nents_xlen(req->dst, assoclen, CHCR_DST_SG_SIZE, 0);
> + dnents += sg_nents_xlen(req->dst, req->cryptlen +
> + (op_type ? -authsize : authsize), CHCR_DST_SG_SIZE,
> + req->assoclen);
> + dnents += MIN_AUTH_SG; // For IV
>
> dst_size = get_space_for_phys_dsgl(dnents);
> kctx_len = (ntohl(KEY_CONTEXT_CTX_LEN_V(aeadctx->key_ctx_hdr)) << 4)