On Tue, Nov 07, 2017 at 09:40:03AM +0000, Gilad Ben-Yossef wrote:
> diff --git a/drivers/staging/ccree/ssi_aead.c 
> b/drivers/staging/ccree/ssi_aead.c
> index f1a3976..e9d03ee 100644
> --- a/drivers/staging/ccree/ssi_aead.c
> +++ b/drivers/staging/ccree/ssi_aead.c
> @@ -240,7 +240,7 @@ static void ssi_aead_complete(struct device *dev, void 
> *ssi_req, void __iomem *c
>  
>       if (areq_ctx->gen_ctx.op_type == DRV_CRYPTO_DIRECTION_DECRYPT) {
>               if (memcmp(areq_ctx->mac_buf, areq_ctx->icv_virt_addr,
> -                        ctx->authsize) != 0) {
> +                        ctx->authsize)) {

Keep the != for *cmp functions.  It makes it way more readable.  The
idiom is:

        if (memcmp(a, b, size) != 0)  <-- this means a != b
        if (memcmp(a, b, size) < 0)   <-- this means a < b
        if (memcmp(a, b, size) == 0)  <-- this means a == b

>                       dev_dbg(dev, "Payload authentication failure, 
> (auth-size=%d, cipher=%d)\n",
>                               ctx->authsize, ctx->cipher_mode);
>                       /* In case of payload authentication failure, MUST NOT
> @@ -458,7 +458,7 @@ ssi_get_plain_hmac_key(struct crypto_aead *tfm, const u8 
> *key, unsigned int keyl
>               hashmode = DRV_HASH_HW_SHA256;
>       }
>  
> -     if (likely(keylen != 0)) {
> +     if (likely(keylen)) {

You can keep the zero here as well if you want.  keylen is a number and
zero is a number.  If you want to remove it that's fine too.

>               key_dma_addr = dma_map_single(dev, (void *)key, keylen, 
> DMA_TO_DEVICE);
>               if (unlikely(dma_mapping_error(dev, key_dma_addr))) {
>                       dev_err(dev, "Mapping key va=0x%p len=%u for DMA 
> failed\n",
> @@ -517,7 +517,7 @@ ssi_get_plain_hmac_key(struct crypto_aead *tfm, const u8 
> *key, unsigned int keyl
>                                     keylen, NS_BIT, 0);
>                       idx++;
>  
> -                     if ((blocksize - keylen) != 0) {
> +                     if (blocksize - keylen) {

Same.  Numbers can be compared to zero and it's fine.

>                               hw_desc_init(&desc[idx]);
>                               set_din_const(&desc[idx], 0,
>                                             (blocksize - keylen));
> @@ -539,10 +539,10 @@ ssi_get_plain_hmac_key(struct crypto_aead *tfm, const 
> u8 *key, unsigned int keyl
>       }
>  
>       rc = send_request(ctx->drvdata, &ssi_req, desc, idx, 0);
> -     if (unlikely(rc != 0))
> +     if (unlikely(rc))

Where-as for these ones "rc" is not a number we can use for math so the
!= 0 is just a double negative and slightly confusing, so removing it
is the right thing.

regards,
dan carpenter

Reply via email to