From: Inki Dae <inki.dae@samsung.com>
To: Gustavo Padovan <gustavo@padovan.org>
Cc: linux-samsung-soc@vger.kernel.org,
Gustavo Padovan <gustavo.padovan@collabora.co.uk>,
dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 0/7] drm/exynos: clean ups
Date: Thu, 12 Mar 2015 23:42:37 +0900 [thread overview]
Message-ID: <5501A5DD.1040707@samsung.com> (raw)
In-Reply-To: <1424352158-728-1-git-send-email-gustavo@padovan.org>
On 2015년 02월 19일 22:22, Gustavo Padovan wrote:
> From: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
>
> Hi,
>
> Here goes some clean ups to the exynos drivers. The main clean ups is
> the presetting and zpos making the property immutable and the removal
> of *_win_data structures.
With your cleanup patch set, the plane test doesn't work. Have you ever
tested plane feature with modetest application?
I typed like below and it looks like no problem but I cannot see
anything on Panel except for cute four penguins.
# modetest -P 29:100x100+0+0
trying to open device 'i915'...failed.
trying to open device 'radeon'...failed.
trying to open device 'nouveau'...failed.
trying to open device 'vmwgfx'...failed.
trying to open device 'omapdrm'...failed.
trying to open device 'exynos'...success.
testing 100x100@XR24 overlay plane 25
And for two plane test,
# modetest -P 29:100x100+0+0 -P 29:100x100+0+100
trying to open device 'i915'...failed.
trying to open device 'radeon'...failed.
trying to open device 'nouveau'...failed.
trying to open device 'vmwgfx'...failed.
trying to open device 'omapdrm'...failed.
trying to open device 'exynos'...success.
testing 100x100@XR24 overlay plane 25
testing 100x100@XR24 overlay plane 26
Still no screen.
Thanks,
Inki Dae
>
>
> Gustavo Padovan (6):
> drm/exynos: remove unused exynos_crtc->win_enable() callback
> drm/exynos: remove struct *_win_data abstraction on planes
> drm/exynos: preset zpos value for overlay planes
> drm/exynos: make zpos property immutable
> drm/exynos: remove exynos_plane_destroy()
> drm/exynos: remove leftover functions declarations
>
> Mandeep Singh Baines (1):
> drm/exynos: track vblank events on a per crtc basis
>
> drivers/gpu/drm/exynos/exynos7_drm_decon.c | 176 +++++++++--------------
> drivers/gpu/drm/exynos/exynos_drm_crtc.c | 101 ++++++--------
> drivers/gpu/drm/exynos/exynos_drm_crtc.h | 7 +-
> drivers/gpu/drm/exynos/exynos_drm_drv.c | 27 ----
> drivers/gpu/drm/exynos/exynos_drm_drv.h | 20 +--
> drivers/gpu/drm/exynos/exynos_drm_fimd.c | 195 ++++++++++----------------
> drivers/gpu/drm/exynos/exynos_drm_plane.c | 66 +++------
> drivers/gpu/drm/exynos/exynos_drm_plane.h | 7 +-
> drivers/gpu/drm/exynos/exynos_drm_vidi.c | 134 +++++-------------
> drivers/gpu/drm/exynos/exynos_mixer.c | 217 ++++++++++-------------------
> 10 files changed, 311 insertions(+), 639 deletions(-)
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2015-03-12 14:42 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-19 13:22 [PATCH 0/7] drm/exynos: clean ups Gustavo Padovan
2015-02-19 13:22 ` [PATCH 1/7] drm/exynos: remove unused exynos_crtc->win_enable() callback Gustavo Padovan
2015-02-19 13:22 ` [PATCH 2/7] drm/exynos: remove struct *_win_data abstraction on planes Gustavo Padovan
2015-02-19 13:22 ` [PATCH 3/7] drm/exynos: preset zpos value for overlay planes Gustavo Padovan
2015-02-19 13:22 ` [PATCH 4/7] drm/exynos: make zpos property immutable Gustavo Padovan
2015-03-11 15:10 ` Inki Dae
2015-02-19 13:22 ` [PATCH 5/7] drm/exynos: remove exynos_plane_destroy() Gustavo Padovan
2015-02-19 13:22 ` [PATCH 6/7] drm/exynos: remove leftover functions declarations Gustavo Padovan
2015-02-19 13:22 ` [PATCH 7/7] drm/exynos: track vblank events on a per crtc basis Gustavo Padovan
2015-03-12 14:42 ` Inki Dae [this message]
2015-03-12 14:57 ` [PATCH 0/7] drm/exynos: clean ups Gustavo Padovan
2015-03-12 18:45 ` Gustavo Padovan
2015-03-13 2:33 ` Inki Dae
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5501A5DD.1040707@samsung.com \
--to=inki.dae@samsung.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=gustavo.padovan@collabora.co.uk \
--cc=gustavo@padovan.org \
--cc=linux-samsung-soc@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.