From mboxrd@z Thu Jan  1 00:00:00 1970
From: Joonyoung Shim <jy0922.shim@samsung.com>
Subject: Re: [PATCH 1/3] drm: use common drm_gem_dmabuf_release in i915/exynos
	drivers
Date: Wed, 07 Aug 2013 19:29:29 +0900
Message-ID: <52022189.4080606@samsung.com>
References: <1375866908-5000-1-git-send-email-daniel.vetter@ffwll.ch>
	<1375866908-5000-2-git-send-email-daniel.vetter@ffwll.ch>
	<014c01ce9352$1e559690$5b00c3b0$%dae@samsung.com>
	<CAKMK7uFytvP4kidbtQXY1CBTki46kaoJcKFOu3wWSA2Tj7k+Jg@mail.gmail.com>
	<52021F05.6070401@samsung.com>
	<20130807102138.GV22035@phenom.ffwll.local>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org>
In-reply-to: <20130807102138.GV22035@phenom.ffwll.local>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/dri-devel>,
	<mailto:dri-devel-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/dri-devel>
List-Post: <mailto:dri-devel@lists.freedesktop.org>
List-Help: <mailto:dri-devel-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/dri-devel>,
	<mailto:dri-devel-request@lists.freedesktop.org?subject=subscribe>
Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org
Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org
To: Daniel Vetter <daniel@ffwll.ch>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>, Intel Graphics Development <intel-gfx@lists.freedesktop.org>, DRI Development <dri-devel@lists.freedesktop.org>
List-Id: dri-devel@lists.freedesktop.org

On 08/07/2013 07:21 PM, Daniel Vetter wrote:
> On Wed, Aug 07, 2013 at 07:18:45PM +0900, Joonyoung Shim wrote:
>> On 08/07/2013 06:55 PM, Daniel Vetter wrote:
>>> On Wed, Aug 7, 2013 at 11:40 AM, Inki Dae <inki.dae@samsung.com> wrote:
>>>>> -----Original Message-----
>>>>> From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch]
>>>>> Sent: Wednesday, August 07, 2013 6:15 PM
>>>>> To: DRI Development
>>>>> Cc: Intel Graphics Development; Daniel Vetter; Inki Dae
>>>>> Subject: [PATCH 1/3] drm: use common drm_gem_dmabuf_release in i915/exynos
>>>>> drivers
>>>>>
>>>>> Note that this is slightly tricky since both drivers store their
>>>>> native objects in dma_buf->priv. But both also embed the base
>>>>> drm_gem_object at the first position, so the implicit cast is ok.
>>>>>
>>>>> To use the release helper we need to export it, too.
>>>> Yeah, may I repost this patch with additional work?  We also need to export
>>>> with a gem object instead of specific one like you did.
>> I think dmabuf stuff of exynos can be replaced to common drm_gem_dmabuf.
>> Already dmabuf stuff of drm_gem_cma_helper.c was substituted to common
>> drm_gem_dmabuf with low-level hook functions to use prime helpers.
> Ah, but that can easily be done on top of this, right?
> -Daniel

I think it doesn't matter.