From: Stephane Grosjean <s.grosj...@peak-system.com>

This patch replaces the plain integers used for flags in
pcan_usb_pro_encode_msg() by macros which are already defined.

Link: 
https://lore.kernel.org/r/20210309082128.23125-4-s.grosj...@peak-system.com
Signed-off-by: Stephane Grosjean <s.grosj...@peak-system.com>
[mkl: split into two patches]
Signed-off-by: Marc Kleine-Budde <m...@pengutronix.de>
---
 drivers/net/can/usb/peak_usb/pcan_usb_pro.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/can/usb/peak_usb/pcan_usb_pro.c 
b/drivers/net/can/usb/peak_usb/pcan_usb_pro.c
index 18fa180ecc81..902900d4f7c1 100644
--- a/drivers/net/can/usb/peak_usb/pcan_usb_pro.c
+++ b/drivers/net/can/usb/peak_usb/pcan_usb_pro.c
@@ -776,9 +776,9 @@ static int pcan_usb_pro_encode_msg(struct peak_usb_device 
*dev,
 
        flags = 0;
        if (cf->can_id & CAN_EFF_FLAG)
-               flags |= 0x02;
+               flags |= PCAN_USBPRO_EXT;
        if (cf->can_id & CAN_RTR_FLAG)
-               flags |= 0x01;
+               flags |= PCAN_USBPRO_RTR;
 
        pcan_msg_add_rec(&usb_msg, data_type, 0, flags, len, cf->can_id,
                         cf->data);
-- 
2.30.2


Reply via email to