pinctrl: ralink: mt76x8: fix pinmux function
authorWeihao Li <cn.liweihao@gmail.com>
Mon, 27 May 2024 02:20:36 +0000 (10:20 +0800)
committerLinus Walleij <linus.walleij@linaro.org>
Mon, 17 Jun 2024 07:23:02 +0000 (09:23 +0200)
commit0a2bcac1a22d9b78852a6039775455a798a095a0
tree6edfb59f6e5181deb454ce89d1c8209a8bde64f8
parent72636eb9c1ea97d86b35abc79d9ffc58286d7db7
pinctrl: ralink: mt76x8: fix pinmux function

The current version of the pinctrl driver has some issues:

1. Duplicated "gpio" pmx function

The common code will add a "gpio" pmx functon to every pin group, so
it's not necessary to define a separate "gpio" pmx function in pin
groups.

2. Duplicated pmx function name

There are some same function name in different pin groups, which will
cause some problems. For example, when we want to use PAD_GPIO0 as
refclk output function, the common clk framework code will search the
entire pin function lists, then return the first one matched, in this
case the matched function list only include the PAD_CO_CLKO pin group
because there are three "refclk" pin function, which is added by
refclk_grp, spi_cs1_grp and gpio_grp.

To solve this problem, a simple way is just add a pingrp refix to
function name like mt7620 pinctrl driver does.

3. Useless "-" or "rsvd" functon

It's really unnecessary to add a reserved pin mux function to the
function lists, because we never use it.

Signed-off-by: Weihao Li <cn.liweihao@gmail.com>
Link: https://lore.kernel.org/r/20240527022036.31985-1-user@blabla
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/mediatek/pinctrl-mt76x8.c