On Thu, Mar 2, 2023 at 1:52 PM Keguang Zhang <keguang.zhang@xxxxxxxxx> wrote: > > This patch replace __raw_readl() & __raw_writel() with readl() & writel(). > Please say WHY you're doing this. Bart > Signed-off-by: Keguang Zhang <keguang.zhang@xxxxxxxxx> > --- > V1 -> V2: Split this change to a separate patch > --- > drivers/gpio/gpio-loongson1.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpio/gpio-loongson1.c b/drivers/gpio/gpio-loongson1.c > index 8862c9ea0d41..b6c11caa3ade 100644 > --- a/drivers/gpio/gpio-loongson1.c > +++ b/drivers/gpio/gpio-loongson1.c > @@ -23,8 +23,8 @@ static int ls1x_gpio_request(struct gpio_chip *gc, unsigned int offset) > unsigned long flags; > > raw_spin_lock_irqsave(&gc->bgpio_lock, flags); > - __raw_writel(__raw_readl(gpio_reg_base + GPIO_CFG) | BIT(offset), > - gpio_reg_base + GPIO_CFG); > + writel(readl(gpio_reg_base + GPIO_CFG) | BIT(offset), > + gpio_reg_base + GPIO_CFG); > raw_spin_unlock_irqrestore(&gc->bgpio_lock, flags); > > return 0; > @@ -35,8 +35,8 @@ static void ls1x_gpio_free(struct gpio_chip *gc, unsigned int offset) > unsigned long flags; > > raw_spin_lock_irqsave(&gc->bgpio_lock, flags); > - __raw_writel(__raw_readl(gpio_reg_base + GPIO_CFG) & ~BIT(offset), > - gpio_reg_base + GPIO_CFG); > + writel(readl(gpio_reg_base + GPIO_CFG) & ~BIT(offset), > + gpio_reg_base + GPIO_CFG); > raw_spin_unlock_irqrestore(&gc->bgpio_lock, flags); > } > > -- > 2.34.1 >
