* [GIT,PULL] mediatek drm fixes for 5.2
@ 2019-06-12 7:51 CK Hu
2019-06-12 16:25 ` Daniel Vetter
0 siblings, 1 reply; 5+ messages in thread
From: CK Hu @ 2019-06-12 7:51 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter, dri-devel, linux-mediatek
Cc: Yongqiang Niu, Hsin-Yi Wang
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 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
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(-)
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [GIT,PULL] mediatek drm fixes for 5.2
2019-06-12 7:51 [GIT,PULL] mediatek drm fixes for 5.2 CK Hu
@ 2019-06-12 16:25 ` Daniel Vetter
2019-06-13 6:31 ` CK Hu
0 siblings, 1 reply; 5+ messages in thread
From: Daniel Vetter @ 2019-06-12 16:25 UTC (permalink / raw)
To: CK Hu
Cc: dri-devel, linux-mediatek, Yongqiang Niu, Hsin-Yi Wang, Daniel Vetter
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.
>
> 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.
>
> 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@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [GIT,PULL] mediatek drm fixes for 5.2
2019-06-12 16:25 ` Daniel Vetter
@ 2019-06-13 6:31 ` CK Hu
2019-06-13 8:27 ` Daniel Vetter
0 siblings, 1 reply; 5+ messages in thread
From: CK Hu @ 2019-06-13 6:31 UTC (permalink / raw)
To: Daniel Vetter
Cc: dri-devel, linux-mediatek, Yongqiang Niu, Hsin-Yi Wang, Daniel Vetter
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/
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@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [GIT,PULL] mediatek drm fixes for 5.2
2019-06-13 6:31 ` CK Hu
@ 2019-06-13 8:27 ` Daniel Vetter
2019-06-14 2:06 ` CK Hu
0 siblings, 1 reply; 5+ messages in thread
From: Daniel Vetter @ 2019-06-13 8:27 UTC (permalink / raw)
To: CK Hu
Cc: Yongqiang Niu, dri-devel, linux-mediatek, Hsin-Yi Wang, Daniel Vetter
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@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> >
>
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [GIT,PULL] mediatek drm fixes for 5.2
2019-06-13 8:27 ` Daniel Vetter
@ 2019-06-14 2:06 ` CK Hu
0 siblings, 0 replies; 5+ messages in thread
From: CK Hu @ 2019-06-14 2:06 UTC (permalink / raw)
To: Daniel Vetter
Cc: dri-devel, linux-mediatek, Yongqiang Niu, Hsin-Yi Wang, Daniel Vetter
On Thu, 2019-06-13 at 10:27 +0200, Daniel Vetter wrote:
> 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.
OK, I find that drm_gem_prime_mmap() call driver->fops->mmap(), so the
customization could be done in driver->fops->mmap and let
drm_driver.gem_prime_mmap to be drm_gem_prime_mmap. I would try to do
so.
Regards,
CK
> -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@lists.freedesktop.org
> > > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> > >
> >
> >
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-06-14 2:06 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-12 7:51 [GIT,PULL] mediatek drm fixes for 5.2 CK Hu
2019-06-12 16:25 ` Daniel Vetter
2019-06-13 6:31 ` CK Hu
2019-06-13 8:27 ` Daniel Vetter
2019-06-14 2:06 ` CK Hu
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.