* [PATCH v2 0/6] arm64: berlin: add pinctrl support
@ 2015-09-21 10:04 Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 1/6] pinctrl: berlin: regmap as an extra argument of berlin_pinctrl_probe() Jisheng Zhang
` (5 more replies)
0 siblings, 6 replies; 15+ messages in thread
From: Jisheng Zhang @ 2015-09-21 10:04 UTC (permalink / raw)
To: sebastian.hesselbarth, linus.walleij, antoine.tenart,
catalin.marinas, will.deacon, mark.rutland, robh+dt, pawel.moll,
ijc+devicetree, galak
Cc: devicetree, linux-arm-kernel, linux-gpio, linux-kernel, Jisheng Zhang
This series adds support for Marvell berlin4ct pin-controller, allowing
to configure the pin muxing from the device tree.
Since v1:
- use generic name for pin functions
- a new commit to add pinmux for uart0
- correct pinctrl usage in dts
Jisheng Zhang (6):
pinctrl: berlin: regmap as an extra argument of berlin_pinctrl_probe()
pinctrl: berlin: add the berlin4ct pinctrl driver
arm64: berlin: add the pinctrl dependency for Marvell Berlin SoCs
pinctrl: dt-binding: document berlin4ct SoC pinctrl
arm64: dts: berlin4ct: add the pinctrl node
arm64: dts: berlin4ct: add default pinmux for uart0
.../devicetree/bindings/pinctrl/berlin,pinctrl.txt | 5 +-
arch/arm64/Kconfig.platforms | 1 +
arch/arm64/boot/dts/marvell/berlin4ct.dtsi | 22 +
drivers/pinctrl/berlin/Kconfig | 5 +
drivers/pinctrl/berlin/Makefile | 1 +
drivers/pinctrl/berlin/berlin-bg2.c | 10 +-
drivers/pinctrl/berlin/berlin-bg2cd.c | 10 +-
drivers/pinctrl/berlin/berlin-bg2q.c | 10 +-
drivers/pinctrl/berlin/berlin.c | 9 +-
drivers/pinctrl/berlin/berlin.h | 1 +
drivers/pinctrl/berlin/berlin4ct.c | 503 +++++++++++++++++++++
11 files changed, 565 insertions(+), 12 deletions(-)
create mode 100644 drivers/pinctrl/berlin/berlin4ct.c
--
2.5.3
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH v2 1/6] pinctrl: berlin: regmap as an extra argument of berlin_pinctrl_probe()
2015-09-21 10:04 [PATCH v2 0/6] arm64: berlin: add pinctrl support Jisheng Zhang
@ 2015-09-21 10:04 ` Jisheng Zhang
2015-10-02 10:37 ` Linus Walleij
2015-10-02 13:04 ` Antoine Tenart
2015-09-21 10:04 ` [PATCH v2 2/6] pinctrl: berlin: add the berlin4ct pinctrl driver Jisheng Zhang
` (4 subsequent siblings)
5 siblings, 2 replies; 15+ messages in thread
From: Jisheng Zhang @ 2015-09-21 10:04 UTC (permalink / raw)
To: sebastian.hesselbarth, linus.walleij, antoine.tenart,
catalin.marinas, will.deacon, mark.rutland, robh+dt, pawel.moll,
ijc+devicetree, galak
Cc: devicetree, linux-arm-kernel, linux-gpio, linux-kernel, Jisheng Zhang
Let berlin_pinctrl_probe() accepts an extra argument: regmap, this is to
prepare for the next berlin4ct support, where we won't use simple-mfd
any more.
Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
---
drivers/pinctrl/berlin/berlin-bg2.c | 10 +++++++++-
drivers/pinctrl/berlin/berlin-bg2cd.c | 10 +++++++++-
drivers/pinctrl/berlin/berlin-bg2q.c | 10 +++++++++-
drivers/pinctrl/berlin/berlin.c | 9 +--------
drivers/pinctrl/berlin/berlin.h | 1 +
5 files changed, 29 insertions(+), 11 deletions(-)
diff --git a/drivers/pinctrl/berlin/berlin-bg2.c b/drivers/pinctrl/berlin/berlin-bg2.c
index 274c553..4aede2e 100644
--- a/drivers/pinctrl/berlin/berlin-bg2.c
+++ b/drivers/pinctrl/berlin/berlin-bg2.c
@@ -10,6 +10,7 @@
* warranty of any kind, whether express or implied.
*/
+#include <linux/mfd/syscon.h>
#include <linux/module.h>
#include <linux/of_device.h>
#include <linux/platform_device.h>
@@ -233,8 +234,15 @@ static int berlin2_pinctrl_probe(struct platform_device *pdev)
{
const struct of_device_id *match =
of_match_device(berlin2_pinctrl_match, &pdev->dev);
+ struct device *dev = &pdev->dev;
+ struct device_node *parent_np = of_get_parent(dev->of_node);
+ struct regmap *regmap = syscon_node_to_regmap(parent_np);
- return berlin_pinctrl_probe(pdev, match->data);
+ of_node_put(parent_np);
+ if (IS_ERR(regmap))
+ return PTR_ERR(regmap);
+
+ return berlin_pinctrl_probe(pdev, regmap, match->data);
}
static struct platform_driver berlin2_pinctrl_driver = {
diff --git a/drivers/pinctrl/berlin/berlin-bg2cd.c b/drivers/pinctrl/berlin/berlin-bg2cd.c
index 0cb793a..6378dd8 100644
--- a/drivers/pinctrl/berlin/berlin-bg2cd.c
+++ b/drivers/pinctrl/berlin/berlin-bg2cd.c
@@ -10,6 +10,7 @@
* warranty of any kind, whether express or implied.
*/
+#include <linux/mfd/syscon.h>
#include <linux/module.h>
#include <linux/of_device.h>
#include <linux/platform_device.h>
@@ -176,8 +177,15 @@ static int berlin2cd_pinctrl_probe(struct platform_device *pdev)
{
const struct of_device_id *match =
of_match_device(berlin2cd_pinctrl_match, &pdev->dev);
+ struct device *dev = &pdev->dev;
+ struct device_node *parent_np = of_get_parent(dev->of_node);
+ struct regmap *regmap = syscon_node_to_regmap(parent_np);
- return berlin_pinctrl_probe(pdev, match->data);
+ of_node_put(parent_np);
+ if (IS_ERR(regmap))
+ return PTR_ERR(regmap);
+
+ return berlin_pinctrl_probe(pdev, regmap, match->data);
}
static struct platform_driver berlin2cd_pinctrl_driver = {
diff --git a/drivers/pinctrl/berlin/berlin-bg2q.c b/drivers/pinctrl/berlin/berlin-bg2q.c
index a466054..907bdf0 100644
--- a/drivers/pinctrl/berlin/berlin-bg2q.c
+++ b/drivers/pinctrl/berlin/berlin-bg2q.c
@@ -10,6 +10,7 @@
* warranty of any kind, whether express or implied.
*/
+#include <linux/mfd/syscon.h>
#include <linux/module.h>
#include <linux/of_device.h>
#include <linux/platform_device.h>
@@ -395,8 +396,15 @@ static int berlin2q_pinctrl_probe(struct platform_device *pdev)
{
const struct of_device_id *match =
of_match_device(berlin2q_pinctrl_match, &pdev->dev);
+ struct device *dev = &pdev->dev;
+ struct device_node *parent_np = of_get_parent(dev->of_node);
+ struct regmap *regmap = syscon_node_to_regmap(parent_np);
- return berlin_pinctrl_probe(pdev, match->data);
+ of_node_put(parent_np);
+ if (IS_ERR(regmap))
+ return PTR_ERR(regmap);
+
+ return berlin_pinctrl_probe(pdev, regmap, match->data);
}
static struct platform_driver berlin2q_pinctrl_driver = {
diff --git a/drivers/pinctrl/berlin/berlin.c b/drivers/pinctrl/berlin/berlin.c
index f495806..0557bbe 100644
--- a/drivers/pinctrl/berlin/berlin.c
+++ b/drivers/pinctrl/berlin/berlin.c
@@ -11,7 +11,6 @@
*/
#include <linux/io.h>
-#include <linux/mfd/syscon.h>
#include <linux/module.h>
#include <linux/of.h>
#include <linux/of_address.h>
@@ -293,19 +292,13 @@ static struct pinctrl_desc berlin_pctrl_desc = {
};
int berlin_pinctrl_probe(struct platform_device *pdev,
+ struct regmap *regmap,
const struct berlin_pinctrl_desc *desc)
{
struct device *dev = &pdev->dev;
- struct device_node *parent_np = of_get_parent(dev->of_node);
struct berlin_pinctrl *pctrl;
- struct regmap *regmap;
int ret;
- regmap = syscon_node_to_regmap(parent_np);
- of_node_put(parent_np);
- if (IS_ERR(regmap))
- return PTR_ERR(regmap);
-
pctrl = devm_kzalloc(dev, sizeof(*pctrl), GFP_KERNEL);
if (!pctrl)
return -ENOMEM;
diff --git a/drivers/pinctrl/berlin/berlin.h b/drivers/pinctrl/berlin/berlin.h
index e1aa841..bf5781a 100644
--- a/drivers/pinctrl/berlin/berlin.h
+++ b/drivers/pinctrl/berlin/berlin.h
@@ -56,6 +56,7 @@ struct berlin_pinctrl_function {
#define BERLIN_PINCTRL_FUNCTION_UNKNOWN {}
int berlin_pinctrl_probe(struct platform_device *pdev,
+ struct regmap *regmap,
const struct berlin_pinctrl_desc *desc);
#endif /* __PINCTRL_BERLIN_H */
--
2.5.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v2 2/6] pinctrl: berlin: add the berlin4ct pinctrl driver
2015-09-21 10:04 [PATCH v2 0/6] arm64: berlin: add pinctrl support Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 1/6] pinctrl: berlin: regmap as an extra argument of berlin_pinctrl_probe() Jisheng Zhang
@ 2015-09-21 10:04 ` Jisheng Zhang
2015-10-02 13:21 ` Antoine Tenart
2015-09-21 10:04 ` [PATCH v2 3/6] arm64: berlin: add the pinctrl dependency for Marvell Berlin SoCs Jisheng Zhang
` (3 subsequent siblings)
5 siblings, 1 reply; 15+ messages in thread
From: Jisheng Zhang @ 2015-09-21 10:04 UTC (permalink / raw)
To: sebastian.hesselbarth, linus.walleij, antoine.tenart,
catalin.marinas, will.deacon, mark.rutland, robh+dt, pawel.moll,
ijc+devicetree, galak
Cc: devicetree, linux-arm-kernel, linux-gpio, linux-kernel, Jisheng Zhang
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
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");
--
2.5.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v2 3/6] arm64: berlin: add the pinctrl dependency for Marvell Berlin SoCs
2015-09-21 10:04 [PATCH v2 0/6] arm64: berlin: add pinctrl support Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 1/6] pinctrl: berlin: regmap as an extra argument of berlin_pinctrl_probe() Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 2/6] pinctrl: berlin: add the berlin4ct pinctrl driver Jisheng Zhang
@ 2015-09-21 10:04 ` Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 4/6] pinctrl: dt-binding: document berlin4ct SoC pinctrl Jisheng Zhang
` (2 subsequent siblings)
5 siblings, 0 replies; 15+ messages in thread
From: Jisheng Zhang @ 2015-09-21 10:04 UTC (permalink / raw)
To: sebastian.hesselbarth, linus.walleij, antoine.tenart,
catalin.marinas, will.deacon, mark.rutland, robh+dt, pawel.moll,
ijc+devicetree, galak
Cc: devicetree, linux-arm-kernel, linux-gpio, linux-kernel, Jisheng Zhang
This is to add the pinctrl dependency for Marvell Berlin SoCs.
Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
---
arch/arm64/Kconfig.platforms | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index c6e2c75..3d17ee2 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -9,6 +9,7 @@ config ARCH_BERLIN
bool "Marvell Berlin SoC Family"
select ARCH_REQUIRE_GPIOLIB
select DW_APB_ICTL
+ select PINCTRL
help
This enables support for Marvell Berlin SoC Family
--
2.5.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v2 4/6] pinctrl: dt-binding: document berlin4ct SoC pinctrl
2015-09-21 10:04 [PATCH v2 0/6] arm64: berlin: add pinctrl support Jisheng Zhang
` (2 preceding siblings ...)
2015-09-21 10:04 ` [PATCH v2 3/6] arm64: berlin: add the pinctrl dependency for Marvell Berlin SoCs Jisheng Zhang
@ 2015-09-21 10:04 ` Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 5/6] arm64: dts: berlin4ct: add the pinctrl node Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 6/6] arm64: dts: berlin4ct: add default pinmux for uart0 Jisheng Zhang
5 siblings, 0 replies; 15+ messages in thread
From: Jisheng Zhang @ 2015-09-21 10:04 UTC (permalink / raw)
To: sebastian.hesselbarth, linus.walleij, antoine.tenart,
catalin.marinas, will.deacon, mark.rutland, robh+dt, pawel.moll,
ijc+devicetree, galak
Cc: devicetree, linux-arm-kernel, linux-gpio, linux-kernel, Jisheng Zhang
Add berlin4ct to existing berlin pinctrl device tree binding.
Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
---
Documentation/devicetree/bindings/pinctrl/berlin,pinctrl.txt | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/pinctrl/berlin,pinctrl.txt b/Documentation/devicetree/bindings/pinctrl/berlin,pinctrl.txt
index a8bb5e2..0580860 100644
--- a/Documentation/devicetree/bindings/pinctrl/berlin,pinctrl.txt
+++ b/Documentation/devicetree/bindings/pinctrl/berlin,pinctrl.txt
@@ -20,7 +20,10 @@ Required properties:
"marvell,berlin2cd-soc-pinctrl",
"marvell,berlin2cd-system-pinctrl",
"marvell,berlin2q-soc-pinctrl",
- "marvell,berlin2q-system-pinctrl"
+ "marvell,berlin2q-system-pinctrl",
+ "marvell,berlin4ct-avio-pinctrl",
+ "marvell,berlin4ct-sm-pinctrl",
+ "marvell,berlin4ct-soc-pinctrl"
Required subnode-properties:
- groups: a list of strings describing the group names.
--
2.5.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v2 5/6] arm64: dts: berlin4ct: add the pinctrl node
2015-09-21 10:04 [PATCH v2 0/6] arm64: berlin: add pinctrl support Jisheng Zhang
` (3 preceding siblings ...)
2015-09-21 10:04 ` [PATCH v2 4/6] pinctrl: dt-binding: document berlin4ct SoC pinctrl Jisheng Zhang
@ 2015-09-21 10:04 ` Jisheng Zhang
2015-10-02 13:24 ` Antoine Tenart
2015-09-21 10:04 ` [PATCH v2 6/6] arm64: dts: berlin4ct: add default pinmux for uart0 Jisheng Zhang
5 siblings, 1 reply; 15+ messages in thread
From: Jisheng Zhang @ 2015-09-21 10:04 UTC (permalink / raw)
To: sebastian.hesselbarth, linus.walleij, antoine.tenart,
catalin.marinas, will.deacon, mark.rutland, robh+dt, pawel.moll,
ijc+devicetree, galak
Cc: devicetree, linux-arm-kernel, linux-gpio, linux-kernel, Jisheng Zhang
Add the avio, soc, sm pinctrl nodes for Marvell berlin4ct SoC.
Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
---
arch/arm64/boot/dts/marvell/berlin4ct.dtsi | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
index a3b5f1d..c374f1d 100644
--- a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
+++ b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
@@ -225,6 +225,16 @@
};
};
+ soc_pinctrl: pinctrl@ea8000 {
+ compatible = "marvell,berlin4ct-soc-pinctrl";
+ reg = <0xea8000 0x14>;
+ };
+
+ avio_pinctrl: pinctrl@ea8400 {
+ compatible = "marvell,berlin4ct-avio-pinctrl";
+ reg = <0xea8400 0x8>;
+ };
+
apb@fc0000 {
compatible = "simple-bus";
#address-cells = <1>;
@@ -280,5 +290,10 @@
status = "disabled";
};
};
+
+ sm_pinctrl: pinctrl@fe2200 {
+ compatible = "marvell,berlin4ct-sm-pinctrl";
+ reg = <0xfe2200 0xc>;
+ };
};
};
--
2.5.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v2 6/6] arm64: dts: berlin4ct: add default pinmux for uart0
2015-09-21 10:04 [PATCH v2 0/6] arm64: berlin: add pinctrl support Jisheng Zhang
` (4 preceding siblings ...)
2015-09-21 10:04 ` [PATCH v2 5/6] arm64: dts: berlin4ct: add the pinctrl node Jisheng Zhang
@ 2015-09-21 10:04 ` Jisheng Zhang
2015-10-02 13:26 ` Antoine Tenart
5 siblings, 1 reply; 15+ messages in thread
From: Jisheng Zhang @ 2015-09-21 10:04 UTC (permalink / raw)
To: sebastian.hesselbarth, linus.walleij, antoine.tenart,
catalin.marinas, will.deacon, mark.rutland, robh+dt, pawel.moll,
ijc+devicetree, galak
Cc: devicetree, linux-arm-kernel, linux-gpio, linux-kernel, Jisheng Zhang
Add urt0 txd and rxd muxing setup in the dtsi because uart0 always uses
them to work, no other possibilities.
Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
---
arch/arm64/boot/dts/marvell/berlin4ct.dtsi | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
index c374f1d..e9409ec 100644
--- a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
+++ b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
@@ -288,12 +288,19 @@
clocks = <&osc>;
reg-shift = <2>;
status = "disabled";
+ pinctrl-0 = <&urt0_pmux>;
+ pinctrl-names = "default";
};
};
sm_pinctrl: pinctrl@fe2200 {
compatible = "marvell,berlin4ct-sm-pinctrl";
reg = <0xfe2200 0xc>;
+
+ urt0_pmux: urt0-pmux {
+ groups = "SM_URT0_TXD", "SM_URT0_RXD";
+ function = "urt0";
+ };
};
};
};
--
2.5.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH v2 1/6] pinctrl: berlin: regmap as an extra argument of berlin_pinctrl_probe()
2015-09-21 10:04 ` [PATCH v2 1/6] pinctrl: berlin: regmap as an extra argument of berlin_pinctrl_probe() Jisheng Zhang
@ 2015-10-02 10:37 ` Linus Walleij
2015-10-02 13:04 ` Antoine Tenart
1 sibling, 0 replies; 15+ messages in thread
From: Linus Walleij @ 2015-10-02 10:37 UTC (permalink / raw)
To: Jisheng Zhang, Antoine Tenart
Cc: Sebastian Hesselbarth, Catalin Marinas, Will Deacon,
Mark Rutland, Rob Herring, Paweł Moll, ijc+devicetree,
Kumar Gala, devicetree, linux-arm-kernel, linux-gpio,
linux-kernel
On Mon, Sep 21, 2015 at 3:04 AM, Jisheng Zhang <jszhang@marvell.com> wrote:
> Let berlin_pinctrl_probe() accepts an extra argument: regmap, this is to
> prepare for the next berlin4ct support, where we won't use simple-mfd
> any more.
>
> Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
Waiting for Antoine's review of this patch before applying.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 1/6] pinctrl: berlin: regmap as an extra argument of berlin_pinctrl_probe()
2015-09-21 10:04 ` [PATCH v2 1/6] pinctrl: berlin: regmap as an extra argument of berlin_pinctrl_probe() Jisheng Zhang
2015-10-02 10:37 ` Linus Walleij
@ 2015-10-02 13:04 ` Antoine Tenart
2015-10-08 10:32 ` Jisheng Zhang
1 sibling, 1 reply; 15+ messages in thread
From: Antoine Tenart @ 2015-10-02 13:04 UTC (permalink / raw)
To: Jisheng Zhang
Cc: sebastian.hesselbarth, linus.walleij, antoine.tenart,
catalin.marinas, will.deacon, mark.rutland, robh+dt, pawel.moll,
ijc+devicetree, galak, devicetree, linux-arm-kernel, linux-gpio,
linux-kernel
Hi Jisheng,
Thanks for providing this!
On Mon, Sep 21, 2015 at 06:04:15PM +0800, Jisheng Zhang wrote:
> Let berlin_pinctrl_probe() accepts an extra argument: regmap, this is to
> prepare for the next berlin4ct support, where we won't use simple-mfd
> any more.
Just a question here: why don't we have instead one
berlin_pinctrl_probe() and one berlin_pinctrl_probe_regmap(), both
calling a static probe function in berlin.c, to avoid duplicating code?
Antoine
> Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
> ---
> drivers/pinctrl/berlin/berlin-bg2.c | 10 +++++++++-
> drivers/pinctrl/berlin/berlin-bg2cd.c | 10 +++++++++-
> drivers/pinctrl/berlin/berlin-bg2q.c | 10 +++++++++-
> drivers/pinctrl/berlin/berlin.c | 9 +--------
> drivers/pinctrl/berlin/berlin.h | 1 +
> 5 files changed, 29 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/pinctrl/berlin/berlin-bg2.c b/drivers/pinctrl/berlin/berlin-bg2.c
> index 274c553..4aede2e 100644
> --- a/drivers/pinctrl/berlin/berlin-bg2.c
> +++ b/drivers/pinctrl/berlin/berlin-bg2.c
> @@ -10,6 +10,7 @@
> * warranty of any kind, whether express or implied.
> */
>
> +#include <linux/mfd/syscon.h>
> #include <linux/module.h>
> #include <linux/of_device.h>
> #include <linux/platform_device.h>
> @@ -233,8 +234,15 @@ static int berlin2_pinctrl_probe(struct platform_device *pdev)
> {
> const struct of_device_id *match =
> of_match_device(berlin2_pinctrl_match, &pdev->dev);
> + struct device *dev = &pdev->dev;
> + struct device_node *parent_np = of_get_parent(dev->of_node);
> + struct regmap *regmap = syscon_node_to_regmap(parent_np);
>
> - return berlin_pinctrl_probe(pdev, match->data);
> + of_node_put(parent_np);
> + if (IS_ERR(regmap))
> + return PTR_ERR(regmap);
> +
> + return berlin_pinctrl_probe(pdev, regmap, match->data);
> }
>
> static struct platform_driver berlin2_pinctrl_driver = {
> diff --git a/drivers/pinctrl/berlin/berlin-bg2cd.c b/drivers/pinctrl/berlin/berlin-bg2cd.c
> index 0cb793a..6378dd8 100644
> --- a/drivers/pinctrl/berlin/berlin-bg2cd.c
> +++ b/drivers/pinctrl/berlin/berlin-bg2cd.c
> @@ -10,6 +10,7 @@
> * warranty of any kind, whether express or implied.
> */
>
> +#include <linux/mfd/syscon.h>
> #include <linux/module.h>
> #include <linux/of_device.h>
> #include <linux/platform_device.h>
> @@ -176,8 +177,15 @@ static int berlin2cd_pinctrl_probe(struct platform_device *pdev)
> {
> const struct of_device_id *match =
> of_match_device(berlin2cd_pinctrl_match, &pdev->dev);
> + struct device *dev = &pdev->dev;
> + struct device_node *parent_np = of_get_parent(dev->of_node);
> + struct regmap *regmap = syscon_node_to_regmap(parent_np);
>
> - return berlin_pinctrl_probe(pdev, match->data);
> + of_node_put(parent_np);
> + if (IS_ERR(regmap))
> + return PTR_ERR(regmap);
> +
> + return berlin_pinctrl_probe(pdev, regmap, match->data);
> }
>
> static struct platform_driver berlin2cd_pinctrl_driver = {
> diff --git a/drivers/pinctrl/berlin/berlin-bg2q.c b/drivers/pinctrl/berlin/berlin-bg2q.c
> index a466054..907bdf0 100644
> --- a/drivers/pinctrl/berlin/berlin-bg2q.c
> +++ b/drivers/pinctrl/berlin/berlin-bg2q.c
> @@ -10,6 +10,7 @@
> * warranty of any kind, whether express or implied.
> */
>
> +#include <linux/mfd/syscon.h>
> #include <linux/module.h>
> #include <linux/of_device.h>
> #include <linux/platform_device.h>
> @@ -395,8 +396,15 @@ static int berlin2q_pinctrl_probe(struct platform_device *pdev)
> {
> const struct of_device_id *match =
> of_match_device(berlin2q_pinctrl_match, &pdev->dev);
> + struct device *dev = &pdev->dev;
> + struct device_node *parent_np = of_get_parent(dev->of_node);
> + struct regmap *regmap = syscon_node_to_regmap(parent_np);
>
> - return berlin_pinctrl_probe(pdev, match->data);
> + of_node_put(parent_np);
> + if (IS_ERR(regmap))
> + return PTR_ERR(regmap);
> +
> + return berlin_pinctrl_probe(pdev, regmap, match->data);
> }
>
> static struct platform_driver berlin2q_pinctrl_driver = {
> diff --git a/drivers/pinctrl/berlin/berlin.c b/drivers/pinctrl/berlin/berlin.c
> index f495806..0557bbe 100644
> --- a/drivers/pinctrl/berlin/berlin.c
> +++ b/drivers/pinctrl/berlin/berlin.c
> @@ -11,7 +11,6 @@
> */
>
> #include <linux/io.h>
> -#include <linux/mfd/syscon.h>
> #include <linux/module.h>
> #include <linux/of.h>
> #include <linux/of_address.h>
> @@ -293,19 +292,13 @@ static struct pinctrl_desc berlin_pctrl_desc = {
> };
>
> int berlin_pinctrl_probe(struct platform_device *pdev,
> + struct regmap *regmap,
> const struct berlin_pinctrl_desc *desc)
> {
> struct device *dev = &pdev->dev;
> - struct device_node *parent_np = of_get_parent(dev->of_node);
> struct berlin_pinctrl *pctrl;
> - struct regmap *regmap;
> int ret;
>
> - regmap = syscon_node_to_regmap(parent_np);
> - of_node_put(parent_np);
> - if (IS_ERR(regmap))
> - return PTR_ERR(regmap);
> -
> pctrl = devm_kzalloc(dev, sizeof(*pctrl), GFP_KERNEL);
> if (!pctrl)
> return -ENOMEM;
> diff --git a/drivers/pinctrl/berlin/berlin.h b/drivers/pinctrl/berlin/berlin.h
> index e1aa841..bf5781a 100644
> --- a/drivers/pinctrl/berlin/berlin.h
> +++ b/drivers/pinctrl/berlin/berlin.h
> @@ -56,6 +56,7 @@ struct berlin_pinctrl_function {
> #define BERLIN_PINCTRL_FUNCTION_UNKNOWN {}
>
> int berlin_pinctrl_probe(struct platform_device *pdev,
> + struct regmap *regmap,
> const struct berlin_pinctrl_desc *desc);
>
> #endif /* __PINCTRL_BERLIN_H */
> --
> 2.5.3
>
--
Antoine Ténart, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 2/6] pinctrl: berlin: add the berlin4ct pinctrl driver
2015-09-21 10:04 ` [PATCH v2 2/6] pinctrl: berlin: add the berlin4ct pinctrl driver Jisheng Zhang
@ 2015-10-02 13:21 ` Antoine Tenart
2015-10-08 10:46 ` Jisheng Zhang
0 siblings, 1 reply; 15+ messages in thread
From: Antoine Tenart @ 2015-10-02 13:21 UTC (permalink / raw)
To: Jisheng Zhang
Cc: sebastian.hesselbarth, linus.walleij, antoine.tenart,
catalin.marinas, will.deacon, mark.rutland, robh+dt, pawel.moll,
ijc+devicetree, galak, devicetree, linux-arm-kernel, linux-gpio,
linux-kernel
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
--
Antoine Ténart, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 5/6] arm64: dts: berlin4ct: add the pinctrl node
2015-09-21 10:04 ` [PATCH v2 5/6] arm64: dts: berlin4ct: add the pinctrl node Jisheng Zhang
@ 2015-10-02 13:24 ` Antoine Tenart
0 siblings, 0 replies; 15+ messages in thread
From: Antoine Tenart @ 2015-10-02 13:24 UTC (permalink / raw)
To: Jisheng Zhang
Cc: sebastian.hesselbarth, linus.walleij, antoine.tenart,
catalin.marinas, will.deacon, mark.rutland, robh+dt, pawel.moll,
ijc+devicetree, galak, devicetree, linux-arm-kernel, linux-gpio,
linux-kernel
Jisheng,
On Mon, Sep 21, 2015 at 06:04:19PM +0800, Jisheng Zhang wrote:
> Add the avio, soc, sm pinctrl nodes for Marvell berlin4ct SoC.
>
> Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
> ---
> arch/arm64/boot/dts/marvell/berlin4ct.dtsi | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
> index a3b5f1d..c374f1d 100644
> --- a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
> +++ b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
> @@ -225,6 +225,16 @@
> };
> };
>
> + soc_pinctrl: pinctrl@ea8000 {
We tend to use "pin-controller@" instead of "pinctrl@" in Berlin device
trees.
> + compatible = "marvell,berlin4ct-soc-pinctrl";
> + reg = <0xea8000 0x14>;
> + };
> +
> + avio_pinctrl: pinctrl@ea8400 {
Ditto.
> + compatible = "marvell,berlin4ct-avio-pinctrl";
> + reg = <0xea8400 0x8>;
> + };
> +
> apb@fc0000 {
> compatible = "simple-bus";
> #address-cells = <1>;
> @@ -280,5 +290,10 @@
> status = "disabled";
> };
> };
> +
> + sm_pinctrl: pinctrl@fe2200 {
Ditto.
> + compatible = "marvell,berlin4ct-sm-pinctrl";
> + reg = <0xfe2200 0xc>;
> + };
> };
> };
Antoine
--
Antoine Ténart, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 6/6] arm64: dts: berlin4ct: add default pinmux for uart0
2015-09-21 10:04 ` [PATCH v2 6/6] arm64: dts: berlin4ct: add default pinmux for uart0 Jisheng Zhang
@ 2015-10-02 13:26 ` Antoine Tenart
0 siblings, 0 replies; 15+ messages in thread
From: Antoine Tenart @ 2015-10-02 13:26 UTC (permalink / raw)
To: Jisheng Zhang
Cc: sebastian.hesselbarth, linus.walleij, antoine.tenart,
catalin.marinas, will.deacon, mark.rutland, robh+dt, pawel.moll,
ijc+devicetree, galak, devicetree, linux-arm-kernel, linux-gpio,
linux-kernel
Jisheng,
Please s/urt/uart/ in this patch.
Thanks!
Antoine
On Mon, Sep 21, 2015 at 06:04:20PM +0800, Jisheng Zhang wrote:
> Add urt0 txd and rxd muxing setup in the dtsi because uart0 always uses
> them to work, no other possibilities.
>
> Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
> ---
> arch/arm64/boot/dts/marvell/berlin4ct.dtsi | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
> index c374f1d..e9409ec 100644
> --- a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
> +++ b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi
> @@ -288,12 +288,19 @@
> clocks = <&osc>;
> reg-shift = <2>;
> status = "disabled";
> + pinctrl-0 = <&urt0_pmux>;
> + pinctrl-names = "default";
> };
> };
>
> sm_pinctrl: pinctrl@fe2200 {
> compatible = "marvell,berlin4ct-sm-pinctrl";
> reg = <0xfe2200 0xc>;
> +
> + urt0_pmux: urt0-pmux {
> + groups = "SM_URT0_TXD", "SM_URT0_RXD";
> + function = "urt0";
> + };
> };
> };
> };
> --
> 2.5.3
>
--
Antoine Ténart, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 1/6] pinctrl: berlin: regmap as an extra argument of berlin_pinctrl_probe()
2015-10-02 13:04 ` Antoine Tenart
@ 2015-10-08 10:32 ` Jisheng Zhang
0 siblings, 0 replies; 15+ messages in thread
From: Jisheng Zhang @ 2015-10-08 10:32 UTC (permalink / raw)
To: Antoine Tenart
Cc: sebastian.hesselbarth, linus.walleij, catalin.marinas,
will.deacon, mark.rutland, robh+dt, pawel.moll, ijc+devicetree,
galak, devicetree, linux-arm-kernel, linux-gpio, linux-kernel
Hi Antoine,
On Fri, 2 Oct 2015 15:04:56 +0200
Antoine Tenart <antoine.tenart@free-electrons.com> wrote:
> Hi Jisheng,
>
> Thanks for providing this!
Just back from vocation. Thanks for the review feedbacks
>
> On Mon, Sep 21, 2015 at 06:04:15PM +0800, Jisheng Zhang wrote:
> > Let berlin_pinctrl_probe() accepts an extra argument: regmap, this is to
> > prepare for the next berlin4ct support, where we won't use simple-mfd
> > any more.
>
> Just a question here: why don't we have instead one
> berlin_pinctrl_probe() and one berlin_pinctrl_probe_regmap(), both
> calling a static probe function in berlin.c, to avoid duplicating code?
It's better! Will take this solution in v3
>
> Antoine
>
> > Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
> > ---
> > drivers/pinctrl/berlin/berlin-bg2.c | 10 +++++++++-
> > drivers/pinctrl/berlin/berlin-bg2cd.c | 10 +++++++++-
> > drivers/pinctrl/berlin/berlin-bg2q.c | 10 +++++++++-
> > drivers/pinctrl/berlin/berlin.c | 9 +--------
> > drivers/pinctrl/berlin/berlin.h | 1 +
> > 5 files changed, 29 insertions(+), 11 deletions(-)
> >
> > diff --git a/drivers/pinctrl/berlin/berlin-bg2.c b/drivers/pinctrl/berlin/berlin-bg2.c
> > index 274c553..4aede2e 100644
> > --- a/drivers/pinctrl/berlin/berlin-bg2.c
> > +++ b/drivers/pinctrl/berlin/berlin-bg2.c
> > @@ -10,6 +10,7 @@
> > * warranty of any kind, whether express or implied.
> > */
> >
> > +#include <linux/mfd/syscon.h>
> > #include <linux/module.h>
> > #include <linux/of_device.h>
> > #include <linux/platform_device.h>
> > @@ -233,8 +234,15 @@ static int berlin2_pinctrl_probe(struct platform_device *pdev)
> > {
> > const struct of_device_id *match =
> > of_match_device(berlin2_pinctrl_match, &pdev->dev);
> > + struct device *dev = &pdev->dev;
> > + struct device_node *parent_np = of_get_parent(dev->of_node);
> > + struct regmap *regmap = syscon_node_to_regmap(parent_np);
> >
> > - return berlin_pinctrl_probe(pdev, match->data);
> > + of_node_put(parent_np);
> > + if (IS_ERR(regmap))
> > + return PTR_ERR(regmap);
> > +
> > + return berlin_pinctrl_probe(pdev, regmap, match->data);
> > }
> >
> > static struct platform_driver berlin2_pinctrl_driver = {
> > diff --git a/drivers/pinctrl/berlin/berlin-bg2cd.c b/drivers/pinctrl/berlin/berlin-bg2cd.c
> > index 0cb793a..6378dd8 100644
> > --- a/drivers/pinctrl/berlin/berlin-bg2cd.c
> > +++ b/drivers/pinctrl/berlin/berlin-bg2cd.c
> > @@ -10,6 +10,7 @@
> > * warranty of any kind, whether express or implied.
> > */
> >
> > +#include <linux/mfd/syscon.h>
> > #include <linux/module.h>
> > #include <linux/of_device.h>
> > #include <linux/platform_device.h>
> > @@ -176,8 +177,15 @@ static int berlin2cd_pinctrl_probe(struct platform_device *pdev)
> > {
> > const struct of_device_id *match =
> > of_match_device(berlin2cd_pinctrl_match, &pdev->dev);
> > + struct device *dev = &pdev->dev;
> > + struct device_node *parent_np = of_get_parent(dev->of_node);
> > + struct regmap *regmap = syscon_node_to_regmap(parent_np);
> >
> > - return berlin_pinctrl_probe(pdev, match->data);
> > + of_node_put(parent_np);
> > + if (IS_ERR(regmap))
> > + return PTR_ERR(regmap);
> > +
> > + return berlin_pinctrl_probe(pdev, regmap, match->data);
> > }
> >
> > static struct platform_driver berlin2cd_pinctrl_driver = {
> > diff --git a/drivers/pinctrl/berlin/berlin-bg2q.c b/drivers/pinctrl/berlin/berlin-bg2q.c
> > index a466054..907bdf0 100644
> > --- a/drivers/pinctrl/berlin/berlin-bg2q.c
> > +++ b/drivers/pinctrl/berlin/berlin-bg2q.c
> > @@ -10,6 +10,7 @@
> > * warranty of any kind, whether express or implied.
> > */
> >
> > +#include <linux/mfd/syscon.h>
> > #include <linux/module.h>
> > #include <linux/of_device.h>
> > #include <linux/platform_device.h>
> > @@ -395,8 +396,15 @@ static int berlin2q_pinctrl_probe(struct platform_device *pdev)
> > {
> > const struct of_device_id *match =
> > of_match_device(berlin2q_pinctrl_match, &pdev->dev);
> > + struct device *dev = &pdev->dev;
> > + struct device_node *parent_np = of_get_parent(dev->of_node);
> > + struct regmap *regmap = syscon_node_to_regmap(parent_np);
> >
> > - return berlin_pinctrl_probe(pdev, match->data);
> > + of_node_put(parent_np);
> > + if (IS_ERR(regmap))
> > + return PTR_ERR(regmap);
> > +
> > + return berlin_pinctrl_probe(pdev, regmap, match->data);
> > }
> >
> > static struct platform_driver berlin2q_pinctrl_driver = {
> > diff --git a/drivers/pinctrl/berlin/berlin.c b/drivers/pinctrl/berlin/berlin.c
> > index f495806..0557bbe 100644
> > --- a/drivers/pinctrl/berlin/berlin.c
> > +++ b/drivers/pinctrl/berlin/berlin.c
> > @@ -11,7 +11,6 @@
> > */
> >
> > #include <linux/io.h>
> > -#include <linux/mfd/syscon.h>
> > #include <linux/module.h>
> > #include <linux/of.h>
> > #include <linux/of_address.h>
> > @@ -293,19 +292,13 @@ static struct pinctrl_desc berlin_pctrl_desc = {
> > };
> >
> > int berlin_pinctrl_probe(struct platform_device *pdev,
> > + struct regmap *regmap,
> > const struct berlin_pinctrl_desc *desc)
> > {
> > struct device *dev = &pdev->dev;
> > - struct device_node *parent_np = of_get_parent(dev->of_node);
> > struct berlin_pinctrl *pctrl;
> > - struct regmap *regmap;
> > int ret;
> >
> > - regmap = syscon_node_to_regmap(parent_np);
> > - of_node_put(parent_np);
> > - if (IS_ERR(regmap))
> > - return PTR_ERR(regmap);
> > -
> > pctrl = devm_kzalloc(dev, sizeof(*pctrl), GFP_KERNEL);
> > if (!pctrl)
> > return -ENOMEM;
> > diff --git a/drivers/pinctrl/berlin/berlin.h b/drivers/pinctrl/berlin/berlin.h
> > index e1aa841..bf5781a 100644
> > --- a/drivers/pinctrl/berlin/berlin.h
> > +++ b/drivers/pinctrl/berlin/berlin.h
> > @@ -56,6 +56,7 @@ struct berlin_pinctrl_function {
> > #define BERLIN_PINCTRL_FUNCTION_UNKNOWN {}
> >
> > int berlin_pinctrl_probe(struct platform_device *pdev,
> > + struct regmap *regmap,
> > const struct berlin_pinctrl_desc *desc);
> >
> > #endif /* __PINCTRL_BERLIN_H */
> > --
> > 2.5.3
> >
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 2/6] pinctrl: berlin: add the berlin4ct pinctrl driver
2015-10-02 13:21 ` Antoine Tenart
@ 2015-10-08 10:46 ` Jisheng Zhang
2015-10-09 7:05 ` Jisheng Zhang
0 siblings, 1 reply; 15+ messages in thread
From: Jisheng Zhang @ 2015-10-08 10:46 UTC (permalink / raw)
To: Antoine Tenart
Cc: sebastian.hesselbarth, linus.walleij, catalin.marinas,
will.deacon, mark.rutland, robh+dt, pawel.moll, ijc+devicetree,
galak, devicetree, linux-arm-kernel, linux-gpio, linux-kernel
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
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 2/6] pinctrl: berlin: add the berlin4ct pinctrl driver
2015-10-08 10:46 ` Jisheng Zhang
@ 2015-10-09 7:05 ` Jisheng Zhang
0 siblings, 0 replies; 15+ messages in thread
From: Jisheng Zhang @ 2015-10-09 7:05 UTC (permalink / raw)
To: Antoine Tenart
Cc: sebastian.hesselbarth, linus.walleij, catalin.marinas,
will.deacon, mark.rutland, robh+dt, pawel.moll, ijc+devicetree,
galak, devicetree, linux-arm-kernel, linux-gpio, linux-kernel
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
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2015-10-09 7:10 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-21 10:04 [PATCH v2 0/6] arm64: berlin: add pinctrl support Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 1/6] pinctrl: berlin: regmap as an extra argument of berlin_pinctrl_probe() Jisheng Zhang
2015-10-02 10:37 ` Linus Walleij
2015-10-02 13:04 ` Antoine Tenart
2015-10-08 10:32 ` Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 2/6] pinctrl: berlin: add the berlin4ct pinctrl driver Jisheng Zhang
2015-10-02 13:21 ` Antoine Tenart
2015-10-08 10:46 ` Jisheng Zhang
2015-10-09 7:05 ` Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 3/6] arm64: berlin: add the pinctrl dependency for Marvell Berlin SoCs Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 4/6] pinctrl: dt-binding: document berlin4ct SoC pinctrl Jisheng Zhang
2015-09-21 10:04 ` [PATCH v2 5/6] arm64: dts: berlin4ct: add the pinctrl node Jisheng Zhang
2015-10-02 13:24 ` Antoine Tenart
2015-09-21 10:04 ` [PATCH v2 6/6] arm64: dts: berlin4ct: add default pinmux for uart0 Jisheng Zhang
2015-10-02 13:26 ` Antoine Tenart
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).