can: peak_usb: fix checkpatch warnings
This patch cleans several checkpatch warnings in the peak_usb driver. Link: https://lore.kernel.org/r/20210406111622.1874957-2-mkl@pengutronix.de Acked-by: Stephane Grosjean <s.grosjean@peak-system.com> Tested-by: Stephane Grosjean <s.grosjean@peak-system.com> Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
This commit is contained in:
parent
c664e2137a
commit
bc256b9597
|
@ -401,7 +401,7 @@ static int pcan_usb_update_ts(struct pcan_usb_msg_context *mc)
|
|||
{
|
||||
__le16 tmp16;
|
||||
|
||||
if ((mc->ptr+2) > mc->end)
|
||||
if ((mc->ptr + 2) > mc->end)
|
||||
return -EINVAL;
|
||||
|
||||
memcpy(&tmp16, mc->ptr, 2);
|
||||
|
@ -1039,7 +1039,7 @@ const struct peak_usb_adapter pcan_usb = {
|
|||
CAN_CTRLMODE_BERR_REPORTING |
|
||||
CAN_CTRLMODE_CC_LEN8_DLC,
|
||||
.clock = {
|
||||
.freq = PCAN_USB_CRYSTAL_HZ / 2 ,
|
||||
.freq = PCAN_USB_CRYSTAL_HZ / 2,
|
||||
},
|
||||
.bittiming_const = &pcan_usb_const,
|
||||
|
||||
|
|
|
@ -624,6 +624,7 @@ static int peak_usb_ndo_stop(struct net_device *netdev)
|
|||
/* can set bus off now */
|
||||
if (dev->adapter->dev_set_bus) {
|
||||
int err = dev->adapter->dev_set_bus(dev, 0);
|
||||
|
||||
if (err)
|
||||
return err;
|
||||
}
|
||||
|
|
|
@ -31,7 +31,7 @@
|
|||
/* usb adapters maximum channels per usb interface */
|
||||
#define PCAN_USB_MAX_CHANNEL 2
|
||||
|
||||
/* maximum length of the usb commands sent to/received from the devices */
|
||||
/* maximum length of the usb commands sent to/received from the devices */
|
||||
#define PCAN_USB_MAX_CMD_LEN 32
|
||||
|
||||
struct peak_usb_device;
|
||||
|
|
|
@ -290,7 +290,7 @@ static int pcan_usb_pro_wait_rsp(struct peak_usb_device *dev,
|
|||
pr->data_type);
|
||||
|
||||
/* check if channel in response corresponds too */
|
||||
else if ((req_channel != 0xff) && \
|
||||
else if ((req_channel != 0xff) &&
|
||||
(pr->bus_act.channel != req_channel))
|
||||
netdev_err(dev->netdev,
|
||||
"got rsp %xh but on chan%u: ignored\n",
|
||||
|
|
Loading…
Reference in New Issue