diff mbox series

mailbox: ti-msgmgr: Use %zu for size_t print format

Message ID 20171202102718.18942-1-nm@ti.com
State Accepted
Commit ca64af43924f85d5de1b384709af3a3ea8c7ebbb
Headers show
Series mailbox: ti-msgmgr: Use %zu for size_t print format | expand

Commit Message

Nishanth Menon Dec. 2, 2017, 10:27 a.m. UTC
message->len is of type size_t and %d is incorrect format usage.
Instead use %zu for handling size_t correctly.

Signed-off-by: Nishanth Menon <nm@ti.com>

---
 drivers/mailbox/ti-msgmgr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.14.1

Comments

Lokesh Vutla Dec. 13, 2017, 8:01 a.m. UTC | #1
On Saturday 02 December 2017 03:57 PM, Nishanth Menon wrote:
> message->len is of type size_t and %d is incorrect format usage.

> Instead use %zu for handling size_t correctly.

> 

> Signed-off-by: Nishanth Menon <nm@ti.com>


Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>


Thanks and regards,
Lokesh
Nishanth Menon Dec. 16, 2017, 9:49 p.m. UTC | #2
On 08:01-20171213, Lokesh Vutla wrote:
> 

> 

> On Saturday 02 December 2017 03:57 PM, Nishanth Menon wrote:

> > message->len is of type size_t and %d is incorrect format usage.

> > Instead use %zu for handling size_t correctly.

> > 

> > Signed-off-by: Nishanth Menon <nm@ti.com>

> 

> Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>


Hi Jassi,
   Gentle ping.
-- 
Regards,
Nishanth Menon
diff mbox series

Patch

diff --git a/drivers/mailbox/ti-msgmgr.c b/drivers/mailbox/ti-msgmgr.c
index c8f34d69f03c..78753a87ba4d 100644
--- a/drivers/mailbox/ti-msgmgr.c
+++ b/drivers/mailbox/ti-msgmgr.c
@@ -283,7 +283,7 @@  static int ti_msgmgr_send_data(struct mbox_chan *chan, void *data)
 	desc = inst->desc;
 
 	if (desc->max_message_size < message->len) {
-		dev_err(dev, "Queue %s message length %d > max %d\n",
+		dev_err(dev, "Queue %s message length %zu > max %d\n",
 			qinst->name, message->len, desc->max_message_size);
 		return -EINVAL;
 	}