On Thu, Jun 13, 2019 at 02:31:18PM +0800, CK Hu wrote: > Hi, Daniel: > > On Wed, 2019-06-12 at 18:25 +0200, Daniel Vetter wrote: > > On Wed, Jun 12, 2019 at 03:51:08PM +0800, CK Hu wrote: > > > Hi Dave, Daniel: > > > > > > This include unbind error fix, clock control flow refinement, and PRIME > > > mmap with page offset. > > > > > > Regards, > > > CK > > > > > > The following changes since commit > > > a188339ca5a396acc588e5851ed7e19f66b0ebd9: > > > > > > Linux 5.2-rc1 (2019-05-19 15:47:09 -0700) > > > > > > are available in the Git repository at: > > > > > > https://github.com/ckhu-mediatek/linux.git-tags.git > > > mediatek-drm-fixes-5.2 > > > > For next time around, please make this a signed annotated tag. annotated > > because then it's less typing for me, and signed because it's on a server > > we don't necessarily trust. > > OK, I'd make a signed annotated tag in next time. > > > > > > > > > for you to fetch changes up to 2458d9d6d94be982b917e93c61a89b4426f32e31: > > > > > > drm/mediatek: call mtk_dsi_stop() after mtk_drm_crtc_atomic_disable() > > > (2019-06-04 09:54:42 +0800) > > > > > > ---------------------------------------------------------------- > > > Hsin-Yi Wang (5): > > > drm/mediatek: fix unbind functions > > > drm/mediatek: unbind components in mtk_drm_unbind() > > > drm/mediatek: call drm_atomic_helper_shutdown() when unbinding > > > driver > > > drm/mediatek: clear num_pipes when unbind driver > > > drm/mediatek: call mtk_dsi_stop() after > > > mtk_drm_crtc_atomic_disable() > > > > > > Yongqiang Niu (2): > > > drm/mediatek: adjust ddp clock control flow > > > drm/mediatek: respect page offset for PRIME mmap calls > > > > You might want to look into all the recently added helpers for mmap, I > > don't think mtk does anything special here. > > If you mean using drm_gem_prime_mmap as the implementation of > drm_driver->gem_prime_mmap, I think we have discussed in [1] that > mediatek drm still need its own implementation. > > [1] https://patchwork.kernel.org/patch/10657007/ Yeah for reusing the overall helpers that's clear, but drm_gem_prime_gem_mmap should work for any gem based driver. It doesn't require you to use specific helpers at all. I'm wondering about this since I'm looking right now into this area, and it looks like we could just make drm_gem_prime_mmap the default and rip out all the custom driver implementations. -Daniel > > Regards, > CK > > > > > > > > > > > drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 30 > > > ++++++------------------------ > > > drivers/gpu/drm/mediatek/mtk_drm_drv.c | 8 +++----- > > > drivers/gpu/drm/mediatek/mtk_drm_gem.c | 7 ++++++- > > > drivers/gpu/drm/mediatek/mtk_dsi.c | 12 +++++++++++- > > > 4 files changed, 26 insertions(+), 31 deletions(-) > > > > Pulled, thanks. > > -Daniel > > > > > > > > _______________________________________________ > > > dri-devel mailing list > > > dri-devel@xxxxxxxxxxxxxxxxxxxxx > > > https://lists.freedesktop.org/mailman/listinfo/dri-devel > > > > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
