Received-SPF: pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH] crypto: omap-sham - Fix sparse/compiler warnings
To:     Herbert Xu <herbert@gondor.apana.org.au>,
        Linux Crypto Mailing List <linux-crypto@vger.kernel.org>
References: <20200615113738.GB20552@gondor.apana.org.au>
From:   Tero Kristo <t-kristo@ti.com>
Message-ID: <34a06971-71c4-eb1c-61e3-37512f75113a@ti.com>
Date:   Tue, 16 Jun 2020 13:20:28 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.8.0
MIME-Version: 1.0
In-Reply-To: <20200615113738.GB20552@gondor.apana.org.au>
Content-Type: text/plain; charset="utf-8"; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-crypto-owner@vger.kernel.org
Precedence: bulk

On 15/06/2020 14:37, Herbert Xu wrote:
> This patch fixes sparse endianness warnings as well as compiler
> warnings on 64-bit hosts.
> 
> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

Looks okay to me.

Reviewed-by: Tero Kristo <t-kristo@ti.com>

> 
> diff --git a/drivers/crypto/omap-sham.c b/drivers/crypto/omap-sham.c
> index 82691a057d2a..954d703f2981 100644
> --- a/drivers/crypto/omap-sham.c
> +++ b/drivers/crypto/omap-sham.c
> @@ -357,10 +357,10 @@ static void omap_sham_copy_ready_hash(struct ahash_request *req)
>   
>   	if (big_endian)
>   		for (i = 0; i < d; i++)
> -			hash[i] = be32_to_cpu(in[i]);
> +			hash[i] = be32_to_cpup((__be32 *)in + i);
>   	else
>   		for (i = 0; i < d; i++)
> -			hash[i] = le32_to_cpu(in[i]);
> +			hash[i] = le32_to_cpup((__le32 *)in + i);
>   }
>   
>   static int omap_sham_hw_init(struct omap_sham_dev *dd)
> @@ -522,7 +522,7 @@ static int omap_sham_xmit_cpu(struct omap_sham_dev *dd, size_t length,
>   	int mlen;
>   	struct sg_mapping_iter mi;
>   
> -	dev_dbg(dd->dev, "xmit_cpu: digcnt: %d, length: %d, final: %d\n",
> +	dev_dbg(dd->dev, "xmit_cpu: digcnt: %zd, length: %zd, final: %d\n",
>   						ctx->digcnt, length, final);
>   
>   	dd->pdata->write_ctrl(dd, length, final, 0);
> @@ -588,7 +588,7 @@ static int omap_sham_xmit_dma(struct omap_sham_dev *dd, size_t length,
>   	struct dma_slave_config cfg;
>   	int ret;
>   
> -	dev_dbg(dd->dev, "xmit_dma: digcnt: %d, length: %d, final: %d\n",
> +	dev_dbg(dd->dev, "xmit_dma: digcnt: %zd, length: %zd, final: %d\n",
>   						ctx->digcnt, length, final);
>   
>   	if (!dma_map_sg(dd->dev, ctx->sg, ctx->sg_len, DMA_TO_DEVICE)) {
> @@ -871,7 +871,7 @@ static int omap_sham_prepare_request(struct ahash_request *req, bool update)
>   		nbytes += req->nbytes - rctx->offset;
>   
>   	dev_dbg(rctx->dd->dev,
> -		"%s: nbytes=%d, bs=%d, total=%d, offset=%d, bufcnt=%d\n",
> +		"%s: nbytes=%d, bs=%d, total=%d, offset=%d, bufcnt=%zd\n",
>   		__func__, nbytes, bs, rctx->total, rctx->offset,
>   		rctx->bufcnt);
>   
> @@ -932,7 +932,7 @@ static int omap_sham_update_dma_stop(struct omap_sham_dev *dd)
>   	return 0;
>   }
>   
> -struct omap_sham_dev *omap_sham_find_dev(struct omap_sham_reqctx *ctx)
> +static struct omap_sham_dev *omap_sham_find_dev(struct omap_sham_reqctx *ctx)
>   {
>   	struct omap_sham_dev *dd;
>   
> @@ -1023,7 +1023,7 @@ static int omap_sham_update_req(struct omap_sham_dev *dd)
>   	bool final = (ctx->flags & BIT(FLAGS_FINUP)) &&
>   			!(dd->flags & BIT(FLAGS_HUGE));
>   
> -	dev_dbg(dd->dev, "update_req: total: %u, digcnt: %d, final: %d",
> +	dev_dbg(dd->dev, "update_req: total: %u, digcnt: %zd, final: %d",
>   		ctx->total, ctx->digcnt, final);
>   
>   	if (ctx->total < get_block_size(ctx) ||
> @@ -1036,7 +1036,7 @@ static int omap_sham_update_req(struct omap_sham_dev *dd)
>   		err = omap_sham_xmit_dma(dd, ctx->total, final);
>   
>   	/* wait for dma completion before can take more data */
> -	dev_dbg(dd->dev, "update: err: %d, digcnt: %d\n", err, ctx->digcnt);
> +	dev_dbg(dd->dev, "update: err: %d, digcnt: %zd\n", err, ctx->digcnt);
>   
>   	return err;
>   }
> @@ -1097,7 +1097,7 @@ static int omap_sham_finish(struct ahash_request *req)
>   			err = omap_sham_finish_hmac(req);
>   	}
>   
> -	dev_dbg(dd->dev, "digcnt: %d, bufcnt: %d\n", ctx->digcnt, ctx->bufcnt);
> +	dev_dbg(dd->dev, "digcnt: %zd, bufcnt: %zd\n", ctx->digcnt, ctx->bufcnt);
>   
>   	return err;
>   }
> 

--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki