Commit 22d63be9 authored by Tom Rix's avatar Tom Rix Committed by Marc Kleine-Budde
Browse files

can: mcba_usb: 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].

Link: https://lore.kernel.org/r/20210124150916.1920434-1-trix@redhat.com


Signed-off-by: default avatarTom Rix <trix@redhat.com>
Signed-off-by: default avatarMarc Kleine-Budde <mkl@pengutronix.de>
parent 54eca60b

drivers/net/can/usb/mcba_usb.c

+3 −3
Original line number Diff line number Diff line
@@ -466,7 +466,7 @@ static void mcba_usb_process_ka_usb(struct mcba_priv *priv, 
				    struct mcba_usb_msg_ka_usb *msg)

{

	if (unlikely(priv->usb_ka_first_pass)) {

		netdev_info(priv->netdev, "PIC USB version %hhu.%hhu\n",

		netdev_info(priv->netdev, "PIC USB version %u.%u\n",

			    msg->soft_ver_major, msg->soft_ver_minor);



		priv->usb_ka_first_pass = false;
@@ -492,7 +492,7 @@ static void mcba_usb_process_ka_can(struct mcba_priv *priv, 
				    struct mcba_usb_msg_ka_can *msg)

{

	if (unlikely(priv->can_ka_first_pass)) {

		netdev_info(priv->netdev, "PIC CAN version %hhu.%hhu\n",

		netdev_info(priv->netdev, "PIC CAN version %u.%u\n",

			    msg->soft_ver_major, msg->soft_ver_minor);



		priv->can_ka_first_pass = false;
@@ -554,7 +554,7 @@ static void mcba_usb_process_rx(struct mcba_priv *priv, 
		break;



	default:

		netdev_warn(priv->netdev, "Unsupported msg (0x%hhX)",

		netdev_warn(priv->netdev, "Unsupported msg (0x%X)",

			    msg->cmd_id);

		break;

	}