* [PATCH 1/3] ARM64: dts: meson-gxl: Fix pinctrl periphs gpio-ranges
2017-05-23 14:21 [PATCH 0/3] ARM64: dts: meson: GXBB and GXL pinctrl fixes Neil Armstrong
@ 2017-05-23 14:21 ` Neil Armstrong
2017-05-24 8:39 ` Jerome Brunet
2017-05-23 14:21 ` [PATCH 2/3] ARM64: dts: Fix GXL periphs pinctrl pull-enable register base Neil Armstrong
` (2 subsequent siblings)
3 siblings, 1 reply; 8+ messages in thread
From: Neil Armstrong @ 2017-05-23 14:21 UTC (permalink / raw)
To: linus-amlogic
The gpio-range was badly added on the GXL dtsi, the AO pin count is 10
instead of 14.
Fixes: 84412e4e857f ("ARM64: dts: meson-gxl: Add gpio-ranges properties")
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
index 3efad5f..4871ce5 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
@@ -244,7 +244,7 @@
reg-names = "mux", "pull", "pull-enable", "gpio";
gpio-controller;
#gpio-cells = <2>;
- gpio-ranges = <&pinctrl_periphs 0 14 101>;
+ gpio-ranges = <&pinctrl_periphs 0 10 101>;
};
emmc_pins: emmc {
--
1.9.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 1/3] ARM64: dts: meson-gxl: Fix pinctrl periphs gpio-ranges
2017-05-23 14:21 ` [PATCH 1/3] ARM64: dts: meson-gxl: Fix pinctrl periphs gpio-ranges Neil Armstrong
@ 2017-05-24 8:39 ` Jerome Brunet
0 siblings, 0 replies; 8+ messages in thread
From: Jerome Brunet @ 2017-05-24 8:39 UTC (permalink / raw)
To: linus-amlogic
On Tue, 2017-05-23 at 16:21 +0200, Neil Armstrong wrote:
> The gpio-range was badly added on the GXL dtsi, the AO pin count is 10
> instead of 14.
>
Reviewed-by: Jerome Brunet <jbrunet@baylibre.com>
> Fixes: 84412e4e857f ("ARM64: dts: meson-gxl: Add gpio-ranges properties")
> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> ---
> ?arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 2 +-
> ?1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
> b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
> index 3efad5f..4871ce5 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
> @@ -244,7 +244,7 @@
> ? reg-names = "mux", "pull", "pull-enable", "gpio";
> ? gpio-controller;
> ? #gpio-cells = <2>;
> - gpio-ranges = <&pinctrl_periphs 0 14 101>;
> + gpio-ranges = <&pinctrl_periphs 0 10 101>;
> ? };
> ?
> ? emmc_pins: emmc {
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/3] ARM64: dts: Fix GXL periphs pinctrl pull-enable register base
2017-05-23 14:21 [PATCH 0/3] ARM64: dts: meson: GXBB and GXL pinctrl fixes Neil Armstrong
2017-05-23 14:21 ` [PATCH 1/3] ARM64: dts: meson-gxl: Fix pinctrl periphs gpio-ranges Neil Armstrong
@ 2017-05-23 14:21 ` Neil Armstrong
2017-05-24 8:39 ` Jerome Brunet
2017-05-23 14:21 ` [PATCH 3/3] ARM64: dts: Fix GXBB " Neil Armstrong
2017-05-30 22:07 ` [PATCH 0/3] ARM64: dts: meson: GXBB and GXL pinctrl fixes Kevin Hilman
3 siblings, 1 reply; 8+ messages in thread
From: Neil Armstrong @ 2017-05-23 14:21 UTC (permalink / raw)
To: linus-amlogic
The pull-enable register base was wrongly copied from the GXBB pinctrl node,
but was not used yet.
Fixes: fb0fe92294a9 ("ARM64: dts: meson-gxl: Add pinctrl nodes")
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
index 4871ce5..57fe1f5 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
@@ -239,7 +239,7 @@
gpio: bank at 4b0 {
reg = <0x0 0x004b0 0x0 0x28>,
<0x0 0x004e8 0x0 0x14>,
- <0x0 0x00120 0x0 0x14>,
+ <0x0 0x00520 0x0 0x14>,
<0x0 0x00430 0x0 0x40>;
reg-names = "mux", "pull", "pull-enable", "gpio";
gpio-controller;
--
1.9.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 2/3] ARM64: dts: Fix GXL periphs pinctrl pull-enable register base
2017-05-23 14:21 ` [PATCH 2/3] ARM64: dts: Fix GXL periphs pinctrl pull-enable register base Neil Armstrong
@ 2017-05-24 8:39 ` Jerome Brunet
0 siblings, 0 replies; 8+ messages in thread
From: Jerome Brunet @ 2017-05-24 8:39 UTC (permalink / raw)
To: linus-amlogic
On Tue, 2017-05-23 at 16:21 +0200, Neil Armstrong wrote:
> The pull-enable register base was wrongly copied from the GXBB pinctrl node,
> but was not used yet.
Reviewed-by: Jerome Brunet <jbrunet@baylibre.com>
>
> Fixes: fb0fe92294a9 ("ARM64: dts: meson-gxl: Add pinctrl nodes")
> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> ---
> ?arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 2 +-
> ?1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
> b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
> index 4871ce5..57fe1f5 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
> @@ -239,7 +239,7 @@
> ? gpio: bank at 4b0 {
> ? reg = <0x0 0x004b0 0x0 0x28>,
> ? ??????<0x0 0x004e8 0x0 0x14>,
> - ??????<0x0 0x00120 0x0 0x14>,
> + ??????<0x0 0x00520 0x0 0x14>,
> ? ??????<0x0 0x00430 0x0 0x40>;
> ? reg-names = "mux", "pull", "pull-enable", "gpio";
> ? gpio-controller;
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 3/3] ARM64: dts: Fix GXBB periphs pinctrl pull-enable register base
2017-05-23 14:21 [PATCH 0/3] ARM64: dts: meson: GXBB and GXL pinctrl fixes Neil Armstrong
2017-05-23 14:21 ` [PATCH 1/3] ARM64: dts: meson-gxl: Fix pinctrl periphs gpio-ranges Neil Armstrong
2017-05-23 14:21 ` [PATCH 2/3] ARM64: dts: Fix GXL periphs pinctrl pull-enable register base Neil Armstrong
@ 2017-05-23 14:21 ` Neil Armstrong
2017-05-24 8:40 ` Jerome Brunet
2017-05-30 22:07 ` [PATCH 0/3] ARM64: dts: meson: GXBB and GXL pinctrl fixes Kevin Hilman
3 siblings, 1 reply; 8+ messages in thread
From: Neil Armstrong @ 2017-05-23 14:21 UTC (permalink / raw)
To: linus-amlogic
The pull-enable register base was wrongly copied from the meson8b pinctrl node,
but was not used yet.
Fixes: c328666d58aa ("ARM64: dts: amlogic: Add Meson GX dtsi from GXBB")
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
index 6f8b301..47ca44d 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
@@ -351,7 +351,7 @@
gpio: bank at 4b0 {
reg = <0x0 0x004b0 0x0 0x28>,
<0x0 0x004e8 0x0 0x14>,
- <0x0 0x00120 0x0 0x14>,
+ <0x0 0x00520 0x0 0x14>,
<0x0 0x00430 0x0 0x40>;
reg-names = "mux", "pull", "pull-enable", "gpio";
gpio-controller;
--
1.9.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 3/3] ARM64: dts: Fix GXBB periphs pinctrl pull-enable register base
2017-05-23 14:21 ` [PATCH 3/3] ARM64: dts: Fix GXBB " Neil Armstrong
@ 2017-05-24 8:40 ` Jerome Brunet
0 siblings, 0 replies; 8+ messages in thread
From: Jerome Brunet @ 2017-05-24 8:40 UTC (permalink / raw)
To: linus-amlogic
On Tue, 2017-05-23 at 16:21 +0200, Neil Armstrong wrote:
> The pull-enable register base was wrongly copied from the meson8b pinctrl
> node,
> but was not used yet.
Reviewed-by: Jerome Brunet <jbrunet@baylibre.com>
>
> Fixes: c328666d58aa ("ARM64: dts: amlogic: Add Meson GX dtsi from GXBB")
> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> ---
> ?arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 2 +-
> ?1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
> b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
> index 6f8b301..47ca44d 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
> @@ -351,7 +351,7 @@
> ? gpio: bank at 4b0 {
> ? reg = <0x0 0x004b0 0x0 0x28>,
> ? ??????<0x0 0x004e8 0x0 0x14>,
> - ??????<0x0 0x00120 0x0 0x14>,
> + ??????<0x0 0x00520 0x0 0x14>,
> ? ??????<0x0 0x00430 0x0 0x40>;
> ? reg-names = "mux", "pull", "pull-enable", "gpio";
> ? gpio-controller;
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 0/3] ARM64: dts: meson: GXBB and GXL pinctrl fixes
2017-05-23 14:21 [PATCH 0/3] ARM64: dts: meson: GXBB and GXL pinctrl fixes Neil Armstrong
` (2 preceding siblings ...)
2017-05-23 14:21 ` [PATCH 3/3] ARM64: dts: Fix GXBB " Neil Armstrong
@ 2017-05-30 22:07 ` Kevin Hilman
3 siblings, 0 replies; 8+ messages in thread
From: Kevin Hilman @ 2017-05-30 22:07 UTC (permalink / raw)
To: linus-amlogic
Neil Armstrong <narmstrong@baylibre.com> writes:
> This patchset fixes the following :
> - bad gpio-ranges for GXL
> - bad pull-enable register base for GXBB & GXL
Applied with Jerome's reviewed-by tag added.
Kevin
^ permalink raw reply [flat|nested] 8+ messages in thread