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 linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
