<br><br><div class="gmail_quote">2013/8/7 Daniel Vetter <span dir="ltr"><<a href="mailto:daniel@ffwll.ch" target="_blank">daniel@ffwll.ch</a>></span><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid">
<div class="im">On Wed, Aug 07, 2013 at 07:18:45PM +0900, Joonyoung Shim wrote:<br>
> On 08/07/2013 06:55 PM, Daniel Vetter wrote:<br>
> >On Wed, Aug 7, 2013 at 11:40 AM, Inki Dae <<a href="mailto:inki.dae@samsung.com">inki.dae@samsung.com</a>> wrote:<br>
> >>>-----Original Message-----<br>
> >>>From: Daniel Vetter [mailto:<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>]<br>
> >>>Sent: Wednesday, August 07, 2013 6:15 PM<br>
> >>>To: DRI Development<br>
> >>>Cc: Intel Graphics Development; Daniel Vetter; Inki Dae<br>
> >>>Subject: [PATCH 1/3] drm: use common drm_gem_dmabuf_release in i915/exynos<br>
> >>>drivers<br>
> >>><br>
> >>>Note that this is slightly tricky since both drivers store their<br>
> >>>native objects in dma_buf->priv. But both also embed the base<br>
> >>>drm_gem_object at the first position, so the implicit cast is ok.<br>
> >>><br>
> >>>To use the release helper we need to export it, too.<br>
> >>Yeah, may I repost this patch with additional work?  We also need to export<br>
> >>with a gem object instead of specific one like you did.<br>
><br>
> I think dmabuf stuff of exynos can be replaced to common drm_gem_dmabuf.<br>
> Already dmabuf stuff of drm_gem_cma_helper.c was substituted to common<br>
> drm_gem_dmabuf with low-level hook functions to use prime helpers.<br>
<br>
</div>Ah, but that can easily be done on top of this, right?<br></blockquote><div> </div><div>Daniel, could you remove exynos related codes from your patch set? Your patch set would make exynos broken because you didn't consider exporting with a gem object for exynos like <span tabindex="-1" class="hP" id=":fj">[PATCH 3/3] drm/i915: explicit store base gem object in dma_buf->priv. So I think your patch set is not complete set, and That is why exynos needs the additional work I mentioned above. So I just wanted to repost your patch set + new one.</span></div>
<div><span tabindex="-1" class="hP">However, I think not only exynos could go to common drm_gem_dmabuf directly but also it would make your patch set to be complete set if you remove exynos related codes from your patch set. Otherwise, we have to work twice. one is the additional work for resolving exynos broken issue by your patch set, and other is to replace existing dmabuf stuff of exynos to common drm_gem_dmabuf.</span></div>
<div><span tabindex="-1" class="hP"></span> </div><div><span tabindex="-1" class="hP">Thanks,</span></div><div><span tabindex="-1" class="hP">Inki Dae</span></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid">

<div class="im HOEnZb">-Daniel<br>
--<br>
Daniel Vetter<br>
Software Engineer, Intel Corporation<br>
+41 (0) 79 365 57 48 - <a href="http://blog.ffwll.ch" target="_blank">http://blog.ffwll.ch</a><br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
dri-devel mailing list<br>
<a href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/dri-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
</div></div></blockquote></div><br>