| .. | .. |
|---|
| 79 | 79 | #define USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID 269 |
|---|
| 80 | 80 | #define USB_HYBRID_PRO_CANLIN_PRODUCT_ID 270 |
|---|
| 81 | 81 | |
|---|
| 82 | | -static inline bool kvaser_is_leaf(const struct usb_device_id *id) |
|---|
| 83 | | -{ |
|---|
| 84 | | - return (id->idProduct >= USB_LEAF_DEVEL_PRODUCT_ID && |
|---|
| 85 | | - id->idProduct <= USB_CAN_R_PRODUCT_ID) || |
|---|
| 86 | | - (id->idProduct >= USB_LEAF_LITE_V2_PRODUCT_ID && |
|---|
| 87 | | - id->idProduct <= USB_MINI_PCIE_2HS_PRODUCT_ID); |
|---|
| 88 | | -} |
|---|
| 82 | +static const struct kvaser_usb_driver_info kvaser_usb_driver_info_hydra = { |
|---|
| 83 | + .quirks = 0, |
|---|
| 84 | + .ops = &kvaser_usb_hydra_dev_ops, |
|---|
| 85 | +}; |
|---|
| 89 | 86 | |
|---|
| 90 | | -static inline bool kvaser_is_usbcan(const struct usb_device_id *id) |
|---|
| 91 | | -{ |
|---|
| 92 | | - return id->idProduct >= USB_USBCAN_REVB_PRODUCT_ID && |
|---|
| 93 | | - id->idProduct <= USB_MEMORATOR_PRODUCT_ID; |
|---|
| 94 | | -} |
|---|
| 87 | +static const struct kvaser_usb_driver_info kvaser_usb_driver_info_usbcan = { |
|---|
| 88 | + .quirks = KVASER_USB_QUIRK_HAS_TXRX_ERRORS | |
|---|
| 89 | + KVASER_USB_QUIRK_HAS_SILENT_MODE, |
|---|
| 90 | + .family = KVASER_USBCAN, |
|---|
| 91 | + .ops = &kvaser_usb_leaf_dev_ops, |
|---|
| 92 | +}; |
|---|
| 95 | 93 | |
|---|
| 96 | | -static inline bool kvaser_is_hydra(const struct usb_device_id *id) |
|---|
| 97 | | -{ |
|---|
| 98 | | - return id->idProduct >= USB_BLACKBIRD_V2_PRODUCT_ID && |
|---|
| 99 | | - id->idProduct <= USB_HYBRID_PRO_CANLIN_PRODUCT_ID; |
|---|
| 100 | | -} |
|---|
| 94 | +static const struct kvaser_usb_driver_info kvaser_usb_driver_info_leaf = { |
|---|
| 95 | + .quirks = KVASER_USB_QUIRK_IGNORE_CLK_FREQ, |
|---|
| 96 | + .family = KVASER_LEAF, |
|---|
| 97 | + .ops = &kvaser_usb_leaf_dev_ops, |
|---|
| 98 | +}; |
|---|
| 99 | + |
|---|
| 100 | +static const struct kvaser_usb_driver_info kvaser_usb_driver_info_leaf_err = { |
|---|
| 101 | + .quirks = KVASER_USB_QUIRK_HAS_TXRX_ERRORS | |
|---|
| 102 | + KVASER_USB_QUIRK_IGNORE_CLK_FREQ, |
|---|
| 103 | + .family = KVASER_LEAF, |
|---|
| 104 | + .ops = &kvaser_usb_leaf_dev_ops, |
|---|
| 105 | +}; |
|---|
| 106 | + |
|---|
| 107 | +static const struct kvaser_usb_driver_info kvaser_usb_driver_info_leaf_err_listen = { |
|---|
| 108 | + .quirks = KVASER_USB_QUIRK_HAS_TXRX_ERRORS | |
|---|
| 109 | + KVASER_USB_QUIRK_HAS_SILENT_MODE | |
|---|
| 110 | + KVASER_USB_QUIRK_IGNORE_CLK_FREQ, |
|---|
| 111 | + .family = KVASER_LEAF, |
|---|
| 112 | + .ops = &kvaser_usb_leaf_dev_ops, |
|---|
| 113 | +}; |
|---|
| 114 | + |
|---|
| 115 | +static const struct kvaser_usb_driver_info kvaser_usb_driver_info_leafimx = { |
|---|
| 116 | + .quirks = 0, |
|---|
| 117 | + .ops = &kvaser_usb_leaf_dev_ops, |
|---|
| 118 | +}; |
|---|
| 101 | 119 | |
|---|
| 102 | 120 | static const struct usb_device_id kvaser_usb_table[] = { |
|---|
| 103 | | - /* Leaf USB product IDs */ |
|---|
| 104 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_DEVEL_PRODUCT_ID) }, |
|---|
| 105 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_LITE_PRODUCT_ID) }, |
|---|
| 121 | + /* Leaf M32C USB product IDs */ |
|---|
| 122 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_DEVEL_PRODUCT_ID), |
|---|
| 123 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf }, |
|---|
| 124 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_LITE_PRODUCT_ID), |
|---|
| 125 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf }, |
|---|
| 106 | 126 | { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_PRO_PRODUCT_ID), |
|---|
| 107 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS | |
|---|
| 108 | | - KVASER_USB_HAS_SILENT_MODE }, |
|---|
| 127 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err_listen }, |
|---|
| 109 | 128 | { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_SPRO_PRODUCT_ID), |
|---|
| 110 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS | |
|---|
| 111 | | - KVASER_USB_HAS_SILENT_MODE }, |
|---|
| 129 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err_listen }, |
|---|
| 112 | 130 | { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_PRO_LS_PRODUCT_ID), |
|---|
| 113 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS | |
|---|
| 114 | | - KVASER_USB_HAS_SILENT_MODE }, |
|---|
| 131 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err_listen }, |
|---|
| 115 | 132 | { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_PRO_SWC_PRODUCT_ID), |
|---|
| 116 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS | |
|---|
| 117 | | - KVASER_USB_HAS_SILENT_MODE }, |
|---|
| 133 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err_listen }, |
|---|
| 118 | 134 | { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_PRO_LIN_PRODUCT_ID), |
|---|
| 119 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS | |
|---|
| 120 | | - KVASER_USB_HAS_SILENT_MODE }, |
|---|
| 135 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err_listen }, |
|---|
| 121 | 136 | { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_SPRO_LS_PRODUCT_ID), |
|---|
| 122 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS | |
|---|
| 123 | | - KVASER_USB_HAS_SILENT_MODE }, |
|---|
| 137 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err_listen }, |
|---|
| 124 | 138 | { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_SPRO_SWC_PRODUCT_ID), |
|---|
| 125 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS | |
|---|
| 126 | | - KVASER_USB_HAS_SILENT_MODE }, |
|---|
| 139 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err_listen }, |
|---|
| 127 | 140 | { USB_DEVICE(KVASER_VENDOR_ID, USB_MEMO2_DEVEL_PRODUCT_ID), |
|---|
| 128 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS | |
|---|
| 129 | | - KVASER_USB_HAS_SILENT_MODE }, |
|---|
| 141 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err_listen }, |
|---|
| 130 | 142 | { USB_DEVICE(KVASER_VENDOR_ID, USB_MEMO2_HSHS_PRODUCT_ID), |
|---|
| 131 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS | |
|---|
| 132 | | - KVASER_USB_HAS_SILENT_MODE }, |
|---|
| 143 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err_listen }, |
|---|
| 133 | 144 | { USB_DEVICE(KVASER_VENDOR_ID, USB_UPRO_HSHS_PRODUCT_ID), |
|---|
| 134 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS }, |
|---|
| 135 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_LITE_GI_PRODUCT_ID) }, |
|---|
| 145 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err }, |
|---|
| 146 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_LITE_GI_PRODUCT_ID), |
|---|
| 147 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf }, |
|---|
| 136 | 148 | { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_PRO_OBDII_PRODUCT_ID), |
|---|
| 137 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS | |
|---|
| 138 | | - KVASER_USB_HAS_SILENT_MODE }, |
|---|
| 149 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err_listen }, |
|---|
| 139 | 150 | { USB_DEVICE(KVASER_VENDOR_ID, USB_MEMO2_HSLS_PRODUCT_ID), |
|---|
| 140 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS }, |
|---|
| 151 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err }, |
|---|
| 141 | 152 | { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_LITE_CH_PRODUCT_ID), |
|---|
| 142 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS }, |
|---|
| 153 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err }, |
|---|
| 143 | 154 | { USB_DEVICE(KVASER_VENDOR_ID, USB_BLACKBIRD_SPRO_PRODUCT_ID), |
|---|
| 144 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS }, |
|---|
| 155 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err }, |
|---|
| 145 | 156 | { USB_DEVICE(KVASER_VENDOR_ID, USB_OEM_MERCURY_PRODUCT_ID), |
|---|
| 146 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS }, |
|---|
| 157 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err }, |
|---|
| 147 | 158 | { USB_DEVICE(KVASER_VENDOR_ID, USB_OEM_LEAF_PRODUCT_ID), |
|---|
| 148 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS }, |
|---|
| 159 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err }, |
|---|
| 149 | 160 | { USB_DEVICE(KVASER_VENDOR_ID, USB_CAN_R_PRODUCT_ID), |
|---|
| 150 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS }, |
|---|
| 151 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_LITE_V2_PRODUCT_ID) }, |
|---|
| 152 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_MINI_PCIE_HS_PRODUCT_ID) }, |
|---|
| 153 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID) }, |
|---|
| 154 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_USBCAN_LIGHT_2HS_PRODUCT_ID) }, |
|---|
| 155 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_MINI_PCIE_2HS_PRODUCT_ID) }, |
|---|
| 161 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leaf_err }, |
|---|
| 162 | + |
|---|
| 163 | + /* Leaf i.MX28 USB product IDs */ |
|---|
| 164 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_LITE_V2_PRODUCT_ID), |
|---|
| 165 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leafimx }, |
|---|
| 166 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_MINI_PCIE_HS_PRODUCT_ID), |
|---|
| 167 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leafimx }, |
|---|
| 168 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_LIGHT_HS_V2_OEM_PRODUCT_ID), |
|---|
| 169 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leafimx }, |
|---|
| 170 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_USBCAN_LIGHT_2HS_PRODUCT_ID), |
|---|
| 171 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leafimx }, |
|---|
| 172 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_MINI_PCIE_2HS_PRODUCT_ID), |
|---|
| 173 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_leafimx }, |
|---|
| 156 | 174 | |
|---|
| 157 | 175 | /* USBCANII USB product IDs */ |
|---|
| 158 | 176 | { USB_DEVICE(KVASER_VENDOR_ID, USB_USBCAN2_PRODUCT_ID), |
|---|
| 159 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS }, |
|---|
| 177 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_usbcan }, |
|---|
| 160 | 178 | { USB_DEVICE(KVASER_VENDOR_ID, USB_USBCAN_REVB_PRODUCT_ID), |
|---|
| 161 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS }, |
|---|
| 179 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_usbcan }, |
|---|
| 162 | 180 | { USB_DEVICE(KVASER_VENDOR_ID, USB_MEMORATOR_PRODUCT_ID), |
|---|
| 163 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS }, |
|---|
| 181 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_usbcan }, |
|---|
| 164 | 182 | { USB_DEVICE(KVASER_VENDOR_ID, USB_VCI2_PRODUCT_ID), |
|---|
| 165 | | - .driver_info = KVASER_USB_HAS_TXRX_ERRORS }, |
|---|
| 183 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_usbcan }, |
|---|
| 166 | 184 | |
|---|
| 167 | 185 | /* Minihydra USB product IDs */ |
|---|
| 168 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_BLACKBIRD_V2_PRODUCT_ID) }, |
|---|
| 169 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_MEMO_PRO_5HS_PRODUCT_ID) }, |
|---|
| 170 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_USBCAN_PRO_5HS_PRODUCT_ID) }, |
|---|
| 171 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_USBCAN_LIGHT_4HS_PRODUCT_ID) }, |
|---|
| 172 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_PRO_HS_V2_PRODUCT_ID) }, |
|---|
| 173 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_USBCAN_PRO_2HS_V2_PRODUCT_ID) }, |
|---|
| 174 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_MEMO_2HS_PRODUCT_ID) }, |
|---|
| 175 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_MEMO_PRO_2HS_V2_PRODUCT_ID) }, |
|---|
| 176 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_HYBRID_CANLIN_PRODUCT_ID) }, |
|---|
| 177 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID) }, |
|---|
| 178 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID) }, |
|---|
| 179 | | - { USB_DEVICE(KVASER_VENDOR_ID, USB_HYBRID_PRO_CANLIN_PRODUCT_ID) }, |
|---|
| 186 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_BLACKBIRD_V2_PRODUCT_ID), |
|---|
| 187 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 188 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_MEMO_PRO_5HS_PRODUCT_ID), |
|---|
| 189 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 190 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_USBCAN_PRO_5HS_PRODUCT_ID), |
|---|
| 191 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 192 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_USBCAN_LIGHT_4HS_PRODUCT_ID), |
|---|
| 193 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 194 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_LEAF_PRO_HS_V2_PRODUCT_ID), |
|---|
| 195 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 196 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_USBCAN_PRO_2HS_V2_PRODUCT_ID), |
|---|
| 197 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 198 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_MEMO_2HS_PRODUCT_ID), |
|---|
| 199 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 200 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_MEMO_PRO_2HS_V2_PRODUCT_ID), |
|---|
| 201 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 202 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_HYBRID_CANLIN_PRODUCT_ID), |
|---|
| 203 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 204 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_ATI_USBCAN_PRO_2HS_V2_PRODUCT_ID), |
|---|
| 205 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 206 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_ATI_MEMO_PRO_2HS_V2_PRODUCT_ID), |
|---|
| 207 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 208 | + { USB_DEVICE(KVASER_VENDOR_ID, USB_HYBRID_PRO_CANLIN_PRODUCT_ID), |
|---|
| 209 | + .driver_info = (kernel_ulong_t)&kvaser_usb_driver_info_hydra }, |
|---|
| 180 | 210 | { } |
|---|
| 181 | 211 | }; |
|---|
| 182 | 212 | MODULE_DEVICE_TABLE(usb, kvaser_usb_table); |
|---|
| .. | .. |
|---|
| 267 | 297 | static void kvaser_usb_read_bulk_callback(struct urb *urb) |
|---|
| 268 | 298 | { |
|---|
| 269 | 299 | struct kvaser_usb *dev = urb->context; |
|---|
| 300 | + const struct kvaser_usb_dev_ops *ops = dev->driver_info->ops; |
|---|
| 270 | 301 | int err; |
|---|
| 271 | 302 | unsigned int i; |
|---|
| 272 | 303 | |
|---|
| .. | .. |
|---|
| 283 | 314 | goto resubmit_urb; |
|---|
| 284 | 315 | } |
|---|
| 285 | 316 | |
|---|
| 286 | | - dev->ops->dev_read_bulk_callback(dev, urb->transfer_buffer, |
|---|
| 287 | | - urb->actual_length); |
|---|
| 317 | + ops->dev_read_bulk_callback(dev, urb->transfer_buffer, |
|---|
| 318 | + urb->actual_length); |
|---|
| 288 | 319 | |
|---|
| 289 | 320 | resubmit_urb: |
|---|
| 290 | 321 | usb_fill_bulk_urb(urb, dev->udev, |
|---|
| .. | .. |
|---|
| 378 | 409 | { |
|---|
| 379 | 410 | struct kvaser_usb_net_priv *priv = netdev_priv(netdev); |
|---|
| 380 | 411 | struct kvaser_usb *dev = priv->dev; |
|---|
| 412 | + const struct kvaser_usb_dev_ops *ops = dev->driver_info->ops; |
|---|
| 381 | 413 | int err; |
|---|
| 382 | 414 | |
|---|
| 383 | 415 | err = open_candev(netdev); |
|---|
| 384 | 416 | if (err) |
|---|
| 385 | 417 | return err; |
|---|
| 386 | 418 | |
|---|
| 387 | | - err = kvaser_usb_setup_rx_urbs(dev); |
|---|
| 419 | + err = ops->dev_set_opt_mode(priv); |
|---|
| 388 | 420 | if (err) |
|---|
| 389 | 421 | goto error; |
|---|
| 390 | 422 | |
|---|
| 391 | | - err = dev->ops->dev_set_opt_mode(priv); |
|---|
| 392 | | - if (err) |
|---|
| 393 | | - goto error; |
|---|
| 394 | | - |
|---|
| 395 | | - err = dev->ops->dev_start_chip(priv); |
|---|
| 423 | + err = ops->dev_start_chip(priv); |
|---|
| 396 | 424 | if (err) { |
|---|
| 397 | 425 | netdev_warn(netdev, "Cannot start device, error %d\n", err); |
|---|
| 398 | 426 | goto error; |
|---|
| .. | .. |
|---|
| 421 | 449 | /* This method might sleep. Do not call it in the atomic context |
|---|
| 422 | 450 | * of URB completions. |
|---|
| 423 | 451 | */ |
|---|
| 424 | | -static void kvaser_usb_unlink_tx_urbs(struct kvaser_usb_net_priv *priv) |
|---|
| 452 | +void kvaser_usb_unlink_tx_urbs(struct kvaser_usb_net_priv *priv) |
|---|
| 425 | 453 | { |
|---|
| 426 | 454 | usb_kill_anchored_urbs(&priv->tx_submitted); |
|---|
| 427 | 455 | kvaser_usb_reset_tx_urb_contexts(priv); |
|---|
| .. | .. |
|---|
| 449 | 477 | { |
|---|
| 450 | 478 | struct kvaser_usb_net_priv *priv = netdev_priv(netdev); |
|---|
| 451 | 479 | struct kvaser_usb *dev = priv->dev; |
|---|
| 480 | + const struct kvaser_usb_dev_ops *ops = dev->driver_info->ops; |
|---|
| 452 | 481 | int err; |
|---|
| 453 | 482 | |
|---|
| 454 | 483 | netif_stop_queue(netdev); |
|---|
| 455 | 484 | |
|---|
| 456 | | - err = dev->ops->dev_flush_queue(priv); |
|---|
| 485 | + err = ops->dev_flush_queue(priv); |
|---|
| 457 | 486 | if (err) |
|---|
| 458 | 487 | netdev_warn(netdev, "Cannot flush queue, error %d\n", err); |
|---|
| 459 | 488 | |
|---|
| 460 | | - if (dev->ops->dev_reset_chip) { |
|---|
| 461 | | - err = dev->ops->dev_reset_chip(dev, priv->channel); |
|---|
| 489 | + if (ops->dev_reset_chip) { |
|---|
| 490 | + err = ops->dev_reset_chip(dev, priv->channel); |
|---|
| 462 | 491 | if (err) |
|---|
| 463 | 492 | netdev_warn(netdev, "Cannot reset card, error %d\n", |
|---|
| 464 | 493 | err); |
|---|
| 465 | 494 | } |
|---|
| 466 | 495 | |
|---|
| 467 | | - err = dev->ops->dev_stop_chip(priv); |
|---|
| 496 | + err = ops->dev_stop_chip(priv); |
|---|
| 468 | 497 | if (err) |
|---|
| 469 | 498 | netdev_warn(netdev, "Cannot stop device, error %d\n", err); |
|---|
| 470 | 499 | |
|---|
| .. | .. |
|---|
| 475 | 504 | close_candev(priv->netdev); |
|---|
| 476 | 505 | |
|---|
| 477 | 506 | return 0; |
|---|
| 507 | +} |
|---|
| 508 | + |
|---|
| 509 | +static int kvaser_usb_set_bittiming(struct net_device *netdev) |
|---|
| 510 | +{ |
|---|
| 511 | + struct kvaser_usb_net_priv *priv = netdev_priv(netdev); |
|---|
| 512 | + struct kvaser_usb *dev = priv->dev; |
|---|
| 513 | + const struct kvaser_usb_dev_ops *ops = dev->driver_info->ops; |
|---|
| 514 | + struct can_bittiming *bt = &priv->can.bittiming; |
|---|
| 515 | + |
|---|
| 516 | + struct kvaser_usb_busparams busparams; |
|---|
| 517 | + int tseg1 = bt->prop_seg + bt->phase_seg1; |
|---|
| 518 | + int tseg2 = bt->phase_seg2; |
|---|
| 519 | + int sjw = bt->sjw; |
|---|
| 520 | + int err = -EOPNOTSUPP; |
|---|
| 521 | + |
|---|
| 522 | + busparams.bitrate = cpu_to_le32(bt->bitrate); |
|---|
| 523 | + busparams.sjw = (u8)sjw; |
|---|
| 524 | + busparams.tseg1 = (u8)tseg1; |
|---|
| 525 | + busparams.tseg2 = (u8)tseg2; |
|---|
| 526 | + if (priv->can.ctrlmode & CAN_CTRLMODE_3_SAMPLES) |
|---|
| 527 | + busparams.nsamples = 3; |
|---|
| 528 | + else |
|---|
| 529 | + busparams.nsamples = 1; |
|---|
| 530 | + |
|---|
| 531 | + err = ops->dev_set_bittiming(netdev, &busparams); |
|---|
| 532 | + if (err) |
|---|
| 533 | + return err; |
|---|
| 534 | + |
|---|
| 535 | + err = kvaser_usb_setup_rx_urbs(priv->dev); |
|---|
| 536 | + if (err) |
|---|
| 537 | + return err; |
|---|
| 538 | + |
|---|
| 539 | + err = ops->dev_get_busparams(priv); |
|---|
| 540 | + if (err) { |
|---|
| 541 | + /* Treat EOPNOTSUPP as success */ |
|---|
| 542 | + if (err == -EOPNOTSUPP) |
|---|
| 543 | + err = 0; |
|---|
| 544 | + return err; |
|---|
| 545 | + } |
|---|
| 546 | + |
|---|
| 547 | + if (memcmp(&busparams, &priv->busparams_nominal, |
|---|
| 548 | + sizeof(priv->busparams_nominal)) != 0) |
|---|
| 549 | + err = -EINVAL; |
|---|
| 550 | + |
|---|
| 551 | + return err; |
|---|
| 552 | +} |
|---|
| 553 | + |
|---|
| 554 | +static int kvaser_usb_set_data_bittiming(struct net_device *netdev) |
|---|
| 555 | +{ |
|---|
| 556 | + struct kvaser_usb_net_priv *priv = netdev_priv(netdev); |
|---|
| 557 | + struct kvaser_usb *dev = priv->dev; |
|---|
| 558 | + const struct kvaser_usb_dev_ops *ops = dev->driver_info->ops; |
|---|
| 559 | + struct can_bittiming *dbt = &priv->can.data_bittiming; |
|---|
| 560 | + |
|---|
| 561 | + struct kvaser_usb_busparams busparams; |
|---|
| 562 | + int tseg1 = dbt->prop_seg + dbt->phase_seg1; |
|---|
| 563 | + int tseg2 = dbt->phase_seg2; |
|---|
| 564 | + int sjw = dbt->sjw; |
|---|
| 565 | + int err; |
|---|
| 566 | + |
|---|
| 567 | + if (!ops->dev_set_data_bittiming || |
|---|
| 568 | + !ops->dev_get_data_busparams) |
|---|
| 569 | + return -EOPNOTSUPP; |
|---|
| 570 | + |
|---|
| 571 | + busparams.bitrate = cpu_to_le32(dbt->bitrate); |
|---|
| 572 | + busparams.sjw = (u8)sjw; |
|---|
| 573 | + busparams.tseg1 = (u8)tseg1; |
|---|
| 574 | + busparams.tseg2 = (u8)tseg2; |
|---|
| 575 | + busparams.nsamples = 1; |
|---|
| 576 | + |
|---|
| 577 | + err = ops->dev_set_data_bittiming(netdev, &busparams); |
|---|
| 578 | + if (err) |
|---|
| 579 | + return err; |
|---|
| 580 | + |
|---|
| 581 | + err = kvaser_usb_setup_rx_urbs(priv->dev); |
|---|
| 582 | + if (err) |
|---|
| 583 | + return err; |
|---|
| 584 | + |
|---|
| 585 | + err = ops->dev_get_data_busparams(priv); |
|---|
| 586 | + if (err) |
|---|
| 587 | + return err; |
|---|
| 588 | + |
|---|
| 589 | + if (memcmp(&busparams, &priv->busparams_data, |
|---|
| 590 | + sizeof(priv->busparams_data)) != 0) |
|---|
| 591 | + err = -EINVAL; |
|---|
| 592 | + |
|---|
| 593 | + return err; |
|---|
| 478 | 594 | } |
|---|
| 479 | 595 | |
|---|
| 480 | 596 | static void kvaser_usb_write_bulk_callback(struct urb *urb) |
|---|
| .. | .. |
|---|
| 503 | 619 | { |
|---|
| 504 | 620 | struct kvaser_usb_net_priv *priv = netdev_priv(netdev); |
|---|
| 505 | 621 | struct kvaser_usb *dev = priv->dev; |
|---|
| 622 | + const struct kvaser_usb_dev_ops *ops = dev->driver_info->ops; |
|---|
| 506 | 623 | struct net_device_stats *stats = &netdev->stats; |
|---|
| 507 | 624 | struct kvaser_usb_tx_urb_context *context = NULL; |
|---|
| 508 | 625 | struct urb *urb; |
|---|
| .. | .. |
|---|
| 545 | 662 | goto freeurb; |
|---|
| 546 | 663 | } |
|---|
| 547 | 664 | |
|---|
| 548 | | - buf = dev->ops->dev_frame_to_cmd(priv, skb, &context->dlc, &cmd_len, |
|---|
| 549 | | - context->echo_index); |
|---|
| 665 | + buf = ops->dev_frame_to_cmd(priv, skb, &context->dlc, &cmd_len, |
|---|
| 666 | + context->echo_index); |
|---|
| 550 | 667 | if (!buf) { |
|---|
| 551 | 668 | stats->tx_dropped++; |
|---|
| 552 | 669 | dev_kfree_skb(skb); |
|---|
| .. | .. |
|---|
| 611 | 728 | |
|---|
| 612 | 729 | static void kvaser_usb_remove_interfaces(struct kvaser_usb *dev) |
|---|
| 613 | 730 | { |
|---|
| 731 | + const struct kvaser_usb_dev_ops *ops = dev->driver_info->ops; |
|---|
| 614 | 732 | int i; |
|---|
| 615 | 733 | |
|---|
| 616 | 734 | for (i = 0; i < dev->nchannels; i++) { |
|---|
| .. | .. |
|---|
| 626 | 744 | if (!dev->nets[i]) |
|---|
| 627 | 745 | continue; |
|---|
| 628 | 746 | |
|---|
| 747 | + if (ops->dev_remove_channel) |
|---|
| 748 | + ops->dev_remove_channel(dev->nets[i]); |
|---|
| 749 | + |
|---|
| 629 | 750 | free_candev(dev->nets[i]->netdev); |
|---|
| 630 | 751 | } |
|---|
| 631 | 752 | } |
|---|
| 632 | 753 | |
|---|
| 633 | | -static int kvaser_usb_init_one(struct kvaser_usb *dev, |
|---|
| 634 | | - const struct usb_device_id *id, int channel) |
|---|
| 754 | +static int kvaser_usb_init_one(struct kvaser_usb *dev, int channel) |
|---|
| 635 | 755 | { |
|---|
| 636 | 756 | struct net_device *netdev; |
|---|
| 637 | 757 | struct kvaser_usb_net_priv *priv; |
|---|
| 758 | + const struct kvaser_usb_driver_info *driver_info = dev->driver_info; |
|---|
| 759 | + const struct kvaser_usb_dev_ops *ops = driver_info->ops; |
|---|
| 638 | 760 | int err; |
|---|
| 639 | 761 | |
|---|
| 640 | | - if (dev->ops->dev_reset_chip) { |
|---|
| 641 | | - err = dev->ops->dev_reset_chip(dev, channel); |
|---|
| 762 | + if (ops->dev_reset_chip) { |
|---|
| 763 | + err = ops->dev_reset_chip(dev, channel); |
|---|
| 642 | 764 | if (err) |
|---|
| 643 | 765 | return err; |
|---|
| 644 | 766 | } |
|---|
| 645 | 767 | |
|---|
| 646 | | - netdev = alloc_candev(sizeof(*priv) + |
|---|
| 647 | | - dev->max_tx_urbs * sizeof(*priv->tx_contexts), |
|---|
| 768 | + netdev = alloc_candev(struct_size(priv, tx_contexts, dev->max_tx_urbs), |
|---|
| 648 | 769 | dev->max_tx_urbs); |
|---|
| 649 | 770 | if (!netdev) { |
|---|
| 650 | 771 | dev_err(&dev->intf->dev, "Cannot alloc candev\n"); |
|---|
| .. | .. |
|---|
| 656 | 777 | init_usb_anchor(&priv->tx_submitted); |
|---|
| 657 | 778 | init_completion(&priv->start_comp); |
|---|
| 658 | 779 | init_completion(&priv->stop_comp); |
|---|
| 780 | + init_completion(&priv->flush_comp); |
|---|
| 781 | + init_completion(&priv->get_busparams_comp); |
|---|
| 659 | 782 | priv->can.ctrlmode_supported = 0; |
|---|
| 660 | 783 | |
|---|
| 661 | 784 | priv->dev = dev; |
|---|
| .. | .. |
|---|
| 668 | 791 | priv->can.state = CAN_STATE_STOPPED; |
|---|
| 669 | 792 | priv->can.clock.freq = dev->cfg->clock.freq; |
|---|
| 670 | 793 | priv->can.bittiming_const = dev->cfg->bittiming_const; |
|---|
| 671 | | - priv->can.do_set_bittiming = dev->ops->dev_set_bittiming; |
|---|
| 672 | | - priv->can.do_set_mode = dev->ops->dev_set_mode; |
|---|
| 673 | | - if ((id->driver_info & KVASER_USB_HAS_TXRX_ERRORS) || |
|---|
| 794 | + priv->can.do_set_bittiming = kvaser_usb_set_bittiming; |
|---|
| 795 | + priv->can.do_set_mode = ops->dev_set_mode; |
|---|
| 796 | + if ((driver_info->quirks & KVASER_USB_QUIRK_HAS_TXRX_ERRORS) || |
|---|
| 674 | 797 | (priv->dev->card_data.capabilities & KVASER_USB_CAP_BERR_CAP)) |
|---|
| 675 | | - priv->can.do_get_berr_counter = dev->ops->dev_get_berr_counter; |
|---|
| 676 | | - if (id->driver_info & KVASER_USB_HAS_SILENT_MODE) |
|---|
| 798 | + priv->can.do_get_berr_counter = ops->dev_get_berr_counter; |
|---|
| 799 | + if (driver_info->quirks & KVASER_USB_QUIRK_HAS_SILENT_MODE) |
|---|
| 677 | 800 | priv->can.ctrlmode_supported |= CAN_CTRLMODE_LISTENONLY; |
|---|
| 678 | 801 | |
|---|
| 679 | 802 | priv->can.ctrlmode_supported |= dev->card_data.ctrlmode_supported; |
|---|
| 680 | 803 | |
|---|
| 681 | 804 | if (priv->can.ctrlmode_supported & CAN_CTRLMODE_FD) { |
|---|
| 682 | 805 | priv->can.data_bittiming_const = dev->cfg->data_bittiming_const; |
|---|
| 683 | | - priv->can.do_set_data_bittiming = |
|---|
| 684 | | - dev->ops->dev_set_data_bittiming; |
|---|
| 806 | + priv->can.do_set_data_bittiming = kvaser_usb_set_data_bittiming; |
|---|
| 685 | 807 | } |
|---|
| 686 | 808 | |
|---|
| 687 | 809 | netdev->flags |= IFF_ECHO; |
|---|
| .. | .. |
|---|
| 693 | 815 | |
|---|
| 694 | 816 | dev->nets[channel] = priv; |
|---|
| 695 | 817 | |
|---|
| 818 | + if (ops->dev_init_channel) { |
|---|
| 819 | + err = ops->dev_init_channel(priv); |
|---|
| 820 | + if (err) |
|---|
| 821 | + goto err; |
|---|
| 822 | + } |
|---|
| 823 | + |
|---|
| 696 | 824 | err = register_candev(netdev); |
|---|
| 697 | 825 | if (err) { |
|---|
| 698 | 826 | dev_err(&dev->intf->dev, "Failed to register CAN device\n"); |
|---|
| 699 | | - free_candev(netdev); |
|---|
| 700 | | - dev->nets[channel] = NULL; |
|---|
| 701 | | - return err; |
|---|
| 827 | + goto err; |
|---|
| 702 | 828 | } |
|---|
| 703 | 829 | |
|---|
| 704 | 830 | netdev_dbg(netdev, "device registered\n"); |
|---|
| 705 | 831 | |
|---|
| 706 | 832 | return 0; |
|---|
| 833 | + |
|---|
| 834 | +err: |
|---|
| 835 | + free_candev(netdev); |
|---|
| 836 | + dev->nets[channel] = NULL; |
|---|
| 837 | + return err; |
|---|
| 707 | 838 | } |
|---|
| 708 | 839 | |
|---|
| 709 | 840 | static int kvaser_usb_probe(struct usb_interface *intf, |
|---|
| .. | .. |
|---|
| 712 | 843 | struct kvaser_usb *dev; |
|---|
| 713 | 844 | int err; |
|---|
| 714 | 845 | int i; |
|---|
| 846 | + const struct kvaser_usb_driver_info *driver_info; |
|---|
| 847 | + const struct kvaser_usb_dev_ops *ops; |
|---|
| 848 | + |
|---|
| 849 | + driver_info = (const struct kvaser_usb_driver_info *)id->driver_info; |
|---|
| 850 | + if (!driver_info) |
|---|
| 851 | + return -ENODEV; |
|---|
| 715 | 852 | |
|---|
| 716 | 853 | dev = devm_kzalloc(&intf->dev, sizeof(*dev), GFP_KERNEL); |
|---|
| 717 | 854 | if (!dev) |
|---|
| 718 | 855 | return -ENOMEM; |
|---|
| 719 | 856 | |
|---|
| 720 | | - if (kvaser_is_leaf(id)) { |
|---|
| 721 | | - dev->card_data.leaf.family = KVASER_LEAF; |
|---|
| 722 | | - dev->ops = &kvaser_usb_leaf_dev_ops; |
|---|
| 723 | | - } else if (kvaser_is_usbcan(id)) { |
|---|
| 724 | | - dev->card_data.leaf.family = KVASER_USBCAN; |
|---|
| 725 | | - dev->ops = &kvaser_usb_leaf_dev_ops; |
|---|
| 726 | | - } else if (kvaser_is_hydra(id)) { |
|---|
| 727 | | - dev->ops = &kvaser_usb_hydra_dev_ops; |
|---|
| 728 | | - } else { |
|---|
| 729 | | - dev_err(&intf->dev, |
|---|
| 730 | | - "Product ID (%d) is not a supported Kvaser USB device\n", |
|---|
| 731 | | - id->idProduct); |
|---|
| 732 | | - return -ENODEV; |
|---|
| 733 | | - } |
|---|
| 734 | | - |
|---|
| 735 | 857 | dev->intf = intf; |
|---|
| 858 | + dev->driver_info = driver_info; |
|---|
| 859 | + ops = driver_info->ops; |
|---|
| 736 | 860 | |
|---|
| 737 | | - err = dev->ops->dev_setup_endpoints(dev); |
|---|
| 861 | + err = ops->dev_setup_endpoints(dev); |
|---|
| 738 | 862 | if (err) { |
|---|
| 739 | 863 | dev_err(&intf->dev, "Cannot get usb endpoint(s)"); |
|---|
| 740 | 864 | return err; |
|---|
| .. | .. |
|---|
| 748 | 872 | |
|---|
| 749 | 873 | dev->card_data.ctrlmode_supported = 0; |
|---|
| 750 | 874 | dev->card_data.capabilities = 0; |
|---|
| 751 | | - err = dev->ops->dev_init_card(dev); |
|---|
| 875 | + err = ops->dev_init_card(dev); |
|---|
| 752 | 876 | if (err) { |
|---|
| 753 | 877 | dev_err(&intf->dev, |
|---|
| 754 | 878 | "Failed to initialize card, error %d\n", err); |
|---|
| 755 | 879 | return err; |
|---|
| 756 | 880 | } |
|---|
| 757 | 881 | |
|---|
| 758 | | - err = dev->ops->dev_get_software_info(dev); |
|---|
| 882 | + err = ops->dev_get_software_info(dev); |
|---|
| 759 | 883 | if (err) { |
|---|
| 760 | 884 | dev_err(&intf->dev, |
|---|
| 761 | 885 | "Cannot get software info, error %d\n", err); |
|---|
| 762 | 886 | return err; |
|---|
| 763 | 887 | } |
|---|
| 764 | 888 | |
|---|
| 765 | | - if (dev->ops->dev_get_software_details) { |
|---|
| 766 | | - err = dev->ops->dev_get_software_details(dev); |
|---|
| 889 | + if (ops->dev_get_software_details) { |
|---|
| 890 | + err = ops->dev_get_software_details(dev); |
|---|
| 767 | 891 | if (err) { |
|---|
| 768 | 892 | dev_err(&intf->dev, |
|---|
| 769 | 893 | "Cannot get software details, error %d\n", err); |
|---|
| .. | .. |
|---|
| 781 | 905 | |
|---|
| 782 | 906 | dev_dbg(&intf->dev, "Max outstanding tx = %d URBs\n", dev->max_tx_urbs); |
|---|
| 783 | 907 | |
|---|
| 784 | | - err = dev->ops->dev_get_card_info(dev); |
|---|
| 908 | + err = ops->dev_get_card_info(dev); |
|---|
| 785 | 909 | if (err) { |
|---|
| 786 | 910 | dev_err(&intf->dev, "Cannot get card info, error %d\n", err); |
|---|
| 787 | 911 | return err; |
|---|
| 788 | 912 | } |
|---|
| 789 | 913 | |
|---|
| 790 | | - if (dev->ops->dev_get_capabilities) { |
|---|
| 791 | | - err = dev->ops->dev_get_capabilities(dev); |
|---|
| 914 | + if (ops->dev_get_capabilities) { |
|---|
| 915 | + err = ops->dev_get_capabilities(dev); |
|---|
| 792 | 916 | if (err) { |
|---|
| 793 | 917 | dev_err(&intf->dev, |
|---|
| 794 | 918 | "Cannot get capabilities, error %d\n", err); |
|---|
| .. | .. |
|---|
| 798 | 922 | } |
|---|
| 799 | 923 | |
|---|
| 800 | 924 | for (i = 0; i < dev->nchannels; i++) { |
|---|
| 801 | | - err = kvaser_usb_init_one(dev, id, i); |
|---|
| 925 | + err = kvaser_usb_init_one(dev, i); |
|---|
| 802 | 926 | if (err) { |
|---|
| 803 | 927 | kvaser_usb_remove_interfaces(dev); |
|---|
| 804 | 928 | return err; |
|---|