i2c: rk3x: Remove unlikely() annotations
authorDoug Anderson <dianders@chromium.org>
Fri, 22 Aug 2014 17:43:44 +0000 (10:43 -0700)
committerWolfram Sang <wsa@the-dreams.de>
Mon, 29 Sep 2014 20:41:18 +0000 (22:41 +0200)
Having a transfer more than 32 bits is not all that unlikely.  Remove
the annotation.

The unlikely in the IRQ handler can't gain us much.  It's not in a
loop, so at most it would save 1 instruction per IRQ, which isn't
much.  In fact on the compiler I tested it produced the exact same
code.  Remove it too.

Suggested-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Doug Anderson <dianders@chromium.org>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
drivers/i2c/busses/i2c-rk3x.c

index 93cfc837200b87360e3717e84513d9c8e03c3b70..b41d979656697a4fa4bc1b5ddd2e56d1c70c550a 100644 (file)
@@ -208,7 +208,7 @@ static void rk3x_i2c_prepare_read(struct rk3x_i2c *i2c)
         * The hw can read up to 32 bytes at a time. If we need more than one
         * chunk, send an ACK after the last byte of the current chunk.
         */
-       if (unlikely(len > 32)) {
+       if (len > 32) {
                len = 32;
                con &= ~REG_CON_LASTACK;
        } else {
@@ -403,7 +403,7 @@ static irqreturn_t rk3x_i2c_irq(int irqno, void *dev_id)
        }
 
        /* is there anything left to handle? */
-       if (unlikely((ipd & REG_INT_ALL) == 0))
+       if ((ipd & REG_INT_ALL) == 0)
                goto out;
 
        switch (i2c->state) {