Hi Mr. Paul,

On Wed, Feb 6, 2013 at 3:20 PM, Paul Menzel
<paulepan...@users.sourceforge.net> wrote:
> Am Mittwoch, den 06.02.2013, 09:54 +0530 schrieb Vikas Sajjan:
>> Add support for parsing the display-timing node using video helper
>> function.
>>
>> The DT node parsing and pinctrl selection is done only if 'dev.of_node'
>> exists and the NON-DT logic is still maintained under the 'else' part.
>
> Last thing. It definitely helps review if you add instructions how to
> test your change. Is there some user space command to check if the
> display-timing node parsing works?
>

It is tested on Exynos4412 board by applying dependent patches available at
http://lists.freedesktop.org/archives/dri-devel/2013-January/033998.html
simply test would be to enable "boot logo" in make menuconfig, and
verify whether the logo the appears or not during bootup.

>> Signed-off-by: Leela Krishna Amudala <l.kris...@samsung.com>
>> Signed-off-by: Vikas Sajjan <vikas.saj...@linaro.org>
>> ---
>>  drivers/gpu/drm/exynos/exynos_drm_fimd.c |   41 
>> +++++++++++++++++++++++++++---
>>  1 file changed, 37 insertions(+), 4 deletions(-)
>
> […]
>
>
> Thanks,
>
> Paul
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to