[Nouveau] [PATCH] devinit: lock/unlock crtc regs for all devices, not just pre-nv50
Andreas Reis
andreas.reis at gmail.com
Sun Jan 19 10:46:34 PST 2014
No problem. I applied the patch (plus the other for the MXM issue),
replaced the preexisting nouveau.ko.gz in /lib/mod/… with the new
gzipped one, and rebuilt the initramfs. (With an updated DRIVER_DATE
define just to be sure.)
The HDMI output works fine.
Attached are the dmesg outputs before and after, grepped for
'nouveau'. As you can see, there's no real change.
On Sun, Jan 19, 2014 at 5:14 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> On Sun, Jan 19, 2014 at 4:18 AM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
>> Also make nv_lockvgac work for nv50+ devices. This should fix IO_CONDITION and
>> related VBIOS opcodes that read/write the crtc regs.
>>
>> See https://bugs.freedesktop.org/show_bug.cgi?id=60680
>>
>> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
>> ---
>>
>> Ben, is this what you had in mind? I haven't gotten a chance to test this yet
>> since your tree doesn't build against mine (which is largely based on
>> nouveau/linux-2.6 master + drm-nouveau-next patches merged in). Something
>> about nouveau_acpi/backlight.
>>
>> Pekka, mind testing this out and see if it also fixes your issue? Grab Ben's
>> tree at http://cgit.freedesktop.org/~darktama/nouveau/ and apply this patch,
>> and build against a late 3.13-rcX release, I guess. (Still haven't figured out
>> which tree I need to have for this stuff to build.)
>>
>> ./autogen.sh; cd drm; make
>>
>> should generate a nouveau.ko against your currently-running kernel.
>
> Andreas -- you had the same issue as Pekka, would you mind testing
> this out? I'm not sure when he'll be able to give it a shot. This is
> an alternate way of resolving the issue that produces itself as HDMI
> output not working. You can get a pristine copy of the patch at
> http://lists.freedesktop.org/archives/nouveau/2014-January/015770.html
> -- but do note the instructions above, as it applies to a different
> repo.
>
>>
>> nvkm/engine/disp/vga.c | 9 +++++++--
>> nvkm/subdev/devinit/base.c | 14 ++++++++++++++
>> nvkm/subdev/devinit/nv04.c | 13 ++++++++-----
>> nvkm/subdev/devinit/priv.h | 8 +++++---
>> 4 files changed, 34 insertions(+), 10 deletions(-)
>>
>> diff --git a/nvkm/engine/disp/vga.c b/nvkm/engine/disp/vga.c
>> index 5a1c684..8836c3c 100644
>> --- a/nvkm/engine/disp/vga.c
>> +++ b/nvkm/engine/disp/vga.c
>> @@ -138,10 +138,15 @@ nv_wrvgai(void *obj, int head, u16 port, u8 index, u8 value)
>> bool
>> nv_lockvgac(void *obj, bool lock)
>> {
>> + struct nouveau_device *dev = nv_device(obj);
>> +
>> bool locked = !nv_rdvgac(obj, 0, 0x1f);
>> u8 data = lock ? 0x99 : 0x57;
>> - nv_wrvgac(obj, 0, 0x1f, data);
>> - if (nv_device(obj)->chipset == 0x11) {
>> + if (dev->card_type < NV_50)
>> + nv_wrvgac(obj, 0, 0x1f, data);
>> + else
>> + nv_wrvgac(obj, 0, 0x3f, data);
>> + if (dev->chipset == 0x11) {
>> if (!(nv_rd32(obj, 0x001084) & 0x10000000))
>> nv_wrvgac(obj, 1, 0x1f, data);
>> }
>> diff --git a/nvkm/subdev/devinit/base.c b/nvkm/subdev/devinit/base.c
>> index 6b23d9a..a4df3fa 100644
>> --- a/nvkm/subdev/devinit/base.c
>> +++ b/nvkm/subdev/devinit/base.c
>> @@ -26,6 +26,7 @@
>>
>> #include <subdev/bios.h>
>> #include <subdev/bios/init.h>
>> +#include <subdev/vga.h>
>>
>> #include "priv.h"
>>
>> @@ -38,6 +39,9 @@ _nouveau_devinit_fini(struct nouveau_object *object, bool suspend)
>> if (suspend)
>> devinit->post = true;
>>
>> + /* unlock the extended vga crtc regs */
>> + nv_lockvgac(devinit, false);
>> +
>> return nouveau_subdev_fini(&devinit->base, suspend);
>> }
>>
>> @@ -61,6 +65,16 @@ _nouveau_devinit_init(struct nouveau_object *object)
>> return 0;
>> }
>>
>> +void
>> +_nouveau_devinit_dtor(struct nouveau_object *object)
>> +{
>> + struct nouveau_devinit *devinit = (void *)object;
>> +
>> + /* lock crtc regs */
>> + nv_lockvgac(devinit, true);
>> + nouveau_subdev_destroy(&devinit->base);
>> +}
>> +
>> int
>> nouveau_devinit_create_(struct nouveau_object *parent,
>> struct nouveau_object *engine,
>> diff --git a/nvkm/subdev/devinit/nv04.c b/nvkm/subdev/devinit/nv04.c
>> index 24025e4..7037eae 100644
>> --- a/nvkm/subdev/devinit/nv04.c
>> +++ b/nvkm/subdev/devinit/nv04.c
>> @@ -388,17 +388,21 @@ int
>> nv04_devinit_fini(struct nouveau_object *object, bool suspend)
>> {
>> struct nv04_devinit_priv *priv = (void *)object;
>> + int ret;
>>
>> /* make i2c busses accessible */
>> nv_mask(priv, 0x000200, 0x00000001, 0x00000001);
>>
>> - /* unlock extended vga crtc regs, and unslave crtcs */
>> - nv_lockvgac(priv, false);
>> + ret = nouveau_devinit_fini(&priv->base, suspend);
>> + if (ret)
>> + return ret;
>> +
>> + /* unslave crtcs */
>> if (priv->owner < 0)
>> priv->owner = nv_rdvgaowner(priv);
>> nv_wrvgaowner(priv, 0);
>>
>> - return nouveau_devinit_fini(&priv->base, suspend);
>> + return 0;
>> }
>>
>> int
>> @@ -426,9 +430,8 @@ nv04_devinit_dtor(struct nouveau_object *object)
>> {
>> struct nv04_devinit_priv *priv = (void *)object;
>>
>> - /* restore vga owner saved at first init, and lock crtc regs */
>> + /* restore vga owner saved at first init */
>> nv_wrvgaowner(priv, priv->owner);
>> - nv_lockvgac(priv, true);
>>
>> nouveau_devinit_destroy(&priv->base);
>> }
>> diff --git a/nvkm/subdev/devinit/priv.h b/nvkm/subdev/devinit/priv.h
>> index c4179b6..822a2fb 100644
>> --- a/nvkm/subdev/devinit/priv.h
>> +++ b/nvkm/subdev/devinit/priv.h
>> @@ -15,8 +15,10 @@ struct nouveau_devinit_impl {
>>
>> #define nouveau_devinit_create(p,e,o,d) \
>> nouveau_devinit_create_((p), (e), (o), sizeof(**d), (void **)d)
>> -#define nouveau_devinit_destroy(p) \
>> - nouveau_subdev_destroy(&(p)->base)
>> +#define nouveau_devinit_destroy(p) ({ \
>> + struct nouveau_devinit *d = (p); \
>> + _nouveau_devinit_dtor(nv_object(d)); \
>> +})
>> #define nouveau_devinit_init(p) ({ \
>> struct nouveau_devinit *d = (p); \
>> _nouveau_devinit_init(nv_object(d)); \
>> @@ -28,7 +30,7 @@ struct nouveau_devinit_impl {
>>
>> int nouveau_devinit_create_(struct nouveau_object *, struct nouveau_object *,
>> struct nouveau_oclass *, int, void **);
>> -#define _nouveau_devinit_dtor _nouveau_subdev_dtor
>> +void _nouveau_devinit_dtor(struct nouveau_object *);
>> int _nouveau_devinit_init(struct nouveau_object *);
>> int _nouveau_devinit_fini(struct nouveau_object *, bool suspend);
>>
>> --
>> 1.8.3.2
>>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dmesg_nouveau_new
Type: application/octet-stream
Size: 3928 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/nouveau/attachments/20140119/c45ef770/attachment-0002.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dmesg_nouveau_old
Type: application/octet-stream
Size: 3928 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/nouveau/attachments/20140119/c45ef770/attachment-0003.obj>
More information about the Nouveau
mailing list