| Message ID | 1435914709-15092-2-git-send-email-albeu@free.fr |
|---|---|
| State | New |
| Headers | show |
2015-07-03 12:11 GMT+03:00 Alban Bedel <albeu@free.fr>: > To prepare moving the GPIO driver to drivers/gpio remove the > platform specific pinmux API. As it is not used by any board, > and such functionality should better be implemented using the > pinmux subsystem just removing it seems to be the best option. > For reference: OpenWRT uses this functions to activate UART.
On Sat, 4 Jul 2015 19:58:32 +0300 Sergey Ryazanov <ryazanov.s.a@gmail.com> wrote: > 2015-07-03 12:11 GMT+03:00 Alban Bedel <albeu@free.fr>: > > To prepare moving the GPIO driver to drivers/gpio remove the > > platform specific pinmux API. As it is not used by any board, > > and such functionality should better be implemented using the > > pinmux subsystem just removing it seems to be the best option. > > > For reference: OpenWRT uses this functions to activate UART. The pinctrl-single driver should be usable for all SoC where this code was used. I haven't tried it yet, but it should only be a matter of writing the DTS down. Alban -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Jul 3, 2015 at 11:11 AM, Alban Bedel <albeu@free.fr> wrote: > To prepare moving the GPIO driver to drivers/gpio remove the > platform specific pinmux API. As it is not used by any board, > and such functionality should better be implemented using the > pinmux subsystem just removing it seems to be the best option. > > Signed-off-by: Alban Bedel <albeu@free.fr> Reviewed-by: Linus Walleij <linus.walleij@linaro.org> Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Sat, Jul 4, 2015 at 6:58 PM, Sergey Ryazanov <ryazanov.s.a@gmail.com> wrote: > 2015-07-03 12:11 GMT+03:00 Alban Bedel <albeu@free.fr>: >> To prepare moving the GPIO driver to drivers/gpio remove the >> platform specific pinmux API. As it is not used by any board, >> and such functionality should better be implemented using the >> pinmux subsystem just removing it seems to be the best option. >> > For reference: OpenWRT uses this functions to activate UART. OpenWRT is wholeheartedly and warmly welcome to work upstream, I will help whenever I can. I have seen their patch stack and it makes me sad :( Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/mips/ath79/common.h b/arch/mips/ath79/common.h index e5ea712..ca7cc19 100644 --- a/arch/mips/ath79/common.h +++ b/arch/mips/ath79/common.h @@ -25,9 +25,6 @@ unsigned long ath79_get_sys_clk_rate(const char *id); void ath79_ddr_ctrl_init(void); void ath79_ddr_wb_flush(unsigned int reg); -void ath79_gpio_function_enable(u32 mask); -void ath79_gpio_function_disable(u32 mask); -void ath79_gpio_function_setup(u32 set, u32 clear); void ath79_gpio_init(void); #endif /* __ATH79_COMMON_H */ diff --git a/arch/mips/ath79/gpio.c b/arch/mips/ath79/gpio.c index f59ccb2..c3c92eb 100644 --- a/arch/mips/ath79/gpio.c +++ b/arch/mips/ath79/gpio.c @@ -24,8 +24,6 @@ #include <linux/of_device.h> #include <asm/mach-ath79/ar71xx_regs.h> -#include <asm/mach-ath79/ath79.h> -#include "common.h" static void __iomem *ath79_gpio_base; static u32 ath79_gpio_count; @@ -139,47 +137,6 @@ static struct gpio_chip ath79_gpio_chip = { .base = 0, }; -static void __iomem *ath79_gpio_get_function_reg(void) -{ - u32 reg = 0; - - if (soc_is_ar71xx() || - soc_is_ar724x() || - soc_is_ar913x() || - soc_is_ar933x()) - reg = AR71XX_GPIO_REG_FUNC; - else if (soc_is_ar934x()) - reg = AR934X_GPIO_REG_FUNC; - else - BUG(); - - return ath79_gpio_base + reg; -} - -void ath79_gpio_function_setup(u32 set, u32 clear) -{ - void __iomem *reg = ath79_gpio_get_function_reg(); - unsigned long flags; - - spin_lock_irqsave(&ath79_gpio_lock, flags); - - __raw_writel((__raw_readl(reg) & ~clear) | set, reg); - /* flush write */ - __raw_readl(reg); - - spin_unlock_irqrestore(&ath79_gpio_lock, flags); -} - -void ath79_gpio_function_enable(u32 mask) -{ - ath79_gpio_function_setup(mask, 0); -} - -void ath79_gpio_function_disable(u32 mask) -{ - ath79_gpio_function_setup(0, mask); -} - static const struct of_device_id ath79_gpio_of_match[] = { { .compatible = "qca,ar7100-gpio" }, { .compatible = "qca,ar9340-gpio" },
To prepare moving the GPIO driver to drivers/gpio remove the platform specific pinmux API. As it is not used by any board, and such functionality should better be implemented using the pinmux subsystem just removing it seems to be the best option. Signed-off-by: Alban Bedel <albeu@free.fr> --- arch/mips/ath79/common.h | 3 --- arch/mips/ath79/gpio.c | 43 ------------------------------------------- 2 files changed, 46 deletions(-)