staging: octeon-hcd: remove unnecessary unlikely()
authorIgor Stoppa <igor.stoppa@gmail.com>
Fri, 7 Sep 2018 17:08:20 +0000 (20:08 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 14 Sep 2018 13:21:06 +0000 (15:21 +0200)
WARN_ON() already contains an unlikely(), so it's not necessary to
wrap it into another.

Signed-off-by: Igor Stoppa <igor.stoppa@huawei.com>
Cc: Aaro Koskinen <aaro.koskinen@iki.fi>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: devel@driverdev.osuosl.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/octeon-usb/octeon-hcd.c

index f188e19..9c766f5 100644 (file)
@@ -2770,7 +2770,7 @@ static int cvmx_usb_poll_channel(struct octeon_hcd *usb, int channel)
            (pipe->transfer_dir == CVMX_USB_DIRECTION_OUT))
                pipe->flags |= CVMX_USB_PIPE_FLAGS_NEED_PING;
 
-       if (unlikely(WARN_ON_ONCE(bytes_this_transfer < 0))) {
+       if (WARN_ON_ONCE(bytes_this_transfer < 0)) {
                /*
                 * In some rare cases the DMA engine seems to get stuck and
                 * keeps substracting same byte count over and over again. In