All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrzej Hajda <a.hajda@samsung.com>
To: Inki Dae <inki.dae@samsung.com>
Cc: linux-samsung-soc@vger.kernel.org,
	Marek Szyprowski <m.szyprowski@samsung.com>,
	dri-devel@lists.freedesktop.org,
	Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Subject: [PATCH 04/23] drm/exynos: add exynos_drm_crtc_init function
Date: Fri, 01 Mar 2019 13:20:36 +0100	[thread overview]
Message-ID: <20190301122055.7135-5-a.hajda@samsung.com> (raw)
In-Reply-To: <20190301122055.7135-1-a.hajda@samsung.com>

Since exynos_drm_crtc is a struct which maps 1:1 to underlying device it
is better to put it directly into device's context instead of allocating
it separately. Another benefit is possibility of initialisation of
its fields directly, without expanding exynos_drm_crtc_create which is
already overloaded with number of arguments.
As a first step of the transition initialisation function should be
created.

Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
---
 drivers/gpu/drm/exynos/exynos_drm_crtc.c | 10 ++++++++++
 drivers/gpu/drm/exynos/exynos_drm_crtc.h |  2 ++
 2 files changed, 12 insertions(+)

diff --git a/drivers/gpu/drm/exynos/exynos_drm_crtc.c b/drivers/gpu/drm/exynos/exynos_drm_crtc.c
index 1eebfa3fa8da..4ff1e0ff2255 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_crtc.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_crtc.c
@@ -172,6 +172,16 @@ static const struct drm_crtc_funcs exynos_crtc_funcs = {
 	.disable_vblank = exynos_drm_crtc_disable_vblank,
 };
 
