* [PATCH v1 0/3] Have Tegra's GPIO chip depend explicitly on the pinctrl device
@ 2015-07-01 12:45 Tomeu Vizoso
2015-07-01 12:45 ` [PATCH v1 3/3] ARM: tegra: Add gpio-ranges property Tomeu Vizoso
0 siblings, 1 reply; 3+ messages in thread
From: Tomeu Vizoso @ 2015-07-01 12:45 UTC (permalink / raw)
To: linux-arm-kernel
Hello,
these three patches make sure that there's an explicit dependency from
the GPIO chip in Tegra SoCs to the corresponding pinctrl device, without
having duplicated gpio ranges.
By having an explicit dependency, we can do things such as probing the
pinctrl device before the GPIO chip device to avoid deferred probes.
Thanks,
Tomeu
Tomeu Vizoso (3):
gpio: defer probe if pinctrl cannot be found
pinctrl: tegra: Only set the gpio range if needed
ARM: tegra: Add gpio-ranges property
arch/arm/boot/dts/tegra114.dtsi | 1 +
arch/arm/boot/dts/tegra124.dtsi | 1 +
arch/arm/boot/dts/tegra20.dtsi | 1 +
arch/arm/boot/dts/tegra30.dtsi | 1 +
drivers/gpio/gpiolib-of.c | 27 ++++++++++++++++++---------
drivers/gpio/gpiolib.c | 5 ++++-
drivers/pinctrl/pinctrl-tegra.c | 19 ++++++++++++++++++-
include/linux/of_gpio.h | 4 ++--
8 files changed, 46 insertions(+), 13 deletions(-)
--
2.4.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH v1 3/3] ARM: tegra: Add gpio-ranges property
2015-07-01 12:45 [PATCH v1 0/3] Have Tegra's GPIO chip depend explicitly on the pinctrl device Tomeu Vizoso
@ 2015-07-01 12:45 ` Tomeu Vizoso
2015-07-08 20:55 ` Stephen Warren
0 siblings, 1 reply; 3+ messages in thread
From: Tomeu Vizoso @ 2015-07-01 12:45 UTC (permalink / raw)
To: linux-arm-kernel
Specify how the GPIOs map to the pins in Tegra SoCs, so the dependency is
explicit.
This currently will add a duplicated entry in the map from pins to gpios
in the pinmux controller but it should be harmless and will be fixed in a
later commit.
Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora•com>
---
arch/arm/boot/dts/tegra114.dtsi | 1 +
arch/arm/boot/dts/tegra124.dtsi | 1 +
arch/arm/boot/dts/tegra20.dtsi | 1 +
arch/arm/boot/dts/tegra30.dtsi | 1 +
4 files changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra114.dtsi
index f58a3d9..7e1e171 100644
--- a/arch/arm/boot/dts/tegra114.dtsi
+++ b/arch/arm/boot/dts/tegra114.dtsi
@@ -234,6 +234,7 @@
gpio-controller;
#interrupt-cells = <2>;
interrupt-controller;
+ gpio-ranges = <&pinmux 0 0 246>;
};
apbmisc at 70000800 {
diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi
index 87318a7..4779df3 100644
--- a/arch/arm/boot/dts/tegra124.dtsi
+++ b/arch/arm/boot/dts/tegra124.dtsi
@@ -255,6 +255,7 @@
gpio-controller;
#interrupt-cells = <2>;
interrupt-controller;
+ gpio-ranges = <&pinmux 0 0 251>;
};
apbdma: dma at 0,60020000 {
diff --git a/arch/arm/boot/dts/tegra20.dtsi b/arch/arm/boot/dts/tegra20.dtsi
index f444b67..b44277c 100644
--- a/arch/arm/boot/dts/tegra20.dtsi
+++ b/arch/arm/boot/dts/tegra20.dtsi
@@ -244,6 +244,7 @@
gpio-controller;
#interrupt-cells = <2>;
interrupt-controller;
+ gpio-ranges = <&pinmux 0 0 224>;
};
apbmisc at 70000800 {
diff --git a/arch/arm/boot/dts/tegra30.dtsi b/arch/arm/boot/dts/tegra30.dtsi
index 782b11b..28c547f 100644
--- a/arch/arm/boot/dts/tegra30.dtsi
+++ b/arch/arm/boot/dts/tegra30.dtsi
@@ -349,6 +349,7 @@
gpio-controller;
#interrupt-cells = <2>;
interrupt-controller;
+ gpio-ranges = <&pinmux 0 0 248>;
};
apbmisc at 70000800 {
--
2.4.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* [PATCH v1 3/3] ARM: tegra: Add gpio-ranges property
2015-07-01 12:45 ` [PATCH v1 3/3] ARM: tegra: Add gpio-ranges property Tomeu Vizoso
@ 2015-07-08 20:55 ` Stephen Warren
0 siblings, 0 replies; 3+ messages in thread
From: Stephen Warren @ 2015-07-08 20:55 UTC (permalink / raw)
To: linux-arm-kernel
On 07/01/2015 06:45 AM, Tomeu Vizoso wrote:
> Specify how the GPIOs map to the pins in Tegra SoCs, so the dependency is
> explicit.
>
> This currently will add a duplicated entry in the map from pins to gpios
> in the pinmux controller but it should be harmless and will be fixed in a
> later commit.
Isn't it in an earlier commit now (patch 2/3)? :-)
At a quick glance, I think this approach will be fine, so the series,
Acked-by: Stephen Warren <swarren@nvidia•com>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-07-08 20:55 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-01 12:45 [PATCH v1 0/3] Have Tegra's GPIO chip depend explicitly on the pinctrl device Tomeu Vizoso
2015-07-01 12:45 ` [PATCH v1 3/3] ARM: tegra: Add gpio-ranges property Tomeu Vizoso
2015-07-08 20:55 ` Stephen Warren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox