sunxi: gpio: Return void from setter functions
authorSamuel Holland <samuel@sholland.org>
Thu, 21 Oct 2021 04:52:54 +0000 (23:52 -0500)
committerAndre Przywara <andre.przywara@arm.com>
Sun, 30 Jan 2022 01:04:30 +0000 (01:04 +0000)
The return values of these functions are always zero, and they are
never checked. Since they are not needed, remove them.

Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Signed-off-by: Andre Przywara <andre.przywara@arm.com>
arch/arm/include/asm/arch-sunxi/gpio.h
arch/arm/mach-sunxi/pinmux.c

index f3ab1ae..2b72b22 100644 (file)
@@ -226,8 +226,8 @@ void sunxi_gpio_set_cfgbank(struct sunxi_gpio *pio, int bank_offset, u32 val);
 void sunxi_gpio_set_cfgpin(u32 pin, u32 val);
 int sunxi_gpio_get_cfgbank(struct sunxi_gpio *pio, int bank_offset);
 int sunxi_gpio_get_cfgpin(u32 pin);
-int sunxi_gpio_set_drv(u32 pin, u32 val);
-int sunxi_gpio_set_pull(u32 pin, u32 val);
+void sunxi_gpio_set_drv(u32 pin, u32 val);
+void sunxi_gpio_set_pull(u32 pin, u32 val);
 int sunxi_name_to_gpio(const char *name);
 
 #if !defined CONFIG_SPL_BUILD && defined CONFIG_AXP_GPIO
index 642483f..cf9d9da 100644 (file)
@@ -45,7 +45,7 @@ int sunxi_gpio_get_cfgpin(u32 pin)
        return sunxi_gpio_get_cfgbank(pio, pin);
 }
 
-int sunxi_gpio_set_drv(u32 pin, u32 val)
+void sunxi_gpio_set_drv(u32 pin, u32 val)
 {
        u32 bank = GPIO_BANK(pin);
        u32 index = GPIO_DRV_INDEX(pin);
@@ -53,11 +53,9 @@ int sunxi_gpio_set_drv(u32 pin, u32 val)
        struct sunxi_gpio *pio = BANK_TO_GPIO(bank);
 
        clrsetbits_le32(&pio->drv[0] + index, 0x3 << offset, val << offset);
-
-       return 0;
 }
 
-int sunxi_gpio_set_pull(u32 pin, u32 val)
+void sunxi_gpio_set_pull(u32 pin, u32 val)
 {
        u32 bank = GPIO_BANK(pin);
        u32 index = GPIO_PULL_INDEX(pin);
@@ -65,6 +63,4 @@ int sunxi_gpio_set_pull(u32 pin, u32 val)
        struct sunxi_gpio *pio = BANK_TO_GPIO(bank);
 
        clrsetbits_le32(&pio->pull[0] + index, 0x3 << offset, val << offset);
-
-       return 0;
 }