drm/nve0/fb/gddr5: somewhat better attempt at 100770/10f604/610/614

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

fb/gddr5/nve0: 100770 is like 10f604

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

1 files changed, 47 insertions, 14 deletions

diff --git a/drivers/gpu/drm/nouveau/core/subdev/fb/ramnve0.c b/drivers/gpu/drm/nouveau/core/subdev/fb/ramnve0.c
index 659183b858e2..a4c84d6f50e5 100644
--- a/drivers/gpu/drm/nouveau/core/subdev/fb/ramnve0.c
+++ b/drivers/gpu/drm/nouveau/core/subdev/fb/ramnve0.c
@@ -41,6 +41,14 @@
 
 #include "ramfuc.h"
 
+/* binary driver only executes this path if the condition (a) is true
+ * for any configuration (combination of rammap+ramcfg+timing) that
+ * can be reached on a given card.  for now, we will execute the branch
+ * unconditionally in the hope that a "false everywhere" in the bios
+ * tables doesn't actually mean "don't touch this".
+ */
+#define NOTE00(a) 1
+
 struct nve0_ramfuc {
 	struct ramfuc base;
 
@@ -466,23 +474,41 @@ nve0_ram_calc_gddr5(struct nouveau_fb *pfb, u32 freq)
 	ram_mask(fuc, 0x10f2cc, 0xffffffff, nv_ro32(bios, timing + 0x20));
 	ram_mask(fuc, 0x10f2e8, 0xffffffff, nv_ro32(bios, timing + 0x24));
 
-	/*XXX: what's the condition here? */
-	if (1) {
-		data = (nv_ro08(bios, ramcfg + 0x02) & 0x03) << 8;
+	data = mask = 0x00000000;
+	if (NOTE00(ramcfg_02_03 != 0)) {
+		data |= (nv_ro08(bios, ramcfg + 0x02) & 0x03) << 8;
+		mask |= 0x00000300;
+	}
+	if (NOTE00(ramcfg_01_10)) {
 		if (nv_ro08(bios, ramcfg + 0x01) & 0x10)
 			data |= 0x70000000;
-		ram_mask(fuc, 0x10f604, 0x70000300, data);
+		mask |= 0x70000000;
+	}
+	ram_mask(fuc, 0x10f604, mask, data);
 
-		data = (nv_ro08(bios, timing + 0x30) & 0x07) << 28;
+	data = mask = 0x00000000;
+	if (NOTE00(timing_30_07 != 0)) {
+		data |= (nv_ro08(bios, timing + 0x30) & 0x07) << 28;
+		mask |= 0x70000000;
+	}
+	if (NOTE00(ramcfg_01_01)) {
 		if (nv_ro08(bios, ramcfg + 0x01) & 0x01)
 			data |= 0x00000100;
-		ram_mask(fuc, 0x10f614, 0x70000000, data);
+		mask |= 0x00000100;
+	}
+	ram_mask(fuc, 0x10f614, mask, data);
 
-		data = (nv_ro08(bios, timing + 0x30) & 0x07) << 28;
+	data = mask = 0x00000000;
+	if (NOTE00(timing_30_07 != 0)) {
+		data |= (nv_ro08(bios, timing + 0x30) & 0x07) << 28;
+		mask |= 0x70000000;
+	}
+	if (NOTE00(ramcfg_01_02)) {
 		if (nv_ro08(bios, ramcfg + 0x01) & 0x02)
 			data |= 0x00000100;
-		ram_mask(fuc, 0x10f610, 0x70000000, data);
+		mask |= 0x00000100;
 	}
+	ram_mask(fuc, 0x10f610, mask, data);
 
 	mask = 0x33f00000;
 	data = 0x00000000;
@@ -511,15 +537,22 @@ nve0_ram_calc_gddr5(struct nouveau_fb *pfb, u32 freq)
 	data = nv_ro08(bios, ramcfg + 0x03) & 0x0f;
 	ram_wr32(fuc, 0x10f870, 0x11111111 * data);
 
-	data = nv_ro08(bios, ramcfg + 0x02) & 0x03;
-	if (nv_ro08(bios, ramcfg + 0x01) & 0x10)
-		data |= 0x00000004;
-	if ((ram_rd32(fuc, 0x100770) & 0x00000004) != (data & 0x00000004)) {
-		ram_wr32(fuc, 0x100750, 0x04000009);
+	data = mask = 0x00000000;
+	if (NOTE00(ramcfg_02_03 != 0)) {
+		data |= nv_ro08(bios, ramcfg + 0x02) & 0x03;
+		mask |= 0x00000003;
+	}
+	if (NOTE00(ramcfg_01_10)) {
+		if (nv_ro08(bios, ramcfg + 0x01) & 0x10)
+			data |= 0x00000004;
+		mask |= 0x00000004;
+	}
+
+	if ((ram_mask(fuc, 0x100770, mask, data) & mask & 4) != (data & 4)) {
+		ram_mask(fuc, 0x100750, 0x00000008, 0x00000008);
 		ram_wr32(fuc, 0x100710, 0x00000000);
 		ram_wait(fuc, 0x100710, 0x80000000, 0x80000000, 200000);
 	}
-	ram_mask(fuc, 0x100770, 0x00000007, data);
 
 	data = (nv_ro08(bios, timing + 0x30) & 0x07) << 8;
 	if (nv_ro08(bios, ramcfg + 0x01) & 0x01)