All of lore.kernel.org
 help / color / mirror / Atom feed
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 -v3 00/11] drm/exynos: Add atomic modesetting support
Date: Mon, 06 Apr 2015 19:46:42 +0900	[thread overview]
Message-ID: <55226412.8040807@samsung.com> (raw)
In-Reply-To: <1428084583-14616-1-git-send-email-gustavo@padovan.org>

On 2015년 04월 04일 03:09, Gustavo Padovan wrote:
> From: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> 
> Hi,
> 
> Here goes the full support for atomic modesetting on exynos. I've
> split the patches in the various phases of atomic support.
> 
> These patches sits on top of the clean up patches I've sent yesterday
> to this mailing list[1].
> 
> v2: fixes comments by Joonyoung
>         - remove unused var in patch 09
>         - use ->disable instead of outdated ->dpms in hdmi code
>         - remove WARN_ON from crtc enable/disable
> 
> v3: fixes comment by Joonyoung
> 	- move the removal of drm_helper_disable_unused_functions() to 
> 	separated patch

With this patch series, Kernel booting is halted at end of kernel
booting. I tested this patch series on Trats2 board based on Exynos4412 SoC.

Below is a part of full booting logs, which was halted,
[    1.992015] exynos-drm-ipp exynos-drm-ipp: drm ipp registered
successfully.
[    1.993009] exynos-drm exynos-drm: bound exynos-drm-vidi (ops
vidi_component_ops)
[    1.993036] exynos-drm exynos-drm: bound 11c00000.fimd (ops
fimd_component_ops)
[    1.993385] exynos-drm exynos-drm: bound 11c80000.dsi (ops
exynos_dsi_component_ops)
[    1.993390] [drm] Supports vblank timestamp caching Rev 2 (21.10.2013).
[    1.993393] [drm] No driver support for vblank timestamp query.
[    1.993442] [drm] Initialized exynos 1.0.0 20110530 on minor 0
[    2.043358] WARNING: CPU: 2 PID: 1209 at drivers/clk/clk.c:898
clk_unprepare+0x24/0x2c()
[    2.051412] Modules linked in:
[    2.054422] CPU: 2 PID: 1209 Comm: kworker/2:1 Tainted: G        W
    4.0.0-rc6-00526-gc49d7de-dirty #1278
[    2.064337] Hardware name: SAMSUNG EXYNOS (Flattened Device Tree)
[    2.070428] Workqueue: pm pm_runtime_work>\x1e\x02

After that, I tested it again without FIMD and the booting is ok. So I
guess that this atomic feature has a bug to FIMD driver.

Thanks,
Inki Dae

