Commit c41e21dc authored by Tom Rix's avatar Tom Rix Committed by Greg Kroah-Hartman
Browse files

sgi-xp: remove h from printk format specifier 



This change fixes the checkpatch warning described in this commit
commit cbacb5ab ("docs: printk-formats: Stop encouraging use of
  unnecessary %h[xudi] and %hh[xudi]")

Standard integer promotion is already done and %hx and %hhx is useless
so do not encourage the use of %hh[xudi] or %h[xudi].

Reviewed-By: default avatarSteve Wahl <steve.wahl@hpe.com>
Signed-off-by: default avatarTom Rix <trix@redhat.com>
Link: https://lore.kernel.org/r/20210123160003.1777766-1-trix@redhat.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 26c2e922

drivers/misc/sgi-xp/xpnet.c

+2 −2
Original line number Diff line number Diff line
@@ -208,7 +208,7 @@ xpnet_receive(short partid, int channel, struct xpnet_message *msg) 
	} else {

		dst = (void *)((u64)skb->data & ~(L1_CACHE_BYTES - 1));

		dev_dbg(xpnet, "transferring buffer to the skb->data area;\n\t"

			"xp_remote_memcpy(0x%p, 0x%p, %hu)\n", dst,

			"xp_remote_memcpy(0x%p, 0x%p, %u)\n", dst,

					  (void *)msg->buf_pa, msg->size);



		ret = xp_remote_memcpy(xp_pa(dst), msg->buf_pa, msg->size);
@@ -218,7 +218,7 @@ xpnet_receive(short partid, int channel, struct xpnet_message *msg) 
			 * !!! appears in_use and we can't just call

			 * !!! dev_kfree_skb.

			 */

			dev_err(xpnet, "xp_remote_memcpy(0x%p, 0x%p, 0x%hx) "

			dev_err(xpnet, "xp_remote_memcpy(0x%p, 0x%p, 0x%x) "

				"returned error=0x%x\n", dst,

				(void *)msg->buf_pa, msg->size, ret);