* [PATCH can-next] kvaser_usb: convert USB IDs to hexadecimal values
@ 2023-03-27 17:53 Oliver Hartkopp
2023-03-28 7:21 ` Jimmy Assarsson
2023-03-28 8:46 ` Alexander Dahl
0 siblings, 2 replies; 5+ messages in thread
From: Oliver Hartkopp @ 2023-03-27 17:53 UTC (permalink / raw)
To: linux-can; +Cc: Oliver Hartkopp, Jimmy Assarsson
USB IDs are usually represented in 16 bit hexadecimal values.
To match the common representation in lsusb and for searching USB IDs
in the internet convert the decimal values to hexadecimal.
Signed-off-by: Oliver Hartkopp <socketcan@hartkopp.net>
Cc: Jimmy Assarsson <extja@kvaser.com>
---
.../net/can/usb/kvaser_usb/kvaser_usb_core.c | 102 +++++++++---------
1 file changed, 51 insertions(+), 51 deletions(-)
diff --git a/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c b/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
index d4c5356d5884..d0015f83f924 100644
--- a/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
+++ b/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
@@ -29,67 +29,67 @@
#include <linux/can/netlink.h>
#include "kvaser_usb.h"
/* Kvaser USB vendor id. */
-#define KVASER_VENDOR_ID 0x0bfd
+#define KVASER_VENDOR_ID 0x0BFD
/* Kvaser Leaf USB devices product ids */
-#define USB_LEAF_DEVEL_PRODUCT_ID 10
-#define USB_LEAF_LITE_PRODUCT_ID 11
-#define USB_LEAF_PRO_PRODUCT_ID 12
-#define USB_LEAF_SPRO_PRODUCT_ID 14
-#define USB_LEAF_PRO_LS_PRODUCT_ID 15
-#define USB_LEAF_PRO_SWC_PRODUCT_ID 16
-#define USB_LEAF_PRO_LIN_PRODUCT_ID 17
-#define USB_LEAF_SPRO_LS_PRODUCT_ID 18
-#define USB_LEAF_SPRO_SWC_PRODUCT_ID 19
-#define USB_MEMO2_DEVEL_PRODUCT_ID 22
-#define USB_MEMO2_HSHS_PRODUCT_ID 23
-#define USB_UPRO_HSHS_PRODUCT_ID 24
-#define USB_LEAF_LITE_GI_PRODUCT_ID 25
-#define USB_LEAF_PRO_OBDII_PRODUCT_ID 26
-#define USB_MEMO2_HSLS_PRODUCT_ID 27
-#define USB_LEAF_LITE_CH_PRODUCT_ID 28
-#define USB_BLACKBIRD_SPRO_PRODUCT_ID 29
-#define USB_OEM_MERCURY_PRODUCT_ID 34
-#define USB_OEM_LEAF_PRODUCT_ID 35
-#define USB_CAN_R_PRODUCT_ID 39
-#define USB_LEAF_LITE_V2_PRODUCT_ID 288
-#define USB_MINI_PCIE_HS_PRODUCT_ID 289
-#define USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID 290
-#define USB_USBCAN_LIGHT_2HS_PRODUCT_ID 291
-#define USB_MINI_PCIE_2HS_PRODUCT_ID 292
-#define USB_USBCAN_R_V2_PRODUCT_ID 294
-#define USB_LEAF_LIGHT_R_V2_PRODUCT_ID 295
-#define USB_LEAF_LIGHT_HS_V2_OEM2_PRODUCT_ID 296
+#define USB_LEAF_DEVEL_PRODUCT_ID 0x000A
+#define USB_LEAF_LITE_PRODUCT_ID 0x000B
+#define USB_LEAF_PRO_PRODUCT_ID 0x000C
+#define USB_LEAF_SPRO_PRODUCT_ID 0x000E
+#define USB_LEAF_PRO_LS_PRODUCT_ID 0x000F
+#define USB_LEAF_PRO_SWC_PRODUCT_ID 0x0010
+#define USB_LEAF_PRO_LIN_PRODUCT_ID 0x0011
+#define USB_LEAF_SPRO_LS_PRODUCT_ID 0x0012
+#define USB_LEAF_SPRO_SWC_PRODUCT_ID 0x0013
+#define USB_MEMO2_DEVEL_PRODUCT_ID 0x0016
+#define USB_MEMO2_HSHS_PRODUCT_ID 0x0017
+#define USB_UPRO_HSHS_PRODUCT_ID 0x0018
+#define USB_LEAF_LITE_GI_PRODUCT_ID 0x0019
+#define USB_LEAF_PRO_OBDII_PRODUCT_ID 0x001A
+#define USB_MEMO2_HSLS_PRODUCT_ID 0x001B
+#define USB_LEAF_LITE_CH_PRODUCT_ID 0x001C
+#define USB_BLACKBIRD_SPRO_PRODUCT_ID 0x001D
+#define USB_OEM_MERCURY_PRODUCT_ID 0x0022
+#define USB_OEM_LEAF_PRODUCT_ID 0x0023
+#define USB_CAN_R_PRODUCT_ID 0x0027
+#define USB_LEAF_LITE_V2_PRODUCT_ID 0x0120
+#define USB_MINI_PCIE_HS_PRODUCT_ID 0x0121
+#define USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID 0x0122
+#define USB_USBCAN_LIGHT_2HS_PRODUCT_ID 0x0123
+#define USB_MINI_PCIE_2HS_PRODUCT_ID 0x0124
+#define USB_USBCAN_R_V2_PRODUCT_ID 0x0126
+#define USB_LEAF_LIGHT_R_V2_PRODUCT_ID 0x0127
+#define USB_LEAF_LIGHT_HS_V2_OEM2_PRODUCT_ID 0x0128
/* Kvaser USBCan-II devices product ids */
-#define USB_USBCAN_REVB_PRODUCT_ID 2
-#define USB_VCI2_PRODUCT_ID 3
-#define USB_USBCAN2_PRODUCT_ID 4
-#define USB_MEMORATOR_PRODUCT_ID 5
+#define USB_USBCAN_REVB_PRODUCT_ID 0x0002
+#define USB_VCI2_PRODUCT_ID 0x0003
+#define USB_USBCAN2_PRODUCT_ID 0x0004
+#define USB_MEMORATOR_PRODUCT_ID 0x0005
/* Kvaser Minihydra USB devices product ids */
-#define USB_BLACKBIRD_V2_PRODUCT_ID 258
-#define USB_MEMO_PRO_5HS_PRODUCT_ID 260
-#define USB_USBCAN_PRO_5HS_PRODUCT_ID 261
-#define USB_USBCAN_LIGHT_4HS_PRODUCT_ID 262
-#define USB_LEAF_PRO_HS_V2_PRODUCT_ID 263
-#define USB_USBCAN_PRO_2HS_V2_PRODUCT_ID 264
-#define USB_MEMO_2HS_PRODUCT_ID 265
-#define USB_MEMO_PRO_2HS_V2_PRODUCT_ID 266
-#define USB_HYBRID_2CANLIN_PRODUCT_ID 267
-#define USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID 268
-#define USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID 269
-#define USB_HYBRID_PRO_2CANLIN_PRODUCT_ID 270
-#define USB_U100_PRODUCT_ID 273
-#define USB_U100P_PRODUCT_ID 274
-#define USB_U100S_PRODUCT_ID 275
-#define USB_USBCAN_PRO_4HS_PRODUCT_ID 276
-#define USB_HYBRID_CANLIN_PRODUCT_ID 277
-#define USB_HYBRID_PRO_CANLIN_PRODUCT_ID 278
+#define USB_BLACKBIRD_V2_PRODUCT_ID 0x0102
+#define USB_MEMO_PRO_5HS_PRODUCT_ID 0x0104
+#define USB_USBCAN_PRO_5HS_PRODUCT_ID 0x0105
+#define USB_USBCAN_LIGHT_4HS_PRODUCT_ID 0x0106
+#define USB_LEAF_PRO_HS_V2_PRODUCT_ID 0x0107
+#define USB_USBCAN_PRO_2HS_V2_PRODUCT_ID 0x0108
+#define USB_MEMO_2HS_PRODUCT_ID 0x0109
+#define USB_MEMO_PRO_2HS_V2_PRODUCT_ID 0x010A
+#define USB_HYBRID_2CANLIN_PRODUCT_ID 0x010B
+#define USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID 0x010C
+#define USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID 0x010D
+#define USB_HYBRID_PRO_2CANLIN_PRODUCT_ID 0x010E
+#define USB_U100_PRODUCT_ID 0x0111
+#define USB_U100P_PRODUCT_ID 0x0112
+#define USB_U100S_PRODUCT_ID 0x0113
+#define USB_USBCAN_PRO_4HS_PRODUCT_ID 0x0114
+#define USB_HYBRID_CANLIN_PRODUCT_ID 0x0115
+#define USB_HYBRID_PRO_CANLIN_PRODUCT_ID 0x0116
static const struct kvaser_usb_driver_info kvaser_usb_driver_info_hydra = {
.quirks = KVASER_USB_QUIRK_HAS_HARDWARE_TIMESTAMP,
.ops = &kvaser_usb_hydra_dev_ops,
};
--
2.30.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH can-next] kvaser_usb: convert USB IDs to hexadecimal values
2023-03-27 17:53 [PATCH can-next] kvaser_usb: convert USB IDs to hexadecimal values Oliver Hartkopp
@ 2023-03-28 7:21 ` Jimmy Assarsson
2023-03-29 8:55 ` Oliver Hartkopp
2023-03-28 8:46 ` Alexander Dahl
1 sibling, 1 reply; 5+ messages in thread
From: Jimmy Assarsson @ 2023-03-28 7:21 UTC (permalink / raw)
To: Oliver Hartkopp, linux-can
On 3/27/23 19:53, Oliver Hartkopp wrote:
> USB IDs are usually represented in 16 bit hexadecimal values.
> To match the common representation in lsusb and for searching USB IDs
> in the internet convert the decimal values to hexadecimal.
Thanks for the patch!
I'm about drop the aligned block indentation, in favour of consistent
one space indentation, for the entire driver. Feel free to update the
patch, or I'll fix it in a separate patch later.
Acked-by: Jimmy Assarsson <extja@kvaser.com>
Best regards,
jimmy
> Signed-off-by: Oliver Hartkopp <socketcan@hartkopp.net>
> Cc: Jimmy Assarsson <extja@kvaser.com>
> ---
> .../net/can/usb/kvaser_usb/kvaser_usb_core.c | 102 +++++++++---------
> 1 file changed, 51 insertions(+), 51 deletions(-)
>
> diff --git a/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c b/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
> index d4c5356d5884..d0015f83f924 100644
> --- a/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
> +++ b/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
> @@ -29,67 +29,67 @@
> #include <linux/can/netlink.h>
>
> #include "kvaser_usb.h"
>
> /* Kvaser USB vendor id. */
> -#define KVASER_VENDOR_ID 0x0bfd
> +#define KVASER_VENDOR_ID 0x0BFD
>
> /* Kvaser Leaf USB devices product ids */
> -#define USB_LEAF_DEVEL_PRODUCT_ID 10
> -#define USB_LEAF_LITE_PRODUCT_ID 11
> -#define USB_LEAF_PRO_PRODUCT_ID 12
> -#define USB_LEAF_SPRO_PRODUCT_ID 14
> -#define USB_LEAF_PRO_LS_PRODUCT_ID 15
> -#define USB_LEAF_PRO_SWC_PRODUCT_ID 16
> -#define USB_LEAF_PRO_LIN_PRODUCT_ID 17
> -#define USB_LEAF_SPRO_LS_PRODUCT_ID 18
> -#define USB_LEAF_SPRO_SWC_PRODUCT_ID 19
> -#define USB_MEMO2_DEVEL_PRODUCT_ID 22
> -#define USB_MEMO2_HSHS_PRODUCT_ID 23
> -#define USB_UPRO_HSHS_PRODUCT_ID 24
> -#define USB_LEAF_LITE_GI_PRODUCT_ID 25
> -#define USB_LEAF_PRO_OBDII_PRODUCT_ID 26
> -#define USB_MEMO2_HSLS_PRODUCT_ID 27
> -#define USB_LEAF_LITE_CH_PRODUCT_ID 28
> -#define USB_BLACKBIRD_SPRO_PRODUCT_ID 29
> -#define USB_OEM_MERCURY_PRODUCT_ID 34
> -#define USB_OEM_LEAF_PRODUCT_ID 35
> -#define USB_CAN_R_PRODUCT_ID 39
> -#define USB_LEAF_LITE_V2_PRODUCT_ID 288
> -#define USB_MINI_PCIE_HS_PRODUCT_ID 289
> -#define USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID 290
> -#define USB_USBCAN_LIGHT_2HS_PRODUCT_ID 291
> -#define USB_MINI_PCIE_2HS_PRODUCT_ID 292
> -#define USB_USBCAN_R_V2_PRODUCT_ID 294
> -#define USB_LEAF_LIGHT_R_V2_PRODUCT_ID 295
> -#define USB_LEAF_LIGHT_HS_V2_OEM2_PRODUCT_ID 296
> +#define USB_LEAF_DEVEL_PRODUCT_ID 0x000A
> +#define USB_LEAF_LITE_PRODUCT_ID 0x000B
> +#define USB_LEAF_PRO_PRODUCT_ID 0x000C
> +#define USB_LEAF_SPRO_PRODUCT_ID 0x000E
> +#define USB_LEAF_PRO_LS_PRODUCT_ID 0x000F
> +#define USB_LEAF_PRO_SWC_PRODUCT_ID 0x0010
> +#define USB_LEAF_PRO_LIN_PRODUCT_ID 0x0011
> +#define USB_LEAF_SPRO_LS_PRODUCT_ID 0x0012
> +#define USB_LEAF_SPRO_SWC_PRODUCT_ID 0x0013
> +#define USB_MEMO2_DEVEL_PRODUCT_ID 0x0016
> +#define USB_MEMO2_HSHS_PRODUCT_ID 0x0017
> +#define USB_UPRO_HSHS_PRODUCT_ID 0x0018
> +#define USB_LEAF_LITE_GI_PRODUCT_ID 0x0019
> +#define USB_LEAF_PRO_OBDII_PRODUCT_ID 0x001A
> +#define USB_MEMO2_HSLS_PRODUCT_ID 0x001B
> +#define USB_LEAF_LITE_CH_PRODUCT_ID 0x001C
> +#define USB_BLACKBIRD_SPRO_PRODUCT_ID 0x001D
> +#define USB_OEM_MERCURY_PRODUCT_ID 0x0022
> +#define USB_OEM_LEAF_PRODUCT_ID 0x0023
> +#define USB_CAN_R_PRODUCT_ID 0x0027
> +#define USB_LEAF_LITE_V2_PRODUCT_ID 0x0120
> +#define USB_MINI_PCIE_HS_PRODUCT_ID 0x0121
> +#define USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID 0x0122
> +#define USB_USBCAN_LIGHT_2HS_PRODUCT_ID 0x0123
> +#define USB_MINI_PCIE_2HS_PRODUCT_ID 0x0124
> +#define USB_USBCAN_R_V2_PRODUCT_ID 0x0126
> +#define USB_LEAF_LIGHT_R_V2_PRODUCT_ID 0x0127
> +#define USB_LEAF_LIGHT_HS_V2_OEM2_PRODUCT_ID 0x0128
>
> /* Kvaser USBCan-II devices product ids */
> -#define USB_USBCAN_REVB_PRODUCT_ID 2
> -#define USB_VCI2_PRODUCT_ID 3
> -#define USB_USBCAN2_PRODUCT_ID 4
> -#define USB_MEMORATOR_PRODUCT_ID 5
> +#define USB_USBCAN_REVB_PRODUCT_ID 0x0002
> +#define USB_VCI2_PRODUCT_ID 0x0003
> +#define USB_USBCAN2_PRODUCT_ID 0x0004
> +#define USB_MEMORATOR_PRODUCT_ID 0x0005
>
> /* Kvaser Minihydra USB devices product ids */
> -#define USB_BLACKBIRD_V2_PRODUCT_ID 258
> -#define USB_MEMO_PRO_5HS_PRODUCT_ID 260
> -#define USB_USBCAN_PRO_5HS_PRODUCT_ID 261
> -#define USB_USBCAN_LIGHT_4HS_PRODUCT_ID 262
> -#define USB_LEAF_PRO_HS_V2_PRODUCT_ID 263
> -#define USB_USBCAN_PRO_2HS_V2_PRODUCT_ID 264
> -#define USB_MEMO_2HS_PRODUCT_ID 265
> -#define USB_MEMO_PRO_2HS_V2_PRODUCT_ID 266
> -#define USB_HYBRID_2CANLIN_PRODUCT_ID 267
> -#define USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID 268
> -#define USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID 269
> -#define USB_HYBRID_PRO_2CANLIN_PRODUCT_ID 270
> -#define USB_U100_PRODUCT_ID 273
> -#define USB_U100P_PRODUCT_ID 274
> -#define USB_U100S_PRODUCT_ID 275
> -#define USB_USBCAN_PRO_4HS_PRODUCT_ID 276
> -#define USB_HYBRID_CANLIN_PRODUCT_ID 277
> -#define USB_HYBRID_PRO_CANLIN_PRODUCT_ID 278
> +#define USB_BLACKBIRD_V2_PRODUCT_ID 0x0102
> +#define USB_MEMO_PRO_5HS_PRODUCT_ID 0x0104
> +#define USB_USBCAN_PRO_5HS_PRODUCT_ID 0x0105
> +#define USB_USBCAN_LIGHT_4HS_PRODUCT_ID 0x0106
> +#define USB_LEAF_PRO_HS_V2_PRODUCT_ID 0x0107
> +#define USB_USBCAN_PRO_2HS_V2_PRODUCT_ID 0x0108
> +#define USB_MEMO_2HS_PRODUCT_ID 0x0109
> +#define USB_MEMO_PRO_2HS_V2_PRODUCT_ID 0x010A
> +#define USB_HYBRID_2CANLIN_PRODUCT_ID 0x010B
> +#define USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID 0x010C
> +#define USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID 0x010D
> +#define USB_HYBRID_PRO_2CANLIN_PRODUCT_ID 0x010E
> +#define USB_U100_PRODUCT_ID 0x0111
> +#define USB_U100P_PRODUCT_ID 0x0112
> +#define USB_U100S_PRODUCT_ID 0x0113
> +#define USB_USBCAN_PRO_4HS_PRODUCT_ID 0x0114
> +#define USB_HYBRID_CANLIN_PRODUCT_ID 0x0115
> +#define USB_HYBRID_PRO_CANLIN_PRODUCT_ID 0x0116
>
> static const struct kvaser_usb_driver_info kvaser_usb_driver_info_hydra = {
> .quirks = KVASER_USB_QUIRK_HAS_HARDWARE_TIMESTAMP,
> .ops = &kvaser_usb_hydra_dev_ops,
> };
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH can-next] kvaser_usb: convert USB IDs to hexadecimal values
2023-03-27 17:53 [PATCH can-next] kvaser_usb: convert USB IDs to hexadecimal values Oliver Hartkopp
2023-03-28 7:21 ` Jimmy Assarsson
@ 2023-03-28 8:46 ` Alexander Dahl
2023-03-29 8:53 ` Oliver Hartkopp
1 sibling, 1 reply; 5+ messages in thread
From: Alexander Dahl @ 2023-03-28 8:46 UTC (permalink / raw)
To: Oliver Hartkopp; +Cc: linux-can, Jimmy Assarsson
Hello Oliver,
Am Mon, Mar 27, 2023 at 07:53:44PM +0200 schrieb Oliver Hartkopp:
> USB IDs are usually represented in 16 bit hexadecimal values.
> To match the common representation in lsusb and for searching USB IDs
> in the internet convert the decimal values to hexadecimal.
lsusb representation on my machines is lowercase for the letters a-f.
From a quick grep over drivers/usb I can see no clear preference, but
I'd say more use lowercase.
> Signed-off-by: Oliver Hartkopp <socketcan@hartkopp.net>
> Cc: Jimmy Assarsson <extja@kvaser.com>
> ---
> .../net/can/usb/kvaser_usb/kvaser_usb_core.c | 102 +++++++++---------
> 1 file changed, 51 insertions(+), 51 deletions(-)
>
> diff --git a/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c b/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
> index d4c5356d5884..d0015f83f924 100644
> --- a/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
> +++ b/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
> @@ -29,67 +29,67 @@
> #include <linux/can/netlink.h>
>
> #include "kvaser_usb.h"
>
> /* Kvaser USB vendor id. */
> -#define KVASER_VENDOR_ID 0x0bfd
> +#define KVASER_VENDOR_ID 0x0BFD
That would also make this change not needed.
> /* Kvaser Leaf USB devices product ids */
> -#define USB_LEAF_DEVEL_PRODUCT_ID 10
> -#define USB_LEAF_LITE_PRODUCT_ID 11
> -#define USB_LEAF_PRO_PRODUCT_ID 12
> -#define USB_LEAF_SPRO_PRODUCT_ID 14
> -#define USB_LEAF_PRO_LS_PRODUCT_ID 15
> -#define USB_LEAF_PRO_SWC_PRODUCT_ID 16
> -#define USB_LEAF_PRO_LIN_PRODUCT_ID 17
> -#define USB_LEAF_SPRO_LS_PRODUCT_ID 18
> -#define USB_LEAF_SPRO_SWC_PRODUCT_ID 19
> -#define USB_MEMO2_DEVEL_PRODUCT_ID 22
> -#define USB_MEMO2_HSHS_PRODUCT_ID 23
> -#define USB_UPRO_HSHS_PRODUCT_ID 24
> -#define USB_LEAF_LITE_GI_PRODUCT_ID 25
> -#define USB_LEAF_PRO_OBDII_PRODUCT_ID 26
> -#define USB_MEMO2_HSLS_PRODUCT_ID 27
> -#define USB_LEAF_LITE_CH_PRODUCT_ID 28
> -#define USB_BLACKBIRD_SPRO_PRODUCT_ID 29
> -#define USB_OEM_MERCURY_PRODUCT_ID 34
> -#define USB_OEM_LEAF_PRODUCT_ID 35
> -#define USB_CAN_R_PRODUCT_ID 39
> -#define USB_LEAF_LITE_V2_PRODUCT_ID 288
> -#define USB_MINI_PCIE_HS_PRODUCT_ID 289
> -#define USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID 290
> -#define USB_USBCAN_LIGHT_2HS_PRODUCT_ID 291
> -#define USB_MINI_PCIE_2HS_PRODUCT_ID 292
> -#define USB_USBCAN_R_V2_PRODUCT_ID 294
> -#define USB_LEAF_LIGHT_R_V2_PRODUCT_ID 295
> -#define USB_LEAF_LIGHT_HS_V2_OEM2_PRODUCT_ID 296
> +#define USB_LEAF_DEVEL_PRODUCT_ID 0x000A
> +#define USB_LEAF_LITE_PRODUCT_ID 0x000B
> +#define USB_LEAF_PRO_PRODUCT_ID 0x000C
> +#define USB_LEAF_SPRO_PRODUCT_ID 0x000E
> +#define USB_LEAF_PRO_LS_PRODUCT_ID 0x000F
> +#define USB_LEAF_PRO_SWC_PRODUCT_ID 0x0010
> +#define USB_LEAF_PRO_LIN_PRODUCT_ID 0x0011
> +#define USB_LEAF_SPRO_LS_PRODUCT_ID 0x0012
> +#define USB_LEAF_SPRO_SWC_PRODUCT_ID 0x0013
> +#define USB_MEMO2_DEVEL_PRODUCT_ID 0x0016
> +#define USB_MEMO2_HSHS_PRODUCT_ID 0x0017
> +#define USB_UPRO_HSHS_PRODUCT_ID 0x0018
> +#define USB_LEAF_LITE_GI_PRODUCT_ID 0x0019
> +#define USB_LEAF_PRO_OBDII_PRODUCT_ID 0x001A
> +#define USB_MEMO2_HSLS_PRODUCT_ID 0x001B
> +#define USB_LEAF_LITE_CH_PRODUCT_ID 0x001C
> +#define USB_BLACKBIRD_SPRO_PRODUCT_ID 0x001D
> +#define USB_OEM_MERCURY_PRODUCT_ID 0x0022
> +#define USB_OEM_LEAF_PRODUCT_ID 0x0023
> +#define USB_CAN_R_PRODUCT_ID 0x0027
> +#define USB_LEAF_LITE_V2_PRODUCT_ID 0x0120
> +#define USB_MINI_PCIE_HS_PRODUCT_ID 0x0121
> +#define USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID 0x0122
> +#define USB_USBCAN_LIGHT_2HS_PRODUCT_ID 0x0123
> +#define USB_MINI_PCIE_2HS_PRODUCT_ID 0x0124
> +#define USB_USBCAN_R_V2_PRODUCT_ID 0x0126
> +#define USB_LEAF_LIGHT_R_V2_PRODUCT_ID 0x0127
> +#define USB_LEAF_LIGHT_HS_V2_OEM2_PRODUCT_ID 0x0128
Decimal match hex.
> /* Kvaser USBCan-II devices product ids */
> -#define USB_USBCAN_REVB_PRODUCT_ID 2
> -#define USB_VCI2_PRODUCT_ID 3
> -#define USB_USBCAN2_PRODUCT_ID 4
> -#define USB_MEMORATOR_PRODUCT_ID 5
> +#define USB_USBCAN_REVB_PRODUCT_ID 0x0002
> +#define USB_VCI2_PRODUCT_ID 0x0003
> +#define USB_USBCAN2_PRODUCT_ID 0x0004
> +#define USB_MEMORATOR_PRODUCT_ID 0x0005
Decimal match hex.
> /* Kvaser Minihydra USB devices product ids */
> -#define USB_BLACKBIRD_V2_PRODUCT_ID 258
> -#define USB_MEMO_PRO_5HS_PRODUCT_ID 260
> -#define USB_USBCAN_PRO_5HS_PRODUCT_ID 261
> -#define USB_USBCAN_LIGHT_4HS_PRODUCT_ID 262
> -#define USB_LEAF_PRO_HS_V2_PRODUCT_ID 263
> -#define USB_USBCAN_PRO_2HS_V2_PRODUCT_ID 264
> -#define USB_MEMO_2HS_PRODUCT_ID 265
> -#define USB_MEMO_PRO_2HS_V2_PRODUCT_ID 266
> -#define USB_HYBRID_2CANLIN_PRODUCT_ID 267
> -#define USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID 268
> -#define USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID 269
> -#define USB_HYBRID_PRO_2CANLIN_PRODUCT_ID 270
> -#define USB_U100_PRODUCT_ID 273
> -#define USB_U100P_PRODUCT_ID 274
> -#define USB_U100S_PRODUCT_ID 275
> -#define USB_USBCAN_PRO_4HS_PRODUCT_ID 276
> -#define USB_HYBRID_CANLIN_PRODUCT_ID 277
> -#define USB_HYBRID_PRO_CANLIN_PRODUCT_ID 278
> +#define USB_BLACKBIRD_V2_PRODUCT_ID 0x0102
> +#define USB_MEMO_PRO_5HS_PRODUCT_ID 0x0104
> +#define USB_USBCAN_PRO_5HS_PRODUCT_ID 0x0105
> +#define USB_USBCAN_LIGHT_4HS_PRODUCT_ID 0x0106
> +#define USB_LEAF_PRO_HS_V2_PRODUCT_ID 0x0107
> +#define USB_USBCAN_PRO_2HS_V2_PRODUCT_ID 0x0108
> +#define USB_MEMO_2HS_PRODUCT_ID 0x0109
> +#define USB_MEMO_PRO_2HS_V2_PRODUCT_ID 0x010A
> +#define USB_HYBRID_2CANLIN_PRODUCT_ID 0x010B
> +#define USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID 0x010C
> +#define USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID 0x010D
> +#define USB_HYBRID_PRO_2CANLIN_PRODUCT_ID 0x010E
> +#define USB_U100_PRODUCT_ID 0x0111
> +#define USB_U100P_PRODUCT_ID 0x0112
> +#define USB_U100S_PRODUCT_ID 0x0113
> +#define USB_USBCAN_PRO_4HS_PRODUCT_ID 0x0114
> +#define USB_HYBRID_CANLIN_PRODUCT_ID 0x0115
> +#define USB_HYBRID_PRO_CANLIN_PRODUCT_ID 0x0116
Decimal match hex. So for the actual values:
Reviewed-by: Alexander Dahl <ada@thorsis.com>
Greets
Alex
>
> static const struct kvaser_usb_driver_info kvaser_usb_driver_info_hydra = {
> .quirks = KVASER_USB_QUIRK_HAS_HARDWARE_TIMESTAMP,
> .ops = &kvaser_usb_hydra_dev_ops,
> };
> --
> 2.30.2
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH can-next] kvaser_usb: convert USB IDs to hexadecimal values
2023-03-28 8:46 ` Alexander Dahl
@ 2023-03-29 8:53 ` Oliver Hartkopp
0 siblings, 0 replies; 5+ messages in thread
From: Oliver Hartkopp @ 2023-03-29 8:53 UTC (permalink / raw)
To: Alexander Dahl; +Cc: linux-can, Jimmy Assarsson
On 28.03.23 10:46, Alexander Dahl wrote:
> Hello Oliver,
>
> Am Mon, Mar 27, 2023 at 07:53:44PM +0200 schrieb Oliver Hartkopp:
>> USB IDs are usually represented in 16 bit hexadecimal values.
>> To match the common representation in lsusb and for searching USB IDs
>> in the internet convert the decimal values to hexadecimal.
>
> lsusb representation on my machines is lowercase for the letters a-f.
> From a quick grep over drivers/usb I can see no clear preference, but
> I'd say more use lowercase.
>
That's right!
I will send a V2 having the hex values lower case.
>> Signed-off-by: Oliver Hartkopp <socketcan@hartkopp.net>
>> Cc: Jimmy Assarsson <extja@kvaser.com>
>> ---
>> .../net/can/usb/kvaser_usb/kvaser_usb_core.c | 102 +++++++++---------
>> 1 file changed, 51 insertions(+), 51 deletions(-)
>>
>> diff --git a/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c b/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
>> index d4c5356d5884..d0015f83f924 100644
>> --- a/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
>> +++ b/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
>> @@ -29,67 +29,67 @@
>> #include <linux/can/netlink.h>
>>
>> #include "kvaser_usb.h"
>>
>> /* Kvaser USB vendor id. */
>> -#define KVASER_VENDOR_ID 0x0bfd
>> +#define KVASER_VENDOR_ID 0x0BFD
>
> That would also make this change not needed.
>
ACK
>> /* Kvaser Leaf USB devices product ids */
>> -#define USB_LEAF_DEVEL_PRODUCT_ID 10
>> -#define USB_LEAF_LITE_PRODUCT_ID 11
>> -#define USB_LEAF_PRO_PRODUCT_ID 12
>> -#define USB_LEAF_SPRO_PRODUCT_ID 14
>> -#define USB_LEAF_PRO_LS_PRODUCT_ID 15
>> -#define USB_LEAF_PRO_SWC_PRODUCT_ID 16
>> -#define USB_LEAF_PRO_LIN_PRODUCT_ID 17
>> -#define USB_LEAF_SPRO_LS_PRODUCT_ID 18
>> -#define USB_LEAF_SPRO_SWC_PRODUCT_ID 19
>> -#define USB_MEMO2_DEVEL_PRODUCT_ID 22
>> -#define USB_MEMO2_HSHS_PRODUCT_ID 23
>> -#define USB_UPRO_HSHS_PRODUCT_ID 24
>> -#define USB_LEAF_LITE_GI_PRODUCT_ID 25
>> -#define USB_LEAF_PRO_OBDII_PRODUCT_ID 26
>> -#define USB_MEMO2_HSLS_PRODUCT_ID 27
>> -#define USB_LEAF_LITE_CH_PRODUCT_ID 28
>> -#define USB_BLACKBIRD_SPRO_PRODUCT_ID 29
>> -#define USB_OEM_MERCURY_PRODUCT_ID 34
>> -#define USB_OEM_LEAF_PRODUCT_ID 35
>> -#define USB_CAN_R_PRODUCT_ID 39
>> -#define USB_LEAF_LITE_V2_PRODUCT_ID 288
>> -#define USB_MINI_PCIE_HS_PRODUCT_ID 289
>> -#define USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID 290
>> -#define USB_USBCAN_LIGHT_2HS_PRODUCT_ID 291
>> -#define USB_MINI_PCIE_2HS_PRODUCT_ID 292
>> -#define USB_USBCAN_R_V2_PRODUCT_ID 294
>> -#define USB_LEAF_LIGHT_R_V2_PRODUCT_ID 295
>> -#define USB_LEAF_LIGHT_HS_V2_OEM2_PRODUCT_ID 296
>> +#define USB_LEAF_DEVEL_PRODUCT_ID 0x000A
>> +#define USB_LEAF_LITE_PRODUCT_ID 0x000B
>> +#define USB_LEAF_PRO_PRODUCT_ID 0x000C
>> +#define USB_LEAF_SPRO_PRODUCT_ID 0x000E
>> +#define USB_LEAF_PRO_LS_PRODUCT_ID 0x000F
>> +#define USB_LEAF_PRO_SWC_PRODUCT_ID 0x0010
>> +#define USB_LEAF_PRO_LIN_PRODUCT_ID 0x0011
>> +#define USB_LEAF_SPRO_LS_PRODUCT_ID 0x0012
>> +#define USB_LEAF_SPRO_SWC_PRODUCT_ID 0x0013
>> +#define USB_MEMO2_DEVEL_PRODUCT_ID 0x0016
>> +#define USB_MEMO2_HSHS_PRODUCT_ID 0x0017
>> +#define USB_UPRO_HSHS_PRODUCT_ID 0x0018
>> +#define USB_LEAF_LITE_GI_PRODUCT_ID 0x0019
>> +#define USB_LEAF_PRO_OBDII_PRODUCT_ID 0x001A
>> +#define USB_MEMO2_HSLS_PRODUCT_ID 0x001B
>> +#define USB_LEAF_LITE_CH_PRODUCT_ID 0x001C
>> +#define USB_BLACKBIRD_SPRO_PRODUCT_ID 0x001D
>> +#define USB_OEM_MERCURY_PRODUCT_ID 0x0022
>> +#define USB_OEM_LEAF_PRODUCT_ID 0x0023
>> +#define USB_CAN_R_PRODUCT_ID 0x0027
>> +#define USB_LEAF_LITE_V2_PRODUCT_ID 0x0120
>> +#define USB_MINI_PCIE_HS_PRODUCT_ID 0x0121
>> +#define USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID 0x0122
>> +#define USB_USBCAN_LIGHT_2HS_PRODUCT_ID 0x0123
>> +#define USB_MINI_PCIE_2HS_PRODUCT_ID 0x0124
>> +#define USB_USBCAN_R_V2_PRODUCT_ID 0x0126
>> +#define USB_LEAF_LIGHT_R_V2_PRODUCT_ID 0x0127
>> +#define USB_LEAF_LIGHT_HS_V2_OEM2_PRODUCT_ID 0x0128
>
> Decimal match hex.
>
>> /* Kvaser USBCan-II devices product ids */
>> -#define USB_USBCAN_REVB_PRODUCT_ID 2
>> -#define USB_VCI2_PRODUCT_ID 3
>> -#define USB_USBCAN2_PRODUCT_ID 4
>> -#define USB_MEMORATOR_PRODUCT_ID 5
>> +#define USB_USBCAN_REVB_PRODUCT_ID 0x0002
>> +#define USB_VCI2_PRODUCT_ID 0x0003
>> +#define USB_USBCAN2_PRODUCT_ID 0x0004
>> +#define USB_MEMORATOR_PRODUCT_ID 0x0005
>
> Decimal match hex.
>
>> /* Kvaser Minihydra USB devices product ids */
>> -#define USB_BLACKBIRD_V2_PRODUCT_ID 258
>> -#define USB_MEMO_PRO_5HS_PRODUCT_ID 260
>> -#define USB_USBCAN_PRO_5HS_PRODUCT_ID 261
>> -#define USB_USBCAN_LIGHT_4HS_PRODUCT_ID 262
>> -#define USB_LEAF_PRO_HS_V2_PRODUCT_ID 263
>> -#define USB_USBCAN_PRO_2HS_V2_PRODUCT_ID 264
>> -#define USB_MEMO_2HS_PRODUCT_ID 265
>> -#define USB_MEMO_PRO_2HS_V2_PRODUCT_ID 266
>> -#define USB_HYBRID_2CANLIN_PRODUCT_ID 267
>> -#define USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID 268
>> -#define USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID 269
>> -#define USB_HYBRID_PRO_2CANLIN_PRODUCT_ID 270
>> -#define USB_U100_PRODUCT_ID 273
>> -#define USB_U100P_PRODUCT_ID 274
>> -#define USB_U100S_PRODUCT_ID 275
>> -#define USB_USBCAN_PRO_4HS_PRODUCT_ID 276
>> -#define USB_HYBRID_CANLIN_PRODUCT_ID 277
>> -#define USB_HYBRID_PRO_CANLIN_PRODUCT_ID 278
>> +#define USB_BLACKBIRD_V2_PRODUCT_ID 0x0102
>> +#define USB_MEMO_PRO_5HS_PRODUCT_ID 0x0104
>> +#define USB_USBCAN_PRO_5HS_PRODUCT_ID 0x0105
>> +#define USB_USBCAN_LIGHT_4HS_PRODUCT_ID 0x0106
>> +#define USB_LEAF_PRO_HS_V2_PRODUCT_ID 0x0107
>> +#define USB_USBCAN_PRO_2HS_V2_PRODUCT_ID 0x0108
>> +#define USB_MEMO_2HS_PRODUCT_ID 0x0109
>> +#define USB_MEMO_PRO_2HS_V2_PRODUCT_ID 0x010A
>> +#define USB_HYBRID_2CANLIN_PRODUCT_ID 0x010B
>> +#define USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID 0x010C
>> +#define USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID 0x010D
>> +#define USB_HYBRID_PRO_2CANLIN_PRODUCT_ID 0x010E
>> +#define USB_U100_PRODUCT_ID 0x0111
>> +#define USB_U100P_PRODUCT_ID 0x0112
>> +#define USB_U100S_PRODUCT_ID 0x0113
>> +#define USB_USBCAN_PRO_4HS_PRODUCT_ID 0x0114
>> +#define USB_HYBRID_CANLIN_PRODUCT_ID 0x0115
>> +#define USB_HYBRID_PRO_CANLIN_PRODUCT_ID 0x0116
>
> Decimal match hex. So for the actual values:
:-D
So how did you test that?
In fact I cut out the values and imported them into LibreOffice Calc to
use DEC2HEX() ...
Seems to do the job ;-)
>
> Reviewed-by: Alexander Dahl <ada@thorsis.com>
>
Thanks,
Oliver
> Greets
> Alex
>
>>
>> static const struct kvaser_usb_driver_info kvaser_usb_driver_info_hydra = {
>> .quirks = KVASER_USB_QUIRK_HAS_HARDWARE_TIMESTAMP,
>> .ops = &kvaser_usb_hydra_dev_ops,
>> };
>> --
>> 2.30.2
>>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH can-next] kvaser_usb: convert USB IDs to hexadecimal values
2023-03-28 7:21 ` Jimmy Assarsson
@ 2023-03-29 8:55 ` Oliver Hartkopp
0 siblings, 0 replies; 5+ messages in thread
From: Oliver Hartkopp @ 2023-03-29 8:55 UTC (permalink / raw)
To: Jimmy Assarsson, linux-can
On 28.03.23 09:21, Jimmy Assarsson wrote:
> On 3/27/23 19:53, Oliver Hartkopp wrote:
>> USB IDs are usually represented in 16 bit hexadecimal values.
>> To match the common representation in lsusb and for searching USB IDs
>> in the internet convert the decimal values to hexadecimal.
>
> Thanks for the patch!
>
> I'm about drop the aligned block indentation, in favour of consistent
> one space indentation, for the entire driver. Feel free to update the
> patch, or I'll fix it in a separate patch later.
>
As I will send a V2 with lowercase hex letters anyway I can directly
replace the block indentation with a single space.
> Acked-by: Jimmy Assarsson <extja@kvaser.com>
Many thanks,
Oliver
>
> Best regards,
> jimmy
>
>> Signed-off-by: Oliver Hartkopp <socketcan@hartkopp.net>
>> Cc: Jimmy Assarsson <extja@kvaser.com>
>> ---
>> .../net/can/usb/kvaser_usb/kvaser_usb_core.c | 102 +++++++++---------
>> 1 file changed, 51 insertions(+), 51 deletions(-)
>>
>> diff --git a/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
>> b/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
>> index d4c5356d5884..d0015f83f924 100644
>> --- a/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
>> +++ b/drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c
>> @@ -29,67 +29,67 @@
>> #include <linux/can/netlink.h>
>> #include "kvaser_usb.h"
>> /* Kvaser USB vendor id. */
>> -#define KVASER_VENDOR_ID 0x0bfd
>> +#define KVASER_VENDOR_ID 0x0BFD
>> /* Kvaser Leaf USB devices product ids */
>> -#define USB_LEAF_DEVEL_PRODUCT_ID 10
>> -#define USB_LEAF_LITE_PRODUCT_ID 11
>> -#define USB_LEAF_PRO_PRODUCT_ID 12
>> -#define USB_LEAF_SPRO_PRODUCT_ID 14
>> -#define USB_LEAF_PRO_LS_PRODUCT_ID 15
>> -#define USB_LEAF_PRO_SWC_PRODUCT_ID 16
>> -#define USB_LEAF_PRO_LIN_PRODUCT_ID 17
>> -#define USB_LEAF_SPRO_LS_PRODUCT_ID 18
>> -#define USB_LEAF_SPRO_SWC_PRODUCT_ID 19
>> -#define USB_MEMO2_DEVEL_PRODUCT_ID 22
>> -#define USB_MEMO2_HSHS_PRODUCT_ID 23
>> -#define USB_UPRO_HSHS_PRODUCT_ID 24
>> -#define USB_LEAF_LITE_GI_PRODUCT_ID 25
>> -#define USB_LEAF_PRO_OBDII_PRODUCT_ID 26
>> -#define USB_MEMO2_HSLS_PRODUCT_ID 27
>> -#define USB_LEAF_LITE_CH_PRODUCT_ID 28
>> -#define USB_BLACKBIRD_SPRO_PRODUCT_ID 29
>> -#define USB_OEM_MERCURY_PRODUCT_ID 34
>> -#define USB_OEM_LEAF_PRODUCT_ID 35
>> -#define USB_CAN_R_PRODUCT_ID 39
>> -#define USB_LEAF_LITE_V2_PRODUCT_ID 288
>> -#define USB_MINI_PCIE_HS_PRODUCT_ID 289
>> -#define USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID 290
>> -#define USB_USBCAN_LIGHT_2HS_PRODUCT_ID 291
>> -#define USB_MINI_PCIE_2HS_PRODUCT_ID 292
>> -#define USB_USBCAN_R_V2_PRODUCT_ID 294
>> -#define USB_LEAF_LIGHT_R_V2_PRODUCT_ID 295
>> -#define USB_LEAF_LIGHT_HS_V2_OEM2_PRODUCT_ID 296
>> +#define USB_LEAF_DEVEL_PRODUCT_ID 0x000A
>> +#define USB_LEAF_LITE_PRODUCT_ID 0x000B
>> +#define USB_LEAF_PRO_PRODUCT_ID 0x000C
>> +#define USB_LEAF_SPRO_PRODUCT_ID 0x000E
>> +#define USB_LEAF_PRO_LS_PRODUCT_ID 0x000F
>> +#define USB_LEAF_PRO_SWC_PRODUCT_ID 0x0010
>> +#define USB_LEAF_PRO_LIN_PRODUCT_ID 0x0011
>> +#define USB_LEAF_SPRO_LS_PRODUCT_ID 0x0012
>> +#define USB_LEAF_SPRO_SWC_PRODUCT_ID 0x0013
>> +#define USB_MEMO2_DEVEL_PRODUCT_ID 0x0016
>> +#define USB_MEMO2_HSHS_PRODUCT_ID 0x0017
>> +#define USB_UPRO_HSHS_PRODUCT_ID 0x0018
>> +#define USB_LEAF_LITE_GI_PRODUCT_ID 0x0019
>> +#define USB_LEAF_PRO_OBDII_PRODUCT_ID 0x001A
>> +#define USB_MEMO2_HSLS_PRODUCT_ID 0x001B
>> +#define USB_LEAF_LITE_CH_PRODUCT_ID 0x001C
>> +#define USB_BLACKBIRD_SPRO_PRODUCT_ID 0x001D
>> +#define USB_OEM_MERCURY_PRODUCT_ID 0x0022
>> +#define USB_OEM_LEAF_PRODUCT_ID 0x0023
>> +#define USB_CAN_R_PRODUCT_ID 0x0027
>> +#define USB_LEAF_LITE_V2_PRODUCT_ID 0x0120
>> +#define USB_MINI_PCIE_HS_PRODUCT_ID 0x0121
>> +#define USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID 0x0122
>> +#define USB_USBCAN_LIGHT_2HS_PRODUCT_ID 0x0123
>> +#define USB_MINI_PCIE_2HS_PRODUCT_ID 0x0124
>> +#define USB_USBCAN_R_V2_PRODUCT_ID 0x0126
>> +#define USB_LEAF_LIGHT_R_V2_PRODUCT_ID 0x0127
>> +#define USB_LEAF_LIGHT_HS_V2_OEM2_PRODUCT_ID 0x0128
>> /* Kvaser USBCan-II devices product ids */
>> -#define USB_USBCAN_REVB_PRODUCT_ID 2
>> -#define USB_VCI2_PRODUCT_ID 3
>> -#define USB_USBCAN2_PRODUCT_ID 4
>> -#define USB_MEMORATOR_PRODUCT_ID 5
>> +#define USB_USBCAN_REVB_PRODUCT_ID 0x0002
>> +#define USB_VCI2_PRODUCT_ID 0x0003
>> +#define USB_USBCAN2_PRODUCT_ID 0x0004
>> +#define USB_MEMORATOR_PRODUCT_ID 0x0005
>> /* Kvaser Minihydra USB devices product ids */
>> -#define USB_BLACKBIRD_V2_PRODUCT_ID 258
>> -#define USB_MEMO_PRO_5HS_PRODUCT_ID 260
>> -#define USB_USBCAN_PRO_5HS_PRODUCT_ID 261
>> -#define USB_USBCAN_LIGHT_4HS_PRODUCT_ID 262
>> -#define USB_LEAF_PRO_HS_V2_PRODUCT_ID 263
>> -#define USB_USBCAN_PRO_2HS_V2_PRODUCT_ID 264
>> -#define USB_MEMO_2HS_PRODUCT_ID 265
>> -#define USB_MEMO_PRO_2HS_V2_PRODUCT_ID 266
>> -#define USB_HYBRID_2CANLIN_PRODUCT_ID 267
>> -#define USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID 268
>> -#define USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID 269
>> -#define USB_HYBRID_PRO_2CANLIN_PRODUCT_ID 270
>> -#define USB_U100_PRODUCT_ID 273
>> -#define USB_U100P_PRODUCT_ID 274
>> -#define USB_U100S_PRODUCT_ID 275
>> -#define USB_USBCAN_PRO_4HS_PRODUCT_ID 276
>> -#define USB_HYBRID_CANLIN_PRODUCT_ID 277
>> -#define USB_HYBRID_PRO_CANLIN_PRODUCT_ID 278
>> +#define USB_BLACKBIRD_V2_PRODUCT_ID 0x0102
>> +#define USB_MEMO_PRO_5HS_PRODUCT_ID 0x0104
>> +#define USB_USBCAN_PRO_5HS_PRODUCT_ID 0x0105
>> +#define USB_USBCAN_LIGHT_4HS_PRODUCT_ID 0x0106
>> +#define USB_LEAF_PRO_HS_V2_PRODUCT_ID 0x0107
>> +#define USB_USBCAN_PRO_2HS_V2_PRODUCT_ID 0x0108
>> +#define USB_MEMO_2HS_PRODUCT_ID 0x0109
>> +#define USB_MEMO_PRO_2HS_V2_PRODUCT_ID 0x010A
>> +#define USB_HYBRID_2CANLIN_PRODUCT_ID 0x010B
>> +#define USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID 0x010C
>> +#define USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID 0x010D
>> +#define USB_HYBRID_PRO_2CANLIN_PRODUCT_ID 0x010E
>> +#define USB_U100_PRODUCT_ID 0x0111
>> +#define USB_U100P_PRODUCT_ID 0x0112
>> +#define USB_U100S_PRODUCT_ID 0x0113
>> +#define USB_USBCAN_PRO_4HS_PRODUCT_ID 0x0114
>> +#define USB_HYBRID_CANLIN_PRODUCT_ID 0x0115
>> +#define USB_HYBRID_PRO_CANLIN_PRODUCT_ID 0x0116
>> static const struct kvaser_usb_driver_info
>> kvaser_usb_driver_info_hydra = {
>> .quirks = KVASER_USB_QUIRK_HAS_HARDWARE_TIMESTAMP,
>> .ops = &kvaser_usb_hydra_dev_ops,
>> };
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-03-29 9:00 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-27 17:53 [PATCH can-next] kvaser_usb: convert USB IDs to hexadecimal values Oliver Hartkopp
2023-03-28 7:21 ` Jimmy Assarsson
2023-03-29 8:55 ` Oliver Hartkopp
2023-03-28 8:46 ` Alexander Dahl
2023-03-29 8:53 ` Oliver Hartkopp
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.