On 5/3/22 1:33 AM, Stanley Chu wrote:
Add Nuvoton BMC NPCM845 Pinmux and Pinconf support.Signed-off-by: Stanley Chu <ys...@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