+int exynos_drm_crtc_init(struct exynos_drm_crtc *exynos_crtc,
+			 struct drm_device *drm_dev)
+{
+	struct drm_crtc *crtc = &exynos_crtc->base;
+
+	drm_crtc_helper_add(crtc, &exynos_crtc_helper_funcs);
+	return drm_crtc_init_with_planes(drm_dev, crtc, NULL, NULL,
+					 &exynos_crtc_funcs, NULL);
+}
+
 struct exynos_drm_crtc *exynos_drm_crtc_create(struct drm_device *drm_dev,
 					struct drm_plane *plane,
 					enum exynos_drm_output_type type,
diff --git a/drivers/gpu/drm/exynos/exynos_drm_crtc.h b/drivers/gpu/drm/exynos/exynos_drm_crtc.h
index dec446109e6c..9e35a9e02332 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_crtc.h
+++ b/drivers/gpu/drm/exynos/exynos_drm_crtc.h
@@ -18,6 +18,8 @@
 
 #include "exynos_drm_drv.h"
 
+int exynos_drm_crtc_init(struct exynos_drm_crtc *exynos_crtc,
+			 struct drm_device *drm_dev);
 struct exynos_drm_crtc *exynos_drm_crtc_create(struct drm_device *drm_dev,
 					struct drm_plane *plane,
 					enum exynos_drm_output_type out_type,
-- 
2.17.1

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

  parent reply	other threads:[~2019-03-01 12:20 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20190301122106eucas1p2dc9cbc6d822965bfa3cd2de83f3a7812@eucas1p2.samsung.com>
2019-03-01 12:20 ` [PATCH 00/23] drm/exynos: add support for GSCALER planes on Exynos5433 Andrzej Hajda
     [not found]   ` <CGME20190301122106eucas1p2abe70ca71f4cab803c0edd88ea46996e@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 01/23] drm/exynos: remove exynos_drm_plane.h header Andrzej Hajda
     [not found]   ` <CGME20190301122107eucas1p2a284b545b5b379e61dd960ab66ae2bdb@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 02/23] drm/exynos: remove spare macro Andrzej Hajda
     [not found]   ` <CGME20190301122107eucas1p2d2defd88a0375c1a2dc00860d43aa7fd@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 03/23] drm/exynos: drop exynos_drm_plane_config structure Andrzej Hajda
     [not found]   ` <CGME20190301122107eucas1p2b546a8f156d54c45669bfa9724c4b8c2@eucas1p2.samsung.com>
2019-03-01 12:20     ` Andrzej Hajda [this message]
     [not found]   ` <CGME20190301122107eucas1p21e887d2025b970939cabe52ee215b9fe@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 05/23] drm/exynos/decon5433: embed exynos_drm_crtc directly into context Andrzej Hajda
     [not found]   ` <CGME20190301122108eucas1p2b52e33f1799eefb8ac68d7c4063bb2b9@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 06/23] drm/exynos/decon7: " Andrzej Hajda
     [not found]   ` <CGME20190301122108eucas1p25f8fcb147e5dd0be0d877c55effa7e7e@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 07/23] drm/exynos/fimd: " Andrzej Hajda
     [not found]   ` <CGME20190301122108eucas1p2ef7114bd5e714ce38ff070e404fd51a2@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 08/23] drm/exynos/mixer: " Andrzej Hajda
     [not found]   ` <CGME20190301122109eucas1p22f304f014932064d4b02caa01b4583ac@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 09/23] drm/exynos/vidi: " Andrzej Hajda
     [not found]   ` <CGME20190301122109eucas1p28ae2e947e12d640d1932895fc4d3c14a@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 10/23] drm/exynos: remove standalone exynos_drm_crtc leftovers Andrzej Hajda
     [not found]   ` <CGME20190301122110eucas1p2506e2cc528823cb2276d757ad49cc396@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 11/23] drm/exynos/vidi: remove encoder_to_vidi helper Andrzej Hajda
     [not found]   ` <CGME20190301122110eucas1p2e11aa4aac815820d1bcfc5dae485f060@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 12/23] drm/exynos: unify plane type assignment Andrzej Hajda
     [not found]   ` <CGME20190301122110eucas1p2107b2c2837e9bad58bf21b7869b7e0eb@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 13/23] drm/exynos: set primary and cursor planes in exynos_drm_crtc_init Andrzej Hajda
     [not found]   ` <CGME20190301122111eucas1p2cb70cdfa7e8c1ad6e32af695a2f88b10@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 14/23] drm/exynos: add plane update/disable callbacks for planes Andrzej Hajda
     [not found]   ` <CGME20190301122111eucas1p2c0184ad8117054056ac10e6afd949a9f@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 15/23] drm/exynos: add GSCALER plane capability Andrzej Hajda
     [not found]   ` <CGME20190301122112eucas1p2c0099b31af68e2c72c07640ab994123a@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 16/23] drm/exynos/gscaler: fix id assignement Andrzej Hajda
     [not found]   ` <CGME20190301122112eucas1p2fe4f1ab4dfc3026f1a64bb07c649a590@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 17/23] arm64: dts: exynos: configure GSCALER related clocks Andrzej Hajda
     [not found]   ` <CGME20190301122112eucas1p2d389c3d94c691507d5ac2c3d224e2e32@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 18/23] arm64: dts: exynos: add DSD/GSD clocks to DECONs and GSCALERs Andrzej Hajda
2019-03-05  9:28       ` Krzysztof Kozlowski
2019-03-05  9:36         ` Andrzej Hajda
2019-03-05  9:37           ` Krzysztof Kozlowski
     [not found]   ` <CGME20190301122113eucas1p2c3cce6f8651859bdef9785e3c7042efc@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 19/23] drm/exynos/gscaler: add local path support Andrzej Hajda
     [not found]   ` <CGME20190301122113eucas1p2bd3c00441025c99aa20e933441bd3f21@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 20/23] drm/exynos/decon5433: " Andrzej Hajda
     [not found]   ` <CGME20190301122113eucas1p25f441847f19e4d1a355bffd7633dcd5d@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 21/23] drm/exynos/decon5433: wait for finish previous update Andrzej Hajda
     [not found]   ` <CGME20190301122114eucas1p2f8722a93496077bfa60c7865781763b9@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 22/23] drm/exynos/gscaler: change supported color format BGRX8888 to XBGR8888 Andrzej Hajda
     [not found]   ` <CGME20190301122114eucas1p2c257303e00546f399ee3e5f97962c28c@eucas1p2.samsung.com>
2019-03-01 12:20     ` [PATCH 23/23] drm/exynos/gscaler: fix handling YVU420 pixel format Andrzej Hajda

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=20190301122055.7135-5-a.hajda@samsung.com \
    --to=a.hajda@samsung.com \
    --cc=b.zolnierkie@samsung.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=inki.dae@samsung.com \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=m.szyprowski@samsung.com \
    /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.