From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthias Brugger Subject: Re: [PATCH v1 08/19] arm: dts: mt7623: enable three available UARTs on bananapi-r2 Date: Mon, 19 Mar 2018 00:49:34 +0100 Message-ID: <32322310-2279-03bb-3023-f3803f57b050@gmail.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: sean.wang@mediatek.com, robh+dt@kernel.org, mark.rutland@arm.com, devicetree@vger.kernel.org, linux-mediatek@lists.infradead.org Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org List-Id: devicetree@vger.kernel.org On 02/23/2018 11:16 AM, sean.wang@mediatek.com wrote: > From: Sean Wang > > On bpi-r2 board, totally there're four UARTs which we usually called > uart[0-3] helpful to extend slow-I/O devices. Among those ones, uart2 has > dedicated pin slot which is used to console log. uart[0-1] appear at the > 40-pins connector and uart3 has no pinout, but just has test points (TP47 > for TX and TP48 for RX, respectively) nearby uart2, but we don't enable > uart3 in the patch. The missing pinctrl is also being supplemented for > those newly added devices. > > Signed-off-by: Sean Wang Pushed to v4.16-next/dts32 Thanks! > --- > arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts | 13 +++++++++++-- > 1 file changed, 11 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts > index 8ef91d6..6b760be 100644 > --- a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts > +++ b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts > @@ -470,6 +470,13 @@ > ; > }; > }; > + > + uart2_pins_a: uart@2 { > + pins_dat { > + pinmux = , > + ; > + }; > + }; > }; > > &pwm { > @@ -515,16 +522,18 @@ > &uart0 { > pinctrl-names = "default"; > pinctrl-0 = <&uart0_pins_a>; > - status = "disabled"; > + status = "okay"; > }; > > &uart1 { > pinctrl-names = "default"; > pinctrl-0 = <&uart1_pins_a>; > - status = "disabled"; > + status = "okay"; > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_pins_a>; > status = "okay"; > }; > >