* [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs
@ 2022-07-27 8:22 Sumit Garg
2022-07-27 8:22 ` [PATCH v2 1/2] pinctrl: sdm845: Remove redundant CONFIG_SDM845 check Sumit Garg
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Sumit Garg @ 2022-07-27 8:22 UTC (permalink / raw)
To: u-boot
Cc: robert.marko, luka.kovacic, luka.perkov, rfried.dev, dsankouski,
stephan, sjg, trini, nicolas.dechesne, vinod.koul, mworsfold,
daniel.thompson, pbrobinson, Sumit Garg
This is an initial step towards achieving complete Linux DT sync on Qcom
SoCs/boards. It syncs up DT compatibles for pinctrl and GPIO drivers.
Changes in v2:
- Separate patch for CONFIG_SDM845 check removal.
- Fix pinctrl DT compatibles for db410c and db820c.
Sumit Garg (2):
pinctrl: sdm845: Remove redundant CONFIG_SDM845 check
arm: dts: qcom: Sync pinctrl DT nodes with Linux bindings
arch/arm/dts/dragonboard410c-uboot.dtsi | 2 +-
arch/arm/dts/dragonboard410c.dts | 17 +++-----
arch/arm/dts/dragonboard820c-uboot.dtsi | 2 +-
arch/arm/dts/dragonboard820c.dts | 4 +-
arch/arm/dts/qcom-ipq4019.dtsi | 18 +++------
arch/arm/dts/qcs404-evb.dts | 2 +-
arch/arm/dts/sdm845.dtsi | 2 +-
arch/arm/mach-ipq40xx/pinctrl-snapdragon.c | 31 ++++++++++++++-
arch/arm/mach-snapdragon/Makefile | 2 +-
arch/arm/mach-snapdragon/pinctrl-snapdragon.c | 39 ++++++++++++++++---
drivers/gpio/msm_gpio.c | 10 +----
11 files changed, 83 insertions(+), 46 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2 1/2] pinctrl: sdm845: Remove redundant CONFIG_SDM845 check
2022-07-27 8:22 [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs Sumit Garg
@ 2022-07-27 8:22 ` Sumit Garg
2022-08-07 0:16 ` Ramon Fried
2022-08-26 18:49 ` Tom Rini
2022-07-27 8:22 ` [PATCH v2 2/2] arm: dts: qcom: Sync pinctrl DT nodes with Linux bindings Sumit Garg
2022-08-04 14:39 ` [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs Sumit Garg
2 siblings, 2 replies; 9+ messages in thread
From: Sumit Garg @ 2022-07-27 8:22 UTC (permalink / raw)
To: u-boot
Cc: robert.marko, luka.kovacic, luka.perkov, rfried.dev, dsankouski,
stephan, sjg, trini, nicolas.dechesne, vinod.koul, mworsfold,
daniel.thompson, pbrobinson, Sumit Garg
DT compatible is sufficient to make platform specific differentiation,
so remove redundant CONFIG_SDM845 check.
Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
---
arch/arm/mach-snapdragon/Makefile | 2 +-
arch/arm/mach-snapdragon/pinctrl-snapdragon.c | 2 --
2 files changed, 1 insertion(+), 3 deletions(-)
diff --git a/arch/arm/mach-snapdragon/Makefile b/arch/arm/mach-snapdragon/Makefile
index 0d31f10f68..cbaaf23f6b 100644
--- a/arch/arm/mach-snapdragon/Makefile
+++ b/arch/arm/mach-snapdragon/Makefile
@@ -16,6 +16,6 @@ obj-y += pinctrl-snapdragon.o
obj-y += pinctrl-apq8016.o
obj-y += pinctrl-apq8096.o
obj-y += pinctrl-qcs404.o
-obj-$(CONFIG_SDM845) += pinctrl-sdm845.o
+obj-y += pinctrl-sdm845.o
obj-$(CONFIG_TARGET_QCS404EVB) += clock-qcs404.o
obj-$(CONFIG_TARGET_QCS404EVB) += sysmap-qcs404.o
diff --git a/arch/arm/mach-snapdragon/pinctrl-snapdragon.c b/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
index c2148a5d0a..842e2da0c5 100644
--- a/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
+++ b/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
@@ -116,9 +116,7 @@ static struct pinctrl_ops msm_pinctrl_ops = {
static const struct udevice_id msm_pinctrl_ids[] = {
{ .compatible = "qcom,tlmm-apq8016", .data = (ulong)&apq8016_data },
{ .compatible = "qcom,tlmm-apq8096", .data = (ulong)&apq8096_data },
-#ifdef CONFIG_SDM845
{ .compatible = "qcom,tlmm-sdm845", .data = (ulong)&sdm845_data },
-#endif
{ .compatible = "qcom,tlmm-qcs404", .data = (ulong)&qcs404_data },
{ }
};
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v2 2/2] arm: dts: qcom: Sync pinctrl DT nodes with Linux bindings
2022-07-27 8:22 [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs Sumit Garg
2022-07-27 8:22 ` [PATCH v2 1/2] pinctrl: sdm845: Remove redundant CONFIG_SDM845 check Sumit Garg
@ 2022-07-27 8:22 ` Sumit Garg
2022-08-26 18:49 ` Tom Rini
2022-08-04 14:39 ` [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs Sumit Garg
2 siblings, 1 reply; 9+ messages in thread
From: Sumit Garg @ 2022-07-27 8:22 UTC (permalink / raw)
To: u-boot
Cc: robert.marko, luka.kovacic, luka.perkov, rfried.dev, dsankouski,
stephan, sjg, trini, nicolas.dechesne, vinod.koul, mworsfold,
daniel.thompson, pbrobinson, Sumit Garg
Currently for all Qcom SoCs/boards there are separate compatibles for
GPIO and pinctrl. But this is inconsistent with official (upstream) Linux
bindings which requires only a single compatible "qcom,<SoC name>-pinctrl"
and there is no such compatible property as "qcom,tlmm-<SoC name>".
So fix this inconsistency for Qcom SoCs in order to comply with upstream
DT bindings. This is done via removing compatibles from "msm_gpio" driver
and via binding to "msm_gpio" driver from pinctrl driver in case
"gpio-controller" property is specified for pinctrl node.
Suggested-by: Stephan Gerhold <stephan@gerhold.net>
Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
---
arch/arm/dts/dragonboard410c-uboot.dtsi | 2 +-
arch/arm/dts/dragonboard410c.dts | 17 +++------
arch/arm/dts/dragonboard820c-uboot.dtsi | 2 +-
arch/arm/dts/dragonboard820c.dts | 4 +-
arch/arm/dts/qcom-ipq4019.dtsi | 18 +++------
arch/arm/dts/qcs404-evb.dts | 2 +-
arch/arm/dts/sdm845.dtsi | 2 +-
arch/arm/mach-ipq40xx/pinctrl-snapdragon.c | 31 +++++++++++++++-
arch/arm/mach-snapdragon/pinctrl-snapdragon.c | 37 +++++++++++++++++--
drivers/gpio/msm_gpio.c | 10 +----
10 files changed, 82 insertions(+), 43 deletions(-)
diff --git a/arch/arm/dts/dragonboard410c-uboot.dtsi b/arch/arm/dts/dragonboard410c-uboot.dtsi
index 9c1be2566f..e4fecaa19e 100644
--- a/arch/arm/dts/dragonboard410c-uboot.dtsi
+++ b/arch/arm/dts/dragonboard410c-uboot.dtsi
@@ -14,7 +14,7 @@
soc {
u-boot,dm-pre-reloc;
- qcom,tlmm@1000000 {
+ pinctrl@1000000 {
u-boot,dm-pre-reloc;
uart {
diff --git a/arch/arm/dts/dragonboard410c.dts b/arch/arm/dts/dragonboard410c.dts
index 50523712cb..59cf45eb17 100644
--- a/arch/arm/dts/dragonboard410c.dts
+++ b/arch/arm/dts/dragonboard410c.dts
@@ -60,9 +60,13 @@
reg = <0x60000 0x8000>;
};
- pinctrl: qcom,tlmm@1000000 {
- compatible = "qcom,tlmm-apq8016";
+ soc_gpios: pinctrl@1000000 {
+ compatible = "qcom,msm8916-pinctrl";
reg = <0x1000000 0x400000>;
+ gpio-controller;
+ gpio-count = <122>;
+ gpio-bank-name="soc";
+ #gpio-cells = <2>;
blsp1_uart: uart {
function = "blsp1_uart";
@@ -86,15 +90,6 @@
pinctrl-0 = <&blsp1_uart>;
};
- soc_gpios: pinctrl@1000000 {
- compatible = "qcom,apq8016-pinctrl";
- reg = <0x1000000 0x300000>;
- gpio-controller;
- gpio-count = <122>;
- gpio-bank-name="soc";
- #gpio-cells = <2>;
- };
-
ehci@78d9000 {
compatible = "qcom,ehci-host";
reg = <0x78d9000 0x400>;
diff --git a/arch/arm/dts/dragonboard820c-uboot.dtsi b/arch/arm/dts/dragonboard820c-uboot.dtsi
index 8610d7ec37..2270ac73bf 100644
--- a/arch/arm/dts/dragonboard820c-uboot.dtsi
+++ b/arch/arm/dts/dragonboard820c-uboot.dtsi
@@ -13,7 +13,7 @@
soc {
u-boot,dm-pre-reloc;
- qcom,tlmm@1010000 {
+ pinctrl@1010000 {
u-boot,dm-pre-reloc;
uart {
diff --git a/arch/arm/dts/dragonboard820c.dts b/arch/arm/dts/dragonboard820c.dts
index b72a2471cf..aaca681d2e 100644
--- a/arch/arm/dts/dragonboard820c.dts
+++ b/arch/arm/dts/dragonboard820c.dts
@@ -64,8 +64,8 @@
reg = <0x300000 0x90000>;
};
- pinctrl: qcom,tlmm@1010000 {
- compatible = "qcom,tlmm-apq8096";
+ pinctrl: pinctrl@1010000 {
+ compatible = "qcom,msm8996-pinctrl";
reg = <0x1010000 0x400000>;
blsp8_uart: uart {
diff --git a/arch/arm/dts/qcom-ipq4019.dtsi b/arch/arm/dts/qcom-ipq4019.dtsi
index 7a52ea2c4e..181732d262 100644
--- a/arch/arm/dts/qcom-ipq4019.dtsi
+++ b/arch/arm/dts/qcom-ipq4019.dtsi
@@ -75,9 +75,13 @@
u-boot,dm-pre-reloc;
};
- pinctrl: qcom,tlmm@1000000 {
- compatible = "qcom,tlmm-ipq4019";
+ soc_gpios: pinctrl@1000000 {
+ compatible = "qcom,ipq4019-pinctrl";
reg = <0x1000000 0x300000>;
+ gpio-controller;
+ gpio-count = <100>;
+ gpio-bank-name="soc";
+ #gpio-cells = <2>;
u-boot,dm-pre-reloc;
};
@@ -90,16 +94,6 @@
u-boot,dm-pre-reloc;
};
- soc_gpios: pinctrl@1000000 {
- compatible = "qcom,ipq4019-pinctrl";
- reg = <0x1000000 0x300000>;
- gpio-controller;
- gpio-count = <100>;
- gpio-bank-name="soc";
- #gpio-cells = <2>;
- u-boot,dm-pre-reloc;
- };
-
blsp1_spi1: spi@78b5000 {
compatible = "qcom,spi-qup-v2.2.1";
reg = <0x78b5000 0x600>;
diff --git a/arch/arm/dts/qcs404-evb.dts b/arch/arm/dts/qcs404-evb.dts
index 4f0ae20bdb..09687e1fd3 100644
--- a/arch/arm/dts/qcs404-evb.dts
+++ b/arch/arm/dts/qcs404-evb.dts
@@ -38,7 +38,7 @@
compatible = "simple-bus";
pinctrl_north@1300000 {
- compatible = "qcom,tlmm-qcs404";
+ compatible = "qcom,qcs404-pinctrl";
reg = <0x1300000 0x200000>;
blsp1_uart2: uart {
diff --git a/arch/arm/dts/sdm845.dtsi b/arch/arm/dts/sdm845.dtsi
index df5b6dfcfc..607af277f8 100644
--- a/arch/arm/dts/sdm845.dtsi
+++ b/arch/arm/dts/sdm845.dtsi
@@ -37,7 +37,7 @@
};
tlmm_north: pinctrl_north@3900000 {
- compatible = "qcom,tlmm-sdm845";
+ compatible = "qcom,sdm845-pinctrl";
reg = <0x3900000 0x400000>;
gpio-count = <150>;
gpio-controller;
diff --git a/arch/arm/mach-ipq40xx/pinctrl-snapdragon.c b/arch/arm/mach-ipq40xx/pinctrl-snapdragon.c
index c51a75ee94..036fec93d7 100644
--- a/arch/arm/mach-ipq40xx/pinctrl-snapdragon.c
+++ b/arch/arm/mach-ipq40xx/pinctrl-snapdragon.c
@@ -14,6 +14,8 @@
#include <dm.h>
#include <errno.h>
#include <asm/io.h>
+#include <dm/device_compat.h>
+#include <dm/lists.h>
#include <dm/pinctrl.h>
#include <linux/bitops.h>
#include "pinctrl-snapdragon.h"
@@ -110,6 +112,32 @@ static int msm_pinconf_set(struct udevice *dev, unsigned int pin_selector,
return 0;
}
+static int msm_pinctrl_bind(struct udevice *dev)
+{
+ ofnode node = dev_ofnode(dev);
+ const char *name;
+ int ret;
+
+ ofnode_get_property(node, "gpio-controller", &ret);
+ if (ret < 0)
+ return 0;
+
+ /* Get the name of gpio node */
+ name = ofnode_get_name(node);
+ if (!name)
+ return -EINVAL;
+
+ /* Bind gpio node */
+ ret = device_bind_driver_to_node(dev, "gpio_msm",
+ name, node, NULL);
+ if (ret)
+ return ret;
+
+ dev_dbg(dev, "bind %s\n", name);
+
+ return 0;
+}
+
static struct pinctrl_ops msm_pinctrl_ops = {
.get_pins_count = msm_get_pins_count,
.get_pin_name = msm_get_pin_name,
@@ -123,7 +151,7 @@ static struct pinctrl_ops msm_pinctrl_ops = {
};
static const struct udevice_id msm_pinctrl_ids[] = {
- { .compatible = "qcom,tlmm-ipq4019", .data = (ulong)&ipq4019_data },
+ { .compatible = "qcom,ipq4019-pinctrl", .data = (ulong)&ipq4019_data },
{ }
};
@@ -134,4 +162,5 @@ U_BOOT_DRIVER(pinctrl_snapdraon) = {
.priv_auto = sizeof(struct msm_pinctrl_priv),
.ops = &msm_pinctrl_ops,
.probe = msm_pinctrl_probe,
+ .bind = msm_pinctrl_bind,
};
diff --git a/arch/arm/mach-snapdragon/pinctrl-snapdragon.c b/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
index 842e2da0c5..ab884ab6bf 100644
--- a/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
+++ b/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
@@ -10,6 +10,8 @@
#include <dm.h>
#include <errno.h>
#include <asm/io.h>
+#include <dm/device_compat.h>
+#include <dm/lists.h>
#include <dm/pinctrl.h>
#include <linux/bitops.h>
#include "pinctrl-snapdragon.h"
@@ -113,11 +115,37 @@ static struct pinctrl_ops msm_pinctrl_ops = {
.get_function_name = msm_get_function_name,
};
+static int msm_pinctrl_bind(struct udevice *dev)
+{
+ ofnode node = dev_ofnode(dev);
+ const char *name;
+ int ret;
+
+ ofnode_get_property(node, "gpio-controller", &ret);
+ if (ret < 0)
+ return 0;
+
+ /* Get the name of gpio node */
+ name = ofnode_get_name(node);
+ if (!name)
+ return -EINVAL;
+
+ /* Bind gpio node */
+ ret = device_bind_driver_to_node(dev, "gpio_msm",
+ name, node, NULL);
+ if (ret)
+ return ret;
+
+ dev_dbg(dev, "bind %s\n", name);
+
+ return 0;
+}
+
static const struct udevice_id msm_pinctrl_ids[] = {
- { .compatible = "qcom,tlmm-apq8016", .data = (ulong)&apq8016_data },
- { .compatible = "qcom,tlmm-apq8096", .data = (ulong)&apq8096_data },
- { .compatible = "qcom,tlmm-sdm845", .data = (ulong)&sdm845_data },
- { .compatible = "qcom,tlmm-qcs404", .data = (ulong)&qcs404_data },
+ { .compatible = "qcom,msm8916-pinctrl", .data = (ulong)&apq8016_data },
+ { .compatible = "qcom,msm8996-pinctrl", .data = (ulong)&apq8096_data },
+ { .compatible = "qcom,sdm845-pinctrl", .data = (ulong)&sdm845_data },
+ { .compatible = "qcom,qcs404-pinctrl", .data = (ulong)&qcs404_data },
{ }
};
@@ -128,4 +156,5 @@ U_BOOT_DRIVER(pinctrl_snapdraon) = {
.priv_auto = sizeof(struct msm_pinctrl_priv),
.ops = &msm_pinctrl_ops,
.probe = msm_pinctrl_probe,
+ .bind = msm_pinctrl_bind,
};
diff --git a/drivers/gpio/msm_gpio.c b/drivers/gpio/msm_gpio.c
index a3c3cd7824..51670f2637 100644
--- a/drivers/gpio/msm_gpio.c
+++ b/drivers/gpio/msm_gpio.c
@@ -116,20 +116,12 @@ static int msm_gpio_of_to_plat(struct udevice *dev)
return 0;
}
-static const struct udevice_id msm_gpio_ids[] = {
- { .compatible = "qcom,msm8916-pinctrl" },
- { .compatible = "qcom,apq8016-pinctrl" },
- { .compatible = "qcom,ipq4019-pinctrl" },
- { .compatible = "qcom,sdm845-pinctrl" },
- { }
-};
-
U_BOOT_DRIVER(gpio_msm) = {
.name = "gpio_msm",
.id = UCLASS_GPIO,
- .of_match = msm_gpio_ids,
.of_to_plat = msm_gpio_of_to_plat,
.probe = msm_gpio_probe,
.ops = &gpio_msm_ops,
+ .flags = DM_UC_FLAG_SEQ_ALIAS,
.priv_auto = sizeof(struct msm_gpio_bank),
};
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs
2022-07-27 8:22 [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs Sumit Garg
2022-07-27 8:22 ` [PATCH v2 1/2] pinctrl: sdm845: Remove redundant CONFIG_SDM845 check Sumit Garg
2022-07-27 8:22 ` [PATCH v2 2/2] arm: dts: qcom: Sync pinctrl DT nodes with Linux bindings Sumit Garg
@ 2022-08-04 14:39 ` Sumit Garg
2022-08-16 8:02 ` Sumit Garg
2 siblings, 1 reply; 9+ messages in thread
From: Sumit Garg @ 2022-08-04 14:39 UTC (permalink / raw)
To: u-boot
Cc: robert.marko, luka.kovacic, luka.perkov, rfried.dev, dsankouski,
stephan, sjg, trini, nicolas.dechesne, vinod.koul, mworsfold,
daniel.thompson, pbrobinson
Hi,
On Wed, 27 Jul 2022 at 13:52, Sumit Garg <sumit.garg@linaro.org> wrote:
>
> This is an initial step towards achieving complete Linux DT sync on Qcom
> SoCs/boards. It syncs up DT compatibles for pinctrl and GPIO drivers.
>
> Changes in v2:
> - Separate patch for CONFIG_SDM845 check removal.
> - Fix pinctrl DT compatibles for db410c and db820c.
>
Do we have any further comments on this?
-Sumit
> Sumit Garg (2):
> pinctrl: sdm845: Remove redundant CONFIG_SDM845 check
> arm: dts: qcom: Sync pinctrl DT nodes with Linux bindings
>
> arch/arm/dts/dragonboard410c-uboot.dtsi | 2 +-
> arch/arm/dts/dragonboard410c.dts | 17 +++-----
> arch/arm/dts/dragonboard820c-uboot.dtsi | 2 +-
> arch/arm/dts/dragonboard820c.dts | 4 +-
> arch/arm/dts/qcom-ipq4019.dtsi | 18 +++------
> arch/arm/dts/qcs404-evb.dts | 2 +-
> arch/arm/dts/sdm845.dtsi | 2 +-
> arch/arm/mach-ipq40xx/pinctrl-snapdragon.c | 31 ++++++++++++++-
> arch/arm/mach-snapdragon/Makefile | 2 +-
> arch/arm/mach-snapdragon/pinctrl-snapdragon.c | 39 ++++++++++++++++---
> drivers/gpio/msm_gpio.c | 10 +----
> 11 files changed, 83 insertions(+), 46 deletions(-)
>
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] pinctrl: sdm845: Remove redundant CONFIG_SDM845 check
2022-07-27 8:22 ` [PATCH v2 1/2] pinctrl: sdm845: Remove redundant CONFIG_SDM845 check Sumit Garg
@ 2022-08-07 0:16 ` Ramon Fried
2022-08-26 18:49 ` Tom Rini
1 sibling, 0 replies; 9+ messages in thread
From: Ramon Fried @ 2022-08-07 0:16 UTC (permalink / raw)
To: Sumit Garg
Cc: U-Boot Mailing List, Robert Marko, luka.kovacic, luka.perkov,
Дмитрий
Санковский,
Stephan Gerhold, Simon Glass, Tom Rini, Nicolas Dechesne,
vinod.koul, mworsfold, Daniel Thompson, Peter Robinson
On Wed, Jul 27, 2022 at 11:22 AM Sumit Garg <sumit.garg@linaro.org> wrote:
>
> DT compatible is sufficient to make platform specific differentiation,
> so remove redundant CONFIG_SDM845 check.
>
> Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
> ---
> arch/arm/mach-snapdragon/Makefile | 2 +-
> arch/arm/mach-snapdragon/pinctrl-snapdragon.c | 2 --
> 2 files changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/arch/arm/mach-snapdragon/Makefile b/arch/arm/mach-snapdragon/Makefile
> index 0d31f10f68..cbaaf23f6b 100644
> --- a/arch/arm/mach-snapdragon/Makefile
> +++ b/arch/arm/mach-snapdragon/Makefile
> @@ -16,6 +16,6 @@ obj-y += pinctrl-snapdragon.o
> obj-y += pinctrl-apq8016.o
> obj-y += pinctrl-apq8096.o
> obj-y += pinctrl-qcs404.o
> -obj-$(CONFIG_SDM845) += pinctrl-sdm845.o
> +obj-y += pinctrl-sdm845.o
> obj-$(CONFIG_TARGET_QCS404EVB) += clock-qcs404.o
> obj-$(CONFIG_TARGET_QCS404EVB) += sysmap-qcs404.o
> diff --git a/arch/arm/mach-snapdragon/pinctrl-snapdragon.c b/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
> index c2148a5d0a..842e2da0c5 100644
> --- a/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
> +++ b/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
> @@ -116,9 +116,7 @@ static struct pinctrl_ops msm_pinctrl_ops = {
> static const struct udevice_id msm_pinctrl_ids[] = {
> { .compatible = "qcom,tlmm-apq8016", .data = (ulong)&apq8016_data },
> { .compatible = "qcom,tlmm-apq8096", .data = (ulong)&apq8096_data },
> -#ifdef CONFIG_SDM845
> { .compatible = "qcom,tlmm-sdm845", .data = (ulong)&sdm845_data },
> -#endif
> { .compatible = "qcom,tlmm-qcs404", .data = (ulong)&qcs404_data },
> { }
> };
> --
> 2.25.1
>
Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs
2022-08-04 14:39 ` [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs Sumit Garg
@ 2022-08-16 8:02 ` Sumit Garg
2022-08-25 5:22 ` Sumit Garg
0 siblings, 1 reply; 9+ messages in thread
From: Sumit Garg @ 2022-08-16 8:02 UTC (permalink / raw)
To: trini
Cc: robert.marko, luka.kovacic, luka.perkov, rfried.dev, dsankouski,
stephan, sjg, nicolas.dechesne, vinod.koul, mworsfold,
daniel.thompson, pbrobinson, u-boot
Hi Tom,
On Thu, 4 Aug 2022 at 20:09, Sumit Garg <sumit.garg@linaro.org> wrote:
>
> Hi,
>
> On Wed, 27 Jul 2022 at 13:52, Sumit Garg <sumit.garg@linaro.org> wrote:
> >
> > This is an initial step towards achieving complete Linux DT sync on Qcom
> > SoCs/boards. It syncs up DT compatibles for pinctrl and GPIO drivers.
> >
> > Changes in v2:
> > - Separate patch for CONFIG_SDM845 check removal.
> > - Fix pinctrl DT compatibles for db410c and db820c.
> >
>
> Do we have any further comments on this?
>
I haven't heard any comments on this patch-set. Would it be fine with
you to merge it?
-Sumit
>
> > Sumit Garg (2):
> > pinctrl: sdm845: Remove redundant CONFIG_SDM845 check
> > arm: dts: qcom: Sync pinctrl DT nodes with Linux bindings
> >
> > arch/arm/dts/dragonboard410c-uboot.dtsi | 2 +-
> > arch/arm/dts/dragonboard410c.dts | 17 +++-----
> > arch/arm/dts/dragonboard820c-uboot.dtsi | 2 +-
> > arch/arm/dts/dragonboard820c.dts | 4 +-
> > arch/arm/dts/qcom-ipq4019.dtsi | 18 +++------
> > arch/arm/dts/qcs404-evb.dts | 2 +-
> > arch/arm/dts/sdm845.dtsi | 2 +-
> > arch/arm/mach-ipq40xx/pinctrl-snapdragon.c | 31 ++++++++++++++-
> > arch/arm/mach-snapdragon/Makefile | 2 +-
> > arch/arm/mach-snapdragon/pinctrl-snapdragon.c | 39 ++++++++++++++++---
> > drivers/gpio/msm_gpio.c | 10 +----
> > 11 files changed, 83 insertions(+), 46 deletions(-)
> >
> > --
> > 2.25.1
> >
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs
2022-08-16 8:02 ` Sumit Garg
@ 2022-08-25 5:22 ` Sumit Garg
0 siblings, 0 replies; 9+ messages in thread
From: Sumit Garg @ 2022-08-25 5:22 UTC (permalink / raw)
To: trini
Cc: robert.marko, luka.kovacic, luka.perkov, rfried.dev, dsankouski,
stephan, sjg, nicolas.dechesne, vinod.koul, mworsfold,
daniel.thompson, pbrobinson, u-boot
On Tue, 16 Aug 2022 at 13:32, Sumit Garg <sumit.garg@linaro.org> wrote:
>
> Hi Tom,
>
> On Thu, 4 Aug 2022 at 20:09, Sumit Garg <sumit.garg@linaro.org> wrote:
> >
> > Hi,
> >
> > On Wed, 27 Jul 2022 at 13:52, Sumit Garg <sumit.garg@linaro.org> wrote:
> > >
> > > This is an initial step towards achieving complete Linux DT sync on Qcom
> > > SoCs/boards. It syncs up DT compatibles for pinctrl and GPIO drivers.
> > >
> > > Changes in v2:
> > > - Separate patch for CONFIG_SDM845 check removal.
> > > - Fix pinctrl DT compatibles for db410c and db820c.
> > >
> >
> > Do we have any further comments on this?
> >
>
> I haven't heard any comments on this patch-set. Would it be fine with
> you to merge it?
>
Gentle ping.
-Sumit
>
> >
> > > Sumit Garg (2):
> > > pinctrl: sdm845: Remove redundant CONFIG_SDM845 check
> > > arm: dts: qcom: Sync pinctrl DT nodes with Linux bindings
> > >
> > > arch/arm/dts/dragonboard410c-uboot.dtsi | 2 +-
> > > arch/arm/dts/dragonboard410c.dts | 17 +++-----
> > > arch/arm/dts/dragonboard820c-uboot.dtsi | 2 +-
> > > arch/arm/dts/dragonboard820c.dts | 4 +-
> > > arch/arm/dts/qcom-ipq4019.dtsi | 18 +++------
> > > arch/arm/dts/qcs404-evb.dts | 2 +-
> > > arch/arm/dts/sdm845.dtsi | 2 +-
> > > arch/arm/mach-ipq40xx/pinctrl-snapdragon.c | 31 ++++++++++++++-
> > > arch/arm/mach-snapdragon/Makefile | 2 +-
> > > arch/arm/mach-snapdragon/pinctrl-snapdragon.c | 39 ++++++++++++++++---
> > > drivers/gpio/msm_gpio.c | 10 +----
> > > 11 files changed, 83 insertions(+), 46 deletions(-)
> > >
> > > --
> > > 2.25.1
> > >
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] pinctrl: sdm845: Remove redundant CONFIG_SDM845 check
2022-07-27 8:22 ` [PATCH v2 1/2] pinctrl: sdm845: Remove redundant CONFIG_SDM845 check Sumit Garg
2022-08-07 0:16 ` Ramon Fried
@ 2022-08-26 18:49 ` Tom Rini
1 sibling, 0 replies; 9+ messages in thread
From: Tom Rini @ 2022-08-26 18:49 UTC (permalink / raw)
To: Sumit Garg
Cc: u-boot, robert.marko, luka.kovacic, luka.perkov, rfried.dev,
dsankouski, stephan, sjg, nicolas.dechesne, vinod.koul,
mworsfold, daniel.thompson, pbrobinson
[-- Attachment #1: Type: text/plain, Size: 336 bytes --]
On Wed, Jul 27, 2022 at 01:52:03PM +0530, Sumit Garg wrote:
> DT compatible is sufficient to make platform specific differentiation,
> so remove redundant CONFIG_SDM845 check.
>
> Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
> Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
Applied to u-boot/next, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] arm: dts: qcom: Sync pinctrl DT nodes with Linux bindings
2022-07-27 8:22 ` [PATCH v2 2/2] arm: dts: qcom: Sync pinctrl DT nodes with Linux bindings Sumit Garg
@ 2022-08-26 18:49 ` Tom Rini
0 siblings, 0 replies; 9+ messages in thread
From: Tom Rini @ 2022-08-26 18:49 UTC (permalink / raw)
To: Sumit Garg
Cc: u-boot, robert.marko, luka.kovacic, luka.perkov, rfried.dev,
dsankouski, stephan, sjg, nicolas.dechesne, vinod.koul,
mworsfold, daniel.thompson, pbrobinson
[-- Attachment #1: Type: text/plain, Size: 807 bytes --]
On Wed, Jul 27, 2022 at 01:52:04PM +0530, Sumit Garg wrote:
> Currently for all Qcom SoCs/boards there are separate compatibles for
> GPIO and pinctrl. But this is inconsistent with official (upstream) Linux
> bindings which requires only a single compatible "qcom,<SoC name>-pinctrl"
> and there is no such compatible property as "qcom,tlmm-<SoC name>".
>
> So fix this inconsistency for Qcom SoCs in order to comply with upstream
> DT bindings. This is done via removing compatibles from "msm_gpio" driver
> and via binding to "msm_gpio" driver from pinctrl driver in case
> "gpio-controller" property is specified for pinctrl node.
>
> Suggested-by: Stephan Gerhold <stephan@gerhold.net>
> Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
Applied to u-boot/next, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2022-08-26 18:50 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-27 8:22 [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs Sumit Garg
2022-07-27 8:22 ` [PATCH v2 1/2] pinctrl: sdm845: Remove redundant CONFIG_SDM845 check Sumit Garg
2022-08-07 0:16 ` Ramon Fried
2022-08-26 18:49 ` Tom Rini
2022-07-27 8:22 ` [PATCH v2 2/2] arm: dts: qcom: Sync pinctrl DT nodes with Linux bindings Sumit Garg
2022-08-26 18:49 ` Tom Rini
2022-08-04 14:39 ` [PATCH v2 0/2] Initial step for Linux DT sync on Qcom SoCs Sumit Garg
2022-08-16 8:02 ` Sumit Garg
2022-08-25 5:22 ` Sumit Garg
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).