Rob Clark
2018-Jan-06  15:59 UTC
[Nouveau] [PATCH] drm/nouveau/disp/gf119: add missing drive vfunc ptr
Fixes broken dp on GF119:
  Call Trace:
   ? nvkm_dp_train_drive+0x183/0x2c0 [nouveau]
   nvkm_dp_acquire+0x4f3/0xcd0 [nouveau]
   nv50_disp_super_2_2+0x5d/0x470 [nouveau]
   ? nvkm_devinit_pll_set+0xf/0x20 [nouveau]
   gf119_disp_super+0x19c/0x2f0 [nouveau]
   process_one_work+0x193/0x3c0
   worker_thread+0x35/0x3b0
   kthread+0x125/0x140
   ? process_one_work+0x3c0/0x3c0
   ? kthread_park+0x60/0x60
   ret_from_fork+0x25/0x30
  Code:  Bad RIP value.
  RIP:           (null) RSP: ffffb1e243e4bc38
  CR2: 0000000000000000
Fixes: af85389c614a drm/nouveau/disp: shuffle functions around
Signed-off-by: Rob Clark <robdclark at gmail.com>
---
 drivers/gpu/drm/nouveau/nvkm/engine/disp/sorgf119.c | 1 +
 1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/disp/sorgf119.c
b/drivers/gpu/drm/nouveau/nvkm/engine/disp/sorgf119.c
index a2978a37b4f3..700fc754f28a 100644
--- a/drivers/gpu/drm/nouveau/nvkm/engine/disp/sorgf119.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/disp/sorgf119.c
@@ -174,6 +174,7 @@ gf119_sor = {
 		.links = gf119_sor_dp_links,
 		.power = g94_sor_dp_power,
 		.pattern = gf119_sor_dp_pattern,
+		.drive = gf119_sor_dp_drive,
 		.vcpi = gf119_sor_dp_vcpi,
 		.audio = gf119_sor_dp_audio,
 		.audio_sym = gf119_sor_dp_audio_sym,
-- 
2.14.3
Rob Clark
2018-Jan-06  16:01 UTC
[Nouveau] [PATCH] drm/nouveau/disp/gf119: add missing drive vfunc ptr
On Sat, Jan 6, 2018 at 10:59 AM, Rob Clark <robdclark at gmail.com> wrote:> Fixes broken dp on GF119: > > Call Trace: > ? nvkm_dp_train_drive+0x183/0x2c0 [nouveau] > nvkm_dp_acquire+0x4f3/0xcd0 [nouveau] > nv50_disp_super_2_2+0x5d/0x470 [nouveau] > ? nvkm_devinit_pll_set+0xf/0x20 [nouveau] > gf119_disp_super+0x19c/0x2f0 [nouveau] > process_one_work+0x193/0x3c0 > worker_thread+0x35/0x3b0 > kthread+0x125/0x140 > ? process_one_work+0x3c0/0x3c0 > ? kthread_park+0x60/0x60 > ret_from_fork+0x25/0x30 > Code: Bad RIP value. > RIP: (null) RSP: ffffb1e243e4bc38 > CR2: 0000000000000000 > > Fixes: af85389c614a drm/nouveau/disp: shuffle functions aroundThis should probably also have: Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103421> Signed-off-by: Rob Clark <robdclark at gmail.com> > --- > drivers/gpu/drm/nouveau/nvkm/engine/disp/sorgf119.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/disp/sorgf119.c b/drivers/gpu/drm/nouveau/nvkm/engine/disp/sorgf119.c > index a2978a37b4f3..700fc754f28a 100644 > --- a/drivers/gpu/drm/nouveau/nvkm/engine/disp/sorgf119.c > +++ b/drivers/gpu/drm/nouveau/nvkm/engine/disp/sorgf119.c > @@ -174,6 +174,7 @@ gf119_sor = { > .links = gf119_sor_dp_links, > .power = g94_sor_dp_power, > .pattern = gf119_sor_dp_pattern, > + .drive = gf119_sor_dp_drive, > .vcpi = gf119_sor_dp_vcpi, > .audio = gf119_sor_dp_audio, > .audio_sym = gf119_sor_dp_audio_sym, > -- > 2.14.3 >
Maybe Matching Threads
- [PATCH 0/5] drm/nouveau: add basic HDMI 2.0 support
- [Bug 103421] New: Kernel 4.13+ nouveau breaks screen output
- [Bug 103351] New: Nouveau fails when booting with a screen connected to displayport since 13a86519202c5d119d83640d6f781f3181205d2c
- [PATCH 2/2] kms/gf119-: always use a 256-entry lut for now
- [PATCH AUTOSEL 5.1 031/186] drm/nouveau/kms/gf119-gp10x: push HeadSetControlOutputResource() mthd when encoders change