Hi Tomasz, > -----Original Message----- > From: Tomasz Figa [mailto:t.figa@xxxxxxxxxxx] > Sent: Friday, August 09, 2013 9:57 PM > To: Chanho Park > Cc: inki.dae@xxxxxxxxxxx; kgene.kim@xxxxxxxxxxx; dri- > devel@xxxxxxxxxxxxxxxxxxxxx; kyungmin.park@xxxxxxxxxxx; > mark.rutland@xxxxxxx; l.stach@xxxxxxxxxxxxxx; s.nawrocki@xxxxxxxxxxx; > tomasz.figa@xxxxxxxxx; linux-samsung-soc@xxxxxxxxxxxxxxx; linux-arm- > kernel@xxxxxxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; Thomas Abraham > Subject: Re: [PATCHv2 2/5] ARM: dts: Add rotator node for exynos4210 > > On Friday 09 of August 2013 16:40:50 Chanho Park wrote: > > This patch adds a rotator node for exynos4210. The exynos4210 has > > different limitation of image size compared with later chips. > > > > Signed-off-by: Chanho Park <chanho61.park@xxxxxxxxxxx> > > Cc: Thomas Abraham <thomas.abraham@xxxxxxxxxx> > > Cc: Kukjin Kim <kgene.kim@xxxxxxxxxxx> > > Cc: Inki Dae <inki.dae@xxxxxxxxxxx> > > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > > --- > > arch/arm/boot/dts/exynos4.dtsi | 9 +++++++++ > > 1 file changed, 9 insertions(+) > > > > diff --git a/arch/arm/boot/dts/exynos4.dtsi > > b/arch/arm/boot/dts/exynos4.dtsi index 597cfcf..002b2b8 100644 > > --- a/arch/arm/boot/dts/exynos4.dtsi > > +++ b/arch/arm/boot/dts/exynos4.dtsi > > @@ -243,6 +243,15 @@ > > status = "disabled"; > > }; > > > > + rotator: rotator@12810000 { > > + compatible = "samsung,exynos4210-rotator"; > > + reg = <0x12810000 0x1000>; > > + interrupts = <0 83 0>; > > + clocks = <&clock 278>; > > + clock-names = "rotator"; > > + status = "disabled"; > > Does the rotator need any board-specific information to work? If no, it > should not be disabled by default. Same comment goes to patch 4/5. It does not have any board-specific information. I'll remove the status node. Thanks. Best Regards, Chanho Park -- 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