> 
> 	Gustavo
> ---
> 
> Gustavo Padovan (11):
>   drm/exynos: atomic phase 1: use drm_plane_helper_update()
>   drm/exynos: atomic phase 1: use drm_plane_helper_disable()
>   drm/exynos: atomic phase 1: add .mode_set_nofb() callback
>   drm/exynos: atomic phase 2: wire up state reset(), duplicate() and
>     destroy()
>   drm/exynos: atomic phase 2: keep track of framebuffer pointer
>   drm/exynos: atomic phase 3: atomic updates of planes
>   drm/exynos: atomic phase 3: use atomic .set_config helper
>   drm/exynos: atomic phase 3: convert page flips
>   drm/exynos: remove exported functions from exynos_drm_plane
>   drm/exynos: don't disable unused functions at init
>   drm/exynos: atomic dpms support
> 
>  drivers/gpu/drm/bridge/ptn3460.c              |   4 +
>  drivers/gpu/drm/exynos/exynos_dp_core.c       |   6 +-
>  drivers/gpu/drm/exynos/exynos_drm_connector.c |   6 +-
>  drivers/gpu/drm/exynos/exynos_drm_crtc.c      | 226 ++++++++------------------
>  drivers/gpu/drm/exynos/exynos_drm_dpi.c       |   6 +-
>  drivers/gpu/drm/exynos/exynos_drm_drv.c       |   2 +
>  drivers/gpu/drm/exynos/exynos_drm_drv.h       |   4 +-
>  drivers/gpu/drm/exynos/exynos_drm_dsi.c       |   6 +-
>  drivers/gpu/drm/exynos/exynos_drm_encoder.c   |  27 +--
>  drivers/gpu/drm/exynos/exynos_drm_fb.c        |  12 +-
>  drivers/gpu/drm/exynos/exynos_drm_fbdev.c     |   3 -
>  drivers/gpu/drm/exynos/exynos_drm_plane.c     | 113 +++++++------
>  drivers/gpu/drm/exynos/exynos_drm_plane.h     |  11 --
>  drivers/gpu/drm/exynos/exynos_drm_vidi.c      |   6 +-
>  drivers/gpu/drm/exynos/exynos_hdmi.c          |  10 +-
>  15 files changed, 187 insertions(+), 255 deletions(-)
> 

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

  parent reply	other threads:[~2015-04-06 10:46 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-03 18:09 [PATCH -v3 00/11] drm/exynos: Add atomic modesetting support Gustavo Padovan
2015-04-03 18:09 ` [PATCH -v3 01/11] drm/exynos: atomic phase 1: use drm_plane_helper_update() Gustavo Padovan
2015-04-03 18:09 ` [PATCH -v3 02/11] drm/exynos: atomic phase 1: use drm_plane_helper_disable() Gustavo Padovan
2015-04-03 18:09 ` [PATCH -v3 03/11] drm/exynos: atomic phase 1: add .mode_set_nofb() callback Gustavo Padovan
2015-04-03 18:09 ` [PATCH -v3 04/11] drm/exynos: atomic phase 2: wire up state reset(), duplicate() and destroy() Gustavo Padovan
2015-04-03 18:09 ` [PATCH -v3 05/11] drm/exynos: atomic phase 2: keep track of framebuffer pointer Gustavo Padovan
2015-04-03 18:09 ` [PATCH -v3 06/11] drm/exynos: atomic phase 3: atomic updates of planes Gustavo Padovan
2015-04-03 18:09 ` [PATCH -v3 07/11] drm/exynos: atomic phase 3: use atomic .set_config helper Gustavo Padovan
2015-04-03 18:09 ` [PATCH -v3 08/11] drm/exynos: atomic phase 3: convert page flips Gustavo Padovan
2015-04-03 18:09 ` [PATCH -v3 09/11] drm/exynos: remove exported functions from exynos_drm_plane Gustavo Padovan
2015-04-03 18:09 ` [PATCH -v3 10/11] drm/exynos: don't disable unused functions at init Gustavo Padovan
2015-04-03 18:09 ` [PATCH -v3 11/11] drm/exynos: atomic dpms support Gustavo Padovan
2015-04-05 14:42   ` Inki Dae
2015-04-06 18:26     ` Gustavo Padovan
2015-04-07  7:22       ` Joonyoung Shim
2015-04-07 19:56         ` [PATCH v4] " Gustavo Padovan
2015-04-10 18:57   ` [PATCH] drm/exynos: remove unnecessary calls to disable_plane() Gustavo Padovan
2015-04-13  6:32     ` Joonyoung Shim
2015-04-13 17:35       ` Gustavo Padovan
2015-04-06 10:46 ` Inki Dae [this message]
2015-04-06 15:44   ` [PATCH -v3 00/11] drm/exynos: Add atomic modesetting support Inki Dae
2015-04-07  7:06     ` Inki Dae
2015-04-07 11:14       ` Inki Dae
2015-04-07 17:24         ` Gustavo Padovan
2015-04-07 18:39       ` Gustavo Padovan
2015-04-07 19:14         ` Tobias Jakobi
2015-04-08 15:14         ` Inki Dae
2015-04-09  7:04           ` Joonyoung Shim

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=55226412.8040807@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.