Tomasz Figa wrote: > > Hi, > Hi, > On 02.05.2014 17:44, Doug Anderson wrote: > > Arun, > > > > On Fri, May 2, 2014 at 5:48 AM, Arun Kumar K <arun.kk@xxxxxxxxxxx> wrote: > >> Adds the PWM nodes to 5420 pinctrl dtsi file. > >> > >> Signed-off-by: Arun Kumar K <arun.kk@xxxxxxxxxxx> > >> --- > >> arch/arm/boot/dts/exynos5420-pinctrl.dtsi | 28 > ++++++++++++++++++++++++++++ > >> 1 file changed, 28 insertions(+) > >> > >> diff --git a/arch/arm/boot/dts/exynos5420-pinctrl.dtsi > b/arch/arm/boot/dts/exynos5420-pinctrl.dtsi > >> index e62c8eb..ba686e4 100644 > >> --- a/arch/arm/boot/dts/exynos5420-pinctrl.dtsi > >> +++ b/arch/arm/boot/dts/exynos5420-pinctrl.dtsi > >> @@ -624,6 +624,34 @@ > >> samsung,pin-drv = <0>; > >> }; > >> > >> + pwm0_out: pwm0-out { > >> + samsung,pins = "gpb2-0"; > >> + samsung,pin-function = <2>; > >> + samsung,pin-pud = <0>; > >> + samsung,pin-drv = <0>; > >> + }; > >> + > >> + pwm1_out: pwm1-out { > >> + samsung,pins = "gpb2-1"; > >> + samsung,pin-function = <2>; > >> + samsung,pin-pud = <0>; > >> + samsung,pin-drv = <0>; > >> + }; > >> + > >> + pwm2_out: pwm2-out { > >> + samsung,pins = "gpb2-2"; > >> + samsung,pin-function = <2>; > >> + samsung,pin-pud = <0>; > >> + samsung,pin-drv = <0>; > >> + }; > >> + > >> + pwm3_out: pwm3-out { > >> + samsung,pins = "gpb2-3"; > >> + samsung,pin-function = <2>; > >> + samsung,pin-pud = <0>; > >> + samsung,pin-drv = <0>; > >> + }; > >> + > >> i2c7_hs_bus: i2c7-hs-bus { > >> samsung,pins = "gpb2-2", "gpb2-3"; > >> samsung,pin-function = <3>; > > > > Possibly pwm3-out belongs below this one (since the start pin is > > "larger" than the start pin of i2c7-hs-bus). Tomasz probably has a > > definite opinion on this. > > Well, I'm not opposed to keeping it as is. Having all the pwm nodes > together might be even more readable. > +1 ;-) > So feel free to add my Reviewed-by tag. > Thanks for you two guys' review. Applied, thanks. - Kukjin -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html