Hi Tomasz, > -----Original Message----- > From: Tomasz Figa [mailto:t.figa at samsung.com] > Sent: Friday, August 09, 2013 9:57 PM > To: Chanho Park > Cc: inki.dae at samsung.com; kgene.kim at samsung.com; dri- > devel at lists.freedesktop.org; kyungmin.park at samsung.com; > mark.rutland at arm.com; l.stach at pengutronix.de; s.nawrocki at samsung.com; > tomasz.figa at gmail.com; linux-samsung-soc at vger.kernel.org; linux-arm- > kernel at lists.infradead.org; devicetree at vger.kernel.org; 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 at samsung.com> > > Cc: Thomas Abraham <thomas.abraham at linaro.org> > > Cc: Kukjin Kim <kgene.kim at samsung.com> > > Cc: Inki Dae <inki.dae at samsung.com> > > Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com> > > --- > > 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 at 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