On 01/03/16 23:38, Ilia Mirkin wrote:
On Tue, Mar 1, 2016 at 4:36 PM, Martin Peres <martin.pe...@free.fr> wrote:
On 26/02/16 17:19, Karol Herbst wrote:
currently there is no change, because nobody uses those macros yet, but
they
shouldn't stay broken

Signed-off-by: Karol Herbst <nouv...@karolherbst.de>
---
   drm/nouveau/nvkm/subdev/pmu/fuc/macros.fuc | 4 ++--
   1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drm/nouveau/nvkm/subdev/pmu/fuc/macros.fuc
b/drm/nouveau/nvkm/subdev/pmu/fuc/macros.fuc
index 0d5cbeb..bb59eb4 100644
--- a/drm/nouveau/nvkm/subdev/pmu/fuc/macros.fuc
+++ b/drm/nouveau/nvkm/subdev/pmu/fuc/macros.fuc
@@ -252,12 +252,12 @@
   #endif
     #define st(size, addr, reg) /*
-*/     movw $r0 addr /*
+*/     mov $r0 addr /*

First of all, I know it is annoying, but we *need* to understand exactly
what movw is now doing.

Secondly, I seem to remember that a 32 bit mov was not added until fuc3 or
fuc5

Ah, right, fuc3 was on the GT2xx. Thanks!

something. Have you tried assembling this code on older fuc versions? Pretty
sure it will fail.
That's what the imm32() macro is for.

Right, with the mov replace with the imm32 macro, this patch will have my R-b!
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau

Reply via email to