diff options
author | Christian Gromm <christian.gromm@microchip.com> | 2020-05-05 13:00:28 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-05-05 13:17:59 +0300 |
commit | 188d5b41f81f58f15b69d8958df19779f23be55c (patch) | |
tree | dd00887b5d1f39acb66bafcf1b3e6c092247f5fe /drivers/staging/most | |
parent | 441be56f894d5f31e0747c35bbfb4784aa3b958a (diff) | |
download | linux-188d5b41f81f58f15b69d8958df19779f23be55c.tar.xz |
staging: most: usb: drop unlikely macros
This patch removes the unlikely macros in the error patch of argument
checking, as it has no measurable performance adavantage.
Signed-off-by: Christian Gromm <christian.gromm@microchip.com>
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Link: https://lore.kernel.org/r/1588672829-28883-4-git-send-email-christian.gromm@microchip.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/most')
-rw-r--r-- | drivers/staging/most/usb/usb.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/staging/most/usb/usb.c b/drivers/staging/most/usb/usb.c index 1087ad9c0038..62d773c97591 100644 --- a/drivers/staging/most/usb/usb.c +++ b/drivers/staging/most/usb/usb.c @@ -233,7 +233,7 @@ static int hdm_poison_channel(struct most_interface *iface, int channel) unsigned long flags; spinlock_t *lock; /* temp. lock */ - if (unlikely(channel < 0 || channel >= iface->num_channels)) { + if (channel < 0 || channel >= iface->num_channels) { dev_warn(&mdev->usb_device->dev, "Channel ID out of range.\n"); return -ECHRNG; } @@ -555,9 +555,9 @@ static int hdm_enqueue(struct most_interface *iface, int channel, unsigned long length; void *virt_address; - if (unlikely(!mbo)) + if (!mbo) return -EINVAL; - if (unlikely(iface->num_channels <= channel || channel < 0)) + if (iface->num_channels <= channel || channel < 0) return -ECHRNG; mdev = to_mdev(iface); @@ -670,11 +670,11 @@ static int hdm_configure_channel(struct most_interface *iface, int channel, mdev->clear_work[channel].mdev = mdev; INIT_WORK(&mdev->clear_work[channel].ws, wq_clear_halt); - if (unlikely(!conf)) { + if (!conf) { dev_err(dev, "Bad config pointer.\n"); return -EINVAL; } - if (unlikely(channel < 0 || channel >= iface->num_channels)) { + if (channel < 0 || channel >= iface->num_channels) { dev_err(dev, "Channel ID out of range.\n"); return -EINVAL; } |