| Message ID | 1367992894-20198-4-git-send-email-vikas.sajjan@linaro.org |
|---|---|
| State | New |
| Headers | show |
Hi Vikas, On Wednesday 08 of May 2013 11:31:34 Vikas Sajjan wrote: > Adds display timing node for a DP panel to Arndale Board DTS file > > Signed-off-by: Vikas Sajjan <vikas.sajjan@linaro.org> > --- > arch/arm/boot/dts/exynos5250-arndale.dts | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/arch/arm/boot/dts/exynos5250-arndale.dts > b/arch/arm/boot/dts/exynos5250-arndale.dts index f68b820..c831a5c > 100644 > --- a/arch/arm/boot/dts/exynos5250-arndale.dts > +++ b/arch/arm/boot/dts/exynos5250-arndale.dts > @@ -459,4 +459,20 @@ > samsung,lane-count = <4>; > }; > > + display-timings { > + native-mode = <&timing0>; > + timing0: timing@0 { > + /* 2560x1600 DP panel */ > + clock-frequency = <50000>; > + hactive = <2560>; > + vactive = <1600>; > + hfront-porch = <48>; > + hback-porch = <80>; > + hsync-len = <32>; > + vback-porch = <16>; > + vfront-porch = <8>; > + vsync-len = <6>; > + }; > + }; What display are those timings used for? Shouldn't they be placed inside node of that display? Best regards, Tomasz
Hi Tomasz, On 10 May 2013 05:35, Tomasz Figa <tomasz.figa@gmail.com> wrote: > Hi Vikas, > > On Wednesday 08 of May 2013 11:31:34 Vikas Sajjan wrote: > > Adds display timing node for a DP panel to Arndale Board DTS file > > > > Signed-off-by: Vikas Sajjan <vikas.sajjan@linaro.org> > > --- > > arch/arm/boot/dts/exynos5250-arndale.dts | 16 ++++++++++++++++ > > 1 file changed, 16 insertions(+) > > > > diff --git a/arch/arm/boot/dts/exynos5250-arndale.dts > > b/arch/arm/boot/dts/exynos5250-arndale.dts index f68b820..c831a5c > > 100644 > > --- a/arch/arm/boot/dts/exynos5250-arndale.dts > > +++ b/arch/arm/boot/dts/exynos5250-arndale.dts > > @@ -459,4 +459,20 @@ > > samsung,lane-count = <4>; > > }; > > > > + display-timings { > > + native-mode = <&timing0>; > > + timing0: timing@0 { > > + /* 2560x1600 DP panel */ > > + clock-frequency = <50000>; > > + hactive = <2560>; > > + vactive = <1600>; > > + hfront-porch = <48>; > > + hback-porch = <80>; > > + hsync-len = <32>; > > + vback-porch = <16>; > > + vfront-porch = <8>; > > + vsync-len = <6>; > > + }; > > + }; > > What display are those timings used for? Shouldn't they be placed inside > node of that display? > > This timing information will be used parsed by the FIMD probe with the help of "Video Helper function" as done in this link https://git.kernel.org/cgit/linux/kernel/git/daeinki/drm-exynos.git/commit/?h=exynos-drm-next&id=7f4596f4aebcc9fcf2d50d3fe398508d710c4dd0 Please refer to this patch which is used for exynos4412-origen and exynos5250-smdk5250 respectively in similar way https://git.kernel.org/cgit/linux/kernel/git/kgene/linux-samsung.git/commit/?h=for-next&id=0207775d6ff7e6a6eddb9931f9328f0f0173a338 https://git.kernel.org/cgit/linux/kernel/git/kgene/linux-samsung.git/commit/?h=for-next&id=06c460b73f75894cabfb1f5277f27cddbc92745c Best regards, > Tomasz > >
diff --git a/arch/arm/boot/dts/exynos5250-arndale.dts b/arch/arm/boot/dts/exynos5250-arndale.dts index f68b820..c831a5c 100644 --- a/arch/arm/boot/dts/exynos5250-arndale.dts +++ b/arch/arm/boot/dts/exynos5250-arndale.dts @@ -459,4 +459,20 @@ samsung,lane-count = <4>; }; + display-timings { + native-mode = <&timing0>; + timing0: timing@0 { + /* 2560x1600 DP panel */ + clock-frequency = <50000>; + hactive = <2560>; + vactive = <1600>; + hfront-porch = <48>; + hback-porch = <80>; + hsync-len = <32>; + vback-porch = <16>; + vfront-porch = <8>; + vsync-len = <6>; + }; + }; + };
Adds display timing node for a DP panel to Arndale Board DTS file Signed-off-by: Vikas Sajjan <vikas.sajjan@linaro.org> --- arch/arm/boot/dts/exynos5250-arndale.dts | 16 ++++++++++++++++ 1 file changed, 16 insertions(+)