[media] rc: meson-ir: use readl_relaxed in the interrupt handler
authorHeiner Kallweit <hkallweit1@gmail.com>
Wed, 12 Apr 2017 19:33:57 +0000 (16:33 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 6 Jun 2017 11:56:29 +0000 (08:56 -0300)
We don't need the memory barriers here and an interrupt handler should
be as fast as possible. Therefore switch to readl_relaxed.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Reviewed-by: Neil Armstrong <narmstrong@baylibre.com>
Signed-off-by: Sean Young <sean@mess.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/rc/meson-ir.c

index 664c3efe1f8f8ade6c12e6bbf68b8d4a3b5b4943..dfe9184e996c57bef4a44375fcb0a2e863d54c27 100644 (file)
@@ -85,16 +85,17 @@ static void meson_ir_set_mask(struct meson_ir *ir, unsigned int reg,
 static irqreturn_t meson_ir_irq(int irqno, void *dev_id)
 {
        struct meson_ir *ir = dev_id;
-       u32 duration;
+       u32 duration, status;
        DEFINE_IR_RAW_EVENT(rawir);
 
        spin_lock(&ir->lock);
 
-       duration = readl(ir->reg + IR_DEC_REG1);
+       duration = readl_relaxed(ir->reg + IR_DEC_REG1);
        duration = FIELD_GET(REG1_TIME_IV_MASK, duration);
        rawir.duration = US_TO_NS(duration * MESON_TRATE);
 
-       rawir.pulse = !!(readl(ir->reg + IR_DEC_STATUS) & STATUS_IR_DEC_IN);
+       status = readl_relaxed(ir->reg + IR_DEC_STATUS);
+       rawir.pulse = !!(status & STATUS_IR_DEC_IN);
 
        ir_raw_event_store_with_filter(ir->rc, &rawir);
        ir_raw_event_handle(ir->rc);