https://bugs.freedesktop.org/show_bug.cgi?id=56461

--- Comment #9 from Chris Paulson-Ellis <[email protected]> ---
Hi,

I only just realised that you sent me a second patch (I thought you were
referring to the first patch, so I went and did the bisect as per my previous
message). I've just tried the second patch against nouveau HEAD. Unfortunately
it causes a NULL pointer dereference. I think this is because when this line in
nv04_timer_init...

n = nouveau_hw_get_clock(((struct nouveau_drm *)nouveau_client(priv))->dev,
PLL_CORE);

...calls nouveau_client() it gets NULL, because the parent of nv04_timer_priv
is a nouveau_device, not a nouveau_drm. I can't see any way to get a pointer to
the drm_device from nv04_timer_init, so I don't think calling
nouveau_hw_get_clock from here is possible.

Chris.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Nouveau mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/nouveau

Reply via email to