From: Tomasz Figa
This patch removes unused fields from vop structure.
Signed-off-by: Tomasz Figa
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/gpu/drm/rockchip
1. In
fact, we should do full link training now, not the fast link training.
So we should move the fast link detection at the end of set_bridge.
Cc: Tomasz Figa
Signed-off-by: zain wang
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej
dither down vop data to 8bit if panel's bpc is 8.
Signed-off-by: Mark Yao
[seanpaul resolved conflict in rockchip_drm_vop.c]
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 2 ++
drivers/gpu/drm/rockchip/rockchip_drm_drv.h
't set the enable until the end.
Cc: Kristian H. Kristensen
Signed-off-by: Douglas Anderson
[seanpaul added exynos change]
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 7 +--
drivers/gpu/drm/exy
aul
Signed-off-by: Thierry Escande
Tested-by: Heiko Stuebner
---
drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
index 5d52020de
lay.
Let's change this error to warn.
Cc: Douglas Anderson
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --gi
ynchronize
PSR state changes with other DRM events, i.e. atomic commits and cursor
updates. As a nice side effect, the naming is sorted out and we have
"inhibit" for stopping the software logic and "enable" for hardware
state.
Signed-off-by: Tomasz Figa
Signed-off-by: Thierry
asz Figa
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
index 23317a2269e1..6d45d62466b3 100644
--- a/dri
on all the
time.
Cc: Tomasz Figa
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/bridge
]
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 6 ++--
drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 6 ++--
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 35 +++---
3 files changed, 37 insertions
: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
index ba2506e17f6d..c940a5bb80ac 100644
--- a/drivers/gpu
From: Lin Huang
AUX errors are caused by many different reasons. We may not know what
happened in aux channel on failure, so let's reset aux channel if some
errors occurred.
Cc: 征增 王
Cc: Douglas Anderson
Signed-off-by: Lin Huang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Es
call
rockchip_drm_do_flush(), as they are both supposed to do the same.
Signed-off-by: Tomasz Figa
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 20
1 file changed, 8 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/roc
From: zain wang
We would meet a short black screen when exit PSR with the full link
training, In this case, we should use fast link train instead of full
link training.
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 4 ++--
drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 2 +-
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 5 -
3 files changed, 7 insertions(+), 4 deletions(-)
diff
Marchesin
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 9 +
drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 2 +-
2 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/
quot;, where I had a kgdb breakpoint. Presumably
the device would have eventually given up and shut down anyway, but it
seems better to fix the order to be more correct.
Cc: Kristian H. Kristensen
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
R
From: Lin Huang
We need to check the dpcd write/read return value to see whether the
write/read was successful
Cc: Kristian H. Kristensen
Signed-off-by: Lin Huang
Signed-off-by: zain wang
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by
From: zain wang
There are some different bits between Rockchip and Exynos in register
"AUX_PD". This patch fixes the incorrect operations about it.
Cc: Douglas Anderson
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
--
BIT 5 RK_VID_FIFO_FUNC_EN_N
So, we should do some private operations to Rockchip.
Cc: Tomasz Figa
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 19
From: zain wang
Following the correct power up sequence:
dp_pd=ff => dp_pd=7f => wait 10us => dp_pd=00
Cc: Stéphane Marchesin
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 10 --
dr
aul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c
b/drivers/gpu/drm/bridge/analogix/analogix
ff-by: Thierry Escande
---
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 31 ++---
1 file changed, 2 insertions(+), 29 deletions(-)
diff --git a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
index 7d76ff47028d..3633483
From: Lin Huang
When panel is shut down, we should make sure edp can be disabled to avoid
undefined behavior.
Cc: Stéphane Marchesin
Signed-off-by: Lin Huang
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm
From: Lin Huang
We need to enable video before analogix_dp_is_video_stream_on(), so
we can get the right video stream status.
Cc: 征增 王
Cc: Stéphane Marchesin
Signed-off-by: Lin Huang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm
L
because encoder->crtc is NULL. Avoid this by starting out with
psr->allowed set to false.
Signed-off-by: Kristian H. Kristensen
Signed-off-by: Tomasz Figa
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 79 +
1 file changed
just removed the power_on portion, seanpaul removed
the power off code as well as improved the commit message]
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 10 --
1 file changed, 10 deletions(-)
diff --git a/drivers/gpu
From: zain wang
If we failed disable psr, it would hang the display until next psr
cycle coming. So we should restore psr->state when it failed.
Cc: Tomasz Figa
Signed-off-by: zain wang
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/
From: Tomasz Figa
It is not used anymore after last changes and it was not even correct to
begin with as it assumed a 1:1 relation between a CRTC and encoder,
while in fact a CRTC can be attached to multiple encoders.
Signed-off-by: Tomasz Figa
Signed-off-by: Thierry Escande
---
drivers/gpu
point before
the power is cut. A reboot notifier, despite being a misnomer and
handling shutdowns as well, is a good place to do it.
Signed-off-by: Tomasz Figa
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 24
1 file changed, 24 insertions
Hi,
This patchset includes cleanups, improvements, and bug fixes for
Rockchip DRM driver and PSR support.
this patchset depends and needs to be applied on top of Rockchip rk3399
eDP support [1].
[1] https://lkml.org/lkml/2018/1/10/682
Regards,
Thierry
Changes in v3:
- Addressed some of the com
ff-by: Tomasz Figa
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_fb.c | 61 -
drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 7
2 files changed, 60 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/roc
From: zain wang
There is no register named ANALOGIX_DP_PLL_CTL in Rockchip edp phy reg
list. We should use BIT_4 in ANALOGIX_DP_PD to control the pll power
instead of ANALOGIX_DP_PLL_CTL.
Cc: Douglas Anderson
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
ed up the commit message a bit and renamed *_supported to
*_enabled]
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 15 ---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 2 +-
drivers/gpu/drm/rockchip/analog
w frame in response to the event,
this can completely hide the exit latency. In case of Chrome OS, we
typically get the input notifier 50ms or more before the dirty_fb
triggered exit.
Signed-off-by: Kristian H. Kristensen
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_p
From: Lin Huang
There was a 1ms delay to detect the hpd signal, which is too short to
detect a short pulse. This patch extends this delay to 100ms.
Cc: Stéphane Marchesin
Cc: 征增 王
Signed-off-by: Lin Huang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
From: zain wang
When we enable bridge failed, we have to retry it, otherwise we would get
the abnormal display.
Cc: Stéphane Marchesin
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix
From: zain wang
Add a lock to vop to avoid disabling the crtc while waiting for a line
flag while enabling psr. If we disable in the middle of waiting for the
line flag, we'll end up timing out or worse.
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Es
From: Haixia Shi
The prime fd to handle ioctl was not used with rockchip before. Support
was added in order to pass graphics_Gbm and to support potential uses
within Chrome OS (e.g. zero-copy video decode, camera).
Signed-off-by: Haixia Shi
Signed-off-by: Sean Paul
Signed-off-by: Thierry
id it.
Cc: Stéphane Marchesin
Signed-off-by: zain wang
Signed-off-by: Caesar Wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 7 +-
drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 30 -
drivers/gp
hesin
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 2 +-
drivers/gpu/drm/rockchip/rockchip_drm_drv.h | 2 +-
drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 68 -
3 files changed, 31 insertions(+), 41 dele
From: Lin Huang
We should check AUX_EN bit to confirm the AUX CH operation is completed.
Cc: Stéphane Marchesin
Signed-off-by: Lin Huang
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 25
From: Sean Paul
Change the mode for Sharp lq123p1jx31 panel to something more
rockchip-friendly such that we can use the fixed PLLs to
generate the pixel clock
Cc: Chris Zhong
Cc: Stéphane Marchesin
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/panel/panel
retry
behavior across all platforms.
Cc: Stéphane Marchesin
Cc: 征增 王
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff --
w frame in response to the event,
this can completely hide the exit latency. In case of Chrome OS, we
typically get the input notifier 50ms or more before the dirty_fb
triggered exit.
Signed-off-by: Kristian H. Kristensen
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_p
lay.
Let's change this error to warn.
Cc: Douglas Anderson
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff
From: zain wang
If we failed disable psr, it would hang the display until next psr
cycle coming. So we should restore psr->state when it failed.
Cc: Tomasz Figa
Signed-off-by: zain wang
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/
point before
the power is cut. A reboot notifier, despite being a misnomer and
handling shutdowns as well, is a good place to do it.
Signed-off-by: Tomasz Figa
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 24
1 file changed, 24 insertions
From: zain wang
When we enable bridge failed, we have to retry it, otherwise we would get
the abnormal display.
Cc: Stéphane Marchesin
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix
From: Lin Huang
We need to enable video before analogix_dp_is_video_stream_on(), so
we can get the right video stream status.
Cc: 征增 王
Cc: Stéphane Marchesin
Signed-off-by: Lin Huang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm
ff-by: Tomasz Figa
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_fb.c | 61 -
drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 7
2 files changed, 60 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/roc
From: Tomasz Figa
It is not used anymore after last changes and it was not even correct to
begin with as it assumed a 1:1 relation between a CRTC and encoder,
while in fact a CRTC can be attached to multiple encoders.
Signed-off-by: Tomasz Figa
Signed-off-by: Thierry Escande
---
drivers/gpu
dither down vop data to 8bit if panel's bpc is 8.
Signed-off-by: Mark Yao
[seanpaul resolved conflict in rockchip_drm_vop.c]
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 2 ++
drivers/gpu/drm/rockchip/rockchip_drm_drv.h
ynchronize
PSR state changes with other DRM events, i.e. atomic commits and cursor
updates. As a nice side effect, the naming is sorted out and we have
"inhibit" for stopping the software logic and "enable" for hardware
state.
Signed-off-by: Tomasz Figa
Signed-off-by: Thierry
1. In
fact, we should do full link training now, not the fast link training.
So we should move the fast link detection at the end of set_bridge.
Cc: Tomasz Figa
Signed-off-by: zain wang
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej
aul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c
b/drivers/gpu/drm/bridge/analogix/analogix
retry
behavior across all platforms.
Cc: Stéphane Marchesin
Cc: 征增 王
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff --
hesin
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 2 +-
drivers/gpu/drm/rockchip/rockchip_drm_drv.h | 2 +-
drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 68 -
3 files changed, 31 insertions(+), 41 dele
just removed the power_on portion, seanpaul removed
the power off code as well as improved the commit message]
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 10 --
1 file changed, 10 deletions(-)
diff --git a/drivers/gpu
From: zain wang
There are some different bits between Rockchip and Exynos in register
"AUX_PD". This patch fixes the incorrect operations about it.
Cc: Douglas Anderson
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
--
ed up the commit message a bit and renamed *_supported to
*_enabled]
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 15 ---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 2 +-
drivers/gpu/drm/rockchip/analog
call
rockchip_drm_do_flush(), as they are both supposed to do the same.
Signed-off-by: Tomasz Figa
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 20
1 file changed, 8 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/roc
From: zain wang
We would meet a short black screen when exit PSR with the full link
training, In this case, we should use fast link train instead of full
link training.
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix
't set the enable until the end.
Cc: Kristian H. Kristensen
Signed-off-by: Douglas Anderson
[seanpaul added exynos change]
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 7 +--
drivers/gpu/drm/exy
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 4 ++--
drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 2 +-
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 5 -
3 files changed, 7 insertions(+), 4 deletions(-)
diff
on all the
time.
Cc: Tomasz Figa
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/bridge
From: Tomasz Figa
This patch removes unused fields from vop structure.
Signed-off-by: Tomasz Figa
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/gpu/drm/rockchip
Marchesin
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 9 +
drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 2 +-
2 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/
From: Lin Huang
When panel is shut down, we should make sure edp can be disabled to avoid
undefined behavior.
Cc: Stéphane Marchesin
Signed-off-by: Lin Huang
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm
From: zain wang
Add a lock to vop to avoid disabling the crtc while waiting for a line
flag while enabling psr. If we disable in the middle of waiting for the
line flag, we'll end up timing out or worse.
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Es
From: Lin Huang
There was a 1ms delay to detect the hpd signal, which is too short to
detect a short pulse. This patch extends this delay to 100ms.
Cc: Stéphane Marchesin
Cc: 征增 王
Signed-off-by: Lin Huang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
]
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 6 ++--
drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 6 ++--
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 35 +++---
3 files changed, 37 insertions
: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
index ba2506e17f6d..c940a5bb80ac 100644
--- a/drivers/gpu
From: Lin Huang
AUX errors are caused by many different reasons. We may not know what
happened in aux channel on failure, so let's reset aux channel if some
errors occurred.
Cc: 征增 王
Cc: Douglas Anderson
Signed-off-by: Lin Huang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Es
quot;, where I had a kgdb breakpoint. Presumably
the device would have eventually given up and shut down anyway, but it
seems better to fix the order to be more correct.
Cc: Kristian H. Kristensen
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
R
From: Lin Huang
We need to check the dpcd write/read return value to see whether the
write/read was successful
Cc: Kristian H. Kristensen
Signed-off-by: Lin Huang
Signed-off-by: zain wang
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by
asz Figa
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
index 23317a2269e1..6d45d62466b3 100644
--- a/dri
BIT 5 RK_VID_FIFO_FUNC_EN_N
So, we should do some private operations to Rockchip.
Cc: Tomasz Figa
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 19
aul
Signed-off-by: Thierry Escande
Tested-by: Heiko Stuebner
---
drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
index 5d52020de
L
because encoder->crtc is NULL. Avoid this by starting out with
psr->allowed set to false.
Signed-off-by: Kristian H. Kristensen
Signed-off-by: Tomasz Figa
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 79 +
1 file changed
From: Sean Paul
Change the mode for Sharp lq123p1jx31 panel to something more
rockchip-friendly such that we can use the fixed PLLs to
generate the pixel clock
Cc: Chris Zhong
Cc: Stéphane Marchesin
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/panel/panel
id it.
Cc: Stéphane Marchesin
Signed-off-by: zain wang
Signed-off-by: Caesar Wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 7 +-
drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 30 -
drivers/gp
ff-by: Thierry Escande
---
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 31 ++---
1 file changed, 2 insertions(+), 29 deletions(-)
diff --git a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
index 7d76ff47028d..3633483
Hi,
This patchset includes cleanups, improvements, and bug fixes for
Rockchip DRM driver and PSR support.
this patchset depends and needs to be applied on top of Rockchip rk3399
eDP support [1].
[1] https://lkml.org/lkml/2018/1/10/682
Regards,
Thierry
Changes in v2:
- A few patches have been r
From: zain wang
Add a lock to vop to avoid disabling the crtc while waiting for a line
flag while enabling psr. If we disable in the middle of waiting for the
line flag, we'll end up timing out or worse.
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Es
From: Lin Huang
AUX errors are caused by many different reasons. We may not know what
happened in aux channel on failure, so let's reset aux channel if some
errors occurred.
Cc: 征增 王
Cc: Douglas Anderson
Signed-off-by: Lin Huang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Es
Marchesin
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 9 +
drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 2 +-
2 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/
. Kristensen
Cc: Brian Norris
Signed-off-by: Tomasz Figa
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 8
1 file changed, 8 insertions(+)
diff --git a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
b/drivers/gpu/drm
BIT 5 RK_VID_FIFO_FUNC_EN_N
So, we should do some private operations to Rockchip.
Cc: Tomasz Figa
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 19
From: Lin Huang
There was a 1ms delay to detect the hpd signal, which is too short to
detect a short pulse. This patch extends this delay to 100ms.
Cc: Stéphane Marchesin
Cc: 征增 王
Signed-off-by: Lin Huang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
quot;, where I had a kgdb breakpoint. Presumably
the device would have eventually given up and shut down anyway, but it
seems better to fix the order to be more correct.
Cc: Kristian H. Kristensen
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
R
From: Sean Paul
Change the mode for Sharp lq123p1jx31 panel to something more
rockchip-friendly such that we can use the fixed PLLs to
generate the pixel clock
Cc: Chris Zhong
Cc: Stéphane Marchesin
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/panel/panel
]
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 6 ++--
drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 6 ++--
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 35 +++---
3 files changed, 37 insertions
1. In
fact, we should do full link training now, not the fast link training.
So we should move the fast link detection at the end of set_bridge.
Cc: Tomasz Figa
Signed-off-by: zain wang
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej
on all the
time.
Cc: Tomasz Figa
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/bridge
From: Lin Huang
When panel is shut down, we should make sure edp can be disabled to avoid
undefined behavior.
Cc: Stéphane Marchesin
Signed-off-by: Lin Huang
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm
aul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c
b/drivers/gpu/drm/bridge/analogix/analogix
From: Lin Huang
We need to check the dpcd write/read return value to see whether the
write/read was successful
Cc: Kristian H. Kristensen
Signed-off-by: Lin Huang
Signed-off-by: zain wang
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by
From: zain wang
When we enable bridge failed, we have to retry it, otherwise we would get
the abnormal display.
Cc: Stéphane Marchesin
Signed-off-by: zain wang
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
Reviewed-by: Andrzej Hajda
---
drivers/gpu/drm/bridge/analogix
retry
behavior across all platforms.
Cc: Stéphane Marchesin
Cc: 征增 王
Signed-off-by: Douglas Anderson
Signed-off-by: Sean Paul
Signed-off-by: Thierry Escande
---
drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff --
1 - 100 of 143 matches
Mail list logo