Commit e005da0e authored by Geert Uytterhoeven's avatar Geert Uytterhoeven

pinctrl: rza1: Remove unneeded spinlock acquisitions

rza1_get_bit() is just a single register read.  Hence there's no need to
synchronize it with other register writes to the same bank.
Signed-off-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
Acked-by: default avatarJacopo Mondi <jacopo@jmondi.org>
parent fa4d3671
...@@ -620,14 +620,7 @@ static void rza1_pin_reset(struct rza1_port *port, unsigned int pin) ...@@ -620,14 +620,7 @@ static void rza1_pin_reset(struct rza1_port *port, unsigned int pin)
static inline int rza1_pin_get_direction(struct rza1_port *port, static inline int rza1_pin_get_direction(struct rza1_port *port,
unsigned int pin) unsigned int pin)
{ {
unsigned long irqflags; return !!rza1_get_bit(port, RZA1_PM_REG, pin);
int input;
spin_lock_irqsave(&port->lock, irqflags);
input = rza1_get_bit(port, RZA1_PM_REG, pin);
spin_unlock_irqrestore(&port->lock, irqflags);
return !!input;
} }
/** /**
...@@ -671,14 +664,7 @@ static inline void rza1_pin_set(struct rza1_port *port, unsigned int pin, ...@@ -671,14 +664,7 @@ static inline void rza1_pin_set(struct rza1_port *port, unsigned int pin,
static inline int rza1_pin_get(struct rza1_port *port, unsigned int pin) static inline int rza1_pin_get(struct rza1_port *port, unsigned int pin)
{ {
unsigned long irqflags; return rza1_get_bit(port, RZA1_PPR_REG, pin);
int val;
spin_lock_irqsave(&port->lock, irqflags);
val = rza1_get_bit(port, RZA1_PPR_REG, pin);
spin_unlock_irqrestore(&port->lock, irqflags);
return val;
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment