crypto: crypto4xx - fix dynamic_sa_ctl's sa_contents declaration
authorChristian Lamparter <chunkeey@googlemail.com>
Fri, 25 Aug 2017 13:47:19 +0000 (15:47 +0200)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 22 Sep 2017 09:43:17 +0000 (17:43 +0800)
The driver had a union dynamic_sa_contents in place that
described the meaning of the bits in the sa_contents
variable.

Signed-off-by: Christian Lamparter <chunkeey@googlemail.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/amcc/crypto4xx_alg.c
drivers/crypto/amcc/crypto4xx_sa.c
drivers/crypto/amcc/crypto4xx_sa.h

index 6da92321173bde9e851eb8e269d2d90c3dff9b42..867a9230cd3fef8ac89628617e5ae5630b4ce673 100644 (file)
@@ -149,7 +149,7 @@ static int crypto4xx_setkey_aes(struct crypto_ablkcipher *cipher,
                                 SA_NOT_COPY_HDR);
        crypto4xx_memcpy_le(ctx->sa_in + get_dynamic_sa_offset_key_field(ctx),
                            key, keylen);
-       sa->sa_contents = SA_AES_CONTENTS | (keylen << 2);
+       sa->sa_contents.w = SA_AES_CONTENTS | (keylen << 2);
        sa->sa_command_1.bf.key_len = keylen >> 3;
        ctx->is_hash = 0;
        ctx->direction = DIR_INBOUND;
@@ -219,7 +219,7 @@ static int crypto4xx_hash_alg_init(struct crypto_tfm *tfm,
                                 SA_NOT_COPY_PAD, SA_NOT_COPY_PAYLOAD,
                                 SA_NOT_COPY_HDR);
        ctx->direction = DIR_INBOUND;
-       sa->sa_contents = SA_HASH160_CONTENTS;
+       sa->sa_contents.w = SA_HASH160_CONTENTS;
        sa_in = (struct dynamic_sa_hash160 *) ctx->sa_in;
        /* Need to zero hash digest in SA */
        memset(sa_in->inner_digest, 0, sizeof(sa_in->inner_digest));
index 69182e2cc3ea8a916f7e24fcf17365c32e242c20..78a8ca192d0d299025db0a16372f3f9182ede0d2 100644 (file)
@@ -40,9 +40,9 @@ u32 get_dynamic_sa_offset_state_ptr_field(struct crypto4xx_ctx *ctx)
        union dynamic_sa_contents cts;
 
        if (ctx->direction == DIR_INBOUND)
-               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_in)->sa_contents;
+               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_in)->sa_contents.w;
        else
-               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_out)->sa_contents;
+               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_out)->sa_contents.w;
        offset = cts.bf.key_size
                + cts.bf.inner_size
                + cts.bf.outer_size
@@ -66,9 +66,9 @@ u32 get_dynamic_sa_iv_size(struct crypto4xx_ctx *ctx)
        union dynamic_sa_contents cts;
 
        if (ctx->direction == DIR_INBOUND)
-               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_in)->sa_contents;
+               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_in)->sa_contents.w;
        else
-               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_out)->sa_contents;
+               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_out)->sa_contents.w;
        return (cts.bf.iv0 + cts.bf.iv1 + cts.bf.iv2 + cts.bf.iv3) * 4;
 }
 
@@ -77,9 +77,9 @@ u32 get_dynamic_sa_offset_key_field(struct crypto4xx_ctx *ctx)
        union dynamic_sa_contents cts;
 
        if (ctx->direction == DIR_INBOUND)
-               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_in)->sa_contents;
+               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_in)->sa_contents.w;
        else
-               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_out)->sa_contents;
+               cts.w = ((struct dynamic_sa_ctl *) ctx->sa_out)->sa_contents.w;
 
        return sizeof(struct dynamic_sa_ctl);
 }
index 1352d58d4e340be0b5d84840cce937f20cc89d1e..537d548a20b70667b265292fe8562d89c15019a3 100644 (file)
@@ -169,7 +169,7 @@ union sa_command_1 {
 } __attribute__((packed));
 
 struct dynamic_sa_ctl {
-       u32 sa_contents;
+       union dynamic_sa_contents sa_contents;
        union sa_command_0 sa_command_0;
        union sa_command_1 sa_command_1;
 } __attribute__((packed));