linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] drm: rockchip: add shutdown function to poweroff VOP and HDMI TX
@ 2019-03-08 18:37 Johan Jonker
  2019-03-11 18:14 ` Johan Jonker
  0 siblings, 1 reply; 3+ messages in thread
From: Johan Jonker @ 2019-03-08 18:37 UTC (permalink / raw)
  To: heiko
  Cc: hjc, airlied, daniel, dri-devel, linux-arm-kernel,
	linux-rockchip, linux-kernel

When a MK808 TV-stick with RK3066 processor
is given a poweroff command, the monitor doesn't turn blank,
but freezes. The VOP and HDMI share the same power domain,
so that means that both there clocks were never turned off.
Fix the disable of the VOP and HDMI TX by adding an extra
shutdown function that will unbind both drivers.

Signed-off-by: Johan Jonker <jbx6244@gmail.com>
---
 drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c
index 8d7a634c1..66a8e99a3 100644
--- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c
+++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c
@@ -448,6 +448,13 @@ static int rockchip_drm_platform_remove(struct platform_device *pdev)
 	return 0;
 }
 
+static void rockchip_drm_platform_shutdown(struct platform_device *pdev)
+{
+	component_master_del(&pdev->dev, &rockchip_drm_ops);
+
+	rockchip_drm_match_remove(&pdev->dev);
+}
+
 static const struct of_device_id rockchip_drm_dt_ids[] = {
 	{ .compatible = "rockchip,display-subsystem", },
 	{ /* sentinel */ },
@@ -457,6 +464,7 @@ MODULE_DEVICE_TABLE(of, rockchip_drm_dt_ids);
 static struct platform_driver rockchip_drm_platform_driver = {
 	.probe = rockchip_drm_platform_probe,
 	.remove = rockchip_drm_platform_remove,
+	.shutdown = rockchip_drm_platform_shutdown,
 	.driver = {
 		.name = "rockchip-drm",
 		.of_match_table = rockchip_drm_dt_ids,
-- 
2.11.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] drm: rockchip: add shutdown function to poweroff VOP and HDMI TX
  2019-03-08 18:37 [PATCH] drm: rockchip: add shutdown function to poweroff VOP and HDMI TX Johan Jonker
@ 2019-03-11 18:14 ` Johan Jonker
  2019-04-02 11:47   ` Heiko Stuebner
  0 siblings, 1 reply; 3+ messages in thread
From: Johan Jonker @ 2019-03-11 18:14 UTC (permalink / raw)
  To: heiko
  Cc: hjc, airlied, daniel, dri-devel, linux-arm-kernel,
	linux-rockchip, linux-kernel

Hi,

Just found out that this was patched and reverted before.(lol)
The revert was 3 months ago for RK3399 boards, but other processors
like RK3066 are left without shutdown now.

What's the status for a fix?
Keep using the old patch for now?


drm/rockchip: shutdown drm subsystem on shutdown
https://patchwork.kernel.org/patch/10556151/

[for-4.20] Revert "drm/rockchip: Allow driver to be shutdown on reboot/kexec"
https://patchwork.kernel.org/patch/10714725/

2019-03-08 19:37 GMT+01:00, Johan Jonker <jbx6244@gmail.com>:
> When a MK808 TV-stick with RK3066 processor
> is given a poweroff command, the monitor doesn't turn blank,
> but freezes. The VOP and HDMI share the same power domain,
> so that means that both there clocks were never turned off.
> Fix the disable of the VOP and HDMI TX by adding an extra
> shutdown function that will unbind both drivers.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] drm: rockchip: add shutdown function to poweroff VOP and HDMI TX
  2019-03-11 18:14 ` Johan Jonker
@ 2019-04-02 11:47   ` Heiko Stuebner
  0 siblings, 0 replies; 3+ messages in thread
From: Heiko Stuebner @ 2019-04-02 11:47 UTC (permalink / raw)
  To: Johan Jonker
  Cc: hjc, airlied, daniel, dri-devel, linux-arm-kernel,
	linux-rockchip, linux-kernel

Hi Johan,

Am Montag, 11. März 2019, 19:14:27 CEST schrieb Johan Jonker:
> Hi,
> 
> Just found out that this was patched and reverted before.(lol)
> The revert was 3 months ago for RK3399 boards, but other processors
> like RK3066 are left without shutdown now.
> 
> What's the status for a fix?
> Keep using the old patch for now?

I've just resurrected the patch from Vicente, calling the atomic_shutdown
helper. Sadly I forgot to Cc you there, but you can for example find it at

https://patchwork.freedesktop.org/patch/295946/


Heiko



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-04-02 11:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-08 18:37 [PATCH] drm: rockchip: add shutdown function to poweroff VOP and HDMI TX Johan Jonker
2019-03-11 18:14 ` Johan Jonker
2019-04-02 11:47   ` Heiko Stuebner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).