From d891caeae96397ba7a5ab01d6722996ea3a1dd16 Mon Sep 17 00:00:00 2001 From: Heinrich Schuchardt Date: Fri, 29 Sep 2023 23:53:55 +0200 Subject: [PATCH] gpio/starfive: redundant readl() call In starfive_gpio_direction_output() readl() is called twice to read the gpio direction register. The result of the first read is discarded. Remove the redundant read. Fixes: 908be1b85c8f ("gpio/starfive: add gpio driver and support gpio reset") Signed-off-by: Heinrich Schuchardt Reviewed-by: Anup Patel --- lib/utils/gpio/fdt_gpio_starfive.c | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/utils/gpio/fdt_gpio_starfive.c b/lib/utils/gpio/fdt_gpio_starfive.c index f430b135..bd2a9f21 100644 --- a/lib/utils/gpio/fdt_gpio_starfive.c +++ b/lib/utils/gpio/fdt_gpio_starfive.c @@ -37,7 +37,6 @@ static int starfive_gpio_direction_output(struct gpio_pin *gp, int value) reg_addr = chip->addr + gp->offset; reg_addr &= ~(STARFIVE_GPIO_REG_SHIFT_MASK); - val = readl((void *)(reg_addr)); shift_bits = (gp->offset & STARFIVE_GPIO_REG_SHIFT_MASK) << STARFIVE_GPIO_SHIFT_BITS; bit_mask = STARFIVE_GPIO_MASK << shift_bits;