drm/nouveau/kms/nv50-: use NVIDIA's headers for core crc_set_src()
authorBen Skeggs <bskeggs@redhat.com>
Sun, 19 Jul 2020 21:56:31 +0000 (07:56 +1000)
committerBen Skeggs <bskeggs@redhat.com>
Fri, 24 Jul 2020 08:51:03 +0000 (18:51 +1000)
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Reviewed-by: Lyude Paul <lyude@redhat.com>
drivers/gpu/drm/nouveau/dispnv50/crc907d.c
drivers/gpu/drm/nouveau/dispnv50/crcc37d.c

index 75728dc..3e88422 100644 (file)
@@ -8,6 +8,8 @@
 
 #include <nvif/push507c.h>
 
+#include <nvhw/class/cl907d.h>
+
 #define CRC907D_MAX_ENTRIES 255
 
 struct crc907d_notifier {
@@ -25,30 +27,33 @@ crc907d_set_src(struct nv50_head *head, int or,
                enum nv50_crc_source_type source,
                struct nv50_crc_notifier_ctx *ctx, u32 wndw)
 {
-       struct drm_crtc *crtc = &head->base.base;
        struct nvif_push *push = nv50_disp(head->base.base.dev)->core->chan.push;
        const int i = head->base.index;
-       u32 crc_args = 0xfff00000;
+       u32 crc_args = NVDEF(NV907D, HEAD_SET_CRC_CONTROL, CONTROLLING_CHANNEL, CORE) |
+                      NVDEF(NV907D, HEAD_SET_CRC_CONTROL, EXPECT_BUFFER_COLLAPSE, FALSE) |
+                      NVDEF(NV907D, HEAD_SET_CRC_CONTROL, TIMESTAMP_MODE, FALSE) |
+                      NVDEF(NV907D, HEAD_SET_CRC_CONTROL, SECONDARY_OUTPUT, NONE) |
+                      NVDEF(NV907D, HEAD_SET_CRC_CONTROL, CRC_DURING_SNOOZE, DISABLE);
        int ret;
 
        switch (source) {
        case NV50_CRC_SOURCE_TYPE_SOR:
-               crc_args |= (0x00000f0f + or * 16) << 8;
+               crc_args |= NVDEF(NV907D, HEAD_SET_CRC_CONTROL, PRIMARY_OUTPUT, SOR(or));
                break;
        case NV50_CRC_SOURCE_TYPE_PIOR:
-               crc_args |= (0x000000ff + or * 256) << 8;
+               crc_args |= NVDEF(NV907D, HEAD_SET_CRC_CONTROL, PRIMARY_OUTPUT, PIOR(or));
                break;
        case NV50_CRC_SOURCE_TYPE_DAC:
-               crc_args |= (0x00000ff0 + or) << 8;
+               crc_args |= NVDEF(NV907D, HEAD_SET_CRC_CONTROL, PRIMARY_OUTPUT, DAC(or));
                break;
        case NV50_CRC_SOURCE_TYPE_RG:
-               crc_args |= (0x00000ff8 + drm_crtc_index(crtc)) << 8;
+               crc_args |= NVDEF(NV907D, HEAD_SET_CRC_CONTROL, PRIMARY_OUTPUT, RG(i));
                break;
        case NV50_CRC_SOURCE_TYPE_SF:
-               crc_args |= (0x00000f8f + drm_crtc_index(crtc) * 16) << 8;
+               crc_args |= NVDEF(NV907D, HEAD_SET_CRC_CONTROL, PRIMARY_OUTPUT, SF(i));
                break;
        case NV50_CRC_SOURCE_NONE:
-               crc_args |= 0x000fff00;
+               crc_args |= NVDEF(NV907D, HEAD_SET_CRC_CONTROL, PRIMARY_OUTPUT, NONE);
                break;
        }
 
@@ -56,11 +61,11 @@ crc907d_set_src(struct nv50_head *head, int or,
                return ret;
 
        if (source) {
-               PUSH_NVSQ(push, NV907D, 0x0438 + (i * 0x300), ctx->ntfy.handle);
-               PUSH_NVSQ(push, NV907D, 0x0430 + (i * 0x300), crc_args);
+               PUSH_MTHD(push, NV907D, HEAD_SET_CONTEXT_DMA_CRC(i), ctx->ntfy.handle);
+               PUSH_MTHD(push, NV907D, HEAD_SET_CRC_CONTROL(i), crc_args);
        } else {
-               PUSH_NVSQ(push, NV907D, 0x0430 + (i * 0x300), crc_args);
-               PUSH_NVSQ(push, NV907D, 0x0438 + (i * 0x300), 0);
+               PUSH_MTHD(push, NV907D, HEAD_SET_CRC_CONTROL(i), crc_args);
+               PUSH_MTHD(push, NV907D, HEAD_SET_CONTEXT_DMA_CRC(i), 0);
        }
 
        return 0;
index dd81893..fe68e57 100644 (file)
@@ -8,6 +8,8 @@
 
 #include <nvif/push507c.h>
 
+#include <nvhw/class/clc37d.h>
+
 #define CRCC37D_MAX_ENTRIES 2047
 
 struct crcc37d_notifier {
@@ -39,21 +41,23 @@ crcc37d_set_src(struct nv50_head *head, int or,
 {
        struct nvif_push *push = nv50_disp(head->base.base.dev)->core->chan.push;
        const int i = head->base.index;
-       u32 crc_args;
+       u32 crc_args = NVVAL(NVC37D, HEAD_SET_CRC_CONTROL, CONTROLLING_CHANNEL, wndw) |
+                      NVDEF(NVC37D, HEAD_SET_CRC_CONTROL, EXPECT_BUFFER_COLLAPSE, FALSE) |
+                      NVDEF(NVC37D, HEAD_SET_CRC_CONTROL, SECONDARY_CRC, NONE) |
+                      NVDEF(NVC37D, HEAD_SET_CRC_CONTROL, CRC_DURING_SNOOZE, DISABLE);
        int ret;
 
        switch (source) {
        case NV50_CRC_SOURCE_TYPE_SOR:
-               crc_args = (0x00000050 + or) << 12;
+               crc_args |= NVDEF(NVC37D, HEAD_SET_CRC_CONTROL, PRIMARY_CRC, SOR(or));
                break;
        case NV50_CRC_SOURCE_TYPE_PIOR:
-               crc_args = (0x00000060 + or) << 12;
+               crc_args |= NVDEF(NVC37D, HEAD_SET_CRC_CONTROL, PRIMARY_CRC, PIOR(or));
                break;
        case NV50_CRC_SOURCE_TYPE_SF:
-               crc_args = 0x00000030 << 12;
+               crc_args |= NVDEF(NVC37D, HEAD_SET_CRC_CONTROL, PRIMARY_CRC, SF);
                break;
        default:
-               crc_args = 0;
                break;
        }
 
@@ -61,11 +65,11 @@ crcc37d_set_src(struct nv50_head *head, int or,
                return ret;
 
        if (source) {
-               PUSH_NVSQ(push, NVC37D, 0x2180 + (i * 0x400), ctx->ntfy.handle);
-               PUSH_NVSQ(push, NVC37D, 0x2184 + (i * 0x400), crc_args | wndw);
+               PUSH_MTHD(push, NVC37D, HEAD_SET_CONTEXT_DMA_CRC(i), ctx->ntfy.handle);
+               PUSH_MTHD(push, NVC37D, HEAD_SET_CRC_CONTROL(i), crc_args);
        } else {
-               PUSH_NVSQ(push, NVC37D, 0x2184 + (i * 0x400), 0);
-               PUSH_NVSQ(push, NVC37D, 0x2180 + (i * 0x400), 0);
+               PUSH_MTHD(push, NVC37D, HEAD_SET_CRC_CONTROL(i), 0);
+               PUSH_MTHD(push, NVC37D, HEAD_SET_CONTEXT_DMA_CRC(i), 0);
        }
 
        return 0;