* [PATCH 0/2] add support for uart_AO_B
@ 2017-01-15 22:20 Martin Blumenstingl
2017-01-15 22:20 ` [PATCH 1/2] pinctrl: meson: fix uart_ao_b for GXBB and GXL/GXM Martin Blumenstingl
[not found] ` <20170115222029.8271-1-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>
0 siblings, 2 replies; 7+ messages in thread
From: Martin Blumenstingl @ 2017-01-15 22:20 UTC (permalink / raw)
To: linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-gpio-u79uwXL29TY76Z2rM5mHXA,
linus.walleij-QSEj5FYQhm4dnm+yROfE0A,
khilman-rdvid1DuHRBWk0Htik3J/w
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
carlo-KA+7E9HrN00dnm+yROfE0A, will.deacon-5wv7dgnIgG8,
catalin.marinas-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, Martin Blumenstingl
This adds the uart_AO_B serial port and the related pinctrl nodes to
meson-gx.
Due to a bug in the current GXBB and GXL/GXM pinctrl code uart_AO_B
could not be used when uart_AO(_A) was enabled at the same time. The
reason for this is that uart_AO_A and uart_AO_B were both trying to
request the same pin as their RX pin (GPIOAO_1). uart_AO_B also
requests a second pin for it's RX (GPIOAO_5), which does not make any
sense.
Thus uart_AO_B is changed to only use GPIOAO_5 for RX and GPIOAO_4
for TX, which is also what the Amlogic reference kernel does.
Adding uart_AO_B is preparation work for adding support for the
Khadas VIM and VIM Pro boards, as these are both exposing uart_AO_B
on the pin-header.
Martin Blumenstingl (2):
pinctrl: meson: fix uart_ao_b for GXBB and GXL/GXM
ARM64: dts: meson-gx: add the missing uart_AO_B
arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 8 ++++++++
arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 7 +++++++
arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 7 +++++++
drivers/pinctrl/meson/pinctrl-meson-gxbb.c | 7 +++----
drivers/pinctrl/meson/pinctrl-meson-gxl.c | 7 +++----
5 files changed, 28 insertions(+), 8 deletions(-)
--
2.11.0
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH 1/2] pinctrl: meson: fix uart_ao_b for GXBB and GXL/GXM 2017-01-15 22:20 [PATCH 0/2] add support for uart_AO_B Martin Blumenstingl @ 2017-01-15 22:20 ` Martin Blumenstingl [not found] ` <20170115222029.8271-2-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> [not found] ` <20170115222029.8271-1-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> 1 sibling, 1 reply; 7+ messages in thread From: Martin Blumenstingl @ 2017-01-15 22:20 UTC (permalink / raw) To: linux-amlogic, linux-gpio, linus.walleij, khilman Cc: devicetree, linux-arm-kernel, carlo, will.deacon, catalin.marinas, mark.rutland, robh+dt, Martin Blumenstingl The GXBB and GXL/GXM pinctrl drivers had a configuration which conflicts with uart_ao_a. According to the GXBB ("S905") datasheet the AO UART functions are: - GPIOAO_0: Func1 = UART_TX_AO_A (bit 12), Func2 = UART_TX_AO_B (bit 26) - GPIOAO_1: Func1 = UART_RX_AO_A (bit 11), Func2 = UART_RX_AO_B (bit 25) - GPIOAO_4: Func2 = UART_TX_AO_B (bit 24) - GPIOAO_5: Func2 = UART_RX_AO_B (bit 25) The existing definition for uart_AO_A already uses GPIOAO_0 and GPIOAO_1. The old definition of uart_AO_B however was broken, as it used GPIOAO_0 for TX (which would be fine) and two pins (GPIOAO_1 and GPIOAO_5) for RX (which does not make any sense). This fixes the uart_AO_B configuration by moving it to GPIOAO_4 and GPIOAO_5 (it would be possible to use GPIOAO_0 and GPIOAO_1 in theory, but all existing hardware uses uart_AO_A there). The fix for GXBB and GXL/GXM is identical since it seems that these specific pins are identical on both SoC variants. Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> --- drivers/pinctrl/meson/pinctrl-meson-gxbb.c | 7 +++---- drivers/pinctrl/meson/pinctrl-meson-gxl.c | 7 +++---- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/drivers/pinctrl/meson/pinctrl-meson-gxbb.c b/drivers/pinctrl/meson/pinctrl-meson-gxbb.c index c3928aa3fefa..e0bca4df2a2f 100644 --- a/drivers/pinctrl/meson/pinctrl-meson-gxbb.c +++ b/drivers/pinctrl/meson/pinctrl-meson-gxbb.c @@ -253,9 +253,8 @@ static const unsigned int uart_tx_ao_a_pins[] = { PIN(GPIOAO_0, 0) }; static const unsigned int uart_rx_ao_a_pins[] = { PIN(GPIOAO_1, 0) }; static const unsigned int uart_cts_ao_a_pins[] = { PIN(GPIOAO_2, 0) }; static const unsigned int uart_rts_ao_a_pins[] = { PIN(GPIOAO_3, 0) }; -static const unsigned int uart_tx_ao_b_pins[] = { PIN(GPIOAO_0, 0) }; -static const unsigned int uart_rx_ao_b_pins[] = { PIN(GPIOAO_1, 0), - PIN(GPIOAO_5, 0) }; +static const unsigned int uart_tx_ao_b_pins[] = { PIN(GPIOAO_4, 0) }; +static const unsigned int uart_rx_ao_b_pins[] = { PIN(GPIOAO_5, 0) }; static const unsigned int uart_cts_ao_b_pins[] = { PIN(GPIOAO_2, 0) }; static const unsigned int uart_rts_ao_b_pins[] = { PIN(GPIOAO_3, 0) }; @@ -498,7 +497,7 @@ static struct meson_pmx_group meson_gxbb_aobus_groups[] = { GPIO_GROUP(GPIOAO_13, 0), /* bank AO */ - GROUP(uart_tx_ao_b, 0, 26), + GROUP(uart_tx_ao_b, 0, 24), GROUP(uart_rx_ao_b, 0, 25), GROUP(uart_tx_ao_a, 0, 12), GROUP(uart_rx_ao_a, 0, 11), diff --git a/drivers/pinctrl/meson/pinctrl-meson-gxl.c b/drivers/pinctrl/meson/pinctrl-meson-gxl.c index 25694f7094c7..b69743b07a1d 100644 --- a/drivers/pinctrl/meson/pinctrl-meson-gxl.c +++ b/drivers/pinctrl/meson/pinctrl-meson-gxl.c @@ -214,9 +214,8 @@ static const unsigned int uart_tx_ao_a_pins[] = { PIN(GPIOAO_0, 0) }; static const unsigned int uart_rx_ao_a_pins[] = { PIN(GPIOAO_1, 0) }; static const unsigned int uart_cts_ao_a_pins[] = { PIN(GPIOAO_2, 0) }; static const unsigned int uart_rts_ao_a_pins[] = { PIN(GPIOAO_3, 0) }; -static const unsigned int uart_tx_ao_b_pins[] = { PIN(GPIOAO_0, 0) }; -static const unsigned int uart_rx_ao_b_pins[] = { PIN(GPIOAO_1, 0), - PIN(GPIOAO_5, 0) }; +static const unsigned int uart_tx_ao_b_pins[] = { PIN(GPIOAO_4, 0) }; +static const unsigned int uart_rx_ao_b_pins[] = { PIN(GPIOAO_5, 0) }; static const unsigned int uart_cts_ao_b_pins[] = { PIN(GPIOAO_2, 0) }; static const unsigned int uart_rts_ao_b_pins[] = { PIN(GPIOAO_3, 0) }; @@ -409,7 +408,7 @@ static struct meson_pmx_group meson_gxl_aobus_groups[] = { GPIO_GROUP(GPIOAO_9, 0), /* bank AO */ - GROUP(uart_tx_ao_b, 0, 26), + GROUP(uart_tx_ao_b, 0, 24), GROUP(uart_rx_ao_b, 0, 25), GROUP(uart_tx_ao_a, 0, 12), GROUP(uart_rx_ao_a, 0, 11), -- 2.11.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
[parent not found: <20170115222029.8271-2-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/2] pinctrl: meson: fix uart_ao_b for GXBB and GXL/GXM [not found] ` <20170115222029.8271-2-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> @ 2017-01-18 10:21 ` Linus Walleij 2017-01-18 21:40 ` Kevin Hilman 1 sibling, 0 replies; 7+ messages in thread From: Linus Walleij @ 2017-01-18 10:21 UTC (permalink / raw) To: Martin Blumenstingl Cc: open list:ARM/Amlogic Meson..., linux-gpio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Kevin Hilman, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Carlo Caione, Will Deacon, Catalin Marinas, Mark Rutland, Rob Herring On Sun, Jan 15, 2017 at 11:20 PM, Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> wrote: > The GXBB and GXL/GXM pinctrl drivers had a configuration which conflicts > with uart_ao_a. According to the GXBB ("S905") datasheet the AO UART > functions are: > - GPIOAO_0: Func1 = UART_TX_AO_A (bit 12), Func2 = UART_TX_AO_B (bit 26) > - GPIOAO_1: Func1 = UART_RX_AO_A (bit 11), Func2 = UART_RX_AO_B (bit 25) > - GPIOAO_4: Func2 = UART_TX_AO_B (bit 24) > - GPIOAO_5: Func2 = UART_RX_AO_B (bit 25) > > The existing definition for uart_AO_A already uses GPIOAO_0 and GPIOAO_1. > The old definition of uart_AO_B however was broken, as it used GPIOAO_0 > for TX (which would be fine) and two pins (GPIOAO_1 and GPIOAO_5) for RX > (which does not make any sense). > > This fixes the uart_AO_B configuration by moving it to GPIOAO_4 and > GPIOAO_5 (it would be possible to use GPIOAO_0 and GPIOAO_1 in theory, > but all existing hardware uses uart_AO_A there). > The fix for GXBB and GXL/GXM is identical since it seems that these > specific pins are identical on both SoC variants. > > Signed-off-by: Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> Patch applied for fixes. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] pinctrl: meson: fix uart_ao_b for GXBB and GXL/GXM [not found] ` <20170115222029.8271-2-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> 2017-01-18 10:21 ` Linus Walleij @ 2017-01-18 21:40 ` Kevin Hilman 1 sibling, 0 replies; 7+ messages in thread From: Kevin Hilman @ 2017-01-18 21:40 UTC (permalink / raw) To: Martin Blumenstingl Cc: linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-gpio-u79uwXL29TY76Z2rM5mHXA, linus.walleij-QSEj5FYQhm4dnm+yROfE0A, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, carlo-KA+7E9HrN00dnm+yROfE0A, will.deacon-5wv7dgnIgG8, catalin.marinas-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8, robh+dt-DgEjT+Ai2ygdnm+yROfE0A Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> writes: > The GXBB and GXL/GXM pinctrl drivers had a configuration which conflicts > with uart_ao_a. According to the GXBB ("S905") datasheet the AO UART > functions are: > - GPIOAO_0: Func1 = UART_TX_AO_A (bit 12), Func2 = UART_TX_AO_B (bit 26) > - GPIOAO_1: Func1 = UART_RX_AO_A (bit 11), Func2 = UART_RX_AO_B (bit 25) > - GPIOAO_4: Func2 = UART_TX_AO_B (bit 24) > - GPIOAO_5: Func2 = UART_RX_AO_B (bit 25) > > The existing definition for uart_AO_A already uses GPIOAO_0 and GPIOAO_1. > The old definition of uart_AO_B however was broken, as it used GPIOAO_0 > for TX (which would be fine) and two pins (GPIOAO_1 and GPIOAO_5) for RX > (which does not make any sense). > > This fixes the uart_AO_B configuration by moving it to GPIOAO_4 and > GPIOAO_5 (it would be possible to use GPIOAO_0 and GPIOAO_1 in theory, > but all existing hardware uses uart_AO_A there). > The fix for GXBB and GXL/GXM is identical since it seems that these > specific pins are identical on both SoC variants. > > Signed-off-by: Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> Reviewed-by: Kevin Hilman <khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> > --- > drivers/pinctrl/meson/pinctrl-meson-gxbb.c | 7 +++---- > drivers/pinctrl/meson/pinctrl-meson-gxl.c | 7 +++---- > 2 files changed, 6 insertions(+), 8 deletions(-) > > diff --git a/drivers/pinctrl/meson/pinctrl-meson-gxbb.c b/drivers/pinctrl/meson/pinctrl-meson-gxbb.c > index c3928aa3fefa..e0bca4df2a2f 100644 > --- a/drivers/pinctrl/meson/pinctrl-meson-gxbb.c > +++ b/drivers/pinctrl/meson/pinctrl-meson-gxbb.c > @@ -253,9 +253,8 @@ static const unsigned int uart_tx_ao_a_pins[] = { PIN(GPIOAO_0, 0) }; > static const unsigned int uart_rx_ao_a_pins[] = { PIN(GPIOAO_1, 0) }; > static const unsigned int uart_cts_ao_a_pins[] = { PIN(GPIOAO_2, 0) }; > static const unsigned int uart_rts_ao_a_pins[] = { PIN(GPIOAO_3, 0) }; > -static const unsigned int uart_tx_ao_b_pins[] = { PIN(GPIOAO_0, 0) }; > -static const unsigned int uart_rx_ao_b_pins[] = { PIN(GPIOAO_1, 0), > - PIN(GPIOAO_5, 0) }; > +static const unsigned int uart_tx_ao_b_pins[] = { PIN(GPIOAO_4, 0) }; > +static const unsigned int uart_rx_ao_b_pins[] = { PIN(GPIOAO_5, 0) }; > static const unsigned int uart_cts_ao_b_pins[] = { PIN(GPIOAO_2, 0) }; > static const unsigned int uart_rts_ao_b_pins[] = { PIN(GPIOAO_3, 0) }; > > @@ -498,7 +497,7 @@ static struct meson_pmx_group meson_gxbb_aobus_groups[] = { > GPIO_GROUP(GPIOAO_13, 0), > > /* bank AO */ > - GROUP(uart_tx_ao_b, 0, 26), > + GROUP(uart_tx_ao_b, 0, 24), > GROUP(uart_rx_ao_b, 0, 25), > GROUP(uart_tx_ao_a, 0, 12), > GROUP(uart_rx_ao_a, 0, 11), > diff --git a/drivers/pinctrl/meson/pinctrl-meson-gxl.c b/drivers/pinctrl/meson/pinctrl-meson-gxl.c > index 25694f7094c7..b69743b07a1d 100644 > --- a/drivers/pinctrl/meson/pinctrl-meson-gxl.c > +++ b/drivers/pinctrl/meson/pinctrl-meson-gxl.c > @@ -214,9 +214,8 @@ static const unsigned int uart_tx_ao_a_pins[] = { PIN(GPIOAO_0, 0) }; > static const unsigned int uart_rx_ao_a_pins[] = { PIN(GPIOAO_1, 0) }; > static const unsigned int uart_cts_ao_a_pins[] = { PIN(GPIOAO_2, 0) }; > static const unsigned int uart_rts_ao_a_pins[] = { PIN(GPIOAO_3, 0) }; > -static const unsigned int uart_tx_ao_b_pins[] = { PIN(GPIOAO_0, 0) }; > -static const unsigned int uart_rx_ao_b_pins[] = { PIN(GPIOAO_1, 0), > - PIN(GPIOAO_5, 0) }; > +static const unsigned int uart_tx_ao_b_pins[] = { PIN(GPIOAO_4, 0) }; > +static const unsigned int uart_rx_ao_b_pins[] = { PIN(GPIOAO_5, 0) }; > static const unsigned int uart_cts_ao_b_pins[] = { PIN(GPIOAO_2, 0) }; > static const unsigned int uart_rts_ao_b_pins[] = { PIN(GPIOAO_3, 0) }; > > @@ -409,7 +408,7 @@ static struct meson_pmx_group meson_gxl_aobus_groups[] = { > GPIO_GROUP(GPIOAO_9, 0), > > /* bank AO */ > - GROUP(uart_tx_ao_b, 0, 26), > + GROUP(uart_tx_ao_b, 0, 24), > GROUP(uart_rx_ao_b, 0, 25), > GROUP(uart_tx_ao_a, 0, 12), > GROUP(uart_rx_ao_a, 0, 11), -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <20170115222029.8271-1-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>]
* [PATCH 2/2] ARM64: dts: meson-gx: add the missing uart_AO_B [not found] ` <20170115222029.8271-1-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> @ 2017-01-15 22:20 ` Martin Blumenstingl 2017-01-18 10:22 ` Linus Walleij 2017-01-18 21:43 ` Kevin Hilman 0 siblings, 2 replies; 7+ messages in thread From: Martin Blumenstingl @ 2017-01-15 22:20 UTC (permalink / raw) To: linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-gpio-u79uwXL29TY76Z2rM5mHXA, linus.walleij-QSEj5FYQhm4dnm+yROfE0A, khilman-rdvid1DuHRBWk0Htik3J/w Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, carlo-KA+7E9HrN00dnm+yROfE0A, will.deacon-5wv7dgnIgG8, catalin.marinas-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8, robh+dt-DgEjT+Ai2ygdnm+yROfE0A, Martin Blumenstingl This adds the missing node for the uart_AO_B port to the meson-gx.dtsi (as this is supported by GXBB, GXL and GXM) along with the required pinctrl pins. This is required as some boards are using it (the boards from the Khadas VIM series for example have it exposed on the pin headers). Signed-off-by: Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> --- arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 8 ++++++++ arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 7 +++++++ arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 7 +++++++ 3 files changed, 22 insertions(+) diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi index cddad8c795ec..5ece505dca71 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi @@ -324,6 +324,14 @@ status = "disabled"; }; + uart_AO_B: serial@4e0 { + compatible = "amlogic,meson-uart"; + reg = <0x0 0x004e0 0x0 0x14>; + interrupts = <GIC_SPI 197 IRQ_TYPE_EDGE_RISING>; + clocks = <&xtal>; + status = "disabled"; + }; + ir: ir@580 { compatible = "amlogic,meson-gxbb-ir"; reg = <0x0 0x00580 0x0 0x40>; diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi index 5d686334f692..474435e21759 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi @@ -138,6 +138,13 @@ }; }; + uart_ao_b_pins: uart_ao_b { + mux { + groups = "uart_tx_ao_b", "uart_rx_ao_b"; + function = "uart_ao_b"; + }; + }; + remote_input_ao_pins: remote_input_ao { mux { groups = "remote_input_ao"; diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi index bb2842f8a08f..53ed7a5f50ab 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi @@ -82,6 +82,13 @@ }; }; + uart_ao_b_pins: uart_ao_b { + mux { + groups = "uart_tx_ao_b", "uart_rx_ao_b"; + function = "uart_ao_b"; + }; + }; + remote_input_ao_pins: remote_input_ao { mux { groups = "remote_input_ao"; -- 2.11.0 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] ARM64: dts: meson-gx: add the missing uart_AO_B 2017-01-15 22:20 ` [PATCH 2/2] ARM64: dts: meson-gx: add the missing uart_AO_B Martin Blumenstingl @ 2017-01-18 10:22 ` Linus Walleij 2017-01-18 21:43 ` Kevin Hilman 1 sibling, 0 replies; 7+ messages in thread From: Linus Walleij @ 2017-01-18 10:22 UTC (permalink / raw) To: Martin Blumenstingl Cc: Mark Rutland, devicetree@vger.kernel.org, Kevin Hilman, Will Deacon, linux-gpio@vger.kernel.org, Rob Herring, Catalin Marinas, Carlo Caione, open list:ARM/Amlogic Meson..., linux-arm-kernel@lists.infradead.org On Sun, Jan 15, 2017 at 11:20 PM, Martin Blumenstingl <martin.blumenstingl@googlemail.com> wrote: > This adds the missing node for the uart_AO_B port to the meson-gx.dtsi > (as this is supported by GXBB, GXL and GXM) along with the required > pinctrl pins. This is required as some boards are using it (the boards > from the Khadas VIM series for example have it exposed on the pin > headers). > > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> Acked-by: Linus Walleij <linus.walleij@linaro.org> Please funnel this through the apropriate ARM SoC-feed tree. Yours, Linus Walleij ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] ARM64: dts: meson-gx: add the missing uart_AO_B 2017-01-15 22:20 ` [PATCH 2/2] ARM64: dts: meson-gx: add the missing uart_AO_B Martin Blumenstingl 2017-01-18 10:22 ` Linus Walleij @ 2017-01-18 21:43 ` Kevin Hilman 1 sibling, 0 replies; 7+ messages in thread From: Kevin Hilman @ 2017-01-18 21:43 UTC (permalink / raw) To: Martin Blumenstingl Cc: linux-amlogic, linux-gpio, linus.walleij, devicetree, linux-arm-kernel, carlo, will.deacon, catalin.marinas, mark.rutland, robh+dt Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: > This adds the missing node for the uart_AO_B port to the meson-gx.dtsi > (as this is supported by GXBB, GXL and GXM) along with the required > pinctrl pins. This is required as some boards are using it (the boards > from the Khadas VIM series for example have it exposed on the pin > headers). > > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> Applied to v4.11/dt64. Kevin ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-01-18 21:43 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-15 22:20 [PATCH 0/2] add support for uart_AO_B Martin Blumenstingl
2017-01-15 22:20 ` [PATCH 1/2] pinctrl: meson: fix uart_ao_b for GXBB and GXL/GXM Martin Blumenstingl
[not found] ` <20170115222029.8271-2-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>
2017-01-18 10:21 ` Linus Walleij
2017-01-18 21:40 ` Kevin Hilman
[not found] ` <20170115222029.8271-1-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>
2017-01-15 22:20 ` [PATCH 2/2] ARM64: dts: meson-gx: add the missing uart_AO_B Martin Blumenstingl
2017-01-18 10:22 ` Linus Walleij
2017-01-18 21:43 ` Kevin Hilman
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).