| Message ID | 20200115200250.10849-3-krzk@kernel.org (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | 40b2ec1251c370bc5557568e259d4058c651d405 |
| Headers | show |
| Series | [v3,1/3] i2c: pmcmsp: Use proper printk format for resource_size_t | expand |
On Wed, Jan 15, 2020 at 09:02:50PM +0100, Krzysztof Kozlowski wrote: > size_t should be printed with its own format to be 64-bit friendly and > fix warning when compiling on 64-bit platform (e.g. with COMPILE_TEST): > > drivers/i2c/busses/i2c-highlander.c: In function ‘highlander_i2c_smbus_xfer’: > drivers/i2c/busses/i2c-highlander.c:325:22: warning: > format ‘%d’ expects argument of type ‘int’, > but argument 3 has type ‘size_t {aka long unsigned int}’ [-Wformat=] > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > Applied to for-next, thanks!
diff --git a/drivers/i2c/busses/i2c-highlander.c b/drivers/i2c/busses/i2c-highlander.c index abfe3094c047..803dad70e2a7 100644 --- a/drivers/i2c/busses/i2c-highlander.c +++ b/drivers/i2c/busses/i2c-highlander.c @@ -322,7 +322,7 @@ static int highlander_i2c_smbus_xfer(struct i2c_adapter *adap, u16 addr, tmp |= (SMMR_MODE0 | SMMR_MODE1); break; default: - dev_err(dev->dev, "unsupported xfer size %d\n", dev->buf_len); + dev_err(dev->dev, "unsupported xfer size %zu\n", dev->buf_len); return -EINVAL; }
size_t should be printed with its own format to be 64-bit friendly and fix warning when compiling on 64-bit platform (e.g. with COMPILE_TEST): drivers/i2c/busses/i2c-highlander.c: In function ‘highlander_i2c_smbus_xfer’: drivers/i2c/busses/i2c-highlander.c:325:22: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘size_t {aka long unsigned int}’ [-Wformat=] Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> --- Changes since v2: 1. Rewrite incorrect commit msg. Changes since v1: 1. None --- drivers/i2c/busses/i2c-highlander.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)