Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp688821rwd; Thu, 18 May 2023 02:29:08 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6icjtJbKQ1i3y1HaKzPWQEYbPJP9JNDn1WXv7v3/rq6LEXUKW46M+4lLt0skhDf8jTPDvW X-Received: by 2002:a05:6a20:2583:b0:101:4627:dfe with SMTP id k3-20020a056a20258300b0010146270dfemr1654915pzd.9.1684402147714; Thu, 18 May 2023 02:29:07 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1684402147; cv=none; d=google.com; s=arc-20160816; b=KXGTmS8Rmm0H5f9IYv1E9Cs5mzhmaw5hN3m5NGVzdx5yZsW13s3uSbKIZyxUWfkZzs kmROAV00f1nqapbXdV+jvySoDbAd5qQts8Z1bPsrM2IJvHQU/4BKi8PTqVsaRmGgm/s+ FAJ2zanfuWZ0C8V7YikLZdlKRJZxN3JO1LTFajH711/yPQfQyEPVKNGu2TxoO3HywVoS Z4YVVaEL8jY/RmvYhl+rpeo0WH3yF1m3t7rzlN34hf+p8j5xAWVwahCGnQa9JDhExBnZ OyANaIK61nXZjv0TX62jFHkmYNGU4mF//IW3IOp9Qfp+bfLfE0MT6WUoXbCqxsMKUKM7 fTlQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version; bh=QXwpAXp8JxyXh5HdlW+s9q8xXEvBKKInORpAZEhgCKM=; b=ff6Ybxr9/8pz1fLR28Ss41xkFXR8bTkwz3DMsIwuiMsTlc8VREvflL1CiedUPwsUtH LlYcQcPviE0Jcjlz7qAO9SyPfVK/873Fplcke4zndY16fMQlyAauX1cVS+G38fKsfeCp QLaaLAqh801rOGTmBRxwWn5qSd+yd4XdggmhVCLUQUq2IM493orvJQC1sKtDA4Sub1oT sGNmKIBGLnpENXdbth3OEF953+srxLeRNqnQckMuaXNGBH9gzWQh5zawu/RSBvXwYQvr ahCWOSEOt2DPTAAaIssLhrsBtFv8LdDRWdlMhZVQaL2InIBy0kKAMjh4BHMGa/6lW0kW cJzQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id x24-20020a656ab8000000b0053076900facsi966346pgu.864.2023.05.18.02.28.55; Thu, 18 May 2023 02:29:07 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230031AbjERJJS (ORCPT + 99 others); Thu, 18 May 2023 05:09:18 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43100 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229914AbjERJJQ (ORCPT ); Thu, 18 May 2023 05:09:16 -0400 Received: from mail-pf1-f176.google.com (mail-pf1-f176.google.com [209.85.210.176]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BA9A01FD8; Thu, 18 May 2023 02:09:13 -0700 (PDT) Received: by mail-pf1-f176.google.com with SMTP id d2e1a72fcca58-64a9335a8e7so13934107b3a.0; Thu, 18 May 2023 02:09:13 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684400953; x=1686992953; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=QXwpAXp8JxyXh5HdlW+s9q8xXEvBKKInORpAZEhgCKM=; b=QxXZHBmtWcfFeFzzSyEhLN2/vl+VeszOeyz0jUfHKK2l66RhimONARV4vdzdPBryNG c8mvitzm3Ilr1kZkaEGAnJ08DRBqJnFOMlrXxn75e26Gqh68ZiYbVSZfaHYbSiOgWkg9 4M/Ng/Jba+GVGv9J5s3ZkJq3tii+IppYrcRTKuSsVNcGzR9GeF+2qzf6ypA6ZLgL4S/0 urlZ102cbnJyy46tA4d31/Tm9iDavo9wxvh1Na7pO8qAl+bVvDyYGJEqH6709ahldc/Y 3kMnlsOyE1k8FlbQROGfEGsrtKd9V4b0/M4X7e8JB086zGRO0AdMzAVa8WMgkHQEolzX Ki0w== X-Gm-Message-State: AC+VfDwzC/rffXgboXljH1PGTE+P4gZ74SJUDrJ6xFtq3RXRgTToUera XGa0ktilEqiu/L+3dHSZsxjrh8T3xMguth8r/P3TO2rsEYY= X-Received: by 2002:a17:902:d2ca:b0:1ae:8fa:cd4c with SMTP id n10-20020a170902d2ca00b001ae08facd4cmr2112618plc.7.1684400952729; Thu, 18 May 2023 02:09:12 -0700 (PDT) MIME-Version: 1.0 References: <20230517192251.2405290-1-frank.jungclaus@esd.eu> <20230517192251.2405290-4-frank.jungclaus@esd.eu> In-Reply-To: <20230517192251.2405290-4-frank.jungclaus@esd.eu> From: Vincent MAILHOL Date: Thu, 18 May 2023 18:09:01 +0900 Message-ID: Subject: Re: [PATCH 3/6] can: esd_usb: Use consistent prefixes for macros To: Frank Jungclaus Cc: linux-can@vger.kernel.org, Marc Kleine-Budde , Wolfgang Grandegger , =?UTF-8?Q?Stefan_M=C3=A4tje?= , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-1.4 required=5.0 tests=BAYES_00, FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS, RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, thanks for the series. I went through it and just have two nitpicks. On Thu. 18 May 2023 at 04:27, Frank Jungclaus wrote: > Initiated by a comment from Vincent Mailhol add the consistent prefix No need to start all the series with: Initiated by a comment from Vincent Mailhol while I appreciate the ommage, the Suggested-by tag is sufficient for that. > ESD_USB_ to all macros defined within esd_usb.c. > > For macros specific to esd CAN-USB/2 use ESD_USB_2_ as prefix. > For macros specific to esd CAN-USB/Micro use ESD_USB_M_ as prefix. > > Link: https://lore.kernel.org/all/CAMZ6RqLaDNy-fZ2G0+QMhUEckkXLL+ZyELVSDFmqpd++aBzZQg@mail.gmail.com/ > Suggested-by: Vincent MAILHOL > Signed-off-by: Frank Jungclaus > --- > drivers/net/can/usb/esd_usb.c | 198 +++++++++++++++++----------------- > 1 file changed, 99 insertions(+), 99 deletions(-) > > diff --git a/drivers/net/can/usb/esd_usb.c b/drivers/net/can/usb/esd_usb.c > index 194aa1cf37b5..23a568bfcdc2 100644 > --- a/drivers/net/can/usb/esd_usb.c > +++ b/drivers/net/can/usb/esd_usb.c > @@ -23,33 +23,33 @@ MODULE_DESCRIPTION("CAN driver for esd electronics gmbh CAN-USB/2 and CAN-USB/Mi > MODULE_LICENSE("GPL v2"); > > /* USB vendor and product ID */ > -#define USB_ESDGMBH_VENDOR_ID 0x0ab4 > -#define USB_CANUSB2_PRODUCT_ID 0x0010 > -#define USB_CANUSBM_PRODUCT_ID 0x0011 > +#define ESD_USB_ESDGMBH_VENDOR_ID 0x0ab4 > +#define ESD_USB_CANUSB2_PRODUCT_ID 0x0010 > +#define ESD_USB_CANUSBM_PRODUCT_ID 0x0011 > > /* CAN controller clock frequencies */ > -#define ESD_USB2_CAN_CLOCK (60 * MEGA) /* Hz */ > -#define ESD_USBM_CAN_CLOCK (36 * MEGA) /* Hz */ > +#define ESD_USB_2_CAN_CLOCK (60 * MEGA) /* Hz */ > +#define ESD_USB_M_CAN_CLOCK (36 * MEGA) /* Hz */ > > /* Maximum number of CAN nets */ > #define ESD_USB_MAX_NETS 2 > > /* USB commands */ > -#define CMD_VERSION 1 /* also used for VERSION_REPLY */ > -#define CMD_CAN_RX 2 /* device to host only */ > -#define CMD_CAN_TX 3 /* also used for TX_DONE */ > -#define CMD_SETBAUD 4 /* also used for SETBAUD_REPLY */ > -#define CMD_TS 5 /* also used for TS_REPLY */ > -#define CMD_IDADD 6 /* also used for IDADD_REPLY */ > +#define ESD_USB_CMD_VERSION 1 /* also used for VERSION_REPLY */ > +#define ESD_USB_CMD_CAN_RX 2 /* device to host only */ > +#define ESD_USB_CMD_CAN_TX 3 /* also used for TX_DONE */ > +#define ESD_USB_CMD_SETBAUD 4 /* also used for SETBAUD_REPLY */ > +#define ESD_USB_CMD_TS 5 /* also used for TS_REPLY */ > +#define ESD_USB_CMD_IDADD 6 /* also used for IDADD_REPLY */ > > /* esd CAN message flags - dlc field */ > #define ESD_RTR BIT(4) /* 0x10 */ > > > /* esd CAN message flags - id field */ > -#define ESD_EXTID BIT(29) /* 0x20000000 */ > -#define ESD_EVENT BIT(30) /* 0x40000000 */ > -#define ESD_IDMASK 0x1fffffff > +#define ESD_USB_EXTID BIT(29) /* 0x20000000 */ > +#define ESD_USB_EVENT BIT(30) /* 0x40000000 */ > +#define ESD_USB_IDMASK 0x1fffffff > > /* esd CAN event ids */ > #define ESD_EV_CAN_ERROR_EXT 2 /* CAN controller specific diagnostic data */ > @@ -59,35 +59,35 @@ MODULE_LICENSE("GPL v2"); > #define ESD_USB_UBR BIT(31) /* 0x80000000, User Bit Rate (controller BTR) in bits 0..27 */ > #define ESD_USB_NO_BAUDRATE 0x7fffffff /* bit rate unconfigured */ > > -/* bit timing CAN-USB/2 */ > -#define ESD_USB2_TSEG1_SHIFT 16 > -#define ESD_USB2_TSEG2_SHIFT 20 > -#define ESD_USB2_SJW_SHIFT 14 > -#define ESD_USBM_SJW_SHIFT 24 > -#define ESD_USB2_3_SAMPLES 0x00800000 That one should also use the BIT macro. > +/* bit timing esd CAN-USB */ > +#define ESD_USB_2_TSEG1_SHIFT 16 > +#define ESD_USB_2_TSEG2_SHIFT 20 > +#define ESD_USB_2_SJW_SHIFT 14 > +#define ESD_USB_M_SJW_SHIFT 24 > +#define ESD_USB_3_SAMPLES 0x00800000 > > /* esd IDADD message */ > -#define ESD_ID_ENABLE 0x80 > -#define ESD_MAX_ID_SEGMENT 64 > +#define ESD_USB_ID_ENABLE 0x80 > +#define ESD_USB_MAX_ID_SEGMENT 64 > > /* SJA1000 ECC register (emulated by usb firmware) */ > -#define SJA1000_ECC_SEG 0x1F > -#define SJA1000_ECC_DIR 0x20 > -#define SJA1000_ECC_ERR 0x06 > -#define SJA1000_ECC_BIT 0x00 > -#define SJA1000_ECC_FORM 0x40 > -#define SJA1000_ECC_STUFF 0x80 > -#define SJA1000_ECC_MASK 0xc0 > +#define ESD_USB_SJA1000_ECC_SEG 0x1F > +#define ESD_USB_SJA1000_ECC_DIR 0x20 > +#define ESD_USB_SJA1000_ECC_ERR 0x06 > +#define ESD_USB_SJA1000_ECC_BIT 0x00 > +#define ESD_USB_SJA1000_ECC_FORM 0x40 > +#define ESD_USB_SJA1000_ECC_STUFF 0x80 > +#define ESD_USB_SJA1000_ECC_MASK 0xc0 > > /* esd bus state event codes */ > -#define ESD_BUSSTATE_MASK 0xc0 > -#define ESD_BUSSTATE_WARN 0x40 > -#define ESD_BUSSTATE_ERRPASSIVE 0x80 > -#define ESD_BUSSTATE_BUSOFF 0xc0 > +#define ESD_USB_BUSSTATE_MASK 0xc0 > +#define ESD_USB_BUSSTATE_WARN 0x40 > +#define ESD_USB_BUSSTATE_ERRPASSIVE 0x80 > +#define ESD_USB_BUSSTATE_BUSOFF 0xc0 > > -#define RX_BUFFER_SIZE 1024 > -#define MAX_RX_URBS 4 > -#define MAX_TX_URBS 16 /* must be power of 2 */ > +#define ESD_USB_RX_BUFFER_SIZE 1024 > +#define ESD_USB_MAX_RX_URBS 4 > +#define ESD_USB_MAX_TX_URBS 16 /* must be power of 2 */ > > struct header_msg { > u8 len; /* len is always the total message length in 32bit words */ > @@ -156,7 +156,7 @@ struct id_filter_msg { > u8 cmd; > u8 net; > u8 option; > - __le32 mask[ESD_MAX_ID_SEGMENT + 1]; > + __le32 mask[ESD_USB_MAX_ID_SEGMENT + 1]; > }; > > struct set_baudrate_msg { > @@ -180,8 +180,8 @@ union __packed esd_usb_msg { > }; > > static struct usb_device_id esd_usb_table[] = { > - {USB_DEVICE(USB_ESDGMBH_VENDOR_ID, USB_CANUSB2_PRODUCT_ID)}, > - {USB_DEVICE(USB_ESDGMBH_VENDOR_ID, USB_CANUSBM_PRODUCT_ID)}, > + {USB_DEVICE(ESD_USB_ESDGMBH_VENDOR_ID, ESD_USB_CANUSB2_PRODUCT_ID)}, > + {USB_DEVICE(ESD_USB_ESDGMBH_VENDOR_ID, ESD_USB_CANUSBM_PRODUCT_ID)}, > {} > }; > MODULE_DEVICE_TABLE(usb, esd_usb_table); > @@ -202,8 +202,8 @@ struct esd_usb { > int net_count; > u32 version; > int rxinitdone; > - void *rxbuf[MAX_RX_URBS]; > - dma_addr_t rxbuf_dma[MAX_RX_URBS]; > + void *rxbuf[ESD_USB_MAX_RX_URBS]; > + dma_addr_t rxbuf_dma[ESD_USB_MAX_RX_URBS]; > }; > > struct esd_usb_net_priv { > @@ -211,7 +211,7 @@ struct esd_usb_net_priv { > > atomic_t active_tx_jobs; > struct usb_anchor tx_submitted; > - struct esd_tx_urb_context tx_contexts[MAX_TX_URBS]; > + struct esd_tx_urb_context tx_contexts[ESD_USB_MAX_TX_URBS]; > > struct esd_usb *usb; > struct net_device *netdev; > @@ -226,7 +226,7 @@ static void esd_usb_rx_event(struct esd_usb_net_priv *priv, > struct net_device_stats *stats = &priv->netdev->stats; > struct can_frame *cf; > struct sk_buff *skb; > - u32 id = le32_to_cpu(msg->rx.id) & ESD_IDMASK; > + u32 id = le32_to_cpu(msg->rx.id) & ESD_USB_IDMASK; > > if (id == ESD_EV_CAN_ERROR_EXT) { > u8 state = msg->rx.ev_can_err_ext.status; > @@ -255,15 +255,15 @@ static void esd_usb_rx_event(struct esd_usb_net_priv *priv, > > priv->old_state = state; > > - switch (state & ESD_BUSSTATE_MASK) { > - case ESD_BUSSTATE_BUSOFF: > + switch (state & ESD_USB_BUSSTATE_MASK) { > + case ESD_USB_BUSSTATE_BUSOFF: > new_state = CAN_STATE_BUS_OFF; > can_bus_off(priv->netdev); > break; > - case ESD_BUSSTATE_WARN: > + case ESD_USB_BUSSTATE_WARN: > new_state = CAN_STATE_ERROR_WARNING; > break; > - case ESD_BUSSTATE_ERRPASSIVE: > + case ESD_USB_BUSSTATE_ERRPASSIVE: > new_state = CAN_STATE_ERROR_PASSIVE; > break; > default: > @@ -285,14 +285,14 @@ static void esd_usb_rx_event(struct esd_usb_net_priv *priv, > > cf->can_id |= CAN_ERR_PROT | CAN_ERR_BUSERROR; > > - switch (ecc & SJA1000_ECC_MASK) { > - case SJA1000_ECC_BIT: > + switch (ecc & ESD_USB_SJA1000_ECC_MASK) { > + case ESD_USB_SJA1000_ECC_BIT: > cf->data[2] |= CAN_ERR_PROT_BIT; > break; > - case SJA1000_ECC_FORM: > + case ESD_USB_SJA1000_ECC_FORM: > cf->data[2] |= CAN_ERR_PROT_FORM; > break; > - case SJA1000_ECC_STUFF: > + case ESD_USB_SJA1000_ECC_STUFF: > cf->data[2] |= CAN_ERR_PROT_STUFF; > break; > default: > @@ -300,11 +300,11 @@ static void esd_usb_rx_event(struct esd_usb_net_priv *priv, > } > > /* Error occurred during transmission? */ > - if (!(ecc & SJA1000_ECC_DIR)) > + if (!(ecc & ESD_USB_SJA1000_ECC_DIR)) > cf->data[2] |= CAN_ERR_PROT_TX; > > /* Bit stream position in CAN frame as the error was detected */ > - cf->data[3] = ecc & SJA1000_ECC_SEG; > + cf->data[3] = ecc & ESD_USB_SJA1000_ECC_SEG; > } > > if (skb) { > @@ -331,7 +331,7 @@ static void esd_usb_rx_can_msg(struct esd_usb_net_priv *priv, > > id = le32_to_cpu(msg->rx.id); > > - if (id & ESD_EVENT) { > + if (id & ESD_USB_EVENT) { > esd_usb_rx_event(priv, msg); > } else { > skb = alloc_can_skb(priv->netdev, &cf); > @@ -340,11 +340,11 @@ static void esd_usb_rx_can_msg(struct esd_usb_net_priv *priv, > return; > } > > - cf->can_id = id & ESD_IDMASK; > + cf->can_id = id & ESD_USB_IDMASK; > can_frame_set_cc_len(cf, msg->rx.dlc & ~ESD_RTR, > priv->can.ctrlmode); > > - if (id & ESD_EXTID) > + if (id & ESD_USB_EXTID) > cf->can_id |= CAN_EFF_FLAG; > > if (msg->rx.dlc & ESD_RTR) { > @@ -371,7 +371,7 @@ static void esd_usb_tx_done_msg(struct esd_usb_net_priv *priv, > if (!netif_device_present(netdev)) > return; > > - context = &priv->tx_contexts[msg->txdone.hnd & (MAX_TX_URBS - 1)]; > + context = &priv->tx_contexts[msg->txdone.hnd & (ESD_USB_MAX_TX_URBS - 1)]; > > if (!msg->txdone.status) { > stats->tx_packets++; > @@ -383,7 +383,7 @@ static void esd_usb_tx_done_msg(struct esd_usb_net_priv *priv, > } > > /* Release context */ > - context->echo_index = MAX_TX_URBS; > + context->echo_index = ESD_USB_MAX_TX_URBS; > atomic_dec(&priv->active_tx_jobs); > > netif_wake_queue(netdev); > @@ -418,7 +418,7 @@ static void esd_usb_read_bulk_callback(struct urb *urb) > msg = (union esd_usb_msg *)(urb->transfer_buffer + pos); > > switch (msg->hdr.cmd) { > - case CMD_CAN_RX: > + case ESD_USB_CMD_CAN_RX: > if (msg->rx.net >= dev->net_count) { > dev_err(dev->udev->dev.parent, "format error\n"); > break; > @@ -427,7 +427,7 @@ static void esd_usb_read_bulk_callback(struct urb *urb) > esd_usb_rx_can_msg(dev->nets[msg->rx.net], msg); > break; > > - case CMD_CAN_TX: > + case ESD_USB_CMD_CAN_TX: > if (msg->txdone.net >= dev->net_count) { > dev_err(dev->udev->dev.parent, "format error\n"); > break; > @@ -448,7 +448,7 @@ static void esd_usb_read_bulk_callback(struct urb *urb) > > resubmit_urb: > usb_fill_bulk_urb(urb, dev->udev, usb_rcvbulkpipe(dev->udev, 1), > - urb->transfer_buffer, RX_BUFFER_SIZE, > + urb->transfer_buffer, ESD_USB_RX_BUFFER_SIZE, > esd_usb_read_bulk_callback, dev); > > retval = usb_submit_urb(urb, GFP_ATOMIC); > @@ -557,7 +557,7 @@ static int esd_usb_setup_rx_urbs(struct esd_usb *dev) > if (dev->rxinitdone) > return 0; > > - for (i = 0; i < MAX_RX_URBS; i++) { > + for (i = 0; i < ESD_USB_MAX_RX_URBS; i++) { > struct urb *urb = NULL; > u8 *buf = NULL; > dma_addr_t buf_dma; > @@ -569,7 +569,7 @@ static int esd_usb_setup_rx_urbs(struct esd_usb *dev) > break; > } > > - buf = usb_alloc_coherent(dev->udev, RX_BUFFER_SIZE, GFP_KERNEL, > + buf = usb_alloc_coherent(dev->udev, ESD_USB_RX_BUFFER_SIZE, GFP_KERNEL, > &buf_dma); > if (!buf) { > dev_warn(dev->udev->dev.parent, > @@ -582,7 +582,7 @@ static int esd_usb_setup_rx_urbs(struct esd_usb *dev) > > usb_fill_bulk_urb(urb, dev->udev, > usb_rcvbulkpipe(dev->udev, 1), > - buf, RX_BUFFER_SIZE, > + buf, ESD_USB_RX_BUFFER_SIZE, > esd_usb_read_bulk_callback, dev); > urb->transfer_flags |= URB_NO_TRANSFER_DMA_MAP; > usb_anchor_urb(urb, &dev->rx_submitted); > @@ -590,7 +590,7 @@ static int esd_usb_setup_rx_urbs(struct esd_usb *dev) > err = usb_submit_urb(urb, GFP_KERNEL); > if (err) { > usb_unanchor_urb(urb); > - usb_free_coherent(dev->udev, RX_BUFFER_SIZE, buf, > + usb_free_coherent(dev->udev, ESD_USB_RX_BUFFER_SIZE, buf, > urb->transfer_dma); > goto freeurb; > } > @@ -612,7 +612,7 @@ static int esd_usb_setup_rx_urbs(struct esd_usb *dev) > } > > /* Warn if we've couldn't transmit all the URBs */ > - if (i < MAX_RX_URBS) { > + if (i < ESD_USB_MAX_RX_URBS) { > dev_warn(dev->udev->dev.parent, > "rx performance may be slow\n"); > } > @@ -647,14 +647,14 @@ static int esd_usb_start(struct esd_usb_net_priv *priv) > * the number of the starting bitmask (0..64) to the filter.option > * field followed by only some bitmasks. > */ > - msg->hdr.cmd = CMD_IDADD; > - msg->hdr.len = 2 + ESD_MAX_ID_SEGMENT; > + msg->hdr.cmd = ESD_USB_CMD_IDADD; > + msg->hdr.len = 2 + ESD_USB_MAX_ID_SEGMENT; > msg->filter.net = priv->index; > - msg->filter.option = ESD_ID_ENABLE; /* start with segment 0 */ > - for (i = 0; i < ESD_MAX_ID_SEGMENT; i++) > + msg->filter.option = ESD_USB_ID_ENABLE; /* start with segment 0 */ > + for (i = 0; i < ESD_USB_MAX_ID_SEGMENT; i++) > msg->filter.mask[i] = cpu_to_le32(0xffffffff); > /* enable 29bit extended IDs */ > - msg->filter.mask[ESD_MAX_ID_SEGMENT] = cpu_to_le32(0x00000001); > + msg->filter.mask[ESD_USB_MAX_ID_SEGMENT] = cpu_to_le32(0x00000001); > > err = esd_usb_send_msg(dev, msg); > if (err) > @@ -683,8 +683,8 @@ static void unlink_all_urbs(struct esd_usb *dev) > > usb_kill_anchored_urbs(&dev->rx_submitted); > > - for (i = 0; i < MAX_RX_URBS; ++i) > - usb_free_coherent(dev->udev, RX_BUFFER_SIZE, > + for (i = 0; i < ESD_USB_MAX_RX_URBS; ++i) > + usb_free_coherent(dev->udev, ESD_USB_RX_BUFFER_SIZE, > dev->rxbuf[i], dev->rxbuf_dma[i]); > > for (i = 0; i < dev->net_count; i++) { > @@ -693,8 +693,8 @@ static void unlink_all_urbs(struct esd_usb *dev) > usb_kill_anchored_urbs(&priv->tx_submitted); > atomic_set(&priv->active_tx_jobs, 0); > > - for (j = 0; j < MAX_TX_URBS; j++) > - priv->tx_contexts[j].echo_index = MAX_TX_URBS; > + for (j = 0; j < ESD_USB_MAX_TX_URBS; j++) > + priv->tx_contexts[j].echo_index = ESD_USB_MAX_TX_URBS; > } > } > } > @@ -760,7 +760,7 @@ static netdev_tx_t esd_usb_start_xmit(struct sk_buff *skb, > msg = (union esd_usb_msg *)buf; > > msg->hdr.len = 3; /* minimal length */ > - msg->hdr.cmd = CMD_CAN_TX; > + msg->hdr.cmd = ESD_USB_CMD_CAN_TX; > msg->tx.net = priv->index; > msg->tx.dlc = can_get_cc_dlc(cf, priv->can.ctrlmode); > msg->tx.id = cpu_to_le32(cf->can_id & CAN_ERR_MASK); > @@ -769,15 +769,15 @@ static netdev_tx_t esd_usb_start_xmit(struct sk_buff *skb, > msg->tx.dlc |= ESD_RTR; > > if (cf->can_id & CAN_EFF_FLAG) > - msg->tx.id |= cpu_to_le32(ESD_EXTID); > + msg->tx.id |= cpu_to_le32(ESD_USB_EXTID); > > for (i = 0; i < cf->len; i++) > msg->tx.data[i] = cf->data[i]; > > msg->hdr.len += (cf->len + 3) >> 2; > > - for (i = 0; i < MAX_TX_URBS; i++) { > - if (priv->tx_contexts[i].echo_index == MAX_TX_URBS) { > + for (i = 0; i < ESD_USB_MAX_TX_URBS; i++) { > + if (priv->tx_contexts[i].echo_index == ESD_USB_MAX_TX_URBS) { > context = &priv->tx_contexts[i]; > break; > } > @@ -809,7 +809,7 @@ static netdev_tx_t esd_usb_start_xmit(struct sk_buff *skb, > atomic_inc(&priv->active_tx_jobs); > > /* Slow down tx path */ > - if (atomic_read(&priv->active_tx_jobs) >= MAX_TX_URBS) > + if (atomic_read(&priv->active_tx_jobs) >= ESD_USB_MAX_TX_URBS) > netif_stop_queue(netdev); > > err = usb_submit_urb(urb, GFP_ATOMIC); > @@ -859,18 +859,18 @@ static int esd_usb_close(struct net_device *netdev) > return -ENOMEM; > > /* Disable all IDs (see esd_usb_start()) */ > - msg->hdr.cmd = CMD_IDADD; > - msg->hdr.len = 2 + ESD_MAX_ID_SEGMENT; > + msg->hdr.cmd = ESD_USB_CMD_IDADD; > + msg->hdr.len = 2 + ESD_USB_MAX_ID_SEGMENT; > msg->filter.net = priv->index; > - msg->filter.option = ESD_ID_ENABLE; /* start with segment 0 */ > - for (i = 0; i <= ESD_MAX_ID_SEGMENT; i++) > + msg->filter.option = ESD_USB_ID_ENABLE; /* start with segment 0 */ > + for (i = 0; i <= ESD_USB_MAX_ID_SEGMENT; i++) > msg->filter.mask[i] = 0; > if (esd_usb_send_msg(priv->usb, msg) < 0) > netdev_err(netdev, "sending idadd message failed\n"); > > /* set CAN controller to reset mode */ > msg->hdr.len = 2; > - msg->hdr.cmd = CMD_SETBAUD; > + msg->hdr.cmd = ESD_USB_CMD_SETBAUD; > msg->setbaud.net = priv->index; > msg->setbaud.rsvd = 0; > msg->setbaud.baud = cpu_to_le32(ESD_USB_NO_BAUDRATE); > @@ -928,27 +928,27 @@ static int esd_usb2_set_bittiming(struct net_device *netdev) > canbtr |= (bt->brp - 1) & (btc->brp_max - 1); > > if (le16_to_cpu(priv->usb->udev->descriptor.idProduct) == > - USB_CANUSBM_PRODUCT_ID) > - sjw_shift = ESD_USBM_SJW_SHIFT; > + ESD_USB_CANUSBM_PRODUCT_ID) > + sjw_shift = ESD_USB_M_SJW_SHIFT; > else > - sjw_shift = ESD_USB2_SJW_SHIFT; > + sjw_shift = ESD_USB_2_SJW_SHIFT; > > canbtr |= ((bt->sjw - 1) & (btc->sjw_max - 1)) > << sjw_shift; > canbtr |= ((bt->prop_seg + bt->phase_seg1 - 1) > & (btc->tseg1_max - 1)) > - << ESD_USB2_TSEG1_SHIFT; > + << ESD_USB_2_TSEG1_SHIFT; > canbtr |= ((bt->phase_seg2 - 1) & (btc->tseg2_max - 1)) > - << ESD_USB2_TSEG2_SHIFT; > + << ESD_USB_2_TSEG2_SHIFT; > if (priv->can.ctrlmode & CAN_CTRLMODE_3_SAMPLES) > - canbtr |= ESD_USB2_3_SAMPLES; > + canbtr |= ESD_USB_3_SAMPLES; > > msg = kmalloc(sizeof(*msg), GFP_KERNEL); > if (!msg) > return -ENOMEM; > > msg->hdr.len = 2; > - msg->hdr.cmd = CMD_SETBAUD; > + msg->hdr.cmd = ESD_USB_CMD_SETBAUD; > msg->setbaud.net = priv->index; > msg->setbaud.rsvd = 0; > msg->setbaud.baud = cpu_to_le32(canbtr); > @@ -994,7 +994,7 @@ static int esd_usb_probe_one_net(struct usb_interface *intf, int index) > int err = 0; > int i; > > - netdev = alloc_candev(sizeof(*priv), MAX_TX_URBS); > + netdev = alloc_candev(sizeof(*priv), ESD_USB_MAX_TX_URBS); > if (!netdev) { > dev_err(&intf->dev, "couldn't alloc candev\n"); > err = -ENOMEM; > @@ -1006,8 +1006,8 @@ static int esd_usb_probe_one_net(struct usb_interface *intf, int index) > init_usb_anchor(&priv->tx_submitted); > atomic_set(&priv->active_tx_jobs, 0); > > - for (i = 0; i < MAX_TX_URBS; i++) > - priv->tx_contexts[i].echo_index = MAX_TX_URBS; > + for (i = 0; i < ESD_USB_MAX_TX_URBS; i++) > + priv->tx_contexts[i].echo_index = ESD_USB_MAX_TX_URBS; > > priv->usb = dev; > priv->netdev = netdev; > @@ -1019,10 +1019,10 @@ static int esd_usb_probe_one_net(struct usb_interface *intf, int index) > CAN_CTRLMODE_BERR_REPORTING; > > if (le16_to_cpu(dev->udev->descriptor.idProduct) == > - USB_CANUSBM_PRODUCT_ID) > - priv->can.clock.freq = ESD_USBM_CAN_CLOCK; > + ESD_USB_CANUSBM_PRODUCT_ID) > + priv->can.clock.freq = ESD_USB_M_CAN_CLOCK; > else { > - priv->can.clock.freq = ESD_USB2_CAN_CLOCK; > + priv->can.clock.freq = ESD_USB_2_CAN_CLOCK; > priv->can.ctrlmode_supported |= CAN_CTRLMODE_3_SAMPLES; > } > > @@ -1085,7 +1085,7 @@ static int esd_usb_probe(struct usb_interface *intf, > } > > /* query number of CAN interfaces (nets) */ > - msg->hdr.cmd = CMD_VERSION; > + msg->hdr.cmd = ESD_USB_CMD_VERSION; > msg->hdr.len = 2; > msg->version.rsvd = 0; > msg->version.flags = 0; > -- > 2.25.1 >