diff mbox

[v2,2/6] pinctrl: berlin: add the berlin4ct pinctrl driver

Message ID 1442829860-4647-3-git-send-email-jszhang@marvell.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jisheng Zhang Sept. 21, 2015, 10:04 a.m. UTC 
Add the pin-controller driver for Marvell Berlin BG4CT SoC, with definition
of its groups and functions. This uses the core Berlin pinctrl driver.

Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
---
 drivers/pinctrl/berlin/Kconfig     |   5 +
 drivers/pinctrl/berlin/Makefile    |   1 +
 drivers/pinctrl/berlin/berlin4ct.c | 503 +++++++++++++++++++++++++++++++++++++
 3 files changed, 509 insertions(+)
 create mode 100644 drivers/pinctrl/berlin/berlin4ct.c

Comments

Antoine Tenart Oct. 2, 2015, 1:21 p.m. UTC | #1 
Hi Jisheng,

Sorry for the delay, I was quite busy these days...

Thanks for the nice comments!

I saw your using pin names in the BERLIN_PINCTRL_GROUP macro, like
"EMMC_RSTn". In other berlin pinctrl drivers we use the group name (such
as "G11" or GSM1"). If there is such a thing in the BG4CT, please
update.

On Mon, Sep 21, 2015 at 06:04:16PM +0800, Jisheng Zhang wrote:
[...]
> --- a/drivers/pinctrl/berlin/Kconfig
> +++ b/drivers/pinctrl/berlin/Kconfig
> @@ -17,4 +17,9 @@ config PINCTRL_BERLIN_BG2Q
>  	bool
>  	select PINCTRL_BERLIN
>  
> +config PINCTRL_BERLIN4CT
> +	depends on OF && (ARCH_BERLIN || COMPILE_TEST)
> +	bool "Marvell berlin4ct pin controller driver"
> +	select PINCTRL_BERLIN
> +
>  endif

These dependencies also apply to the other Berlin pinctrl drivers, so
this should be on PINCTRL_BERLIN.

Is there a reason not to name the configuration option
PINCTRL_BERLIN_BG4CT, to be consistent with the existing?

> diff --git a/drivers/pinctrl/berlin/berlin4ct.c b/drivers/pinctrl/berlin/berlin4ct.c
> new file mode 100644
> index 0000000..88e1670
> --- /dev/null
> +++ b/drivers/pinctrl/berlin/berlin4ct.c
> +
> +static const struct berlin_desc_group berlin4ct_soc_pinctrl_groups[] = {
> +	BERLIN_PINCTRL_GROUP("EMMC_RSTn", 0x0, 0x3, 0x00,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "emmc"), /* RSTn */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* GPIO47 */
> +	BERLIN_PINCTRL_GROUP("NAND_IO0", 0x0, 0x3, 0x03,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO0 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD0 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CLK */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO0 */
> +	BERLIN_PINCTRL_GROUP("NAND_IO1", 0x0, 0x3, 0x06,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO1 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD1 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CDn */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO1 */
> +	BERLIN_PINCTRL_GROUP("NAND_IO2", 0x0, 0x3, 0x09,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO2 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD2 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT0 */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO2 */
> +	BERLIN_PINCTRL_GROUP("NAND_IO3", 0x0, 0x3, 0x0c,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO3 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD3 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT1 */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO3 */
> +	BERLIN_PINCTRL_GROUP("NAND_IO4", 0x0, 0x3, 0x0f,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO4 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXC */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT2 */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO4 */
> +	BERLIN_PINCTRL_GROUP("NAND_IO5", 0x0, 0x3, 0x12,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO5 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXCTL */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT3 */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO5 */
> +	BERLIN_PINCTRL_GROUP("NAND_IO6", 0x0, 0x3, 0x15,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO6 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* MDC */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CMD */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO6 */
> +	BERLIN_PINCTRL_GROUP("NAND_IO7", 0x0, 0x3, 0x18,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO7 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* MDIO */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* WP */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO7 */
> +	BERLIN_PINCTRL_GROUP("NAND_ALE", 0x0, 0x3, 0x1b,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* ALE */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD0 */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO8 */
> +	BERLIN_PINCTRL_GROUP("NAND_CLE", 0x4, 0x3, 0x00,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* CLE */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD1 */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO9 */
> +	BERLIN_PINCTRL_GROUP("NAND_WEn", 0x4, 0x3, 0x03,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* WEn */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD2 */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO10 */
> +	BERLIN_PINCTRL_GROUP("NAND_REn", 0x4, 0x3, 0x06,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* REn */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD3 */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO11 */
> +	BERLIN_PINCTRL_GROUP("NAND_WPn", 0x4, 0x3, 0x09,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* WPn */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO12 */
> +	BERLIN_PINCTRL_GROUP("NAND_CEn", 0x4, 0x3, 0x0c,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* CEn */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXC */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO13 */
> +	BERLIN_PINCTRL_GROUP("NAND_RDY", 0x4, 0x3, 0x0f,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* RDY */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXCTL */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO14 */
> +	BERLIN_PINCTRL_GROUP("SD0_CLK", 0x4, 0x3, 0x12,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO29 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CLK*/
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* CLK */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG8 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG8 */
> +	BERLIN_PINCTRL_GROUP("SD0_DAT0", 0x4, 0x3, 0x15,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO30 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT0 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* SOP */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG9 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG9 */
> +	BERLIN_PINCTRL_GROUP("SD0_DAT1", 0x4, 0x3, 0x18,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO31 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT1 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* SD */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG10 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG10 */
> +	BERLIN_PINCTRL_GROUP("SD0_DAT2", 0x4, 0x3, 0x1b,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO32 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT2 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* VALD */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG11 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG11 */
> +	BERLIN_PINCTRL_GROUP("SD0_DAT3", 0x8, 0x3, 0x00,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO33 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT3 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* CLK */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG12 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG12 */
> +	BERLIN_PINCTRL_GROUP("SD0_CDn", 0x8, 0x3, 0x03,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO34 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CDn */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* SOP */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG13 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG13 */
> +	BERLIN_PINCTRL_GROUP("SD0_CMD", 0x8, 0x3, 0x06,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO35 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CMD */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* SD */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG14 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG14 */
> +	BERLIN_PINCTRL_GROUP("SD0_WP", 0x8, 0x3, 0x09,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO36 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* WP */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* VALD */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG15 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG15 */
> +	BERLIN_PINCTRL_GROUP("STS0_CLK", 0x8, 0x3, 0x0c,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO21 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* CLK */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "cpupll"), /* CLKO */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG0 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG0 */
> +	BERLIN_PINCTRL_GROUP("STS0_SOP", 0x8, 0x3, 0x0f,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO22 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* SOP */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "syspll"), /* CLKO */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG1 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG1 */
> +	BERLIN_PINCTRL_GROUP("STS0_SD", 0x8, 0x3, 0x12,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO23 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* SD */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "mempll"), /* CLKO */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG2 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG2 */
> +	BERLIN_PINCTRL_GROUP("STS0_VALD", 0x8, 0x3, 0x15,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO24 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* VALD */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG3 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG3 */
> +	BERLIN_PINCTRL_GROUP("STS1_CLK", 0x8, 0x3, 0x18,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO25 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* CLK */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0"),
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG4 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG4 */
> +	BERLIN_PINCTRL_GROUP("STS1_SOP", 0x8, 0x3, 0x1b,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO26 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* SOP */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1"),
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG5 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG5 */
> +	BERLIN_PINCTRL_GROUP("STS1_SD", 0xc, 0x3, 0x00,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO27 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* SD */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm2"),
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG6 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG6 */
> +	BERLIN_PINCTRL_GROUP("STS1_VALD", 0xc, 0x3, 0x03,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO28 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* VALD */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm3"),
> +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG7 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG7 */
> +	BERLIN_PINCTRL_GROUP("SCRD0_RST", 0xc, 0x3, 0x06,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO15 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* RST */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* CLK */
> +	BERLIN_PINCTRL_GROUP("SCRD0_DCLK", 0xc, 0x3, 0x09,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO16 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* DCLK */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* CMD */
> +	BERLIN_PINCTRL_GROUP("SCRD0_GPIO0", 0xc, 0x3, 0x0c,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO17 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* SCRD0 GPIO0 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* DIO */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT0 */
> +	BERLIN_PINCTRL_GROUP("SCRD0_GPIO1", 0xc, 0x3, 0x0f,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO18 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* SCRD0 GPIO1 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* CLK */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT1 */
> +	BERLIN_PINCTRL_GROUP("SCRD0_DIO", 0xc, 0x3, 0x12,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO19 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* DIO */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* DEN */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT2 */
> +	BERLIN_PINCTRL_GROUP("SCRD0_CRD_PRES", 0xc, 0x3, 0x15,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO20 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* crd pres */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "sd1a")), /* DAT3 */
> +	BERLIN_PINCTRL_GROUP("SPI1_SS0n", 0xc, 0x3, 0x18,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS0n */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO37 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2")), /* CLK */
> +	BERLIN_PINCTRL_GROUP("SPI1_SS1n", 0xc, 0x3, 0x1b,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS1n */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO38 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2"), /* SOP */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "pwm1")),
> +	BERLIN_PINCTRL_GROUP("SPI1_SS2n", 0x10, 0x3, 0x00,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS2n */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO39 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2"), /* SD */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "pwm0")),
> +	BERLIN_PINCTRL_GROUP("SPI1_SS3n", 0x10, 0x3, 0x03,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS3n */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO40 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2")), /* VALD */
> +	BERLIN_PINCTRL_GROUP("SPI1_SCLK", 0x10, 0x3, 0x06,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SCLK */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO41 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* CLK */
> +	BERLIN_PINCTRL_GROUP("SPI1_SDO", 0x10, 0x3, 0x09,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SDO */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO42 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* SOP */
> +	BERLIN_PINCTRL_GROUP("SPI1_SDI", 0x10, 0x3, 0x0c,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SDI */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO43 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* SD */
> +	BERLIN_PINCTRL_GROUP("USB0_DRV_VBUS", 0x10, 0x3, 0x0f,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO44 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "usb0"), /* VBUS */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* VALD */
> +	BERLIN_PINCTRL_GROUP("TW0_SCL", 0x10, 0x3, 0x12,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO45 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "tw0")), /* SCL */
> +	BERLIN_PINCTRL_GROUP("TW0_SDA", 0x10, 0x3, 0x15,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO46 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "tw0")), /* SDA */
> +};
> +
> +static const struct berlin_desc_group berlin4ct_soc_aviopinctrl_groups[] = {
> +	BERLIN_PINCTRL_GROUP("TX_EDDC_SCL", 0x0, 0x3, 0x00,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO0 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "tx_eddc"), /* SCL */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "tw1")), /* SCL */
> +	BERLIN_PINCTRL_GROUP("TX_EDDC_SDA", 0x0, 0x3, 0x03,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO1 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "tx_eddc"), /* SDA */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "tw1")), /* SDA */
> +	BERLIN_PINCTRL_GROUP("I2S1_LRCKO", 0x0, 0x3, 0x06,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO2 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* LRCKO */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* CLK */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG0 */
> +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* CLK */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG0 */
> +	BERLIN_PINCTRL_GROUP("I2S1_BCLKO", 0x0, 0x3, 0x09,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO3 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* BCLKO */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* SOP */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG1 */
> +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* CMD */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG1 */
> +	BERLIN_PINCTRL_GROUP("I2S1_DO", 0x0, 0x3, 0x0c,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO4 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* DO */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* SD */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG2 */
> +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* DAT0 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG2 */
> +	BERLIN_PINCTRL_GROUP("I2S1_MCLK", 0x0, 0x3, 0x0f,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO5 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* MCLK */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* VALD */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* MCLK */
> +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* DAT1 */
> +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG3 */
> +	BERLIN_PINCTRL_GROUP("SPDIFO", 0x0, 0x3, 0x12,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO6 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "spdifo"),
> +			BERLIN_PINCTRL_FUNCTION(0x2, "avpll"), /* CLKO */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "adac")), /* DBG3 */
> +	BERLIN_PINCTRL_GROUP("I2S2_MCLK", 0x0, 0x3, 0x15,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO7 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* MCLK */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "hdmi"), /* FBCLK */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "pdm")), /* CLKO */
> +	BERLIN_PINCTRL_GROUP("I2S2_LRCKI", 0x0, 0x3, 0x18,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO8 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* LRCKI */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0"),
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* CLK */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* LRCK */
> +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b_dat2")),

Why did you suffix this with _dat2? All the others are named "sd1b".

> +	BERLIN_PINCTRL_GROUP("I2S2_BCLKI", 0x0, 0x3, 0x1b,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO9 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* BCLKI */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1"),
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* SOP */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* BCLK */
> +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* DAT3 */
> +	BERLIN_PINCTRL_GROUP("I2S2_DI0", 0x4, 0x3, 0x00,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO10 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* DI0 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm2"),
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* SD */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* SDIN */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "pdm"), /* DI0 */
> +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* CDn */
> +	BERLIN_PINCTRL_GROUP("I2S2_DI1", 0x4, 0x3, 0x03,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO11 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* DI1 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm3"),
> +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* VALD */
> +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* PWMCLK */
> +			BERLIN_PINCTRL_FUNCTION(0x5, "pdm"), /* DI1 */
> +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* WP */
> +};
> +
> +static const struct berlin_desc_group berlin4ct_sysmgr_pinctrl_groups[] = {
> +	BERLIN_PINCTRL_GROUP("SM_TW2_SCL", 0x0, 0x3, 0x00,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO19 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "tw2")), /* SCL */
> +	BERLIN_PINCTRL_GROUP("SM_TW2_SDA", 0x0, 0x3, 0x03,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO20 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "tw2")), /* SDA */
> +	BERLIN_PINCTRL_GROUP("SM_TW3_SCL", 0x0, 0x3, 0x06,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO21 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "tw3")), /* SCL */
> +	BERLIN_PINCTRL_GROUP("SM_TW3_SDA", 0x0, 0x3, 0x09,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO22 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "tw3")), /* SDA */
> +	BERLIN_PINCTRL_GROUP("SM_TMS", 0x0, 0x3, 0x0c,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TMS */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO0 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0")),
> +	BERLIN_PINCTRL_GROUP("SM_TDI", 0x0, 0x3, 0x0f,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TDI */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO1 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1")),
> +	BERLIN_PINCTRL_GROUP("SM_TDO", 0x0, 0x3, 0x12,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TDO */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO2 */
> +	BERLIN_PINCTRL_GROUP("SM_URT0_TXD", 0x0, 0x3, 0x15,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "urt0"), /* TXD */

s/urt/uart/

> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO3 */
> +	BERLIN_PINCTRL_GROUP("SM_URT0_RXD", 0x0, 0x3, 0x18,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "urt0"), /* RXD */

Ditto.

> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO4 */
> +	BERLIN_PINCTRL_GROUP("SM_URT1_TXD", 0x0, 0x3, 0x1b,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO5 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "urt1"), /* TXD */

Ditto.

> +			BERLIN_PINCTRL_FUNCTION(0x2, "eth1_rxclk"), /* RXCLK */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm2"),
> +			BERLIN_PINCTRL_FUNCTION(0x4, "timer0"),
> +			BERLIN_PINCTRL_FUNCTION(0x5, "clk_25m")),
> +	BERLIN_PINCTRL_GROUP("SM_URT1_RXD", 0x4, 0x3, 0x00,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO6 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "urt1"), /* RXD */

Ditto.

> +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm3"),
> +			BERLIN_PINCTRL_FUNCTION(0x4, "timer1")),
> +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS0n", 0x4, 0x3, 0x03,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SS0 n*/
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO7 */
> +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS1n", 0x4, 0x3, 0x06,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO8 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "spi2")), /* SS1n */
> +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS2n", 0x4, 0x3, 0x09,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO9 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "spi2"), /* SS2n */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "eth1"), /* MDC */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm0"),
> +			BERLIN_PINCTRL_FUNCTION(0x4, "timer0"),
> +			BERLIN_PINCTRL_FUNCTION(0x5, "clk_25m")),
> +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS3n", 0x4, 0x3, 0x0c,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO10 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "spi2"), /* SS3n */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "eth1"), /* MDIO */
> +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm1"),
> +			BERLIN_PINCTRL_FUNCTION(0x4, "timer1")),
> +	BERLIN_PINCTRL_GROUP("SM_SPI2_SDO", 0x4, 0x3, 0x0f,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SDO */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO11 */
> +	BERLIN_PINCTRL_GROUP("SM_SPI2_SDI", 0x4, 0x3, 0x12,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SDI */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO12 */
> +	BERLIN_PINCTRL_GROUP("SM_SPI2_SCLK", 0x4, 0x3, 0x15,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SCLK */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO13 */
> +	BERLIN_PINCTRL_GROUP("SM_FE_LED0", 0x4, 0x3, 0x18,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO14 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED0 */

s/fe/led/

> +	BERLIN_PINCTRL_GROUP("SM_FE_LED1", 0x4, 0x3, 0x1b,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "pwr_ok"),

This is named "pwr" in other Berlin pinctrl drivers.

> +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO 15 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED1 */

s/fe/led/

> +	BERLIN_PINCTRL_GROUP("SM_FE_LED2", 0x8, 0x3, 0x00,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO16 */
> +			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED2 */

Ditto.

> +	BERLIN_PINCTRL_GROUP("SM_HDMI_HPD", 0x8, 0x3, 0x03,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO17 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "hdmi")), /* HPD */
> +	BERLIN_PINCTRL_GROUP("SM_HDMI_CEC", 0x8, 0x3, 0x06,
> +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO18 */
> +			BERLIN_PINCTRL_FUNCTION(0x1, "hdmi")), /* CEC */
> +};
> +
> +static const struct berlin_pinctrl_desc berlin4ct_soc_pinctrl_data = {
> +	.groups = berlin4ct_soc_pinctrl_groups,
> +	.ngroups = ARRAY_SIZE(berlin4ct_soc_pinctrl_groups),
> +};
> +
> +static const struct berlin_pinctrl_desc berlin4ct_soc_aviopinctrl_data = {

Shouldn't this be named berlin4ct_avio_pinctrl_data?

> +	.groups = berlin4ct_soc_aviopinctrl_groups,

And berlin4ct_avio_pinctrl_groups?

> +	.ngroups = ARRAY_SIZE(berlin4ct_soc_aviopinctrl_groups),
> +};
> +
> +static const struct berlin_pinctrl_desc berlin4ct_sysmgr_pinctrl_data = {
> +	.groups = berlin4ct_sysmgr_pinctrl_groups,
> +	.ngroups = ARRAY_SIZE(berlin4ct_sysmgr_pinctrl_groups),
> +};
> +
> +static const struct of_device_id berlin4ct_pinctrl_match[] = {
> +	{
> +		.compatible = "marvell,berlin4ct-soc-pinctrl",
> +		.data = &berlin4ct_soc_pinctrl_data,
> +	},
> +	{
> +		.compatible = "marvell,berlin4ct-avio-pinctrl",
> +		.data = &berlin4ct_soc_aviopinctrl_data,
> +	},
> +	{
> +		.compatible = "marvell,berlin4ct-sm-pinctrl",

If "sm" refers to system manager, please update this to
"marvell,berlin4ct-system-pinctrl" to have the same naming as the other
Berlin pinctrl drivers.

Antoine
Jisheng Zhang Oct. 8, 2015, 10:46 a.m. UTC | #2 
Hi Antoine,

On Fri, 2 Oct 2015 15:21:39 +0200
Antoine Tenart <antoine.tenart@free-electrons.com> wrote:

> Hi Jisheng,
> 
> Sorry for the delay, I was quite busy these days...
> 
> Thanks for the nice comments!
> 
> I saw your using pin names in the BERLIN_PINCTRL_GROUP macro, like
> "EMMC_RSTn". In other berlin pinctrl drivers we use the group name (such
> as "G11" or GSM1"). If there is such a thing in the BG4CT, please
> update.

From newer SoCs than BG2Q, ASIC began to use more meaningful names such
as "EMMC_RSTn" as group name, there's no Gy or GSMy any more ;)

> 
> On Mon, Sep 21, 2015 at 06:04:16PM +0800, Jisheng Zhang wrote:
> [...]
> > --- a/drivers/pinctrl/berlin/Kconfig
> > +++ b/drivers/pinctrl/berlin/Kconfig
> > @@ -17,4 +17,9 @@ config PINCTRL_BERLIN_BG2Q
> >  	bool
> >  	select PINCTRL_BERLIN
> >  
> > +config PINCTRL_BERLIN4CT
> > +	depends on OF && (ARCH_BERLIN || COMPILE_TEST)
> > +	bool "Marvell berlin4ct pin controller driver"
> > +	select PINCTRL_BERLIN
> > +
> >  endif
> 
> These dependencies also apply to the other Berlin pinctrl drivers, so
> this should be on PINCTRL_BERLIN.

Will provide a separate patch to add this dependency

> 
> Is there a reason not to name the configuration option
> PINCTRL_BERLIN_BG4CT, to be consistent with the existing?

There's no special reason. I just like the name be consistent with BERLINxYY
used in dts, where, x= 2, 3, 4... YY may be CT, CD, etc. for example berlin4ct.
I could update in V3 if BERLIN_BGxYY is preferred


> 
> > diff --git a/drivers/pinctrl/berlin/berlin4ct.c b/drivers/pinctrl/berlin/berlin4ct.c
> > new file mode 100644
> > index 0000000..88e1670
> > --- /dev/null
> > +++ b/drivers/pinctrl/berlin/berlin4ct.c
> > +
> > +static const struct berlin_desc_group berlin4ct_soc_pinctrl_groups[] = {
> > +	BERLIN_PINCTRL_GROUP("EMMC_RSTn", 0x0, 0x3, 0x00,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "emmc"), /* RSTn */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* GPIO47 */
> > +	BERLIN_PINCTRL_GROUP("NAND_IO0", 0x0, 0x3, 0x03,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO0 */
> > +	BERLIN_PINCTRL_GROUP("NAND_IO1", 0x0, 0x3, 0x06,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CDn */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO1 */
> > +	BERLIN_PINCTRL_GROUP("NAND_IO2", 0x0, 0x3, 0x09,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO2 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD2 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO2 */
> > +	BERLIN_PINCTRL_GROUP("NAND_IO3", 0x0, 0x3, 0x0c,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO3 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD3 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO3 */
> > +	BERLIN_PINCTRL_GROUP("NAND_IO4", 0x0, 0x3, 0x0f,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO4 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXC */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT2 */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO4 */
> > +	BERLIN_PINCTRL_GROUP("NAND_IO5", 0x0, 0x3, 0x12,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO5 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXCTL */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT3 */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO5 */
> > +	BERLIN_PINCTRL_GROUP("NAND_IO6", 0x0, 0x3, 0x15,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO6 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* MDC */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CMD */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO6 */
> > +	BERLIN_PINCTRL_GROUP("NAND_IO7", 0x0, 0x3, 0x18,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO7 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* MDIO */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* WP */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO7 */
> > +	BERLIN_PINCTRL_GROUP("NAND_ALE", 0x0, 0x3, 0x1b,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* ALE */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO8 */
> > +	BERLIN_PINCTRL_GROUP("NAND_CLE", 0x4, 0x3, 0x00,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* CLE */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO9 */
> > +	BERLIN_PINCTRL_GROUP("NAND_WEn", 0x4, 0x3, 0x03,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* WEn */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD2 */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO10 */
> > +	BERLIN_PINCTRL_GROUP("NAND_REn", 0x4, 0x3, 0x06,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* REn */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD3 */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO11 */
> > +	BERLIN_PINCTRL_GROUP("NAND_WPn", 0x4, 0x3, 0x09,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* WPn */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO12 */
> > +	BERLIN_PINCTRL_GROUP("NAND_CEn", 0x4, 0x3, 0x0c,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* CEn */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXC */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO13 */
> > +	BERLIN_PINCTRL_GROUP("NAND_RDY", 0x4, 0x3, 0x0f,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* RDY */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXCTL */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO14 */
> > +	BERLIN_PINCTRL_GROUP("SD0_CLK", 0x4, 0x3, 0x12,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO29 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CLK*/
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* CLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG8 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG8 */
> > +	BERLIN_PINCTRL_GROUP("SD0_DAT0", 0x4, 0x3, 0x15,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO30 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* SOP */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG9 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG9 */
> > +	BERLIN_PINCTRL_GROUP("SD0_DAT1", 0x4, 0x3, 0x18,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO31 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* SD */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG10 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG10 */
> > +	BERLIN_PINCTRL_GROUP("SD0_DAT2", 0x4, 0x3, 0x1b,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO32 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT2 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* VALD */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG11 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG11 */
> > +	BERLIN_PINCTRL_GROUP("SD0_DAT3", 0x8, 0x3, 0x00,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO33 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT3 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* CLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG12 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG12 */
> > +	BERLIN_PINCTRL_GROUP("SD0_CDn", 0x8, 0x3, 0x03,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO34 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CDn */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* SOP */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG13 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG13 */
> > +	BERLIN_PINCTRL_GROUP("SD0_CMD", 0x8, 0x3, 0x06,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO35 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CMD */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* SD */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG14 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG14 */
> > +	BERLIN_PINCTRL_GROUP("SD0_WP", 0x8, 0x3, 0x09,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO36 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* WP */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* VALD */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG15 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG15 */
> > +	BERLIN_PINCTRL_GROUP("STS0_CLK", 0x8, 0x3, 0x0c,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO21 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* CLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "cpupll"), /* CLKO */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG0 */
> > +	BERLIN_PINCTRL_GROUP("STS0_SOP", 0x8, 0x3, 0x0f,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO22 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* SOP */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "syspll"), /* CLKO */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG1 */
> > +	BERLIN_PINCTRL_GROUP("STS0_SD", 0x8, 0x3, 0x12,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO23 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* SD */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "mempll"), /* CLKO */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG2 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG2 */
> > +	BERLIN_PINCTRL_GROUP("STS0_VALD", 0x8, 0x3, 0x15,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO24 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* VALD */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG3 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG3 */
> > +	BERLIN_PINCTRL_GROUP("STS1_CLK", 0x8, 0x3, 0x18,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO25 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* CLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0"),
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG4 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG4 */
> > +	BERLIN_PINCTRL_GROUP("STS1_SOP", 0x8, 0x3, 0x1b,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO26 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* SOP */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1"),
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG5 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG5 */
> > +	BERLIN_PINCTRL_GROUP("STS1_SD", 0xc, 0x3, 0x00,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO27 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* SD */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm2"),
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG6 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG6 */
> > +	BERLIN_PINCTRL_GROUP("STS1_VALD", 0xc, 0x3, 0x03,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO28 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* VALD */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm3"),
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG7 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG7 */
> > +	BERLIN_PINCTRL_GROUP("SCRD0_RST", 0xc, 0x3, 0x06,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO15 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* RST */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* CLK */
> > +	BERLIN_PINCTRL_GROUP("SCRD0_DCLK", 0xc, 0x3, 0x09,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO16 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* DCLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* CMD */
> > +	BERLIN_PINCTRL_GROUP("SCRD0_GPIO0", 0xc, 0x3, 0x0c,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO17 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* SCRD0 GPIO0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* DIO */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT0 */
> > +	BERLIN_PINCTRL_GROUP("SCRD0_GPIO1", 0xc, 0x3, 0x0f,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO18 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* SCRD0 GPIO1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* CLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT1 */
> > +	BERLIN_PINCTRL_GROUP("SCRD0_DIO", 0xc, 0x3, 0x12,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO19 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* DIO */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* DEN */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT2 */
> > +	BERLIN_PINCTRL_GROUP("SCRD0_CRD_PRES", 0xc, 0x3, 0x15,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO20 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* crd pres */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd1a")), /* DAT3 */
> > +	BERLIN_PINCTRL_GROUP("SPI1_SS0n", 0xc, 0x3, 0x18,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS0n */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO37 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2")), /* CLK */
> > +	BERLIN_PINCTRL_GROUP("SPI1_SS1n", 0xc, 0x3, 0x1b,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS1n */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO38 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2"), /* SOP */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "pwm1")),
> > +	BERLIN_PINCTRL_GROUP("SPI1_SS2n", 0x10, 0x3, 0x00,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS2n */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO39 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2"), /* SD */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "pwm0")),
> > +	BERLIN_PINCTRL_GROUP("SPI1_SS3n", 0x10, 0x3, 0x03,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS3n */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO40 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2")), /* VALD */
> > +	BERLIN_PINCTRL_GROUP("SPI1_SCLK", 0x10, 0x3, 0x06,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SCLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO41 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* CLK */
> > +	BERLIN_PINCTRL_GROUP("SPI1_SDO", 0x10, 0x3, 0x09,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SDO */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO42 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* SOP */
> > +	BERLIN_PINCTRL_GROUP("SPI1_SDI", 0x10, 0x3, 0x0c,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SDI */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO43 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* SD */
> > +	BERLIN_PINCTRL_GROUP("USB0_DRV_VBUS", 0x10, 0x3, 0x0f,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO44 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "usb0"), /* VBUS */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* VALD */
> > +	BERLIN_PINCTRL_GROUP("TW0_SCL", 0x10, 0x3, 0x12,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO45 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw0")), /* SCL */
> > +	BERLIN_PINCTRL_GROUP("TW0_SDA", 0x10, 0x3, 0x15,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO46 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw0")), /* SDA */
> > +};
> > +
> > +static const struct berlin_desc_group berlin4ct_soc_aviopinctrl_groups[] = {
> > +	BERLIN_PINCTRL_GROUP("TX_EDDC_SCL", 0x0, 0x3, 0x00,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "tx_eddc"), /* SCL */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "tw1")), /* SCL */
> > +	BERLIN_PINCTRL_GROUP("TX_EDDC_SDA", 0x0, 0x3, 0x03,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "tx_eddc"), /* SDA */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "tw1")), /* SDA */
> > +	BERLIN_PINCTRL_GROUP("I2S1_LRCKO", 0x0, 0x3, 0x06,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO2 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* LRCKO */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* CLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* CLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG0 */
> > +	BERLIN_PINCTRL_GROUP("I2S1_BCLKO", 0x0, 0x3, 0x09,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO3 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* BCLKO */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* SOP */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* CMD */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG1 */
> > +	BERLIN_PINCTRL_GROUP("I2S1_DO", 0x0, 0x3, 0x0c,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO4 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* DO */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* SD */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG2 */
> > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* DAT0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG2 */
> > +	BERLIN_PINCTRL_GROUP("I2S1_MCLK", 0x0, 0x3, 0x0f,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO5 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* MCLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* VALD */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* MCLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* DAT1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG3 */
> > +	BERLIN_PINCTRL_GROUP("SPDIFO", 0x0, 0x3, 0x12,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO6 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "spdifo"),
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "avpll"), /* CLKO */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac")), /* DBG3 */
> > +	BERLIN_PINCTRL_GROUP("I2S2_MCLK", 0x0, 0x3, 0x15,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO7 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* MCLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "hdmi"), /* FBCLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "pdm")), /* CLKO */
> > +	BERLIN_PINCTRL_GROUP("I2S2_LRCKI", 0x0, 0x3, 0x18,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO8 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* LRCKI */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0"),
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* CLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* LRCK */
> > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b_dat2")),
> 
> Why did you suffix this with _dat2? All the others are named "sd1b".

Good catch! Will fix this in v3

> 
> > +	BERLIN_PINCTRL_GROUP("I2S2_BCLKI", 0x0, 0x3, 0x1b,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO9 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* BCLKI */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1"),
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* SOP */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* BCLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* DAT3 */
> > +	BERLIN_PINCTRL_GROUP("I2S2_DI0", 0x4, 0x3, 0x00,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO10 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* DI0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm2"),
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* SD */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* SDIN */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "pdm"), /* DI0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* CDn */
> > +	BERLIN_PINCTRL_GROUP("I2S2_DI1", 0x4, 0x3, 0x03,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO11 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* DI1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm3"),
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* VALD */
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* PWMCLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "pdm"), /* DI1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* WP */
> > +};
> > +
> > +static const struct berlin_desc_group berlin4ct_sysmgr_pinctrl_groups[] = {
> > +	BERLIN_PINCTRL_GROUP("SM_TW2_SCL", 0x0, 0x3, 0x00,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO19 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw2")), /* SCL */
> > +	BERLIN_PINCTRL_GROUP("SM_TW2_SDA", 0x0, 0x3, 0x03,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO20 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw2")), /* SDA */
> > +	BERLIN_PINCTRL_GROUP("SM_TW3_SCL", 0x0, 0x3, 0x06,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO21 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw3")), /* SCL */
> > +	BERLIN_PINCTRL_GROUP("SM_TW3_SDA", 0x0, 0x3, 0x09,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO22 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw3")), /* SDA */
> > +	BERLIN_PINCTRL_GROUP("SM_TMS", 0x0, 0x3, 0x0c,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TMS */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO0 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0")),
> > +	BERLIN_PINCTRL_GROUP("SM_TDI", 0x0, 0x3, 0x0f,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TDI */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO1 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1")),
> > +	BERLIN_PINCTRL_GROUP("SM_TDO", 0x0, 0x3, 0x12,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TDO */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO2 */
> > +	BERLIN_PINCTRL_GROUP("SM_URT0_TXD", 0x0, 0x3, 0x15,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "urt0"), /* TXD */
> 
> s/urt/uart/

ASIC/HW/ guys call this as urt, I just want to keep consistent with them.
I could rename it if "uart" is preferred.

> 
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO3 */
> > +	BERLIN_PINCTRL_GROUP("SM_URT0_RXD", 0x0, 0x3, 0x18,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "urt0"), /* RXD */
> 
> Ditto.
> 
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO4 */
> > +	BERLIN_PINCTRL_GROUP("SM_URT1_TXD", 0x0, 0x3, 0x1b,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO5 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "urt1"), /* TXD */
> 
> Ditto.
> 
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "eth1_rxclk"), /* RXCLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm2"),
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "timer0"),
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "clk_25m")),
> > +	BERLIN_PINCTRL_GROUP("SM_URT1_RXD", 0x4, 0x3, 0x00,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO6 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "urt1"), /* RXD */
> 
> Ditto.
> 
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm3"),
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "timer1")),
> > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS0n", 0x4, 0x3, 0x03,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SS0 n*/
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO7 */
> > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS1n", 0x4, 0x3, 0x06,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO8 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "spi2")), /* SS1n */
> > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS2n", 0x4, 0x3, 0x09,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO9 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "spi2"), /* SS2n */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "eth1"), /* MDC */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm0"),
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "timer0"),
> > +			BERLIN_PINCTRL_FUNCTION(0x5, "clk_25m")),
> > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS3n", 0x4, 0x3, 0x0c,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO10 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "spi2"), /* SS3n */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "eth1"), /* MDIO */
> > +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm1"),
> > +			BERLIN_PINCTRL_FUNCTION(0x4, "timer1")),
> > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SDO", 0x4, 0x3, 0x0f,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SDO */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO11 */
> > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SDI", 0x4, 0x3, 0x12,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SDI */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO12 */
> > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SCLK", 0x4, 0x3, 0x15,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SCLK */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO13 */
> > +	BERLIN_PINCTRL_GROUP("SM_FE_LED0", 0x4, 0x3, 0x18,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO14 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED0 */
> 
> s/fe/led/
> 
> > +	BERLIN_PINCTRL_GROUP("SM_FE_LED1", 0x4, 0x3, 0x1b,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "pwr_ok"),
> 
> This is named "pwr" in other Berlin pinctrl drivers.

OK. will fix in v3

> 
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO 15 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED1 */
> 
> s/fe/led/
> 
> > +	BERLIN_PINCTRL_GROUP("SM_FE_LED2", 0x8, 0x3, 0x00,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO16 */
> > +			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED2 */
> 
> Ditto.
> 
> > +	BERLIN_PINCTRL_GROUP("SM_HDMI_HPD", 0x8, 0x3, 0x03,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO17 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "hdmi")), /* HPD */
> > +	BERLIN_PINCTRL_GROUP("SM_HDMI_CEC", 0x8, 0x3, 0x06,
> > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO18 */
> > +			BERLIN_PINCTRL_FUNCTION(0x1, "hdmi")), /* CEC */
> > +};
> > +
> > +static const struct berlin_pinctrl_desc berlin4ct_soc_pinctrl_data = {
> > +	.groups = berlin4ct_soc_pinctrl_groups,
> > +	.ngroups = ARRAY_SIZE(berlin4ct_soc_pinctrl_groups),
> > +};
> > +
> > +static const struct berlin_pinctrl_desc berlin4ct_soc_aviopinctrl_data = {
> 
> Shouldn't this be named berlin4ct_avio_pinctrl_data?
> 
> > +	.groups = berlin4ct_soc_aviopinctrl_groups,
> 
> And berlin4ct_avio_pinctrl_groups?

Will fix in v3

> 
> > +	.ngroups = ARRAY_SIZE(berlin4ct_soc_aviopinctrl_groups),
> > +};
> > +
> > +static const struct berlin_pinctrl_desc berlin4ct_sysmgr_pinctrl_data = {
> > +	.groups = berlin4ct_sysmgr_pinctrl_groups,
> > +	.ngroups = ARRAY_SIZE(berlin4ct_sysmgr_pinctrl_groups),
> > +};
> > +
> > +static const struct of_device_id berlin4ct_pinctrl_match[] = {
> > +	{
> > +		.compatible = "marvell,berlin4ct-soc-pinctrl",
> > +		.data = &berlin4ct_soc_pinctrl_data,
> > +	},
> > +	{
> > +		.compatible = "marvell,berlin4ct-avio-pinctrl",
> > +		.data = &berlin4ct_soc_aviopinctrl_data,
> > +	},
> > +	{
> > +		.compatible = "marvell,berlin4ct-sm-pinctrl",
> 
> If "sm" refers to system manager, please update this to
> "marvell,berlin4ct-system-pinctrl" to have the same naming as the other
> Berlin pinctrl drivers.

Yes, sm here means system manager. will fix in v3

Thanks a lot for review,
Jisheng
Jisheng Zhang Oct. 9, 2015, 7:05 a.m. UTC | #3 
Hi Antoine,

On Thu, 8 Oct 2015 18:46:53 +0800
Jisheng Zhang <jszhang@marvell.com> wrote:

> Hi Antoine,
> 
> On Fri, 2 Oct 2015 15:21:39 +0200
> Antoine Tenart <antoine.tenart@free-electrons.com> wrote:
> 
> > Hi Jisheng,
> > 
> > Sorry for the delay, I was quite busy these days...
> > 
> > Thanks for the nice comments!
> > 
> > I saw your using pin names in the BERLIN_PINCTRL_GROUP macro, like
> > "EMMC_RSTn". In other berlin pinctrl drivers we use the group name (such
> > as "G11" or GSM1"). If there is such a thing in the BG4CT, please
> > update.
> 
> From newer SoCs than BG2Q, ASIC began to use more meaningful names such
> as "EMMC_RSTn" as group name, there's no Gy or GSMy any more ;)
> 
> > 
> > On Mon, Sep 21, 2015 at 06:04:16PM +0800, Jisheng Zhang wrote:
> > [...]
> > > --- a/drivers/pinctrl/berlin/Kconfig
> > > +++ b/drivers/pinctrl/berlin/Kconfig
> > > @@ -17,4 +17,9 @@ config PINCTRL_BERLIN_BG2Q
> > >  	bool
> > >  	select PINCTRL_BERLIN
> > >  
> > > +config PINCTRL_BERLIN4CT
> > > +	depends on OF && (ARCH_BERLIN || COMPILE_TEST)
> > > +	bool "Marvell berlin4ct pin controller driver"
> > > +	select PINCTRL_BERLIN
> > > +
> > >  endif
> > 
> > These dependencies also apply to the other Berlin pinctrl drivers, so
> > this should be on PINCTRL_BERLIN.
> 
> Will provide a separate patch to add this dependency

Hmm, other berlin pinctrl driver options are not user visible and are selected
by MACH_BERLIN_BGxx, but here option for BG4CT is user visible, so it's better
to keep the dependency as this patch does. I also found similar pattern in
other pinctrl drivers, what do you think?

Thanks,
Jisheng

> 
> > 
> > Is there a reason not to name the configuration option
> > PINCTRL_BERLIN_BG4CT, to be consistent with the existing?
> 
> There's no special reason. I just like the name be consistent with BERLINxYY
> used in dts, where, x= 2, 3, 4... YY may be CT, CD, etc. for example berlin4ct.
> I could update in V3 if BERLIN_BGxYY is preferred
> 
> 
> > 
> > > diff --git a/drivers/pinctrl/berlin/berlin4ct.c b/drivers/pinctrl/berlin/berlin4ct.c
> > > new file mode 100644
> > > index 0000000..88e1670
> > > --- /dev/null
> > > +++ b/drivers/pinctrl/berlin/berlin4ct.c
> > > +
> > > +static const struct berlin_desc_group berlin4ct_soc_pinctrl_groups[] = {
> > > +	BERLIN_PINCTRL_GROUP("EMMC_RSTn", 0x0, 0x3, 0x00,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "emmc"), /* RSTn */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* GPIO47 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_IO0", 0x0, 0x3, 0x03,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO0 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_IO1", 0x0, 0x3, 0x06,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CDn */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO1 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_IO2", 0x0, 0x3, 0x09,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO2 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD2 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO2 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_IO3", 0x0, 0x3, 0x0c,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO3 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD3 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO3 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_IO4", 0x0, 0x3, 0x0f,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO4 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXC */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT2 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO4 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_IO5", 0x0, 0x3, 0x12,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO5 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXCTL */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT3 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO5 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_IO6", 0x0, 0x3, 0x15,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO6 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* MDC */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CMD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO6 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_IO7", 0x0, 0x3, 0x18,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO7 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* MDIO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* WP */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO7 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_ALE", 0x0, 0x3, 0x1b,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* ALE */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO8 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_CLE", 0x4, 0x3, 0x00,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* CLE */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO9 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_WEn", 0x4, 0x3, 0x03,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* WEn */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD2 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO10 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_REn", 0x4, 0x3, 0x06,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* REn */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD3 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO11 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_WPn", 0x4, 0x3, 0x09,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* WPn */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO12 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_CEn", 0x4, 0x3, 0x0c,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* CEn */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXC */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO13 */
> > > +	BERLIN_PINCTRL_GROUP("NAND_RDY", 0x4, 0x3, 0x0f,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* RDY */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXCTL */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO14 */
> > > +	BERLIN_PINCTRL_GROUP("SD0_CLK", 0x4, 0x3, 0x12,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO29 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CLK*/
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* CLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG8 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG8 */
> > > +	BERLIN_PINCTRL_GROUP("SD0_DAT0", 0x4, 0x3, 0x15,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO30 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* SOP */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG9 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG9 */
> > > +	BERLIN_PINCTRL_GROUP("SD0_DAT1", 0x4, 0x3, 0x18,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO31 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* SD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG10 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG10 */
> > > +	BERLIN_PINCTRL_GROUP("SD0_DAT2", 0x4, 0x3, 0x1b,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO32 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT2 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* VALD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG11 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG11 */
> > > +	BERLIN_PINCTRL_GROUP("SD0_DAT3", 0x8, 0x3, 0x00,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO33 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT3 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* CLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG12 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG12 */
> > > +	BERLIN_PINCTRL_GROUP("SD0_CDn", 0x8, 0x3, 0x03,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO34 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CDn */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* SOP */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG13 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG13 */
> > > +	BERLIN_PINCTRL_GROUP("SD0_CMD", 0x8, 0x3, 0x06,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO35 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CMD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* SD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG14 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG14 */
> > > +	BERLIN_PINCTRL_GROUP("SD0_WP", 0x8, 0x3, 0x09,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO36 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* WP */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* VALD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG15 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG15 */
> > > +	BERLIN_PINCTRL_GROUP("STS0_CLK", 0x8, 0x3, 0x0c,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO21 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* CLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "cpupll"), /* CLKO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG0 */
> > > +	BERLIN_PINCTRL_GROUP("STS0_SOP", 0x8, 0x3, 0x0f,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO22 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* SOP */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "syspll"), /* CLKO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG1 */
> > > +	BERLIN_PINCTRL_GROUP("STS0_SD", 0x8, 0x3, 0x12,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO23 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* SD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "mempll"), /* CLKO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG2 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG2 */
> > > +	BERLIN_PINCTRL_GROUP("STS0_VALD", 0x8, 0x3, 0x15,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO24 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* VALD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG3 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG3 */
> > > +	BERLIN_PINCTRL_GROUP("STS1_CLK", 0x8, 0x3, 0x18,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO25 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* CLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG4 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG4 */
> > > +	BERLIN_PINCTRL_GROUP("STS1_SOP", 0x8, 0x3, 0x1b,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO26 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* SOP */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG5 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG5 */
> > > +	BERLIN_PINCTRL_GROUP("STS1_SD", 0xc, 0x3, 0x00,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO27 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* SD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm2"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG6 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG6 */
> > > +	BERLIN_PINCTRL_GROUP("STS1_VALD", 0xc, 0x3, 0x03,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO28 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* VALD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm3"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG7 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG7 */
> > > +	BERLIN_PINCTRL_GROUP("SCRD0_RST", 0xc, 0x3, 0x06,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO15 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* RST */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* CLK */
> > > +	BERLIN_PINCTRL_GROUP("SCRD0_DCLK", 0xc, 0x3, 0x09,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO16 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* DCLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* CMD */
> > > +	BERLIN_PINCTRL_GROUP("SCRD0_GPIO0", 0xc, 0x3, 0x0c,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO17 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* SCRD0 GPIO0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* DIO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT0 */
> > > +	BERLIN_PINCTRL_GROUP("SCRD0_GPIO1", 0xc, 0x3, 0x0f,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO18 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* SCRD0 GPIO1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* CLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT1 */
> > > +	BERLIN_PINCTRL_GROUP("SCRD0_DIO", 0xc, 0x3, 0x12,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO19 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* DIO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* DEN */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT2 */
> > > +	BERLIN_PINCTRL_GROUP("SCRD0_CRD_PRES", 0xc, 0x3, 0x15,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO20 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* crd pres */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "sd1a")), /* DAT3 */
> > > +	BERLIN_PINCTRL_GROUP("SPI1_SS0n", 0xc, 0x3, 0x18,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS0n */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO37 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2")), /* CLK */
> > > +	BERLIN_PINCTRL_GROUP("SPI1_SS1n", 0xc, 0x3, 0x1b,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS1n */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO38 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2"), /* SOP */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "pwm1")),
> > > +	BERLIN_PINCTRL_GROUP("SPI1_SS2n", 0x10, 0x3, 0x00,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS2n */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO39 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2"), /* SD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "pwm0")),
> > > +	BERLIN_PINCTRL_GROUP("SPI1_SS3n", 0x10, 0x3, 0x03,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS3n */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO40 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts2")), /* VALD */
> > > +	BERLIN_PINCTRL_GROUP("SPI1_SCLK", 0x10, 0x3, 0x06,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SCLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO41 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* CLK */
> > > +	BERLIN_PINCTRL_GROUP("SPI1_SDO", 0x10, 0x3, 0x09,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SDO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO42 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* SOP */
> > > +	BERLIN_PINCTRL_GROUP("SPI1_SDI", 0x10, 0x3, 0x0c,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SDI */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO43 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* SD */
> > > +	BERLIN_PINCTRL_GROUP("USB0_DRV_VBUS", 0x10, 0x3, 0x0f,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO44 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "usb0"), /* VBUS */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* VALD */
> > > +	BERLIN_PINCTRL_GROUP("TW0_SCL", 0x10, 0x3, 0x12,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO45 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw0")), /* SCL */
> > > +	BERLIN_PINCTRL_GROUP("TW0_SDA", 0x10, 0x3, 0x15,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO46 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw0")), /* SDA */
> > > +};
> > > +
> > > +static const struct berlin_desc_group berlin4ct_soc_aviopinctrl_groups[] = {
> > > +	BERLIN_PINCTRL_GROUP("TX_EDDC_SCL", 0x0, 0x3, 0x00,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "tx_eddc"), /* SCL */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "tw1")), /* SCL */
> > > +	BERLIN_PINCTRL_GROUP("TX_EDDC_SDA", 0x0, 0x3, 0x03,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "tx_eddc"), /* SDA */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "tw1")), /* SDA */
> > > +	BERLIN_PINCTRL_GROUP("I2S1_LRCKO", 0x0, 0x3, 0x06,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO2 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* LRCKO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* CLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* CLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG0 */
> > > +	BERLIN_PINCTRL_GROUP("I2S1_BCLKO", 0x0, 0x3, 0x09,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO3 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* BCLKO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* SOP */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* CMD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG1 */
> > > +	BERLIN_PINCTRL_GROUP("I2S1_DO", 0x0, 0x3, 0x0c,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO4 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* DO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* SD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG2 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* DAT0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG2 */
> > > +	BERLIN_PINCTRL_GROUP("I2S1_MCLK", 0x0, 0x3, 0x0f,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO5 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* MCLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* VALD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* MCLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* DAT1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG3 */
> > > +	BERLIN_PINCTRL_GROUP("SPDIFO", 0x0, 0x3, 0x12,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO6 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "spdifo"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "avpll"), /* CLKO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac")), /* DBG3 */
> > > +	BERLIN_PINCTRL_GROUP("I2S2_MCLK", 0x0, 0x3, 0x15,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO7 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* MCLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "hdmi"), /* FBCLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "pdm")), /* CLKO */
> > > +	BERLIN_PINCTRL_GROUP("I2S2_LRCKI", 0x0, 0x3, 0x18,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO8 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* LRCKI */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* CLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* LRCK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b_dat2")),
> > 
> > Why did you suffix this with _dat2? All the others are named "sd1b".
> 
> Good catch! Will fix this in v3
> 
> > 
> > > +	BERLIN_PINCTRL_GROUP("I2S2_BCLKI", 0x0, 0x3, 0x1b,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO9 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* BCLKI */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* SOP */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* BCLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* DAT3 */
> > > +	BERLIN_PINCTRL_GROUP("I2S2_DI0", 0x4, 0x3, 0x00,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO10 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* DI0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm2"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* SD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* SDIN */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "pdm"), /* DI0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* CDn */
> > > +	BERLIN_PINCTRL_GROUP("I2S2_DI1", 0x4, 0x3, 0x03,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO11 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* DI1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm3"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* VALD */
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* PWMCLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "pdm"), /* DI1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* WP */
> > > +};
> > > +
> > > +static const struct berlin_desc_group berlin4ct_sysmgr_pinctrl_groups[] = {
> > > +	BERLIN_PINCTRL_GROUP("SM_TW2_SCL", 0x0, 0x3, 0x00,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO19 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw2")), /* SCL */
> > > +	BERLIN_PINCTRL_GROUP("SM_TW2_SDA", 0x0, 0x3, 0x03,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO20 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw2")), /* SDA */
> > > +	BERLIN_PINCTRL_GROUP("SM_TW3_SCL", 0x0, 0x3, 0x06,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO21 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw3")), /* SCL */
> > > +	BERLIN_PINCTRL_GROUP("SM_TW3_SDA", 0x0, 0x3, 0x09,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO22 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "tw3")), /* SDA */
> > > +	BERLIN_PINCTRL_GROUP("SM_TMS", 0x0, 0x3, 0x0c,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TMS */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO0 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0")),
> > > +	BERLIN_PINCTRL_GROUP("SM_TDI", 0x0, 0x3, 0x0f,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TDI */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO1 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1")),
> > > +	BERLIN_PINCTRL_GROUP("SM_TDO", 0x0, 0x3, 0x12,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TDO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO2 */
> > > +	BERLIN_PINCTRL_GROUP("SM_URT0_TXD", 0x0, 0x3, 0x15,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "urt0"), /* TXD */
> > 
> > s/urt/uart/
> 
> ASIC/HW/ guys call this as urt, I just want to keep consistent with them.
> I could rename it if "uart" is preferred.
> 
> > 
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO3 */
> > > +	BERLIN_PINCTRL_GROUP("SM_URT0_RXD", 0x0, 0x3, 0x18,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "urt0"), /* RXD */
> > 
> > Ditto.
> > 
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO4 */
> > > +	BERLIN_PINCTRL_GROUP("SM_URT1_TXD", 0x0, 0x3, 0x1b,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO5 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "urt1"), /* TXD */
> > 
> > Ditto.
> > 
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "eth1_rxclk"), /* RXCLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm2"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "timer0"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "clk_25m")),
> > > +	BERLIN_PINCTRL_GROUP("SM_URT1_RXD", 0x4, 0x3, 0x00,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO6 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "urt1"), /* RXD */
> > 
> > Ditto.
> > 
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm3"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "timer1")),
> > > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS0n", 0x4, 0x3, 0x03,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SS0 n*/
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO7 */
> > > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS1n", 0x4, 0x3, 0x06,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO8 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "spi2")), /* SS1n */
> > > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS2n", 0x4, 0x3, 0x09,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO9 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "spi2"), /* SS2n */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "eth1"), /* MDC */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm0"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "timer0"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x5, "clk_25m")),
> > > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SS3n", 0x4, 0x3, 0x0c,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO10 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "spi2"), /* SS3n */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "eth1"), /* MDIO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x3, "pwm1"),
> > > +			BERLIN_PINCTRL_FUNCTION(0x4, "timer1")),
> > > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SDO", 0x4, 0x3, 0x0f,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SDO */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO11 */
> > > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SDI", 0x4, 0x3, 0x12,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SDI */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO12 */
> > > +	BERLIN_PINCTRL_GROUP("SM_SPI2_SCLK", 0x4, 0x3, 0x15,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SCLK */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO13 */
> > > +	BERLIN_PINCTRL_GROUP("SM_FE_LED0", 0x4, 0x3, 0x18,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO14 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED0 */
> > 
> > s/fe/led/
> > 
> > > +	BERLIN_PINCTRL_GROUP("SM_FE_LED1", 0x4, 0x3, 0x1b,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "pwr_ok"),
> > 
> > This is named "pwr" in other Berlin pinctrl drivers.
> 
> OK. will fix in v3
> 
> > 
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO 15 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED1 */
> > 
> > s/fe/led/
> > 
> > > +	BERLIN_PINCTRL_GROUP("SM_FE_LED2", 0x8, 0x3, 0x00,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO16 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED2 */
> > 
> > Ditto.
> > 
> > > +	BERLIN_PINCTRL_GROUP("SM_HDMI_HPD", 0x8, 0x3, 0x03,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO17 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "hdmi")), /* HPD */
> > > +	BERLIN_PINCTRL_GROUP("SM_HDMI_CEC", 0x8, 0x3, 0x06,
> > > +			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO18 */
> > > +			BERLIN_PINCTRL_FUNCTION(0x1, "hdmi")), /* CEC */
> > > +};
> > > +
> > > +static const struct berlin_pinctrl_desc berlin4ct_soc_pinctrl_data = {
> > > +	.groups = berlin4ct_soc_pinctrl_groups,
> > > +	.ngroups = ARRAY_SIZE(berlin4ct_soc_pinctrl_groups),
> > > +};
> > > +
> > > +static const struct berlin_pinctrl_desc berlin4ct_soc_aviopinctrl_data = {
> > 
> > Shouldn't this be named berlin4ct_avio_pinctrl_data?
> > 
> > > +	.groups = berlin4ct_soc_aviopinctrl_groups,
> > 
> > And berlin4ct_avio_pinctrl_groups?
> 
> Will fix in v3
> 
> > 
> > > +	.ngroups = ARRAY_SIZE(berlin4ct_soc_aviopinctrl_groups),
> > > +};
> > > +
> > > +static const struct berlin_pinctrl_desc berlin4ct_sysmgr_pinctrl_data = {
> > > +	.groups = berlin4ct_sysmgr_pinctrl_groups,
> > > +	.ngroups = ARRAY_SIZE(berlin4ct_sysmgr_pinctrl_groups),
> > > +};
> > > +
> > > +static const struct of_device_id berlin4ct_pinctrl_match[] = {
> > > +	{
> > > +		.compatible = "marvell,berlin4ct-soc-pinctrl",
> > > +		.data = &berlin4ct_soc_pinctrl_data,
> > > +	},
> > > +	{
> > > +		.compatible = "marvell,berlin4ct-avio-pinctrl",
> > > +		.data = &berlin4ct_soc_aviopinctrl_data,
> > > +	},
> > > +	{
> > > +		.compatible = "marvell,berlin4ct-sm-pinctrl",
> > 
> > If "sm" refers to system manager, please update this to
> > "marvell,berlin4ct-system-pinctrl" to have the same naming as the other
> > Berlin pinctrl drivers.
> 
> Yes, sm here means system manager. will fix in v3
> 
> Thanks a lot for review,
> Jisheng
diff mbox

Patch

diff --git a/drivers/pinctrl/berlin/Kconfig b/drivers/pinctrl/berlin/Kconfig
index b18322b..eac4d28 100644
--- a/drivers/pinctrl/berlin/Kconfig
+++ b/drivers/pinctrl/berlin/Kconfig
@@ -17,4 +17,9 @@  config PINCTRL_BERLIN_BG2Q
 	bool
 	select PINCTRL_BERLIN
 
+config PINCTRL_BERLIN4CT
+	depends on OF && (ARCH_BERLIN || COMPILE_TEST)
+	bool "Marvell berlin4ct pin controller driver"
+	select PINCTRL_BERLIN
+
 endif
diff --git a/drivers/pinctrl/berlin/Makefile b/drivers/pinctrl/berlin/Makefile
index deb0c6b..c8fe2a0 100644
--- a/drivers/pinctrl/berlin/Makefile
+++ b/drivers/pinctrl/berlin/Makefile
@@ -2,3 +2,4 @@  obj-$(CONFIG_PINCTRL_BERLIN)		+= berlin.o
 obj-$(CONFIG_PINCTRL_BERLIN_BG2)	+= berlin-bg2.o
 obj-$(CONFIG_PINCTRL_BERLIN_BG2CD)	+= berlin-bg2cd.o
 obj-$(CONFIG_PINCTRL_BERLIN_BG2Q)	+= berlin-bg2q.o
+obj-$(CONFIG_PINCTRL_BERLIN4CT)		+= berlin4ct.o
diff --git a/drivers/pinctrl/berlin/berlin4ct.c b/drivers/pinctrl/berlin/berlin4ct.c
new file mode 100644
index 0000000..88e1670
--- /dev/null
+++ b/drivers/pinctrl/berlin/berlin4ct.c
@@ -0,0 +1,503 @@ 
+/*
+ * Marvell berlin4ct pinctrl driver
+ *
+ * Copyright (C) 2015 Marvell Technology Group Ltd.
+ *
+ * Author: Jisheng Zhang <jszhang@marvell.com>
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms and conditions of the GNU General Public License,
+ * version 2, as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
+ * more details.
+ *
+ * You should have received a copy of the GNU General Public License along with
+ * this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <linux/module.h>
+#include <linux/of_device.h>
+#include <linux/platform_device.h>
+#include <linux/regmap.h>
+
+#include "berlin.h"
+
+static const struct berlin_desc_group berlin4ct_soc_pinctrl_groups[] = {
+	BERLIN_PINCTRL_GROUP("EMMC_RSTn", 0x0, 0x3, 0x00,
+			BERLIN_PINCTRL_FUNCTION(0x0, "emmc"), /* RSTn */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* GPIO47 */
+	BERLIN_PINCTRL_GROUP("NAND_IO0", 0x0, 0x3, 0x03,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO0 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD0 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CLK */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO0 */
+	BERLIN_PINCTRL_GROUP("NAND_IO1", 0x0, 0x3, 0x06,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO1 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD1 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CDn */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO1 */
+	BERLIN_PINCTRL_GROUP("NAND_IO2", 0x0, 0x3, 0x09,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO2 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD2 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT0 */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO2 */
+	BERLIN_PINCTRL_GROUP("NAND_IO3", 0x0, 0x3, 0x0c,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO3 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXD3 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT1 */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO3 */
+	BERLIN_PINCTRL_GROUP("NAND_IO4", 0x0, 0x3, 0x0f,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO4 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXC */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT2 */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO4 */
+	BERLIN_PINCTRL_GROUP("NAND_IO5", 0x0, 0x3, 0x12,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO5 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* RXCTL */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* DAT3 */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO5 */
+	BERLIN_PINCTRL_GROUP("NAND_IO6", 0x0, 0x3, 0x15,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO6 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* MDC */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* CMD */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO6 */
+	BERLIN_PINCTRL_GROUP("NAND_IO7", 0x0, 0x3, 0x18,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* IO7 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* MDIO */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sd1"), /* WP */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO7 */
+	BERLIN_PINCTRL_GROUP("NAND_ALE", 0x0, 0x3, 0x1b,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* ALE */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD0 */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO8 */
+	BERLIN_PINCTRL_GROUP("NAND_CLE", 0x4, 0x3, 0x00,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* CLE */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD1 */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO9 */
+	BERLIN_PINCTRL_GROUP("NAND_WEn", 0x4, 0x3, 0x03,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* WEn */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD2 */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO10 */
+	BERLIN_PINCTRL_GROUP("NAND_REn", 0x4, 0x3, 0x06,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* REn */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXD3 */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO11 */
+	BERLIN_PINCTRL_GROUP("NAND_WPn", 0x4, 0x3, 0x09,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* WPn */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO12 */
+	BERLIN_PINCTRL_GROUP("NAND_CEn", 0x4, 0x3, 0x0c,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* CEn */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXC */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO13 */
+	BERLIN_PINCTRL_GROUP("NAND_RDY", 0x4, 0x3, 0x0f,
+			BERLIN_PINCTRL_FUNCTION(0x0, "nand"), /* RDY */
+			BERLIN_PINCTRL_FUNCTION(0x1, "rgmii"), /* TXCTL */
+			BERLIN_PINCTRL_FUNCTION(0x3, "gpio")), /* GPIO14 */
+	BERLIN_PINCTRL_GROUP("SD0_CLK", 0x4, 0x3, 0x12,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO29 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CLK*/
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* CLK */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG8 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG8 */
+	BERLIN_PINCTRL_GROUP("SD0_DAT0", 0x4, 0x3, 0x15,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO30 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT0 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* SOP */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG9 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG9 */
+	BERLIN_PINCTRL_GROUP("SD0_DAT1", 0x4, 0x3, 0x18,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO31 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT1 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* SD */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG10 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG10 */
+	BERLIN_PINCTRL_GROUP("SD0_DAT2", 0x4, 0x3, 0x1b,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO32 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT2 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts4"), /* VALD */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG11 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG11 */
+	BERLIN_PINCTRL_GROUP("SD0_DAT3", 0x8, 0x3, 0x00,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO33 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* DAT3 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* CLK */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG12 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG12 */
+	BERLIN_PINCTRL_GROUP("SD0_CDn", 0x8, 0x3, 0x03,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO34 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CDn */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* SOP */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG13 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG13 */
+	BERLIN_PINCTRL_GROUP("SD0_CMD", 0x8, 0x3, 0x06,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO35 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* CMD */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* SD */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG14 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG14 */
+	BERLIN_PINCTRL_GROUP("SD0_WP", 0x8, 0x3, 0x09,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO36 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sd0"), /* WP */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts5"), /* VALD */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG15 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG15 */
+	BERLIN_PINCTRL_GROUP("STS0_CLK", 0x8, 0x3, 0x0c,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO21 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* CLK */
+			BERLIN_PINCTRL_FUNCTION(0x2, "cpupll"), /* CLKO */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG0 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG0 */
+	BERLIN_PINCTRL_GROUP("STS0_SOP", 0x8, 0x3, 0x0f,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO22 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* SOP */
+			BERLIN_PINCTRL_FUNCTION(0x2, "syspll"), /* CLKO */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG1 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG1 */
+	BERLIN_PINCTRL_GROUP("STS0_SD", 0x8, 0x3, 0x12,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO23 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* SD */
+			BERLIN_PINCTRL_FUNCTION(0x2, "mempll"), /* CLKO */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG2 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG2 */
+	BERLIN_PINCTRL_GROUP("STS0_VALD", 0x8, 0x3, 0x15,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO24 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sts0"), /* VALD */
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG3 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG3 */
+	BERLIN_PINCTRL_GROUP("STS1_CLK", 0x8, 0x3, 0x18,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO25 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* CLK */
+			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0"),
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG4 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG4 */
+	BERLIN_PINCTRL_GROUP("STS1_SOP", 0x8, 0x3, 0x1b,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO26 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* SOP */
+			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1"),
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG5 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG5 */
+	BERLIN_PINCTRL_GROUP("STS1_SD", 0xc, 0x3, 0x00,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO27 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* SD */
+			BERLIN_PINCTRL_FUNCTION(0x2, "pwm2"),
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG6 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG6 */
+	BERLIN_PINCTRL_GROUP("STS1_VALD", 0xc, 0x3, 0x03,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO28 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sts1"), /* VALD */
+			BERLIN_PINCTRL_FUNCTION(0x2, "pwm3"),
+			BERLIN_PINCTRL_FUNCTION(0x5, "v4g"), /* DBG7 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "phy")), /* DBG7 */
+	BERLIN_PINCTRL_GROUP("SCRD0_RST", 0xc, 0x3, 0x06,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO15 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* RST */
+			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* CLK */
+	BERLIN_PINCTRL_GROUP("SCRD0_DCLK", 0xc, 0x3, 0x09,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO16 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* DCLK */
+			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* CMD */
+	BERLIN_PINCTRL_GROUP("SCRD0_GPIO0", 0xc, 0x3, 0x0c,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO17 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* SCRD0 GPIO0 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* DIO */
+			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT0 */
+	BERLIN_PINCTRL_GROUP("SCRD0_GPIO1", 0xc, 0x3, 0x0f,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO18 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* SCRD0 GPIO1 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* CLK */
+			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT1 */
+	BERLIN_PINCTRL_GROUP("SCRD0_DIO", 0xc, 0x3, 0x12,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO19 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* DIO */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sif"), /* DEN */
+			BERLIN_PINCTRL_FUNCTION(0x3, "sd1a")), /* DAT2 */
+	BERLIN_PINCTRL_GROUP("SCRD0_CRD_PRES", 0xc, 0x3, 0x15,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO20 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "scrd0"), /* crd pres */
+			BERLIN_PINCTRL_FUNCTION(0x1, "sd1a")), /* DAT3 */
+	BERLIN_PINCTRL_GROUP("SPI1_SS0n", 0xc, 0x3, 0x18,
+			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS0n */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO37 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts2")), /* CLK */
+	BERLIN_PINCTRL_GROUP("SPI1_SS1n", 0xc, 0x3, 0x1b,
+			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS1n */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO38 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts2"), /* SOP */
+			BERLIN_PINCTRL_FUNCTION(0x4, "pwm1")),
+	BERLIN_PINCTRL_GROUP("SPI1_SS2n", 0x10, 0x3, 0x00,
+			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS2n */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO39 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts2"), /* SD */
+			BERLIN_PINCTRL_FUNCTION(0x4, "pwm0")),
+	BERLIN_PINCTRL_GROUP("SPI1_SS3n", 0x10, 0x3, 0x03,
+			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SS3n */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO40 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts2")), /* VALD */
+	BERLIN_PINCTRL_GROUP("SPI1_SCLK", 0x10, 0x3, 0x06,
+			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SCLK */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO41 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* CLK */
+	BERLIN_PINCTRL_GROUP("SPI1_SDO", 0x10, 0x3, 0x09,
+			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SDO */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO42 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* SOP */
+	BERLIN_PINCTRL_GROUP("SPI1_SDI", 0x10, 0x3, 0x0c,
+			BERLIN_PINCTRL_FUNCTION(0x0, "spi1"), /* SDI */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* GPIO43 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* SD */
+	BERLIN_PINCTRL_GROUP("USB0_DRV_VBUS", 0x10, 0x3, 0x0f,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO44 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "usb0"), /* VBUS */
+			BERLIN_PINCTRL_FUNCTION(0x2, "sts3")), /* VALD */
+	BERLIN_PINCTRL_GROUP("TW0_SCL", 0x10, 0x3, 0x12,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO45 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "tw0")), /* SCL */
+	BERLIN_PINCTRL_GROUP("TW0_SDA", 0x10, 0x3, 0x15,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* GPIO46 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "tw0")), /* SDA */
+};
+
+static const struct berlin_desc_group berlin4ct_soc_aviopinctrl_groups[] = {
+	BERLIN_PINCTRL_GROUP("TX_EDDC_SCL", 0x0, 0x3, 0x00,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO0 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "tx_eddc"), /* SCL */
+			BERLIN_PINCTRL_FUNCTION(0x2, "tw1")), /* SCL */
+	BERLIN_PINCTRL_GROUP("TX_EDDC_SDA", 0x0, 0x3, 0x03,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO1 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "tx_eddc"), /* SDA */
+			BERLIN_PINCTRL_FUNCTION(0x2, "tw1")), /* SDA */
+	BERLIN_PINCTRL_GROUP("I2S1_LRCKO", 0x0, 0x3, 0x06,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO2 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* LRCKO */
+			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* CLK */
+			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG0 */
+			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* CLK */
+			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG0 */
+	BERLIN_PINCTRL_GROUP("I2S1_BCLKO", 0x0, 0x3, 0x09,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO3 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* BCLKO */
+			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* SOP */
+			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG1 */
+			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* CMD */
+			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG1 */
+	BERLIN_PINCTRL_GROUP("I2S1_DO", 0x0, 0x3, 0x0c,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO4 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* DO */
+			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* SD */
+			BERLIN_PINCTRL_FUNCTION(0x4, "adac"), /* DBG2 */
+			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* DAT0 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG2 */
+	BERLIN_PINCTRL_GROUP("I2S1_MCLK", 0x0, 0x3, 0x0f,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO5 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "i2s1"), /* MCLK */
+			BERLIN_PINCTRL_FUNCTION(0x3, "sts6"), /* VALD */
+			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* MCLK */
+			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b"), /* DAT1 */
+			BERLIN_PINCTRL_FUNCTION(0x7, "avio")), /* DBG3 */
+	BERLIN_PINCTRL_GROUP("SPDIFO", 0x0, 0x3, 0x12,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO6 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "spdifo"),
+			BERLIN_PINCTRL_FUNCTION(0x2, "avpll"), /* CLKO */
+			BERLIN_PINCTRL_FUNCTION(0x4, "adac")), /* DBG3 */
+	BERLIN_PINCTRL_GROUP("I2S2_MCLK", 0x0, 0x3, 0x15,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO7 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* MCLK */
+			BERLIN_PINCTRL_FUNCTION(0x4, "hdmi"), /* FBCLK */
+			BERLIN_PINCTRL_FUNCTION(0x5, "pdm")), /* CLKO */
+	BERLIN_PINCTRL_GROUP("I2S2_LRCKI", 0x0, 0x3, 0x18,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO8 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* LRCKI */
+			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0"),
+			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* CLK */
+			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* LRCK */
+			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b_dat2")),
+	BERLIN_PINCTRL_GROUP("I2S2_BCLKI", 0x0, 0x3, 0x1b,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO9 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* BCLKI */
+			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1"),
+			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* SOP */
+			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* BCLK */
+			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* DAT3 */
+	BERLIN_PINCTRL_GROUP("I2S2_DI0", 0x4, 0x3, 0x00,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO10 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* DI0 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "pwm2"),
+			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* SD */
+			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* SDIN */
+			BERLIN_PINCTRL_FUNCTION(0x5, "pdm"), /* DI0 */
+			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* CDn */
+	BERLIN_PINCTRL_GROUP("I2S2_DI1", 0x4, 0x3, 0x03,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* AVIO GPIO11 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "i2s2"), /* DI1 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "pwm3"),
+			BERLIN_PINCTRL_FUNCTION(0x3, "sts7"), /* VALD */
+			BERLIN_PINCTRL_FUNCTION(0x4, "adac_test"), /* PWMCLK */
+			BERLIN_PINCTRL_FUNCTION(0x5, "pdm"), /* DI1 */
+			BERLIN_PINCTRL_FUNCTION(0x6, "sd1b")), /* WP */
+};
+
+static const struct berlin_desc_group berlin4ct_sysmgr_pinctrl_groups[] = {
+	BERLIN_PINCTRL_GROUP("SM_TW2_SCL", 0x0, 0x3, 0x00,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO19 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "tw2")), /* SCL */
+	BERLIN_PINCTRL_GROUP("SM_TW2_SDA", 0x0, 0x3, 0x03,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO20 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "tw2")), /* SDA */
+	BERLIN_PINCTRL_GROUP("SM_TW3_SCL", 0x0, 0x3, 0x06,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO21 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "tw3")), /* SCL */
+	BERLIN_PINCTRL_GROUP("SM_TW3_SDA", 0x0, 0x3, 0x09,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO22 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "tw3")), /* SDA */
+	BERLIN_PINCTRL_GROUP("SM_TMS", 0x0, 0x3, 0x0c,
+			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TMS */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO0 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "pwm0")),
+	BERLIN_PINCTRL_GROUP("SM_TDI", 0x0, 0x3, 0x0f,
+			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TDI */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO1 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "pwm1")),
+	BERLIN_PINCTRL_GROUP("SM_TDO", 0x0, 0x3, 0x12,
+			BERLIN_PINCTRL_FUNCTION(0x0, "jtag"), /* TDO */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO2 */
+	BERLIN_PINCTRL_GROUP("SM_URT0_TXD", 0x0, 0x3, 0x15,
+			BERLIN_PINCTRL_FUNCTION(0x0, "urt0"), /* TXD */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO3 */
+	BERLIN_PINCTRL_GROUP("SM_URT0_RXD", 0x0, 0x3, 0x18,
+			BERLIN_PINCTRL_FUNCTION(0x0, "urt0"), /* RXD */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO4 */
+	BERLIN_PINCTRL_GROUP("SM_URT1_TXD", 0x0, 0x3, 0x1b,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO5 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "urt1"), /* TXD */
+			BERLIN_PINCTRL_FUNCTION(0x2, "eth1_rxclk"), /* RXCLK */
+			BERLIN_PINCTRL_FUNCTION(0x3, "pwm2"),
+			BERLIN_PINCTRL_FUNCTION(0x4, "timer0"),
+			BERLIN_PINCTRL_FUNCTION(0x5, "clk_25m")),
+	BERLIN_PINCTRL_GROUP("SM_URT1_RXD", 0x4, 0x3, 0x00,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO6 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "urt1"), /* RXD */
+			BERLIN_PINCTRL_FUNCTION(0x3, "pwm3"),
+			BERLIN_PINCTRL_FUNCTION(0x4, "timer1")),
+	BERLIN_PINCTRL_GROUP("SM_SPI2_SS0n", 0x4, 0x3, 0x03,
+			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SS0 n*/
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO7 */
+	BERLIN_PINCTRL_GROUP("SM_SPI2_SS1n", 0x4, 0x3, 0x06,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO8 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "spi2")), /* SS1n */
+	BERLIN_PINCTRL_GROUP("SM_SPI2_SS2n", 0x4, 0x3, 0x09,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO9 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "spi2"), /* SS2n */
+			BERLIN_PINCTRL_FUNCTION(0x2, "eth1"), /* MDC */
+			BERLIN_PINCTRL_FUNCTION(0x3, "pwm0"),
+			BERLIN_PINCTRL_FUNCTION(0x4, "timer0"),
+			BERLIN_PINCTRL_FUNCTION(0x5, "clk_25m")),
+	BERLIN_PINCTRL_GROUP("SM_SPI2_SS3n", 0x4, 0x3, 0x0c,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO10 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "spi2"), /* SS3n */
+			BERLIN_PINCTRL_FUNCTION(0x2, "eth1"), /* MDIO */
+			BERLIN_PINCTRL_FUNCTION(0x3, "pwm1"),
+			BERLIN_PINCTRL_FUNCTION(0x4, "timer1")),
+	BERLIN_PINCTRL_GROUP("SM_SPI2_SDO", 0x4, 0x3, 0x0f,
+			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SDO */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO11 */
+	BERLIN_PINCTRL_GROUP("SM_SPI2_SDI", 0x4, 0x3, 0x12,
+			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SDI */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO12 */
+	BERLIN_PINCTRL_GROUP("SM_SPI2_SCLK", 0x4, 0x3, 0x15,
+			BERLIN_PINCTRL_FUNCTION(0x0, "spi2"), /* SCLK */
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio")), /* SM GPIO13 */
+	BERLIN_PINCTRL_GROUP("SM_FE_LED0", 0x4, 0x3, 0x18,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO14 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED0 */
+	BERLIN_PINCTRL_GROUP("SM_FE_LED1", 0x4, 0x3, 0x1b,
+			BERLIN_PINCTRL_FUNCTION(0x0, "pwr_ok"),
+			BERLIN_PINCTRL_FUNCTION(0x1, "gpio"), /* SM GPIO 15 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED1 */
+	BERLIN_PINCTRL_GROUP("SM_FE_LED2", 0x8, 0x3, 0x00,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO16 */
+			BERLIN_PINCTRL_FUNCTION(0x2, "fe")), /* LED2 */
+	BERLIN_PINCTRL_GROUP("SM_HDMI_HPD", 0x8, 0x3, 0x03,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO17 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "hdmi")), /* HPD */
+	BERLIN_PINCTRL_GROUP("SM_HDMI_CEC", 0x8, 0x3, 0x06,
+			BERLIN_PINCTRL_FUNCTION(0x0, "gpio"), /* SM GPIO18 */
+			BERLIN_PINCTRL_FUNCTION(0x1, "hdmi")), /* CEC */
+};
+
+static const struct berlin_pinctrl_desc berlin4ct_soc_pinctrl_data = {
+	.groups = berlin4ct_soc_pinctrl_groups,
+	.ngroups = ARRAY_SIZE(berlin4ct_soc_pinctrl_groups),
+};
+
+static const struct berlin_pinctrl_desc berlin4ct_soc_aviopinctrl_data = {
+	.groups = berlin4ct_soc_aviopinctrl_groups,
+	.ngroups = ARRAY_SIZE(berlin4ct_soc_aviopinctrl_groups),
+};
+
+static const struct berlin_pinctrl_desc berlin4ct_sysmgr_pinctrl_data = {
+	.groups = berlin4ct_sysmgr_pinctrl_groups,
+	.ngroups = ARRAY_SIZE(berlin4ct_sysmgr_pinctrl_groups),
+};
+
+static const struct of_device_id berlin4ct_pinctrl_match[] = {
+	{
+		.compatible = "marvell,berlin4ct-soc-pinctrl",
+		.data = &berlin4ct_soc_pinctrl_data,
+	},
+	{
+		.compatible = "marvell,berlin4ct-avio-pinctrl",
+		.data = &berlin4ct_soc_aviopinctrl_data,
+	},
+	{
+		.compatible = "marvell,berlin4ct-sm-pinctrl",
+		.data = &berlin4ct_sysmgr_pinctrl_data,
+	},
+	{}
+};
+MODULE_DEVICE_TABLE(of, berlin4ct_pinctrl_match);
+
+static int berlin4ct_pinctrl_probe(struct platform_device *pdev)
+{
+	const struct of_device_id *match =
+		of_match_device(berlin4ct_pinctrl_match, &pdev->dev);
+	struct regmap_config *rmconfig;
+	struct regmap *regmap;
+	struct resource *res;
+	void __iomem *base;
+
+	rmconfig = devm_kzalloc(&pdev->dev, sizeof(*rmconfig), GFP_KERNEL);
+	if (!rmconfig)
+		return -ENOMEM;
+
+	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+	base = devm_ioremap_resource(&pdev->dev, res);
+	if (IS_ERR(base))
+		return PTR_ERR(base);
+
+	rmconfig->reg_bits = 32,
+	rmconfig->val_bits = 32,
+	rmconfig->reg_stride = 4,
+	rmconfig->max_register = resource_size(res);
+
+	regmap = devm_regmap_init_mmio(&pdev->dev, base, rmconfig);
+	if (IS_ERR(regmap))
+		return PTR_ERR(regmap);
+
+	return berlin_pinctrl_probe(pdev, regmap, match->data);
+}
+
+static struct platform_driver berlin4ct_pinctrl_driver = {
+	.probe	= berlin4ct_pinctrl_probe,
+	.driver	= {
+		.name = "berlin4ct-pinctrl",
+		.of_match_table = berlin4ct_pinctrl_match,
+	},
+};
+module_platform_driver(berlin4ct_pinctrl_driver);
+
+MODULE_AUTHOR("Jisheng Zhang <jszhang@marvell.com>");
+MODULE_DESCRIPTION("Marvell berlin4ct pinctrl driver");
+MODULE_LICENSE("GPL");