<p dir="ltr"><br>
On Oct 29, 2013 8:16 AM, "Inki Dae" <<a href="mailto:inki.dae@samsung.com">inki.dae@samsung.com</a>> wrote:<br>
><br>
> 2013/10/28 Sean Paul <<a href="mailto:seanpaul@google.com">seanpaul@google.com</a>>:<br>
> > Hi Inki,<br>
> > I noticed that you merged "drm/exynos: change callback argument of sub<br>
> > driver with device" to your tree without posting it to me, or<br>
> > dri-devel, first. I think it would have been prudent to send for<br>
> > review/comments considering that:<br>
> ><br>
> > a) we don't agree on this, and were in discussion about it last week, and<br>
> > b) it essentially reverts my "Pass exynos_drm_manager in..." patch,<br>
> > and changes key elements of my "Split manager/display/subdrv" patch,<br>
> ><br>
><br>
> Don't worry about that. I have already removed my patch: it was really<br>
> my mistake. And it doesn't need to discuss that issue anymore. I<br>
> understood enough. :)<br>
></p>
<p dir="ltr">Hi InKi,<br>
I sent this yesterday before your patches were posted to dri-devel, I guess it got lost in the ether.</p>
<p dir="ltr">Sean</p>
<p dir="ltr">> > Lastly, I was working on fixing the vidi errors in the patchset,<br>
> > instead of just on top. For bisect reasons, this is probably<br>
> > desirable.<br>
><br>
> I had already commented about that, and my patch has already be<br>
> removed from exynos-drm-next. Please re-post that fixup. :)<br>
><br>
> Thanks,<br>
> Inki Dae<br>
><br>
> ><br>
> > Sean<br>
> > _______________________________________________<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">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
</p>