drm/nve0/fb/gddr5: found LP3 setting

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

diff --git a/drivers/gpu/drm/nouveau/core/subdev/fb/gddr5.c b/drivers/gpu/drm/nouveau/core/subdev/fb/gddr5.c
index 1427ae3828d1695ca4f5b881eb1ba25de5f6f26e..f2bbe9e98bda9e826c0ecbcbe3a40a47c2a106ad 100644 (file)
--- a/drivers/gpu/drm/nouveau/core/subdev/fb/gddr5.c
+++ b/drivers/gpu/drm/nouveau/core/subdev/fb/gddr5.c
@@ -29,7 +29,7 @@ int
 nouveau_gddr5_calc(struct nouveau_ram *ram, bool nuts)
 {
        struct nouveau_bios *bios = nouveau_bios(ram);
-       int pd, lf, xd, vh, vr, vo;
+       int pd, lf, xd, vh, vr, vo, l3;
        int WL, CL, WR, at[2], dt, ds;
        int rq = ram->freq < 1000000; /* XXX */
 
@@ -41,6 +41,7 @@ nouveau_gddr5_calc(struct nouveau_ram *ram, bool nuts)
                vh =  (nv_ro08(bios, ram->ramcfg.data + 0x02) & 0x10) >> 4;
                vr =  (nv_ro08(bios, ram->ramcfg.data + 0x02) & 0x04) >> 2;
                vo =   nv_ro08(bios, ram->ramcfg.data + 0x06) & 0xff;
+               l3 = !(nv_ro08(bios, ram->ramcfg.data + 0x07) & 0x02);
                break;
        default:
                return -ENOSYS;
@@ -88,8 +89,8 @@ nouveau_gddr5_calc(struct nouveau_ram *ram, bool nuts)
        ram->mr[3] &= ~0x020;
        ram->mr[3] |= (rq & 0x01) << 5;
 
-       /*XXX: LP3, where's the bit?  Let's hardcode to off for now */
        ram->mr[5] &= ~0x004;
+       ram->mr[5] |= (l3 << 2);
 
        if (!vo)
                vo = (ram->mr[6] & 0xff0) >> 4;

diff --git a/drivers/gpu/drm/nouveau/core/subdev/fb/ramnve0.c b/drivers/gpu/drm/nouveau/core/subdev/fb/ramnve0.c
index e0d63af19107d7c8fce2c81b305f74ae35b9d8b1..6682ec04990084d1cf76e62ec40b95363a3b3b79 100644 (file)
--- a/drivers/gpu/drm/nouveau/core/subdev/fb/ramnve0.c
+++ b/drivers/gpu/drm/nouveau/core/subdev/fb/ramnve0.c
@@ -638,16 +638,16 @@ nve0_ram_calc_gddr5(struct nouveau_fb *pfb, u32 freq)
                data = 0xa40e0000;
        }
        nve0_ram_train(fuc, 0xbc0f0000, data);
-       ram_nsec(fuc, 1000);
+       if (1) /* XXX: not always? */
+               ram_nsec(fuc, 1000);
 
        if (ram->mode == 2) { /*XXX*/
                ram_mask(fuc, 0x10f800, 0x00000004, 0x00000004);
        }
 
-       /* MR5: (re)enable LP3 if necessary
-        * XXX: need to find the switch, keeping off for now
-        */
-       ram_mask(fuc, mr[5], 0xfff, ram->base.mr[5]);
+       /* LP3 */
+       if (ram_mask(fuc, mr[5], 0x004, ram->base.mr[5]) != ram->base.mr[5])
+               ram_nsec(fuc, 1000);
 
        if (ram->mode != 2) {
                ram_mask(fuc, 0x10f830, 0x01000000, 0x01000000);