[PATCH v6] pinctrl: nuvoton: Add NPCM8xx pinctrl driver
Sean Anderson
seanga2 at gmail.com
Fri May 27 06:06:45 CEST 2022
On 5/3/22 1:33 AM, Stanley Chu wrote:
> Add Nuvoton BMC NPCM845 Pinmux and Pinconf support.
>
> Signed-off-by: Stanley Chu <yschu at nuvoton.com>
> ---
> v6:
> - sync pin name with Linux driver
> - add support for gpi35/gpi36/gpio183~189
> v5:
> - lower-case hex consistently
> - use uint type for pin list in the group_config struct
> v4:
> - correct the pin flags, add slew rate control suuport for rgmii pins
> v3:
> - separate group names and function names in different tables
> to allow for adding additional functions
> v2:
> - drop the WDnRCRB/CORSTCB register access, it is not for
> GPIO modules reset control
> ---
> drivers/pinctrl/Kconfig | 1 +
> drivers/pinctrl/Makefile | 1 +
> drivers/pinctrl/nuvoton/Kconfig | 12 +
> drivers/pinctrl/nuvoton/Makefile | 1 +
> drivers/pinctrl/nuvoton/pinctrl-npcm8xx.c | 1225 +++++++++++++++++++++
> 5 files changed, 1240 insertions(+)
> create mode 100644 drivers/pinctrl/nuvoton/Kconfig
> create mode 100644 drivers/pinctrl/nuvoton/Makefile
> create mode 100644 drivers/pinctrl/nuvoton/pinctrl-npcm8xx.c
>
> diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig
> index 13033198f9..e14e885c3e 100644
> --- a/drivers/pinctrl/Kconfig
> +++ b/drivers/pinctrl/Kconfig
> @@ -339,6 +339,7 @@ source "drivers/pinctrl/mscc/Kconfig"
> source "drivers/pinctrl/mtmips/Kconfig"
> source "drivers/pinctrl/mvebu/Kconfig"
> source "drivers/pinctrl/nexell/Kconfig"
> +source "drivers/pinctrl/nuvoton/Kconfig"
> source "drivers/pinctrl/nxp/Kconfig"
> source "drivers/pinctrl/renesas/Kconfig"
> source "drivers/pinctrl/rockchip/Kconfig"
> diff --git a/drivers/pinctrl/Makefile b/drivers/pinctrl/Makefile
> index 9b4978253b..aa31f31c16 100644
> --- a/drivers/pinctrl/Makefile
> +++ b/drivers/pinctrl/Makefile
> @@ -12,6 +12,7 @@ obj-$(CONFIG_ARCH_ASPEED) += aspeed/
> obj-$(CONFIG_ARCH_ATH79) += ath79/
> obj-$(CONFIG_PINCTRL_INTEL) += intel/
> obj-$(CONFIG_ARCH_MTMIPS) += mtmips/
> +obj-$(CONFIG_ARCH_NPCM) += nuvoton/
> obj-$(CONFIG_ARCH_RMOBILE) += renesas/
> obj-$(CONFIG_PINCTRL_SANDBOX) += pinctrl-sandbox.o
> obj-$(CONFIG_PINCTRL_SUNXI) += sunxi/
> diff --git a/drivers/pinctrl/nuvoton/Kconfig b/drivers/pinctrl/nuvoton/Kconfig
> new file mode 100644
> index 0000000000..519539d6ae
> --- /dev/null
> +++ b/drivers/pinctrl/nuvoton/Kconfig
> @@ -0,0 +1,12 @@
> +config PINCTRL_NPCM8XX
> + bool "Pinctrl driver for Nuvoton NPCM8XX"
> + depends on DM && PINCTRL_GENERIC && ARCH_NPCM8XX
> + help
> + Support pin muxing and pin configuration on
> + Nuvoton NPCM8XX SoC.
> +
> + The NPCM8XX contains 256 GPIO pins. Most of them are
> + multiplexed with other system functions. These pins can
> + be configured as either GPIO pin or alternate function.
> + It also supports basic configurations such as pull up/down,
> + drive-strength, and slew rate control for some of the pins.
> diff --git a/drivers/pinctrl/nuvoton/Makefile b/drivers/pinctrl/nuvoton/Makefile
> new file mode 100644
> index 0000000000..a6dfdf3672
> --- /dev/null
> +++ b/drivers/pinctrl/nuvoton/Makefile
> @@ -0,0 +1 @@
> +obj-$(CONFIG_PINCTRL_NPCM8XX) += pinctrl-npcm8xx.o
> diff --git a/drivers/pinctrl/nuvoton/pinctrl-npcm8xx.c b/drivers/pinctrl/nuvoton/pinctrl-npcm8xx.c
> new file mode 100644
> index 0000000000..cc49310506
> --- /dev/null
> +++ b/drivers/pinctrl/nuvoton/pinctrl-npcm8xx.c
> @@ -0,0 +1,1225 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Copyright (c) 2022 Nuvoton Technology Corp.
> + */
> +
> +#include <dm.h>
> +#include <errno.h>
> +#include <regmap.h>
> +#include <syscon.h>
> +#include <asm/io.h>
> +#include <dm/device_compat.h>
> +#include <dm/pinctrl.h>
> +#include <linux/bitfield.h>
> +#include <asm/arch/rst.h>
> +
> +/* GCR register offsets */
> +#define WD0RCR 0x38
> +#define WD1RCR 0x3c
> +#define WD2RCR 0x40
> +#define SWRSTC1 0x44
> +#define SWRSTC2 0x48
> +#define SWRSTC3 0x4c
> +#define SWRSTC4 0x50
> +#define CORSTC 0x5c
> +#define FLOCKR1 0x74
> +#define INTCR4 0xc0
> +#define I2CSEGSEL 0xe0
> +#define MFSEL1 0x260
> +#define MFSEL2 0x264
> +#define MFSEL3 0x268
> +#define MFSEL4 0x26c
> +#define MFSEL5 0x270
> +#define MFSEL6 0x274
> +#define MFSEL7 0x278
> +
> +/* GPIO register offsets */
> +#define GPIO_POL 0x08 /* Polarity */
> +#define GPIO_DOUT 0x0c /* Data OUT */
> +#define GPIO_OTYP 0x14 /* Output Type */
> +#define GPIO_PU 0x1c /* Pull-up */
> +#define GPIO_PD 0x20 /* Pull-down */
> +#define GPIO_DBNC 0x24 /* Debounce */
> +#define GPIO_EVEN 0x40 /* Event Enable */
> +#define GPIO_EVST 0x4c /* Event Status */
> +#define GPIO_IEM 0x58 /* Input Enable */
> +#define GPIO_OSRC 0x5c /* Output Slew-Rate Control */
> +#define GPIO_ODSC 0x60 /* Output Drive Strength Control */
> +#define GPIO_OES 0x70 /* Output Enable Set */
> +#define GPIO_OEC 0x74 /* Output Enable Clear */
> +
> +#define NPCM8XX_GPIO_PER_BANK 32
> +#define GPIOX_OFFSET 16
> +
> +/* The lists contain alternate GPIO pins of the function */
> +/* Serial Interfaces */
> +static const uint hsi1a_pins[] = { 43, 63 };
u8? Seems like you could save a lot of space pretty easily
> +static const uint hsi1b_pins[] = { 44, 62 };
> +static const uint hsi1c_pins[] = { 45, 46, 47, 61 };
> +static const uint hsi2a_pins[] = { 48, 49 };
> +static const uint hsi2b_pins[] = { 50, 51 };
> +static const uint hsi2c_pins[] = { 52, 53, 54, 55 };
> +static const uint bmcuart0a_pins[] = { 41, 42 };
> +static const uint bmcuart0b_pins[] = { 48, 49 };
> +static const uint bmcuart1_pins[] = { 43, 63 };
> +static const uint nbu1crts_pins[] = { 44, 62 };
> +static const uint bu2_pins[] = { 96, 97};
> +static const uint bu4_pins[] = { 54, 55 };
> +static const uint bu4b_pins[] = { 98, 99 };
> +static const uint bu5_pins[] = { 52, 53 };
> +static const uint bu5b_pins[] = { 100, 101 };
> +static const uint bu6_pins[] = { 50, 51 };
> +
> +/* SPI/FIU/FLM */
> +static const uint spi0cs1_pins[] = { 32 };
> +static const uint spi1_pins[] = { 175, 176, 177, 203 };
> +static const uint spi1cs1_pins[] = { 233 };
> +static const uint spi1d23_pins[] = { 191, 192 };
> +static const uint spi1cs2_pins[] = { 191 };
> +static const uint spi1cs3_pins[] = { 192 };
> +static const uint spi3_pins[] = { 183, 184, 185, 186 };
> +static const uint spi3cs1_pins[] = { 187 };
> +static const uint spi3quad_pins[] = { 188, 189 };
> +static const uint spi3cs2_pins[] = { 188 };
> +static const uint spi3cs3_pins[] = { 189 };
> +static const uint spix_pins[] = { 224, 225, 226, 227, 229, 230 };
> +static const uint spixcs1_pins[] = { 228 };
> +static const uint pspi_pins[] = { 17, 18, 19 };
> +static const uint fm2_pins[] = { 224, 225, 226, 227, 228, 229, 230 };
> +static const uint fm1_pins[] = { 175, 176, 177, 203, 191, 192, 233 };
> +static const uint fm0_pins[] = { 194, 195, 196, 202, 199, 198, 197 };
> +
> +/* I2C */
> +static const uint smb0_pins[] = { 115, 114 };
> +static const uint smb0b_pins[] = { 195, 194 };
> +static const uint smb0c_pins[] = { 202, 196 };
> +static const uint smb0d_pins[] = { 198, 199 };
> +static const uint smb0den_pins[] = { 197 };
> +static const uint smb1_pins[] = { 117, 116 };
> +static const uint smb1b_pins[] = { 126, 127 };
> +static const uint smb1c_pins[] = { 124, 125 };
> +static const uint smb1d_pins[] = { 4, 5 };
> +static const uint smb2_pins[] = { 119, 118 };
> +static const uint smb2b_pins[] = { 122, 123 };
> +static const uint smb2c_pins[] = { 120, 121 };
> +static const uint smb2d_pins[] = { 6, 7 };
> +static const uint smb3_pins[] = { 30, 31 };
> +static const uint smb3b_pins[] = { 39, 40 };
> +static const uint smb3c_pins[] = { 37, 38 };
> +static const uint smb3d_pins[] = { 59, 60 };
> +static const uint smb4_pins[] = { 28, 29 };
> +static const uint smb4b_pins[] = { 18, 19 };
> +static const uint smb4c_pins[] = { 20, 21 };
> +static const uint smb4d_pins[] = { 22, 23 };
> +static const uint smb5_pins[] = { 26, 27 };
> +static const uint smb5b_pins[] = { 13, 12 };
> +static const uint smb5c_pins[] = { 15, 14 };
> +static const uint smb5d_pins[] = { 94, 93 };
> +static const uint smb6_pins[] = { 172, 171 };
> +static const uint smb6b_pins[] = { 2, 3 };
> +static const uint smb6c_pins[] = { 0, 1 };
> +static const uint smb6d_pins[] = { 10, 11 };
> +static const uint smb7_pins[] = { 174, 173 };
> +static const uint smb7b_pins[] = { 16, 141 };
> +static const uint smb7c_pins[] = { 24, 25 };
> +static const uint smb7d_pins[] = { 142, 143 };
> +static const uint smb8_pins[] = { 129, 128 };
> +static const uint smb9_pins[] = { 131, 130 };
> +static const uint smb10_pins[] = { 133, 132 };
> +static const uint smb11_pins[] = { 135, 134 };
> +static const uint smb12_pins[] = { 221, 220 };
> +static const uint smb13_pins[] = { 223, 222 };
> +static const uint smb14_pins[] = { 22, 23 };
> +static const uint smb14b_pins[] = { 32, 187 };
> +static const uint smb15_pins[] = { 20, 21 };
> +static const uint smb15b_pins[] = { 192, 191 };
> +static const uint smb16_pins[] = { 10, 11 };
> +static const uint smb16b_pins[] = { 218, 219 };
> +static const uint smb17_pins[] = { 3, 2 };
> +static const uint smb18_pins[] = { 0, 1 };
> +static const uint smb19_pins[] = { 60, 59 };
> +static const uint smb20_pins[] = { 234, 235 };
> +static const uint smb21_pins[] = { 169, 170 };
> +static const uint smb22_pins[] = { 40, 39 };
> +static const uint smb23_pins[] = { 38, 37 };
> +static const uint smb23b_pins[] = { 134, 134 };
> +
> +/* I3C */
> +static const uint i3c5_pins[] = { 106, 107 };
> +static const uint i3c4_pins[] = { 33, 34 };
> +static const uint i3c3_pins[] = { 246, 247 };
> +static const uint i3c2_pins[] = { 244, 245 };
> +static const uint i3c1_pins[] = { 242, 243 };
> +static const uint i3c0_pins[] = { 240, 241 };
> +
> +/* Fan/PWM */
> +static const uint fanin0_pins[] = { 64 };
> +static const uint fanin1_pins[] = { 65 };
> +static const uint fanin2_pins[] = { 66 };
> +static const uint fanin3_pins[] = { 67 };
> +static const uint fanin4_pins[] = { 68 };
> +static const uint fanin5_pins[] = { 69 };
> +static const uint fanin6_pins[] = { 70 };
> +static const uint fanin7_pins[] = { 71 };
> +static const uint fanin8_pins[] = { 72 };
> +static const uint fanin9_pins[] = { 73 };
> +static const uint fanin10_pins[] = { 74 };
> +static const uint fanin11_pins[] = { 75 };
> +static const uint fanin12_pins[] = { 76 };
> +static const uint fanin13_pins[] = { 77 };
> +static const uint fanin14_pins[] = { 78 };
> +static const uint fanin15_pins[] = { 79 };
> +static const uint faninx_pins[] = { 175, 176, 177, 203 };
> +static const uint pwm0_pins[] = { 80 };
> +static const uint pwm1_pins[] = { 81 };
> +static const uint pwm2_pins[] = { 82 };
> +static const uint pwm3_pins[] = { 83 };
> +static const uint pwm4_pins[] = { 144 };
> +static const uint pwm5_pins[] = { 145 };
> +static const uint pwm6_pins[] = { 146 };
> +static const uint pwm7_pins[] = { 147 };
> +static const uint pwm8_pins[] = { 220 };
> +static const uint pwm9_pins[] = { 221 };
> +static const uint pwm10_pins[] = { 234 };
> +static const uint pwm11_pins[] = { 235 };
> +
> +/* Network */
> +static const uint rg1mdio_pins[] = { 108, 109 }; /* SGMII1 */
> +static const uint rg2_pins[] = { 110, 111, 112, 113, 208, 209, 210, 211, 212,
> + 213, 214, 215 }; /* RGMII2 */
> +static const uint rg2mdio_pins[] = { 216, 217 };
> +static const uint rg2refck_pins[] = { 250 };
> +static const uint r1_pins[] = { 178, 179, 180, 181, 182, 193, 201 }; /* RMII1 */
> +static const uint r1err_pins[] = { 56 };
> +static const uint r1oen_pins[] = { 56 };
> +static const uint r1md_pins[] = { 57, 58 };
> +static const uint r1en_pins[] = { };
> +static const uint r2_pins[] = { 84, 85, 86, 87, 88, 89, 200 }; /* RMII2 */
> +static const uint r2md_pins[] = { 91, 92 };
> +static const uint r2err_pins[] = { 90 };
> +static const uint r2oen_pins[] = { 90 };
> +static const uint r2en_pins[] = { };
> +static const uint rmii3_pins[] = { 110, 111, 209, 210, 211, 214, 215 };
> +static const uint r3rxer_pins[] = { 212 };
> +static const uint r3oen_pins[] = { 213 };
> +static const uint r3en_pins[] = { };
> +
> +/* MMC */
> +static const uint mmc_pins[] = { 152, 154, 156, 157, 158, 159 };
> +static const uint mmc8_pins[] = { 148, 149, 150, 151 };
> +static const uint mmcwp_pins[] = { 153 };
> +static const uint mmccd_pins[] = { 155 };
> +static const uint mmcrst_pins[] = { 155 };
> +
> +/* JTAG */
> +static const uint jtag2_pins[] = { 43, 44, 45, 46, 47 };
> +static const uint j2j3_pins[] = { 44, 62, 45, 46 };
> +static const uint jm1_pins[] = { 136, 137, 138, 139, 140 };
> +static const uint jm2_pins[] = { 248 };
> +
> +/* Serial I/O Expander*/
> +static const uint iox1_pins[] = { 0, 1, 2, 3 };
> +static const uint iox2_pins[] = { 4, 5, 6, 7 };
> +static const uint ioxh_pins[] = { 10, 11, 24, 25 };
> +
> +/* LPC/eSPI */
> +static const uint lpc_pins[] = { 95, 161, 163, 164, 165, 166, 167 };
> +static const uint espi_pins[] = { 95, 161, 163, 164, 165, 166, 167, 168 };
> +static const uint clkrun_pins[] = { 162 };
> +static const uint serirq_pins[] = { 168 };
> +static const uint nprd_smi_pins[] = { 190 };
> +static const uint scipme_pins[] = { 169 };
> +static const uint sci_pins[] = { 170 };
> +
> +/* VGA */
> +static const uint vgadig_pins[] = { 102, 103, 104, 105 };
> +static const uint gspi_pins[] = { 12, 13, 14, 15 };
> +
> +/* PCIe */
> +static const uint clkreq_pins[] = { 231 };
> +
> +/* MISC */
> +static const uint ga20kbc_pins[] = { 94, 93 };
> +static const uint clkout_pins[] = { 160 };
> +static const uint wdog1_pins[] = { 218 };
> +static const uint wdog2_pins[] = { 219 };
> +static const uint ddr_pins[] = { 110, 111, 112, 113, 208, 209, 210, 211, 212,
> + 213, 214, 215, 216, 217, 250 };
> +
> +/* TIP/COP */
> +static const uint tp_gpio7_pins[] = { 96 };
> +static const uint tp_gpio6_pins[] = { 97 };
> +static const uint tp_gpio5_pins[] = { 98 };
> +static const uint tp_gpio4_pins[] = { 99 };
> +static const uint tp_gpio3_pins[] = { 100 };
> +static const uint tp_gpio2_pins[] = { 16 };
> +static const uint tp_gpio1_pins[] = { 9 };
> +static const uint tp_gpio0_pins[] = { 8 };
> +static const uint tp_gpio5b_pins[] = { 58 };
> +static const uint tp_gpio4b_pins[] = { 57 };
> +static const uint tp_gpio2b_pins[] = { 101 };
> +static const uint tp_gpio1b_pins[] = { 92 };
> +static const uint tp_gpio0b_pins[] = { 91 };
> +static const uint tp_uart_pins[] = { 50, 51 };
> +static const uint tp_smb2_pins[] = { 24, 25 };
> +static const uint tp_smb1_pins[] = { 142, 143 };
> +static const uint tp_jtag3_pins[] = { 44, 62, 45, 46 };
> +static const uint cp1gpio2c_pins[] = { 101 };
> +static const uint cp1gpio3c_pins[] = { 100 };
> +static const uint cp1gpio0b_pins[] = { 127 };
> +static const uint cp1gpio1b_pins[] = { 126 };
> +static const uint cp1gpio2b_pins[] = { 125 };
> +static const uint cp1gpio3b_pins[] = { 124 };
> +static const uint cp1gpio4b_pins[] = { 99 };
> +static const uint cp1gpio5b_pins[] = { 98 };
> +static const uint cp1gpio6b_pins[] = { 97 };
> +static const uint cp1gpio7b_pins[] = { 96 };
> +static const uint cp1gpio0_pins[] = { };
> +static const uint cp1gpio1_pins[] = { };
> +static const uint cp1gpio2_pins[] = { };
> +static const uint cp1gpio3_pins[] = { };
> +static const uint cp1gpio4_pins[] = { };
> +static const uint cp1gpio5_pins[] = { 17 };
> +static const uint cp1gpio6_pins[] = { 91 };
> +static const uint cp1gpio7_pins[] = { 92 };
> +static const uint cp1utxd_pins[] = { 42 };
> +static const uint cp1urxd_pins[] = { 41 };
> +
> +/* General IO */
> +static const uint gpi35_pins[] = { 35 };
> +static const uint gpi36_pins[] = { 36 };
> +static const uint gpio1836_pins[] = { 183, 184, 185, 186 };
> +static const uint gpio1889_pins[] = { 188, 189 };
> +static const uint gpio187_pins[] = { 187 };
> +static const uint lkgpo0_pins[] = { 16 };
> +static const uint lkgpo1_pins[] = { 8 };
> +static const uint lkgpo2_pins[] = { 9 };
> +static const uint hgpio0_pins[] = { 20 };
> +static const uint hgpio1_pins[] = { 21 };
> +static const uint hgpio2_pins[] = { 22 };
> +static const uint hgpio3_pins[] = { 23 };
> +static const uint hgpio4_pins[] = { 24 };
> +static const uint hgpio5_pins[] = { 25 };
> +static const uint hgpio6_pins[] = { 59 };
> +static const uint hgpio7_pins[] = { 60 };
> +static char *gpio_func_name = "gpio";
> +static char **npcm8xx_funcs;
> +static int npcm8xx_num_funcs;
This should be driver private data
> +
> +struct npcm8xx_pinctrl_priv {
> + void __iomem *gpio_base;
> + struct regmap *gcr_regmap;
> + struct regmap *rst_regmap;
> +};
> +
> +struct group_config {
> + char *name;
> + const uint *pins;
> + uint npins;
> + u32 reg; /* Register of setting func */
> + u32 bit;
> +};
> +
> +#define GRP(x, _reg, _bit) { \
> + .name = #x, \
> + .pins = x## _pins, \
> + .npins = ARRAY_SIZE(x## _pins), \
> + .reg = _reg, \
> + .bit = _bit, \
> + }
I wonder if you could write this like
#define GRP(_name, _reg, _bit, ...) { \
.name = #_name, \
.pins = { ##__VA_ARGS__ }, \
.npins = ARRAY_SIZE({ ##__VA_ARGS__ }), \
.reg = _reg, \
.bit = _bit, \
}
that way you could do something like
GRP(jtag2, MFSEL4, 0, 43, 44, 45, 46, 47),
and then eveything is specified on one line
although, I don't think you can specify array literals like that (only strings), so you'd have to do something like
#define PIN_LIST \
GRP(...), \
etc
#define GRP(_name, _reg, _bit, ...) \
static const uint _name##_pins[] = { ##__VA_ARGS__ };
PIN_LIST
#undef GRP
#define GRP(...) \
same as above...
static const struct group_config npcm8xx_groups[] = {
PIN_LIST
};
> +static const struct group_config npcm8xx_groups[] = {
> + GRP(smb3, MFSEL1, 0),
> + GRP(smb4, MFSEL1, 1),
> + GRP(smb5, MFSEL1, 2),
> + GRP(spi0cs1, MFSEL1, 3),
> + GRP(hsi1c, MFSEL1, 4),
> + GRP(hsi2c, MFSEL1, 5),
> + GRP(smb0, MFSEL1, 6),
> + GRP(smb1, MFSEL1, 7),
> + GRP(smb2, MFSEL1, 8),
> + GRP(bmcuart0a, MFSEL1, 9),
> + GRP(hsi1a, MFSEL1, 10),
> + GRP(hsi2a, MFSEL1, 11),
> + GRP(r1err, MFSEL1, 12),
> + GRP(r1md, MFSEL1, 13),
> + GRP(r2, MFSEL1, 14),
> + GRP(r2err, MFSEL1, 15),
> + GRP(r2md, MFSEL1, 16),
> + GRP(ga20kbc, MFSEL1, 17),
> + GRP(clkout, MFSEL1, 21),
> + GRP(sci, MFSEL1, 22),
> + GRP(gspi, MFSEL1, 24),
> + GRP(lpc, MFSEL1, 26),
> + GRP(hsi1b, MFSEL1, 28),
> + GRP(hsi2b, MFSEL1, 29),
> + GRP(iox1, MFSEL1, 30),
> + GRP(serirq, MFSEL1, 31),
> + GRP(fanin0, MFSEL2, 0),
> + GRP(fanin1, MFSEL2, 1),
> + GRP(fanin2, MFSEL2, 2),
> + GRP(fanin3, MFSEL2, 3),
> + GRP(fanin4, MFSEL2, 4),
> + GRP(fanin5, MFSEL2, 5),
> + GRP(fanin6, MFSEL2, 6),
> + GRP(fanin7, MFSEL2, 7),
> + GRP(fanin8, MFSEL2, 8),
> + GRP(fanin9, MFSEL2, 9),
> + GRP(fanin10, MFSEL2, 10),
> + GRP(fanin11, MFSEL2, 11),
> + GRP(fanin12, MFSEL2, 12),
> + GRP(fanin13, MFSEL2, 13),
> + GRP(fanin14, MFSEL2, 14),
> + GRP(fanin15, MFSEL2, 15),
> + GRP(pwm0, MFSEL2, 16),
> + GRP(pwm1, MFSEL2, 17),
> + GRP(pwm2, MFSEL2, 18),
> + GRP(pwm3, MFSEL2, 19),
> + GRP(pwm4, MFSEL2, 20),
> + GRP(pwm5, MFSEL2, 21),
> + GRP(pwm6, MFSEL2, 22),
> + GRP(pwm7, MFSEL2, 23),
> + GRP(hgpio0, MFSEL2, 24),
> + GRP(hgpio1, MFSEL2, 25),
> + GRP(hgpio2, MFSEL2, 26),
> + GRP(hgpio3, MFSEL2, 27),
> + GRP(hgpio4, MFSEL2, 28),
> + GRP(hgpio5, MFSEL2, 29),
> + GRP(hgpio6, MFSEL2, 30),
> + GRP(hgpio7, MFSEL2, 31),
> + GRP(scipme, MFSEL3, 0),
> + GRP(smb6, MFSEL3, 1),
> + GRP(smb7, MFSEL3, 2),
> + GRP(faninx, MFSEL3, 3),
> + GRP(spi1, MFSEL3, 4),
> + GRP(smb12, MFSEL3, 5),
> + GRP(smb13, MFSEL3, 6),
> + GRP(smb14, MFSEL3, 7),
> + GRP(smb15, MFSEL3, 8),
> + GRP(r1, MFSEL3, 9),
> + GRP(mmc, MFSEL3, 10),
> + GRP(mmc8, MFSEL3, 11),
> + GRP(pspi, MFSEL3, 13),
> + GRP(iox2, MFSEL3, 14),
> + GRP(clkrun, MFSEL3, 16),
> + GRP(ioxh, MFSEL3, 18),
> + GRP(wdog1, MFSEL3, 19),
> + GRP(wdog2, MFSEL3, 20),
> + GRP(i3c5, MFSEL3, 22),
> + GRP(bmcuart1, MFSEL3, 24),
> + GRP(mmccd, MFSEL3, 25),
> + GRP(ddr, MFSEL3, 26),
> + GRP(jtag2, MFSEL4, 0),
> + GRP(bmcuart0b, MFSEL4, 1),
> + GRP(mmcrst, MFSEL4, 6),
> + GRP(espi, MFSEL4, 8),
> + GRP(clkreq, MFSEL4, 9),
> + GRP(smb8, MFSEL4, 11),
> + GRP(smb9, MFSEL4, 12),
> + GRP(smb10, MFSEL4, 13),
> + GRP(smb11, MFSEL4, 14),
> + GRP(spi3, MFSEL4, 16),
> + GRP(spi3cs1, MFSEL4, 17),
> + GRP(spi3cs2, MFSEL4, 18),
> + GRP(spi3cs3, MFSEL4, 19),
> + GRP(spi3quad, MFSEL4, 20),
> + GRP(rg1mdio, MFSEL4, 21),
> + GRP(bu2, MFSEL4, 22),
> + GRP(rg2mdio, MFSEL4, 23),
> + GRP(rg2, MFSEL4, 24),
> + GRP(spix, MFSEL4, 27),
> + GRP(spixcs1, MFSEL4, 28),
> + GRP(spi1cs1, MFSEL5, 0),
> + GRP(jm2, MFSEL5, 1),
> + GRP(j2j3, MFSEL5, 2),
> + GRP(spi1d23, MFSEL5, 3),
> + GRP(spi1cs2, MFSEL5, 4),
> + GRP(spi1cs3, MFSEL5, 5),
> + GRP(bu6, MFSEL5, 6),
> + GRP(bu5, MFSEL5, 7),
> + GRP(bu4, MFSEL5, 8),
> + GRP(r1oen, MFSEL5, 9),
> + GRP(r2oen, MFSEL5, 10),
> + GRP(rmii3, MFSEL5, 11),
> + GRP(bu5b, MFSEL5, 12),
> + GRP(bu4b, MFSEL5, 13),
> + GRP(r3oen, MFSEL5, 14),
> + GRP(jm1, MFSEL5, 15),
> + GRP(gpi35, MFSEL5, 16),
> + GRP(i3c0, MFSEL5, 17),
> + GRP(gpi36, MFSEL5, 18),
> + GRP(i3c1, MFSEL5, 19),
> + GRP(tp_gpio4b, MFSEL5, 20),
> + GRP(i3c2, MFSEL5, 21),
> + GRP(tp_gpio5b, MFSEL5, 22),
> + GRP(i3c3, MFSEL5, 23),
> + GRP(smb16, MFSEL5, 24),
> + GRP(smb17, MFSEL5, 25),
> + GRP(smb18, MFSEL5, 26),
> + GRP(smb19, MFSEL5, 27),
> + GRP(smb20, MFSEL5, 28),
> + GRP(smb21, MFSEL5, 29),
> + GRP(smb22, MFSEL5, 30),
> + GRP(smb23, MFSEL5, 31),
> + GRP(smb23b, MFSEL6, 0),
> + GRP(cp1utxd, MFSEL6, 1),
> + GRP(cp1gpio0, MFSEL6, 2),
> + GRP(cp1gpio1, MFSEL6, 3),
> + GRP(cp1gpio2, MFSEL6, 4),
> + GRP(cp1gpio3, MFSEL6, 5),
> + GRP(cp1gpio4, MFSEL6, 6),
> + GRP(cp1gpio5, MFSEL6, 7),
> + GRP(cp1gpio6, MFSEL6, 8),
> + GRP(cp1gpio7, MFSEL6, 9),
> + GRP(i3c4, MFSEL6, 10),
> + GRP(pwm8, MFSEL6, 11),
> + GRP(pwm9, MFSEL6, 12),
> + GRP(pwm10, MFSEL6, 13),
> + GRP(pwm11, MFSEL6, 14),
> + GRP(nbu1crts, MFSEL6, 15),
> + GRP(fm0, MFSEL6, 16),
> + GRP(fm1, MFSEL6, 17),
> + GRP(fm2, MFSEL6, 18),
> + GRP(gpio1836, MFSEL6, 19),
> + GRP(cp1gpio0b, MFSEL6, 20),
> + GRP(cp1gpio1b, MFSEL6, 21),
> + GRP(cp1gpio2b, MFSEL6, 22),
> + GRP(cp1gpio3b, MFSEL6, 23),
> + GRP(cp1gpio7b, MFSEL6, 24),
> + GRP(cp1gpio6b, MFSEL6, 25),
> + GRP(cp1gpio5b, MFSEL6, 26),
> + GRP(cp1gpio4b, MFSEL6, 27),
> + GRP(cp1gpio3c, MFSEL6, 28),
> + GRP(cp1gpio2c, MFSEL6, 29),
> + GRP(r3rxer, MFSEL6, 30),
> + GRP(cp1urxd, MFSEL6, 31),
> + GRP(tp_gpio0, MFSEL7, 0),
> + GRP(tp_gpio1, MFSEL7, 1),
> + GRP(tp_gpio2, MFSEL7, 2),
> + GRP(tp_gpio3, MFSEL7, 3),
> + GRP(tp_gpio4, MFSEL7, 4),
> + GRP(tp_gpio5, MFSEL7, 5),
> + GRP(tp_gpio6, MFSEL7, 6),
> + GRP(tp_gpio7, MFSEL7, 7),
> + GRP(tp_gpio0b, MFSEL7, 8),
> + GRP(tp_gpio1b, MFSEL7, 9),
> + GRP(tp_gpio2b, MFSEL7, 10),
> + GRP(tp_smb1, MFSEL7, 11),
> + GRP(tp_uart, MFSEL7, 12),
> + GRP(tp_jtag3, MFSEL7, 13),
> + GRP(gpio187, MFSEL7, 24),
> + GRP(gpio1889, MFSEL7, 25),
> + GRP(smb14b, MFSEL7, 26),
> + GRP(smb15b, MFSEL7, 27),
> + GRP(tp_smb2, MFSEL7, 28),
> + GRP(vgadig, MFSEL7, 29),
> + GRP(smb16b, MFSEL7, 30),
> + GRP(smb0b, I2CSEGSEL, 0),
> + GRP(smb0c, I2CSEGSEL, 1),
> + GRP(smb0d, I2CSEGSEL, 2),
> + GRP(smb1b, I2CSEGSEL, 5),
> + GRP(smb1c, I2CSEGSEL, 6),
> + GRP(smb1d, I2CSEGSEL, 7),
> + GRP(smb2b, I2CSEGSEL, 8),
> + GRP(smb2c, I2CSEGSEL, 9),
> + GRP(smb2d, I2CSEGSEL, 10),
> + GRP(smb3b, I2CSEGSEL, 11),
> + GRP(smb3c, I2CSEGSEL, 12),
> + GRP(smb3d, I2CSEGSEL, 13),
> + GRP(smb4b, I2CSEGSEL, 14),
> + GRP(smb4c, I2CSEGSEL, 15),
> + GRP(smb4d, I2CSEGSEL, 16),
> + GRP(smb5b, I2CSEGSEL, 19),
> + GRP(smb5c, I2CSEGSEL, 20),
> + GRP(smb5d, I2CSEGSEL, 21),
> + GRP(smb0den, I2CSEGSEL, 22),
> + GRP(smb6b, I2CSEGSEL, 24),
> + GRP(smb6c, I2CSEGSEL, 25),
> + GRP(smb6d, I2CSEGSEL, 26),
> + GRP(smb7b, I2CSEGSEL, 27),
> + GRP(smb7c, I2CSEGSEL, 28),
> + GRP(smb7d, I2CSEGSEL, 29),
> + GRP(lkgpo0, FLOCKR1, 0),
> + GRP(lkgpo1, FLOCKR1, 4),
> + GRP(lkgpo2, FLOCKR1, 8),
> + GRP(nprd_smi, FLOCKR1, 20),
> + GRP(mmcwp, FLOCKR1, 24),
> + GRP(rg2refck, INTCR4, 6),
> + GRP(r1en, INTCR4, 12),
> + GRP(r2en, INTCR4, 13),
> + GRP(r3en, INTCR4, 14),
> +};
> +
> +/* Pin flags */
> +#define SLEW BIT(0) /* Has Slew Control */
> +#define GPIO_ALT BIT(1) /* GPIO function is enabled by setting alternate */
> +#define DSLO_MASK GENMASK(11, 8) /* Drive strength */
> +#define DSHI_MASK GENMASK(15, 12)
> +#define GPIO_IDX_MASK GENMASK(18, 16)
> +#define GPIO_IDX(x) ((x) << 16) /* index of alt_func[] for gpio function */
> +#define DS(lo, hi) (((lo) << 8) | ((hi) << 12))
> +#define DSLO(x) FIELD_GET(DSLO_MASK, x) /* Low DS value */
> +#define DSHI(x) FIELD_GET(DSHI_MASK, x) /* High DS value */
> +#define GPIO_IDX_VAL(x) FIELD_GET(GPIO_IDX_MASK, x)
> +
> +#define MAX_ALT_FUNCS 5 /* Max alternate functions */
> +struct pin_info {
> + int gpio_num;
> + char *name;
> + char *alt_func[MAX_ALT_FUNCS];
> + int num_funcs;
> + u32 flags;
> +};
> +
> +static const struct pin_info npcm8xx_pins[] = {
> + {0, "GPIO0/IOX1_DI/SMB6C_SDA/SMB18_SDA", {"iox1", "smb6c", "smb18"}, 3, SLEW},
> + {1, "GPIO1/IOX1_LD/SMB6C_SCL/SMB18_SCL", {"iox1", "smb6c", "smb18"}, 3, SLEW},
> + {2, "GPIO2/IOX1_CK/SMB6B_SDA/SMB17_SDA", {"iox1", "smb6b", "smb17"}, 3, SLEW},
> + {3, "GPIO3/IOX1_DO/SMB6B_SCL/SMB17_SCL", {"iox1", "smb6b", "smb17"}, 3, SLEW},
> + {4, "GPIO4/IOX2_DI/SMB1D_SDA", {"iox2", "smb1d"}, 2, SLEW},
> + {5, "GPIO5/IOX2_LD/SMB1D_SCL", {"iox2", "smb1d"}, 2, SLEW},
> + {6, "GPIO6/IOX2_CK/SMB2D_SDA", {"iox2", "smb2d"}, 2, SLEW},
> + {7, "GPIO7/IOX2_D0/SMB2D_SCL", {"iox2", "smb2d"}, 2, SLEW},
> + {8, "GPIO8/LKGPO1/TP_GPIO0", {"lkgpo1", "tp_gpio0b"}, 2, DS(8, 12)},
> + {9, "GPIO9/LKGPO2/TP_GPIO1", {"lkgpo2", "tp_gpio1b"}, 2, DS(8, 12)},
> + {10, "GPIO10/IOXH_LD/SMB6D_SCL/SMB16_SCL", {"ioxh", "smb6d", "smb16"}, 3, SLEW},
> + {11, "GPIO11/IOXH_CK/SMB6D_SDA/SMB16_SDA", {"ioxh", "smb6d", "smb16"}, 3, SLEW},
> + {12, "GPIO12/GSPI_CK/SMB5B_SCL", {"gspi", "smb5d"}, 2, SLEW},
> + {13, "GPIO13/GSPI_DO/SMB5B_SDA", {"gspi", "smb5d"}, 2, SLEW},
> + {14, "GPIO14/GSPI_DI/SMB5C_SCL", {"gspi", "smb5c"}, 2, SLEW},
> + {15, "GPIO15/GSPI_CS/SMB5C_SDA", {"gspi", "smb5c"}, 2, SLEW},
> + {16, "GPIO16/SMB7B_SDA/LKGPO0/TP_GPIO2", {"lkgpo0", "smb7b", "tp_gpio2b"}, 3, SLEW},
> + {17, "GPIO17/PSPI_DI/CP1_GPIO5", {"pspi", "cp1gpio5"}, 2, SLEW},
> + {18, "GPIO18/PSPI_D0/SMB4B_SDA", {"pspi", "smb4b"}, 2, SLEW},
> + {19, "GPIO19/PSPI_CK/SMB4B_SCL", {"pspi", "smb4b"}, 2, SLEW},
> + {20, "GPIO20/H_GPIO0/SMB4C_SDA/SMB15_SDA", {"hgpio0", "smb15", "smb4c"}, 3, SLEW},
> + {21, "GPIO21/H_GPIO1/SMB4C_SCL/SMB15_SCL", {"hgpio1", "smb15", "smb4c"}, 3, SLEW},
> + {22, "GPIO22/H_GPIO2/SMB4D_SDA/SMB14_SDA", {"hgpio2", "smb14", "smb4d"}, 3, SLEW},
> + {23, "GPIO23/H_GPIO3/SMB4D_SCL/SMB14_SCL", {"hgpio3", "smb14", "smb4d"}, 3, SLEW},
> + {24, "GPIO24/IOXH_DO/H_GPIO4/SMB7C_SCL/TP_SMB2_SCL",
> + {"hgpio4", "ioxh", "smb7c", "tp_smb2"}, 4, SLEW},
> + {25, "GPIO25/IOXH_DI/H_GPIO4/SMB7C_SDA/TP_SMB2_SDA", {"hgpio5", "ioxh", "smb7c"}, 3, SLEW},
> + {26, "GPIO26/SMB5_SDA", {"smb5"}, 1, 0},
> + {27, "GPIO27/SMB5_SCL", {"smb5"}, 1, 0},
> + {28, "GPIO28/SMB4_SDA", {"smb4"}, 1, 0},
> + {29, "GPIO29/SMB4_SCL", {"smb4"}, 1, 0},
> + {30, "GPIO30/SMB3_SDA", {"smb3"}, 1, 0},
> + {31, "GPIO31/SMB3_SCL", {"smb3"}, 1, 0},
> + {32, "GPIO32/SMB14_SCL/SPI0_nCS1", {"smb14b", "spi0cs1"}, 2, SLEW},
> + {33, "I3C4_SCL", {"i3c4"}, 1, SLEW},
> + {34, "I3C4_SDA", {"i3c4"}, 1, SLEW},
> + {35, "GPI35/MCBPCK", {"gpi35"}, 1, GPIO_ALT | GPIO_IDX(0)},
> + {36, "GPI36/SYSBPCK", {"gpi36"}, 1, GPIO_ALT | GPIO_IDX(0)},
> + {37, "GPIO37/SMB3C_SDA/SMB23_SDA", {"smb3c", "smb23"}, 2, SLEW},
> + {38, "GPIO38/SMB3C_SCL/SMB23_SCL", {"smb3c", "smb23"}, 2, SLEW},
> + {39, "GPIO39/SMB3B_SDA/SMB22_SDA", {"smb3b", "smb22"}, 2, SLEW},
> + {40, "GPIO40/SMB3B_SCL/SMB22_SCL", {"smb3b", "smb22"}, 2, SLEW},
> + {41, "GPIO41/BU0_RXD/CP1U_RXD", {"bmcuart0a", "cp1urxd"}, 2, 0},
> + {42, "GPIO42/BU0_TXD/CP1U_TXD", {"bmcuart0a", "cp1utxd"}, 2, DS(2, 4)},
> + {43, "GPIO43/SI1_RXD/BU1_RXD", {"hsi1a", "bmcuart1"}, 2, 0},
> + {44, "GPIO44/SI1_nCTS/BU1_nCTS/CP_TDI/TP_TDI/CP_TP_TDI",
> + {"hsi1b", "nbu1crts", "jtag2", "tp_jtag3", "j2j3"}, 5, 0},
> + {45, "GPIO45/SI1_nDCD/CP_TMS_SWIO/TP_TMS_SWIO/CP_TP_TMS_SWIO",
> + {"hsi1c", "jtag2", "j2j3", "tp_jtag3"}, 4, DS(2, 8)},
> + {46, "GPIO46/SI1_nDSR/CP_TCK_SWCLK/TP_TCK_SWCLK/CP_TP_TCK_SWCLK",
> + {"hsi1c", "jtag2", "j2j3", "tp_jtag3"}, 4, 0},
> + {47, "GPIO47/SI1n_RI1", {"hsi1c",}, 1, DS(2, 8)},
> + {48, "GPIO48/SI2_TXD/BU0_TXD/STRAP5", {"hsi2a", "bmcuart0b"}, 2, 0},
> + {49, "GPIO49/SI2_RXD/BU0_RXD", {"hsi2a", "bmcuart0b"}, 2, 0},
> + {50, "GPIO50/SI2_nCTS/BU6_TXD/TPU_TXD", {"hsi2b", "bu6", "tp_uart"}, 3, 0},
> + {51, "GPIO51/SI2_nRTS/BU6_RXD/TPU_RXD", {"hsi2b", "bu6", "tp_uart"}, 3, 0},
> + {52, "GPIO52/SI2_nDCD/BU5_RXD", {"hsi2c", "bu5"}, 2, 0},
> + {53, "GPIO53/SI2_nDTR_BOUT2/BU5_TXD", {"hsi2c", "bu5"}, 2, 0},
> + {54, "GPIO54/SI2_nDSR/BU4_TXD", {"hsi2c", "bu4"}, 2, 0},
> + {55, "GPIO55/SI2_RI2/BU4_RXD", {"hsi2c", "bu4"}, 2, 0},
> + {56, "GPIO56/R1_RXERR/R1_OEN", {"r1err", "r1oen"}, 2, 0},
> + {57, "GPIO57/R1_MDC/TP_GPIO4", {"r1md", "tp_gpio4b"}, 2, DS(2, 4)},
> + {58, "GPIO58/R1_MDIO/TP_GPIO5", {"r1md", "tp_gpio5b"}, 2, DS(2, 4)},
> + {59, "GPIO59/H_GPIO06/SMB3D_SDA/SMB19_SDA", {"hgpio6", "smb3d", "smb19"}, 3, 0},
> + {60, "GPIO60/H_GPIO07/SMB3D_SCL/SMB19_SCL", {"hgpio7", "smb3d", "smb19"}, 3, 0},
> + {61, "GPIO61/SI1_nDTR_BOUT", {"hsi1c"}, 1, 0},
> + {62, "GPIO62/SI1_nRTS/BU1_nRTS/CP_TDO_SWO/TP_TDO_SWO/CP_TP_TDO_SWO",
> + {"hsi1b", "jtag2", "j2j3", "nbu1crts", "tp_jtag3"}, 5, 0},
> + {63, "GPIO63/BU1_TXD1/SI1_TXD", {"hsi1a", "bmcuart1"}, 2, 0},
> + {64, "GPIO64/FANIN0", {"fanin0"}, 1, 0},
> + {65, "GPIO65/FANIN1", {"fanin1"}, 1, 0},
> + {66, "GPIO66/FANIN2", {"fanin2"}, 1, 0},
> + {67, "GPIO67/FANIN3", {"fanin3"}, 1, 0},
> + {68, "GPIO68/FANIN4", {"fanin4"}, 1, 0},
> + {69, "GPIO69/FANIN5", {"fanin5"}, 1, 0},
> + {70, "GPIO70/FANIN6", {"fanin6"}, 1, 0},
> + {71, "GPIO71/FANIN7", {"fanin7"}, 1, 0},
> + {72, "GPIO72/FANIN8", {"fanin8"}, 1, 0},
> + {73, "GPIO73/FANIN9", {"fanin9"}, 1, 0},
> + {74, "GPIO74/FANIN10", {"fanin10"}, 1, 0},
> + {75, "GPIO75/FANIN11", {"fanin11"}, 1, 0},
> + {76, "GPIO76/FANIN12", {"fanin12"}, 1, 0},
> + {77, "GPIO77/FANIN13", {"fanin13"}, 1, 0},
> + {78, "GPIO78/FANIN14", {"fanin14"}, 1, 0},
> + {79, "GPIO79/FANIN15", {"fanin15"}, 1, 0},
> + {80, "GPIO80/PWM0", {"pwm0"}, 1, DS(4, 8)},
> + {81, "GPIO81/PWM1", {"pwm1"}, 1, DS(4, 8)},
> + {82, "GPIO82/PWM2", {"pwm2"}, 1, DS(4, 8)},
> + {83, "GPIO83/PWM3", {"pwm3"}, 1, DS(4, 8)},
> + {84, "GPIO84/R2_TXD0", {"r2"}, 1, DS(4, 8) | SLEW},
> + {85, "GPIO85/R2_TXD1", {"r2"}, 1, DS(4, 8) | SLEW},
> + {86, "GPIO86/R2_TXEN", {"r2"}, 1, DS(4, 8) | SLEW},
> + {87, "GPIO87/R2_RXD0", {"r2"}, 1, 0},
> + {88, "GPIO88/R2_RXD1", {"r2"}, 1, 0},
> + {89, "GPIO89/R2_CRSDV", {"r2"}, 1, 0},
> + {90, "GPIO90/R2_RXERR/R2_OEN", {"r2err", "r2oen"}, 2, 0},
> + {91, "GPIO91/R2_MDC/CP1_GPIO6/TP_GPIO0", {"r2md", "cp1gpio6", "tp_gpio0"}, 3, DS(2, 4)},
> + {92, "GPIO92/R2_MDIO/CP1_GPIO7/TP_GPIO1", {"r2md", "cp1gpio7", "tp_gpio1"}, 3, DS(2, 4)},
> + {93, "GPIO93/GA20/SMB5D_SCL", {"ga20kbc", "smb5d"}, 2, 0},
> + {94, "GPIO94/nKBRST/SMB5D_SDA", {"ga20kbc", "smb5d"}, 2, 0},
> + {95, "GPIO95/nESPIRST/LPC_nLRESET", {"lpc", "espi"}, 2, 0},
> + {96, "GPIO96/CP1_GPIO7/BU2_TXD/TP_GPIO7", {"cp1gpio7b", "bu2", "tp_gpio7"}, 3, SLEW},
> + {97, "GPIO97/CP1_GPIO6/BU2_RXD/TP_GPIO6", {"cp1gpio6b", "bu2", "tp_gpio6"}, 3, SLEW},
> + {98, "GPIO98/CP1_GPIO5/BU4_TXD/TP_GPIO5", {"bu4b", "cp1gpio5b", "tp_gpio5"}, 3, SLEW},
> + {99, "GPIO99/CP1_GPIO4/BU4_RXD/TP_GPIO4", {"bu4b", "cp1gpio4b", "tp_gpio4"}, 3, SLEW},
> + {100, "GPIO100/CP1_GPIO3/BU5_TXD/TP_GPIO3", {"bu5b", "cp1gpio3c", "tp_gpio3"}, 3, SLEW},
> + {101, "GPIO101/CP1_GPIO2/BU5_RXD/TP_GPIO2", {"bu5b", "cp1gpio2c", "tp_gpio2"}, 3, SLEW},
> + {102, "GPIO102/HSYNC", {"vgadig"}, 1, DS(4, 8)},
> + {103, "GPIO103/VSYNC", {"vgadig"}, 1, DS(4, 8)},
> + {104, "GPIO104/DDC_SCL", {"vgadig"}, 1, 0},
> + {105, "GPIO105/DDC_SDA", {"vgadig"}, 1, 0},
> + {106, "GPIO106/I3C5_SCL", {"i3c5"}, 1, SLEW},
> + {107, "GPIO107/I3C5_SDA", {"i3c5"}, 1, SLEW},
> + {108, "GPIO108/SG1_MDC", {"rg1mdio"}, 1, SLEW},
> + {109, "GPIO109/SG1_MDIO", {"rg1mdio"}, 1, SLEW},
> + {110, "GPIO110/RG2_TXD0/DDRV0/R3_TXD0", {"rg2", "ddr", "rmii3"}, 3, SLEW},
> + {111, "GPIO111/RG2_TXD1/DDRV1/R3_TXD1", {"rg2", "ddr", "rmii3"}, 3, SLEW},
> + {112, "GPIO112/RG2_TXD2/DDRV2", {"rg2", "ddr"}, 2, SLEW},
> + {113, "GPIO113/RG2_TXD3/DDRV3", {"rg2", "ddr"}, 2, SLEW},
> + {114, "GPIO114/SMB0_SCL", {"smb0"}, 1, 0},
> + {115, "GPIO115/SMB0_SDA", {"smb0"}, 1, 0},
> + {116, "GPIO116/SMB1_SCL", {"smb1"}, 1, 0},
> + {117, "GPIO117/SMB1_SDA", {"smb1"}, 1, 0},
> + {118, "GPIO118/SMB2_SCL", {"smb2"}, 1, 0},
> + {119, "GPIO119/SMB2_SDA", {"smb2"}, 1, 0},
> + {120, "GPIO120/SMB2C_SDA", {"smb2c"}, 1, SLEW},
> + {121, "GPIO121/SMB2C_SCL", {"smb2c"}, 1, SLEW},
> + {122, "GPIO122/SMB2B_SDA", {"smb2b"}, 1, SLEW},
> + {123, "GPIO123/SMB2B_SCL", {"smb2b"}, 1, SLEW},
> + {124, "GPIO124/SMB1C_SDA/CP1_GPIO3", {"smb1c", "cp1gpio3b"}, 2, SLEW},
> + {125, "GPIO125/SMB1C_SCL/CP1_GPIO2", {"smb1c", "cp1gpio2b"}, 2, SLEW},
> + {126, "GPIO126/SMB1B_SDA/CP1_GPIO1", {"smb1b", "cp1gpio1b"}, 2, SLEW},
> + {127, "GPIO127/SMB1B_SCL/CP1_GPIO0", {"smb1b", "cp1gpio0b"}, 2, SLEW},
> + {128, "GPIO128/SMB824_SCL", {"smb8"}, 1, 0},
> + {129, "GPIO129/SMB824_SDA", {"smb8"}, 1, 0},
> + {130, "GPIO130/SMB925_SCL", {"smb9"}, 1, 0},
> + {131, "GPIO131/SMB925_SDA", {"smb9"}, 1, 0},
> + {132, "GPIO132/SMB1026_SCL", {"smb10"}, 1, 0},
> + {133, "GPIO133/SMB1026_SDA", {"smb10"}, 1, 0},
> + {134, "GPIO134/SMB11_SCL", {"smb11", "smb23b"}, 2, 0},
> + {135, "GPIO135/SMB11_SDA", {"smb11", "smb23b"}, 2, 0},
> + {136, "GPIO136/JM1_TCK", {"jm1"}, 1, SLEW},
> + {137, "GPIO137/JM1_TDO", {"jm1"}, 1, SLEW},
> + {138, "GPIO138/JM1_TMS", {"jm1"}, 1, SLEW},
> + {139, "GPIO139/JM1_TDI", {"jm1"}, 1, SLEW},
> + {140, "GPIO140/JM1_nTRST", {"jm1"}, 1, SLEW},
> + {141, "GPIO141/SMB7B_SCL", {"smb7b"}, 1, 0},
> + {142, "GPIO142/SMB7D_SCL/TPSMB1_SCL", {"smb7d", "tp_smb1"}, 2, SLEW},
> + {143, "GPIO143/SMB7D_SDA/TPSMB1_SDA", {"smb7d", "tp_smb1"}, 2, SLEW},
> + {144, "GPIO144/PWM4", {"pwm4"}, 1, DS(4, 8)},
> + {145, "GPIO145/PWM5", {"pwm5"}, 1, DS(4, 8)},
> + {146, "GPIO146/PWM6", {"pwm6"}, 1, DS(4, 8)},
> + {147, "GPIO147/PWM7", {"pwm7"}, 1, DS(4, 8)},
> + {148, "GPIO148/MMC_DT4", {"mmc8"}, 1, DS(8, 12) | SLEW},
> + {149, "GPIO149/MMC_DT5", {"mmc8"}, 1, DS(8, 12) | SLEW},
> + {150, "GPIO150/MMC_DT6", {"mmc8"}, 1, DS(8, 12) | SLEW},
> + {151, "GPIO151/MMC_DT7", {"mmc8"}, 1, DS(8, 12) | SLEW},
> + {152, "GPIO152/MMC_CLK", {"mmc"}, 1, DS(8, 12) | SLEW},
> + {153, "GPIO153/MMC_WP", {"mmcwp"}, 1, 0},
> + {154, "GPIO154/MMC_CMD", {"mmc"}, 1, DS(8, 12) | SLEW},
> + {155, "GPIO155/MMC_nCD/MMC_nRSTLK", {"mmccd", "mmcrst"}, 2, 0},
> + {156, "GPIO156/MMC_DT0", {"mmc"}, 1, DS(8, 12) | SLEW},
> + {157, "GPIO157/MMC_DT1", {"mmc"}, 1, DS(8, 12) | SLEW},
> + {158, "GPIO158/MMC_DT2", {"mmc"}, 1, DS(8, 12) | SLEW},
> + {159, "GPIO159/MMC_DT3", {"mmc"}, 1, DS(8, 12) | SLEW},
> + {160, "GPIO160/CLKOUT/RNGOSCOUT/GFXBYPCK", {"clkout"}, 1, DS(8, 12) | SLEW},
> + {161, "GPIO161/ESPI_nCS/LPC_nLFRAME", {"espi", "lpc"}, 2, 0},
> + {162, "GPIO162/LPC_nCLKRUN", {"clkrun"}, 1, DS(8, 12)},
> + {163, "GPIO163/ESPI_CK/LPC_LCLK", {"espi", "lpc"}, 2, 0},
> + {164, "GPIO164/ESPI_IO0/LPC_LAD0", {"espi", "lpc"}, 2, 0},
> + {165, "GPIO165/ESPI_IO1/LPC_LAD1", {"espi", "lpc"}, 2, 0},
> + {166, "GPIO166/ESPI_IO2/LPC_LAD2", {"espi", "lpc"}, 2, 0},
> + {167, "GPIO167/ESPI_IO3/LPC_LAD3", {"espi", "lpc"}, 2, 0},
> + {168, "GPIO168/ESPI_nALERT/SERIRQ", {"espi", "serirq"}, 2, 0},
> + {169, "GPIO169/nSCIPME/SMB21_SCL", {"scipme", "smb21"}, 2, 0},
> + {170, "GPIO170/nSMI/SMB21_SDA", {"sci", "smb21"}, 2, 0},
> + {171, "GPIO171/SMB6_SCL", {"smb6"}, 1, 0},
> + {172, "GPIO172/SMB6_SDA", {"smb6"}, 1, 0},
> + {173, "GPIO173/SMB7_SCL", {"smb7"}, 1, 0},
> + {174, "GPIO174/SMB7_SDA", {"smb7"}, 1, 0},
> + {175, "GPIO175/SPI1_CK/FANIN19/FM1_CK", {"spi1", "faninx", "fm1"}, 3, DS(8, 12)},
> + {176, "GPIO176/SPI1_DO/FANIN18/FM1_DO/STRAP9", {"spi1", "faninx", "fm1"}, 3, DS(8, 12)},
> + {177, "GPIO177/SPI1_DI/FANIN17/FM1_D1/STRAP10", {"spi1", "faninx", "fm1"}, 3, DS(8, 12)},
> + {178, "GPIO178/R1_TXD0", {"r1"}, 1, DS(8, 12) | SLEW},
> + {179, "GPIO179/R1_TXD1", {"r1"}, 1, DS(8, 12) | SLEW},
> + {180, "GPIO180/R1_TXEN", {"r1"}, 1, DS(8, 12) | SLEW},
> + {181, "GPIO181/R1_RXD0", {"r1"}, 1, 0},
> + {182, "GPIO182/R1_RXD1", {"r1"}, 1, 0},
> + {183, "GPIO183/SPI3_SEL", {"spi3", "gpio1836"}, 2,
> + DS(8, 12) | SLEW | GPIO_ALT | GPIO_IDX(1)},
> + {184, "GPIO184/SPI3_D0/STRAP13", {"spi3", "gpio1836"}, 2,
> + DS(8, 12) | SLEW | GPIO_ALT | GPIO_IDX(1)},
> + {185, "GPIO185/SPI3_D1", {"spi3", "gpio1836"}, 2,
> + DS(8, 12) | SLEW | GPIO_ALT | GPIO_IDX(1)},
> + {186, "GPIO186/SPI3_nCS0", {"spi3", "gpio1836"}, 2,
> + DS(8, 12) | SLEW | GPIO_ALT | GPIO_IDX(1)},
> + {187, "GPIO187/SPI3_nCS1_SMB14_SDA", {"spi3cs1", "smb14b", "gpio187"}, 3,
> + SLEW | GPIO_ALT | GPIO_IDX(2)},
> + {188, "GPIO188/SPI3_D2/SPI3_nCS2", {"spi3quad", "spi3cs2", "gpio1889"}, 3,
> + DS(8, 12) | SLEW | GPIO_ALT | GPIO_IDX(2)},
> + {189, "GPIO189/SPI3_D3/SPI3_nCS3", {"spi3quad", "spi3cs3", "gpio1889"}, 3,
> + DS(8, 12) | SLEW | GPIO_ALT | GPIO_IDX(2)},
> + {190, "GPIO190/nPRD_SMI", {"nprd_smi"}, 1, DS(2, 4)},
> + {191, "GPIO191/SPI1_D1/FANIN17/FM1_D1/STRAP10",
> + {"spi1d23", "spi1cs2", "fm1", "smb15"}, 4, SLEW},
> + {192, "GPIO192/SPI1_D3/SPI_nCS3/FM1_D3/SMB15_SCL",
> + {"spi1d23", "spi1cs3", "fm1", "smb15"}, 4, SLEW},
> + {193, "GPIO193/R1_CRSDV", {"r1"}, 1, 0},
> + {194, "GPIO194/SMB0B_SCL/FM0_CK", {"smb0b", "fm0"}, 2, SLEW},
> + {195, "GPIO195/SMB0B_SDA/FM0_D0", {"smb0b", "fm0"}, 2, SLEW},
> + {196, "GPIO196/SMB0C_SCL/FM0_D1", {"smb0c", "fm0"}, 2, SLEW},
> + {197, "GPIO197/SMB0DEN/FM0_D3", {"smb0den", "fm0"}, 2, SLEW},
> + {198, "GPIO198/SMB0D_SDA/FM0_D2", {"smb0d", "fm0"}, 2, SLEW},
> + {199, "GPIO199/SMB0D_SCL/FM0_CSO", {"smb0d", "fm0"}, 2, SLEW},
> + {200, "GPIO200/R2_CK", {"r2"}, 1, 0},
> + {201, "GPIO201/R1_CK", {"r1"}, 1, 0},
> + {202, "GPIO202/SMB0C_SDA/FM0_CSI", {"smb0c", "fm0"}, 2, SLEW},
> + {203, "GPIO203/SPI1_nCS0/FANIN16/FM1_CSI", {"faninx", "spi1", "fm1"}, 3, DS(8, 12)},
> + {208, "GPIO208/RG2_TXC/DVCK", {"rg2", "ddr"}, 2, SLEW},
> + {209, "GPIO209/RG2_TXCTL/DDRV4/R3_TXEN", {"rg2", "ddr", "rmii3"}, 3, SLEW},
> + {210, "GPIO210/RG2_RXD0/DDRV5/R3_RXD0", {"rg2", "ddr", "rmii3"}, 3, DS(8, 12) | SLEW},
> + {211, "GPIO211/RG2_RXD1/DDRV6/R3_RXD1", {"rg2", "ddr", "rmii3"}, 3, DS(8, 12) | SLEW},
> + {212, "GPIO212/RG2_RXD2/DDRV7/R3_RXD2", {"rg2", "ddr", "r3rxer"}, 3, DS(8, 12) | SLEW},
> + {213, "GPIO213/RG2_RXD3/DDRV8/R3_OEN", {"rg2", "ddr", "r3oen"}, 3, DS(8, 12) | SLEW},
> + {214, "GPIO214/RG2_RXC/DDRV9/R3_CK", {"rg2", "ddr", "rmii3"}, 3, DS(8, 12) | SLEW},
> + {215, "GPIO215/RG2_RXCTL/DDRV10/R3_CRSDV", {"rg2", "ddr", "rmii3"}, 3, DS(8, 12) | SLEW},
> + {216, "GPIO216/RG2_MDC/DDRV11", {"rg2mdio", "ddr"}, 2, DS(8, 12) | SLEW},
> + {217, "GPIO217/RG2_MDIO/DVHSYNC", {"rg2mdio", "ddr"}, 2, DS(8, 12) | SLEW},
> + {218, "GPIO218/nWDO1/SMB16_SCL", {"wdog1", "smb16"}, 2, SLEW},
> + {219, "GPIO219/nWDO2/SMB16_SDA", {"wdog2", "smb16"}, 2, SLEW},
> + {220, "GPIO220/SMB12_SCL/PWM8", {"smb12", "pwm8"}, 2, SLEW},
> + {221, "GPIO221/SMB12_SDA/PWM9", {"smb12", "pwm9"}, 2, SLEW},
> + {222, "GPIO222/SMB13_SCL", {"smb13"}, 1, SLEW},
> + {223, "GPIO223/SMB13_SDA", {"smb13"}, 1, SLEW},
> + {224, "GPIO224/SPIX_CK/FM2_CK", {"spix", "fm2"}, 2, DS(8, 12) | SLEW},
> + {225, "GPO225/SPIX_D0/FM2_D0/STRAP1", {"spix", "fm2"}, 2, DS(8, 12) | SLEW},
> + {226, "GPO226/SPIX_D1/FM2_D1/STRAP2", {"spix", "fm2"}, 2, DS(8, 12) | SLEW},
> + {227, "GPIO227/SPIX_nCS0/FM2_CSI", {"spix", "fm2"}, 2, DS(8, 12) | SLEW},
> + {228, "GPIO228/SPIX_nCS1/FM2_CSO", {"spixcs1", "fm2"}, 2, DS(8, 12) | SLEW},
> + {229, "GPO229/SPIX_D2/FM2_D2/STRAP3", {"spix", "fm2"}, 2, DS(8, 12) | SLEW},
> + {230, "GPO230/SPIX_D3/FM2_D3/STRAP6", {"spix", "fm2"}, 2, DS(8, 12) | SLEW},
> + {231, "GPIO231/EP_nCLKREQ", {"clkreq"}, 1, DS(4, 12) | SLEW},
> + {233, "GPIO233/SPI1_nCS1/FM1_CSO", {"spi1cs1", "fm1"}, 2, 0},
> + {234, "GPIO234/PWM10/SMB20_SCL", {"pwm10", "smb20"}, 2, SLEW},
> + {235, "GPIO235/PWM11/SMB20_SDA", {"pwm11", "smb20"}, 2, SLEW},
> + {240, "GPIO240/I3C0_SCL", {"i3c0"}, 2, SLEW},
> + {241, "GPIO241/I3C0_SDA", {"i3c0"}, 2, SLEW},
> + {242, "GPIO242/I3C1_SCL", {"i3c1"}, 2, SLEW},
> + {243, "GPIO243/I3C1_SDA", {"i3c1"}, 2, SLEW},
> + {244, "GPIO244/I3C2_SCL", {"i3c2"}, 2, SLEW},
> + {245, "GPIO245/I3C2_SDA", {"i3c2"}, 2, SLEW},
> + {246, "GPIO246/I3C3_SCL", {"i3c3"}, 2, SLEW},
> + {247, "GPIO247/I3C3_SDA", {"i3c3"}, 2, SLEW},
> + {250, "GPIO250/RG2_REFCK/DVVSYNC", {"ddr", "rg2refck"}, 2, DS(8, 12) | SLEW},
> +};
> +
> +static const struct group_config *npcm8xx_group_get(const char *name)
> +{
> + int i;
> +
> + for (i = 0; i < ARRAY_SIZE(npcm8xx_groups); i++) {
> + if (!strcmp(npcm8xx_groups[i].name, name))
> + return &npcm8xx_groups[i];
> + }
> +
> + return NULL;
> +}
> +
> +static int npcm8xx_get_pin_selector(int gpio)
> +{
> + int i;
> +
> + for (i = 0; i < ARRAY_SIZE(npcm8xx_pins); i++) {
> + if (npcm8xx_pins[i].gpio_num == gpio)
> + return i;
> + }
> +
> + return -ENOENT;
> +}
> +
> +static int npcm8xx_group_set_func(struct udevice *dev,
> + const struct group_config *group,
> + char *func)
> +{
> + struct npcm8xx_pinctrl_priv *priv = dev_get_priv(dev);
> +
> + dev_dbg(dev, "set_func [grp %s][func %s]\n", group->name, func);
> + if (strcmp(group->name, func))
> + regmap_update_bits(priv->gcr_regmap, group->reg,
> + BIT(group->bit), 0);
> + else
> + regmap_update_bits(priv->gcr_regmap, group->reg,
> + BIT(group->bit), BIT(group->bit));
> +
> + return 0;
> +}
> +
> +static int npcm8xx_pinmux_set(struct udevice *dev,
> + unsigned int pin_selector,
> + unsigned int func_selector)
> +{
> + const struct pin_info *pin;
> + char *func = npcm8xx_funcs[func_selector];
> + const struct group_config *group;
> + int i;
> +
> + pin = &npcm8xx_pins[pin_selector];
> + dev_dbg(dev, "set_mux [pin %s][func %s]\n", pin->name, func);
> +
> + for (i = 0; i < pin->num_funcs; i++) {
> + group = npcm8xx_group_get(pin->alt_func[i]);
> + if (!group)
> + break;
> + npcm8xx_group_set_func(dev, group, func);
> + }
> +
> + return 0;
> +}
So why do you go int -> name -> strcmp here? Couldn't you just store
the valid func_selectors? E.g. use the PIN_LIST thing from above and
do something like
#define GRP(_name, _reg, _bit, ...) \
NPCM8XX_FUNC_##_name,
enum npcm8xx_func_selectors {
PIN_LIST
};
and then you define your pins like
{0, "GPIO0/IOX1_DI/SMB6C_SDA/SMB18_SDA", { NPCM8XX_FUNC_IOX1, NPCM8XX_FUNC_SMB6C, NPCM8XX_FUNC_SM18}, 3, SLEW}
(with some macro magic)
> +static int npcm8xx_pinmux_group_set(struct udevice *dev,
> + unsigned int group_selector,
> + unsigned int func_selector)
> +{
> + const struct group_config *group;
> + int pin_selector;
> + int i;
> +
> + dev_dbg(dev, "set_mux [grp %s][func %s]\n",
> + npcm8xx_groups[group_selector].name,
> + npcm8xx_funcs[func_selector]);
> + group = &npcm8xx_groups[group_selector];
> +
> + if (!group->npins) {
> + /* No alternate GPIO pins, just set the function */
> + npcm8xx_group_set_func(dev, group,
> + npcm8xx_funcs[func_selector]);
> + return 0;
> + }
> +
> + for (i = 0; i < group->npins; i++) {
> + pin_selector = npcm8xx_get_pin_selector(group->pins[i]);
> + if (pin_selector < 0) {
> + dev_dbg(dev, "invalid pin %d\n", group->pins[i]);
> + return -EINVAL;
> + }
> + npcm8xx_pinmux_set(dev, pin_selector, func_selector);
> + }
> +
> + return 0;
> +}
> +
> +static int npcm8xx_get_pins_count(struct udevice *dev)
> +{
> + return ARRAY_SIZE(npcm8xx_pins);
> +}
> +
> +static const char *npcm8xx_get_pin_name(struct udevice *dev,
> + unsigned int selector)
> +{
> + return npcm8xx_pins[selector].name;
> +}
> +
> +static int npcm8xx_get_groups_count(struct udevice *dev)
> +{
> + return ARRAY_SIZE(npcm8xx_groups);
> +}
> +
> +static const char *npcm8xx_get_group_name(struct udevice *dev,
> + unsigned int selector)
> +{
> + return npcm8xx_groups[selector].name;
> +}
> +
> +static int npcm8xx_get_functions_count(struct udevice *dev)
> +{
> + return npcm8xx_num_funcs;
> +}
> +
> +static const char *npcm8xx_get_function_name(struct udevice *dev,
> + unsigned int selector)
> +{
> + return npcm8xx_funcs[selector];
> +}
> +
> +#if CONFIG_IS_ENABLED(PINCONF)
> +#define PIN_CONFIG_PERSIST_STATE (PIN_CONFIG_END + 1)
> +#define PIN_CONFIG_POLARITY_STATE (PIN_CONFIG_END + 2)
> +#define PIN_CONFIG_EVENT_CLEAR (PIN_CONFIG_END + 3)
> +
> +static const struct pinconf_param npcm8xx_conf_params[] = {
> + { "bias-disable", PIN_CONFIG_BIAS_DISABLE, 0 },
> + { "bias-pull-up", PIN_CONFIG_BIAS_PULL_UP, 1 },
> + { "bias-pull-down", PIN_CONFIG_BIAS_PULL_DOWN, 1 },
> + { "input-enable", PIN_CONFIG_INPUT_ENABLE, 1 },
> + { "output-enable", PIN_CONFIG_OUTPUT_ENABLE, 1 },
> + { "output-high", PIN_CONFIG_OUTPUT, 1, },
> + { "output-low", PIN_CONFIG_OUTPUT, 0, },
> + { "drive-open-drain", PIN_CONFIG_DRIVE_OPEN_DRAIN, 1 },
> + { "drive-push-pull", PIN_CONFIG_DRIVE_PUSH_PULL, 1 },
> + { "persist-enable", PIN_CONFIG_PERSIST_STATE, 1 },
> + { "persist-disable", PIN_CONFIG_PERSIST_STATE, 0 },
> + { "input-debounce", PIN_CONFIG_INPUT_DEBOUNCE, 0 },
> + { "active-high", PIN_CONFIG_POLARITY_STATE, 0 },
> + { "active-low", PIN_CONFIG_POLARITY_STATE, 1 },
> + { "drive-strength", PIN_CONFIG_DRIVE_STRENGTH, 0 },
> + { "slew-rate", PIN_CONFIG_SLEW_RATE, 0},
> + { "event-clear", PIN_CONFIG_EVENT_CLEAR, 0},
> +};
> +
> +/* Support for retaining the state after soft reset */
> +static int npcm8xx_gpio_reset_persist(struct udevice *dev, uint bank,
> + uint enable)
> +{
> + struct npcm8xx_pinctrl_priv *priv = dev_get_priv(dev);
> + u8 offset = bank + GPIOX_OFFSET;
> +
> + dev_dbg(dev, "set gpio persist, bank %d, enable %d\n", bank, enable);
> +
> + if (enable) {
> + regmap_update_bits(priv->rst_regmap, WD0RCR, BIT(offset), 0);
> + regmap_update_bits(priv->rst_regmap, WD1RCR, BIT(offset), 0);
> + regmap_update_bits(priv->rst_regmap, WD2RCR, BIT(offset), 0);
> + regmap_update_bits(priv->rst_regmap, CORSTC, BIT(offset), 0);
> + } else {
> + regmap_update_bits(priv->rst_regmap, WD0RCR, BIT(offset),
> + BIT(offset));
> + regmap_update_bits(priv->rst_regmap, WD1RCR, BIT(offset),
> + BIT(offset));
> + regmap_update_bits(priv->rst_regmap, WD2RCR, BIT(offset),
> + BIT(offset));
> + regmap_update_bits(priv->rst_regmap, CORSTC, BIT(offset),
> + BIT(offset));
> + }
> +
> + return 0;
> +}
> +
> +static bool is_gpio_persist(struct udevice *dev, uint bank)
> +{
> + struct npcm8xx_pinctrl_priv *priv = dev_get_priv(dev);
> + u8 offset = bank + GPIOX_OFFSET;
> + u32 val;
> + int status;
> +
> + status = npcm_get_reset_status();
> + dev_dbg(dev, "reset status: 0x%x\n", status);
> +
> + if (status & CORST)
> + regmap_read(priv->rst_regmap, CORSTC, &val);
> + else if (status & WD0RST)
> + regmap_read(priv->rst_regmap, WD0RCR, &val);
> + else if (status & WD1RST)
> + regmap_read(priv->rst_regmap, WD1RCR, &val);
> + else if (status & WD2RST)
> + regmap_read(priv->rst_regmap, WD2RCR, &val);
> + else
> + return false;
> +
> + return !(val & BIT(offset));
> +}
> +
> +static void npcm8xx_set_gpio_func(struct udevice *dev, unsigned int selector)
> +{
> + const struct pin_info *pin = &npcm8xx_pins[selector];
> + const struct group_config *group;
> + char *func = gpio_func_name;
> + int i;
> +
> + /* gpio is enabled by setting alternate function */
> + if (pin->flags & GPIO_ALT)
> + func = pin->alt_func[GPIO_IDX_VAL(pin->flags)];
> +
> + for (i = 0; i < pin->num_funcs; i++) {
> + group = npcm8xx_group_get(pin->alt_func[i]);
> + if (!group)
> + break;
> + npcm8xx_group_set_func(dev, group, func);
> + }
> +}
> +
> +static int npcm8xx_pinconf_set(struct udevice *dev, unsigned int selector,
> + unsigned int param, unsigned int arg)
> +{
> + struct npcm8xx_pinctrl_priv *priv = dev_get_priv(dev);
> + uint pin = npcm8xx_pins[selector].gpio_num;
> + uint bank = pin / NPCM8XX_GPIO_PER_BANK;
> + uint gpio = (pin % NPCM8XX_GPIO_PER_BANK);
> + void __iomem *base = priv->gpio_base + (0x1000 * bank);
> + u32 flags = npcm8xx_pins[selector].flags;
> + int ret = 0;
> +
> + dev_dbg(dev, "set_conf [pin %d][param 0x%x, arg 0x%x]\n",
> + pin, param, arg);
> +
> + /* Configure pin as gpio function */
> + npcm8xx_set_gpio_func(dev, selector);
> +
> + if (is_gpio_persist(dev, bank)) {
> + dev_dbg(dev, "retain the state\n");
> + return 0;
> + }
> +
> + switch (param) {
> + case PIN_CONFIG_BIAS_DISABLE:
> + dev_dbg(dev, "set pin %d bias disable\n", pin);
> + clrbits_le32(base + GPIO_PU, BIT(gpio));
> + clrbits_le32(base + GPIO_PD, BIT(gpio));
> + break;
> + case PIN_CONFIG_BIAS_PULL_DOWN:
> + dev_dbg(dev, "set pin %d bias pull down\n", pin);
> + clrbits_le32(base + GPIO_PU, BIT(gpio));
> + setbits_le32(base + GPIO_PD, BIT(gpio));
> + break;
> + case PIN_CONFIG_BIAS_PULL_UP:
> + dev_dbg(dev, "set pin %d bias pull up\n", pin);
> + setbits_le32(base + GPIO_PU, BIT(gpio));
> + clrbits_le32(base + GPIO_PD, BIT(gpio));
> + break;
> + case PIN_CONFIG_INPUT_ENABLE:
> + dev_dbg(dev, "set pin %d input enable\n", pin);
> + setbits_le32(base + GPIO_OEC, BIT(gpio));
> + setbits_le32(base + GPIO_IEM, BIT(gpio));
> + break;
> + case PIN_CONFIG_OUTPUT_ENABLE:
> + dev_dbg(dev, "set pin %d output enable\n", pin);
> + clrbits_le32(base + GPIO_IEM, BIT(gpio));
> + setbits_le32(base + GPIO_OES, BIT(gpio));
> + case PIN_CONFIG_OUTPUT:
> + dev_dbg(dev, "set pin %d output %d\n", pin, arg);
> + clrbits_le32(base + GPIO_IEM, BIT(gpio));
> + setbits_le32(base + GPIO_OES, BIT(gpio));
> + if (arg)
> + setbits_le32(base + GPIO_DOUT, BIT(gpio));
> + else
> + clrbits_le32(base + GPIO_DOUT, BIT(gpio));
> + break;
> + case PIN_CONFIG_DRIVE_PUSH_PULL:
> + dev_dbg(dev, "set pin %d push pull\n", pin);
> + clrbits_le32(base + GPIO_OTYP, BIT(gpio));
> + break;
> + case PIN_CONFIG_DRIVE_OPEN_DRAIN:
> + dev_dbg(dev, "set pin %d open drain\n", pin);
> + setbits_le32(base + GPIO_OTYP, BIT(gpio));
> + break;
> + case PIN_CONFIG_INPUT_DEBOUNCE:
> + dev_dbg(dev, "set pin %d input debounce\n", pin);
> + setbits_le32(base + GPIO_DBNC, BIT(gpio));
> + break;
> + case PIN_CONFIG_POLARITY_STATE:
> + dev_dbg(dev, "set pin %d active %d\n", pin, arg);
> + if (arg)
> + setbits_le32(base + GPIO_POL, BIT(gpio));
> + else
> + clrbits_le32(base + GPIO_POL, BIT(gpio));
> + break;
> + case PIN_CONFIG_DRIVE_STRENGTH:
> + dev_dbg(dev, "set pin %d driver strength %d\n", pin, arg);
> + if (DSLO(flags) == arg)
> + clrbits_le32(base + GPIO_ODSC, BIT(gpio));
> + else if (DSHI(flags) == arg)
> + setbits_le32(base + GPIO_ODSC, BIT(gpio));
> + else
> + ret = -EOPNOTSUPP;
> + break;
> + case PIN_CONFIG_SLEW_RATE:
> + dev_dbg(dev, "set pin %d slew rate %d\n", pin, arg);
> + if (!(flags & SLEW)) {
> + ret = -EOPNOTSUPP;
> + break;
> + }
> + if (arg)
> + setbits_le32(base + GPIO_OSRC, BIT(gpio));
> + else
> + clrbits_le32(base + GPIO_OSRC, BIT(gpio));
> + break;
> + case PIN_CONFIG_EVENT_CLEAR:
> + dev_dbg(dev, "set pin %d event clear\n", pin);
> + clrbits_le32(base + GPIO_EVEN, BIT(gpio));
> + setbits_le32(base + GPIO_EVST, BIT(gpio));
> + break;
> + case PIN_CONFIG_PERSIST_STATE:
> + npcm8xx_gpio_reset_persist(dev, bank, arg);
> + break;
> +
> + default:
> + ret = -EOPNOTSUPP;
> + }
> +
> + return ret;
> +}
> +#endif
> +
> +static struct pinctrl_ops npcm8xx_pinctrl_ops = {
> + .set_state = pinctrl_generic_set_state,
> + .get_pins_count = npcm8xx_get_pins_count,
> + .get_pin_name = npcm8xx_get_pin_name,
> + .get_groups_count = npcm8xx_get_groups_count,
> + .get_group_name = npcm8xx_get_group_name,
> + .get_functions_count = npcm8xx_get_functions_count,
> + .get_function_name = npcm8xx_get_function_name,
> + .pinmux_set = npcm8xx_pinmux_set,
> + .pinmux_group_set = npcm8xx_pinmux_group_set,
> +#if CONFIG_IS_ENABLED(PINCONF)
> + .pinconf_num_params = ARRAY_SIZE(npcm8xx_conf_params),
> + .pinconf_params = npcm8xx_conf_params,
> + .pinconf_set = npcm8xx_pinconf_set,
> + .pinconf_group_set = npcm8xx_pinconf_set,
> +#endif
> +};
> +
> +static int npcm8xx_pinctrl_probe(struct udevice *dev)
> +{
> + struct npcm8xx_pinctrl_priv *priv = dev_get_priv(dev);
> + int i;
> +
> + priv->gpio_base = dev_read_addr_ptr(dev);
> + if (!priv->gpio_base)
> + return -EINVAL;
> +
> + priv->gcr_regmap = syscon_regmap_lookup_by_phandle(dev, "syscon-gcr");
> + if (IS_ERR(priv->gcr_regmap))
> + return -EINVAL;
> +
> + priv->rst_regmap = syscon_regmap_lookup_by_phandle(dev, "syscon-rst");
> + if (IS_ERR(priv->rst_regmap))
> + return -EINVAL;
> +
> + /* initialize function names, all group functions + gpio function */
> + npcm8xx_num_funcs = ARRAY_SIZE(npcm8xx_groups) + 1;
> + npcm8xx_funcs = malloc(npcm8xx_num_funcs * sizeof(char *));
> + if (!npcm8xx_funcs)
> + return -ENOMEM;
> + npcm8xx_funcs[0] = gpio_func_name;
> + for (i = 1; i < npcm8xx_num_funcs; i++)
> + npcm8xx_funcs[i] = npcm8xx_groups[i - 1].name;
So why do you allocate this? Can't you just do the npcm8xx_groups lookup directly in pinmux_set?
> + return 0;
> +}
> +
> +static const struct udevice_id npcm8xx_pinctrl_ids[] = {
> + { .compatible = "nuvoton,npcm845-pinctrl" },
> + { }
> +};
> +
> +U_BOOT_DRIVER(pinctrl_npcm8xx) = {
> + .name = "nuvoton_npcm8xx_pinctrl",
> + .id = UCLASS_PINCTRL,
> + .of_match = npcm8xx_pinctrl_ids,
> + .priv_auto = sizeof(struct npcm8xx_pinctrl_priv),
> + .ops = &npcm8xx_pinctrl_ops,
> + .probe = npcm8xx_pinctrl_probe,
> +};
>
--Sean
More information about the U-Boot
mailing list