From mboxrd@z Thu Jan 1 00:00:00 1970 From: peter.griffin@linaro.org (Peter Griffin) Date: Wed, 25 Jan 2017 11:54:26 +0000 Subject: [STLinux Kernel] [PATCH 7/8] ARM: dts: STiH407-family: Use new Pinctrl groups In-Reply-To: <20170124134310.27512-8-lee.jones@linaro.org> References: <20170124134310.27512-1-lee.jones@linaro.org> <20170124134310.27512-8-lee.jones@linaro.org> Message-ID: <20170125115426.GH5680@griffinp-ThinkPad-X1-Carbon-2nd> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Lee, On Tue, 24 Jan 2017, Lee Jones wrote: > Having just defined some new Pinctrl groups for when when HW flow- > control is {en,dis}abled, let's reference them for use within the > driver. > > Signed-off-by: Lee Jones Same as previous comment, your enabling hw flow control for all stih407 family boards. I've not checked the schematics for them but as hw flow control is dependent on board wiring, IMO this should be added in board specific file. regards, Peter. > --- > arch/arm/boot/dts/stih407-family.dtsi | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi > index c8b2944..9789978 100644 > --- a/arch/arm/boot/dts/stih407-family.dtsi > +++ b/arch/arm/boot/dts/stih407-family.dtsi > @@ -222,8 +222,9 @@ > compatible = "st,asc"; > reg = <0x9830000 0x2c>; > interrupts = ; > - pinctrl-names = "default"; > - pinctrl-0 = <&pinctrl_serial0>; > + pinctrl-names = "default", "manual-rts"; > + pinctrl-0 = <&pinctrl_serial0_flowctrl>; > + pinctrl-1 = <&pinctrl_serial0>; > clocks = <&clk_s_c0_flexgen CLK_EXT2F_A9>; > > status = "disabled";