diff options
| author | Jisheng Zhang <jszhang@marvell.com> | 2017-04-13 11:46:39 +0200 |
|---|---|---|
| committer | Linus Walleij <linus.walleij@linaro.org> | 2017-04-24 15:08:53 +0200 |
| commit | 5664aa1c9ef59895eee1af15e36140d09d20714d (patch) | |
| tree | ec8d59c696f294700cd80ae30a5af410c1c013e0 /drivers/gpio/gpio-dwapb.c | |
| parent | gpio: aspeed: Add open-source and open-drain support (diff) | |
| download | linux-5664aa1c9ef59895eee1af15e36140d09d20714d.tar.xz linux-5664aa1c9ef59895eee1af15e36140d09d20714d.zip | |
gpio: dwapb: use dwapb_read instead of readl_relaxed
Commit 67809b974a07 ("GPIO: gpio-dwapb: Change readl&writel to
dwapb_read&dwapb_write") missed this readl_relaxed() usage, I'm not
sure the reason, maybe for performance reason? But if we do care
the performance, we could use the relaxed io in dwapb_read and
dwapb_write.
Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/gpio/gpio-dwapb.c')
| -rw-r--r-- | drivers/gpio/gpio-dwapb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpio/gpio-dwapb.c b/drivers/gpio/gpio-dwapb.c index 53ade0b69f49..f051c4552af5 100644 --- a/drivers/gpio/gpio-dwapb.c +++ b/drivers/gpio/gpio-dwapb.c @@ -164,7 +164,7 @@ static void dwapb_toggle_trigger(struct dwapb_gpio *gpio, unsigned int offs) static u32 dwapb_do_irq(struct dwapb_gpio *gpio) { - u32 irq_status = readl_relaxed(gpio->regs + GPIO_INTSTATUS); + u32 irq_status = dwapb_read(gpio, GPIO_INTSTATUS); u32 ret = irq_status; while (irq_status) { |