All of lore.kernel.org
 help / color / mirror / Atom feed
From: Inki Dae <inki.dae@samsung.com>
To: Andrzej Hajda <a.hajda@samsung.com>
Cc: Kukjin Kim <kgene.kim@samsung.com>,
	Seung-Woo Kim <sw0312.kim@samsung.com>,
	open list <linux-kernel@vger.kernel.org>,
	"open list:DRM DRIVERS FOR E..."
	<dri-devel@lists.freedesktop.org>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	"moderated list:ARM/S5P EXYNOS AR..."
	<linux-samsung-soc@vger.kernel.org>,
	Marek Szyprowski <m.szyprowski@samsung.com>
Subject: Re: [PATCH 0/4] drm/exynos/dsi: remove global display variable
Date: Thu, 13 Nov 2014 17:50:04 +0900	[thread overview]
Message-ID: <546470BC.4020104@samsung.com> (raw)
In-Reply-To: <1412683271-15835-1-git-send-email-a.hajda@samsung.com>

On 2014년 10월 07일 21:01, Andrzej Hajda wrote:
> Hi Inki,
> 
> Many Exynos DRM drivers uses global variables to represent associated devices
> in Exynos DRM internal framework. It is quite confusing, it adds data duplication
> and finally it does not allow to handle more than one device in system.
> It seems better to embed such structures in private context of the device.
> 
> The patchset is based on exynos_drm_next plus my patch [1]:
> 'drm/exynos: remove explicit encoder/connector de-initialization'.
> 
> If the patchset is OK for you I can prepare similar patches for other Exynos DRM components.

Sorry for late. Applied. Can you prepare similar patches for other? If
so, I'd happy.

Thanks,
Inki Dae

> 
> [1]: https://lkml.org/lkml/2014/9/22/148
> 
> Regards
> Andrzej
> 
> 
> Andrzej Hajda (4):
>   drm/exynos/dsi: remove global variable exynos_dsi_display
>   drm/exynos/dsi: simplify device pointer evaluation
>   drm/exynos/dsi: remove redundant encoder field
>   drm/exynos/dsi: stop using display->ctx pointer
> 
>  drivers/gpu/drm/exynos/exynos_drm_dsi.c | 96 ++++++++++++++++-----------------
>  1 file changed, 47 insertions(+), 49 deletions(-)
> 

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

WARNING: multiple messages have this Message-ID (diff)
From: Inki Dae <inki.dae@samsung.com>
To: Andrzej Hajda <a.hajda@samsung.com>
Cc: Marek Szyprowski <m.szyprowski@samsung.com>,
	Joonyoung Shim <jy0922.shim@samsung.com>,
	Seung-Woo Kim <sw0312.kim@samsung.com>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	Kukjin Kim <kgene.kim@samsung.com>,
	"open list:DRM DRIVERS FOR E..."
	<dri-devel@lists.freedesktop.org>,
	"moderated list:ARM/S5P EXYNOS AR..." 
	<linux-samsung-soc@vger.kernel.org>,
	open list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 0/4] drm/exynos/dsi: remove global display variable
Date: Thu, 13 Nov 2014 17:50:04 +0900	[thread overview]
Message-ID: <546470BC.4020104@samsung.com> (raw)
In-Reply-To: <1412683271-15835-1-git-send-email-a.hajda@samsung.com>

On 2014년 10월 07일 21:01, Andrzej Hajda wrote:
> Hi Inki,
> 
> Many Exynos DRM drivers uses global variables to represent associated devices
> in Exynos DRM internal framework. It is quite confusing, it adds data duplication
> and finally it does not allow to handle more than one device in system.
> It seems better to embed such structures in private context of the device.
> 
> The patchset is based on exynos_drm_next plus my patch [1]:
> 'drm/exynos: remove explicit encoder/connector de-initialization'.
> 
> If the patchset is OK for you I can prepare similar patches for other Exynos DRM components.

Sorry for late. Applied. Can you prepare similar patches for other? If
so, I'd happy.

Thanks,
Inki Dae

> 
> [1]: https://lkml.org/lkml/2014/9/22/148
> 
> Regards
> Andrzej
> 
> 
> Andrzej Hajda (4):
>   drm/exynos/dsi: remove global variable exynos_dsi_display
>   drm/exynos/dsi: simplify device pointer evaluation
>   drm/exynos/dsi: remove redundant encoder field
>   drm/exynos/dsi: stop using display->ctx pointer
> 
>  drivers/gpu/drm/exynos/exynos_drm_dsi.c | 96 ++++++++++++++++-----------------
>  1 file changed, 47 insertions(+), 49 deletions(-)
> 


  parent reply	other threads:[~2014-11-13  8:50 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-07 12:01 [PATCH 0/4] drm/exynos/dsi: remove global display variable Andrzej Hajda
2014-10-07 12:01 ` Andrzej Hajda
2014-10-07 12:01 ` [PATCH 1/4] drm/exynos/dsi: remove global variable exynos_dsi_display Andrzej Hajda
2014-10-07 12:01   ` Andrzej Hajda
2014-10-07 12:01 ` [PATCH 2/4] drm/exynos/dsi: simplify device pointer evaluation Andrzej Hajda
2014-10-07 12:01   ` Andrzej Hajda
2014-10-07 12:01 ` [PATCH 3/4] drm/exynos/dsi: remove redundant encoder field Andrzej Hajda
2014-10-07 12:01   ` Andrzej Hajda
2014-10-07 12:01 ` [PATCH 4/4] drm/exynos/dsi: stop using display->ctx pointer Andrzej Hajda
2014-10-07 12:01   ` Andrzej Hajda
2014-10-30 12:39 ` [PATCH 0/4] drm/exynos/dsi: remove global display variable Andrzej Hajda
2014-10-30 12:39   ` Andrzej Hajda
2014-10-30 12:59   ` Inki Dae
2014-10-30 12:59     ` Inki Dae
2014-11-13  8:50 ` Inki Dae [this message]
2014-11-13  8:50   ` Inki Dae
2014-11-13 14:14   ` Andrzej Hajda
2014-11-13 14:14     ` 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=546470BC.4020104@samsung.com \
    --to=inki.dae@samsung.com \
    --cc=a.hajda@samsung.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=kgene.kim@samsung.com \
    --cc=kyungmin.park@samsung.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=m.szyprowski@samsung.com \
    --cc=sw0312.kim@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.