Message ID | 20220905052449.1830669-1-neal_liu@aspeedtech.com (mailing list archive) |
---|---|
State | Changes Requested |
Delegated to: | Herbert Xu |
Headers | show |
Series | [v2] crypto: aspeed: fix format unexpected build warning | expand |
On 9/4/22 22:24, Neal Liu wrote: > This fixes the following similar build warning when > enabling compile test: > > aspeed-hace-hash.c:188:9: warning: format '%x' expects argument of type > 'unsigned int', but argument 7 has type 'size_t' {aka 'long unsigned int'} > [-Wformat=] > > Reported-by: kernel test robot <lkp@intel.com> > Signed-off-by: Neal Liu <neal_liu@aspeedtech.com> Acked-by: Randy Dunlap <rdunlap@infradead.org> Thanks. > --- > v2: Use "%zu" format to print size_t. > > drivers/crypto/aspeed/aspeed-hace-hash.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/crypto/aspeed/aspeed-hace-hash.c b/drivers/crypto/aspeed/aspeed-hace-hash.c > index 0a44ffc0e13b..6e833d238253 100644 > --- a/drivers/crypto/aspeed/aspeed-hace-hash.c > +++ b/drivers/crypto/aspeed/aspeed-hace-hash.c > @@ -185,7 +185,7 @@ static int aspeed_ahash_dma_prepare_sg(struct aspeed_hace_dev *hace_dev) > remain = (rctx->total + rctx->bufcnt) % rctx->block_size; > length = rctx->total + rctx->bufcnt - remain; > > - AHASH_DBG(hace_dev, "%s:0x%x, %s:0x%x, %s:0x%x, %s:0x%x\n", > + AHASH_DBG(hace_dev, "%s:0x%x, %s:%zu, %s:0x%x, %s:0x%x\n", > "rctx total", rctx->total, "bufcnt", rctx->bufcnt, > "length", length, "remain", remain); > > @@ -324,7 +324,7 @@ static int aspeed_hace_ahash_trigger(struct aspeed_hace_dev *hace_dev, > struct ahash_request *req = hash_engine->req; > struct aspeed_sham_reqctx *rctx = ahash_request_ctx(req); > > - AHASH_DBG(hace_dev, "src_dma:0x%x, digest_dma:0x%x, length:0x%x\n", > + AHASH_DBG(hace_dev, "src_dma:%zu, digest_dma:%zu, length:%zu\n", > hash_engine->src_dma, hash_engine->digest_dma, > hash_engine->src_length); >
On Mon, Sep 05, 2022 at 01:24:49PM +0800, Neal Liu wrote: > > @@ -324,7 +324,7 @@ static int aspeed_hace_ahash_trigger(struct aspeed_hace_dev *hace_dev, > struct ahash_request *req = hash_engine->req; > struct aspeed_sham_reqctx *rctx = ahash_request_ctx(req); > > - AHASH_DBG(hace_dev, "src_dma:0x%x, digest_dma:0x%x, length:0x%x\n", > + AHASH_DBG(hace_dev, "src_dma:%zu, digest_dma:%zu, length:%zu\n", > hash_engine->src_dma, hash_engine->digest_dma, > hash_engine->src_length); Shouldn't we use %pad instead of %zu for dma_addr_t (you'll also need to pass in a dma_addr_t pointer instead of the value)? Thanks,
On 9/6/22 19:13, Herbert Xu wrote: > On Mon, Sep 05, 2022 at 01:24:49PM +0800, Neal Liu wrote: >> >> @@ -324,7 +324,7 @@ static int aspeed_hace_ahash_trigger(struct aspeed_hace_dev *hace_dev, >> struct ahash_request *req = hash_engine->req; >> struct aspeed_sham_reqctx *rctx = ahash_request_ctx(req); >> >> - AHASH_DBG(hace_dev, "src_dma:0x%x, digest_dma:0x%x, length:0x%x\n", >> + AHASH_DBG(hace_dev, "src_dma:%zu, digest_dma:%zu, length:%zu\n", >> hash_engine->src_dma, hash_engine->digest_dma, >> hash_engine->src_length); > > Shouldn't we use %pad instead of %zu for dma_addr_t (you'll also > need to pass in a dma_addr_t pointer instead of the value)? That sounds correct. I only looked at the size_t parts...
> On 9/6/22 19:13, Herbert Xu wrote: > > On Mon, Sep 05, 2022 at 01:24:49PM +0800, Neal Liu wrote: > >> > >> @@ -324,7 +324,7 @@ static int aspeed_hace_ahash_trigger(struct > aspeed_hace_dev *hace_dev, > >> struct ahash_request *req = hash_engine->req; > >> struct aspeed_sham_reqctx *rctx = ahash_request_ctx(req); > >> > >> - AHASH_DBG(hace_dev, "src_dma:0x%x, digest_dma:0x%x, > length:0x%x\n", > >> + AHASH_DBG(hace_dev, "src_dma:%zu, digest_dma:%zu, length:%zu\n", > >> hash_engine->src_dma, hash_engine->digest_dma, > >> hash_engine->src_length); > > > > Shouldn't we use %pad instead of %zu for dma_addr_t (you'll also need > > to pass in a dma_addr_t pointer instead of the value)? > > That sounds correct. > I only looked at the size_t parts... True. I'll revise it and send next patch, thanks.
diff --git a/drivers/crypto/aspeed/aspeed-hace-hash.c b/drivers/crypto/aspeed/aspeed-hace-hash.c index 0a44ffc0e13b..6e833d238253 100644 --- a/drivers/crypto/aspeed/aspeed-hace-hash.c +++ b/drivers/crypto/aspeed/aspeed-hace-hash.c @@ -185,7 +185,7 @@ static int aspeed_ahash_dma_prepare_sg(struct aspeed_hace_dev *hace_dev) remain = (rctx->total + rctx->bufcnt) % rctx->block_size; length = rctx->total + rctx->bufcnt - remain; - AHASH_DBG(hace_dev, "%s:0x%x, %s:0x%x, %s:0x%x, %s:0x%x\n", + AHASH_DBG(hace_dev, "%s:0x%x, %s:%zu, %s:0x%x, %s:0x%x\n", "rctx total", rctx->total, "bufcnt", rctx->bufcnt, "length", length, "remain", remain); @@ -324,7 +324,7 @@ static int aspeed_hace_ahash_trigger(struct aspeed_hace_dev *hace_dev, struct ahash_request *req = hash_engine->req; struct aspeed_sham_reqctx *rctx = ahash_request_ctx(req); - AHASH_DBG(hace_dev, "src_dma:0x%x, digest_dma:0x%x, length:0x%x\n", + AHASH_DBG(hace_dev, "src_dma:%zu, digest_dma:%zu, length:%zu\n", hash_engine->src_dma, hash_engine->digest_dma, hash_engine->src_length);
This fixes the following similar build warning when enabling compile test: aspeed-hace-hash.c:188:9: warning: format '%x' expects argument of type 'unsigned int', but argument 7 has type 'size_t' {aka 'long unsigned int'} [-Wformat=] Reported-by: kernel test robot <lkp@intel.com> Signed-off-by: Neal Liu <neal_liu@aspeedtech.com> --- v2: Use "%zu" format to print size_t. drivers/crypto/aspeed/aspeed-hace-hash.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)