drm/nouveau/fb: finalise big page size selection in constructor

MMU will need to know this during its constructor, so we can't delay
deciding this until init-time.

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

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/base.c b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/base.c
index a7049c041594b6511f52b33331a73e2bc8e3582d..fa0fad10b62cd31c34dd5605217e315d6043aeaa 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/base.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/base.c
@@ -135,8 +135,13 @@ nvkm_fb_init(struct nvkm_subdev *subdev)
 
        if (fb->func->init)
                fb->func->init(fb);
-       if (fb->func->init_page)
-               fb->func->init_page(fb);
+
+       if (fb->func->init_page) {
+               ret = fb->func->init_page(fb);
+               if (WARN_ON(ret))
+                       return ret;
+       }
+
        if (fb->func->init_unkn)
                fb->func->init_unkn(fb);
        return 0;
@@ -176,7 +181,8 @@ nvkm_fb_ctor(const struct nvkm_fb_func *func, struct nvkm_device *device,
        nvkm_subdev_ctor(&nvkm_fb, device, index, &fb->subdev);
        fb->func = func;
        fb->tile.regions = fb->func->tile.regions;
-       fb->page = nvkm_longopt(device->cfgopt, "NvFbBigPage", 0);
+       fb->page = nvkm_longopt(device->cfgopt, "NvFbBigPage",
+                               fb->func->default_bigpage);
 }
 
 int

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf100.c b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf100.c
index a239e73562c8123dae0d432e5ed006bb396a13ce..a308a99ef64e09a419f0382bd631c366871cfcfb 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf100.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf100.c
@@ -80,20 +80,17 @@ gf100_fb_oneinit(struct nvkm_fb *base)
        return 0;
 }
 
-void
+int
 gf100_fb_init_page(struct nvkm_fb *fb)
 {
        struct nvkm_device *device = fb->subdev.device;
        switch (fb->page) {
-       case 16:
-               nvkm_mask(device, 0x100c80, 0x00000001, 0x00000001);
-               break;
-       case 17:
+       case 16: nvkm_mask(device, 0x100c80, 0x00000001, 0x00000001); break;
+       case 17: nvkm_mask(device, 0x100c80, 0x00000001, 0x00000000); break;
        default:
-               nvkm_mask(device, 0x100c80, 0x00000001, 0x00000000);
-               fb->page = 17;
-               break;
+               return -EINVAL;
        }
+       return 0;
 }
 
 void
@@ -144,6 +141,7 @@ gf100_fb = {
        .intr = gf100_fb_intr,
        .ram_new = gf100_ram_new,
        .memtype_valid = gf100_fb_memtype_valid,
+       .default_bigpage = 17,
 };
 
 int

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf100.h b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf100.h
index 412eb89834e8fb041e9b433fee6809de061e6454..e3cf0515bb70a09493cc2a298e92506f324497ce 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf100.h
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf100.h
@@ -17,7 +17,5 @@ void gf100_fb_intr(struct nvkm_fb *);
 
 void gp100_fb_init(struct nvkm_fb *);
 
-void gm200_fb_init_page(struct nvkm_fb *fb);
 void gm200_fb_init(struct nvkm_fb *base);
-
 #endif

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf108.c b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf108.c
index 56af84aa333b0940e792ddaa8a26912f1f07cbdc..4c012fa02de193c246ed5ba18c1321c82ea51358 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf108.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf108.c
@@ -33,6 +33,7 @@ gf108_fb = {
        .intr = gf100_fb_intr,
        .ram_new = gf108_ram_new,
        .memtype_valid = gf100_fb_memtype_valid,
+       .default_bigpage = 17,
 };
 
 int

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gk104.c b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gk104.c
index 4245e2e6e604501fa9ead8ad0ed3582f9afee6cc..7ff647499acfe18c3ca6e124ffc54b18d3c39c52 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gk104.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gk104.c
@@ -33,6 +33,7 @@ gk104_fb = {
        .intr = gf100_fb_intr,
        .ram_new = gk104_ram_new,
        .memtype_valid = gf100_fb_memtype_valid,
+       .default_bigpage = 17,
 };
 
 int

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gk20a.c b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gk20a.c
index 5d34d6136616ac5c58e280c53a6f2a1f0d09b5a9..59308575b1d4fc34f136067c810b8e3b0adaa303 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gk20a.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gk20a.c
@@ -31,6 +31,7 @@ gk20a_fb = {
        .init_page = gf100_fb_init_page,
        .intr = gf100_fb_intr,
        .memtype_valid = gf100_fb_memtype_valid,
+       .default_bigpage = 17,
 };
 
 int

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm107.c b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm107.c
index db699025f5464711c8b02972c32ea32ee5076a59..005aa9d6db3226938d2c9f8450ceb3cccbec90d0 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm107.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm107.c
@@ -33,6 +33,7 @@ gm107_fb = {
        .intr = gf100_fb_intr,
        .ram_new = gm107_ram_new,
        .memtype_valid = gf100_fb_memtype_valid,
+       .default_bigpage = 17,
 };
 
 int

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm200.c b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm200.c
index d83da5ddbc1e82cf477dca32919e43abac8f1435..cc3f90304c1c594b5a7e4a4a1f3b0ce98ecdfa26 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm200.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm200.c
@@ -26,22 +26,18 @@
 
 #include <core/memory.h>
 
-void
+int
 gm200_fb_init_page(struct nvkm_fb *fb)
 {
        struct nvkm_device *device = fb->subdev.device;
        switch (fb->page) {
-       case 16:
-               nvkm_mask(device, 0x100c80, 0x00000801, 0x00000001);
-               break;
-       case 17:
-               nvkm_mask(device, 0x100c80, 0x00000801, 0x00000000);
-               break;
+       case 16: nvkm_mask(device, 0x100c80, 0x00000801, 0x00000001); break;
+       case 17: nvkm_mask(device, 0x100c80, 0x00000801, 0x00000000); break;
+       case  0: nvkm_mask(device, 0x100c80, 0x00000800, 0x00000800); break;
        default:
-               nvkm_mask(device, 0x100c80, 0x00000800, 0x00000800);
-               fb->page = 0;
-               break;
+               return -EINVAL;
        }
+       return 0;
 }
 
 void
@@ -70,6 +66,7 @@ gm200_fb = {
        .intr = gf100_fb_intr,
        .ram_new = gm200_ram_new,
        .memtype_valid = gf100_fb_memtype_valid,
+       .default_bigpage = 0 /* per-instance. */,
 };
 
 int

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm20b.c b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm20b.c
index b87c233bcd6db05c971c783b3337852812d03c74..e66a0a871043b5c6095c6bda584cf395cfe8f1bf 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm20b.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gm20b.c
@@ -31,6 +31,7 @@ gm20b_fb = {
        .init_page = gm200_fb_init_page,
        .intr = gf100_fb_intr,
        .memtype_valid = gf100_fb_memtype_valid,
+       .default_bigpage = 0 /* per-instance. */,
 };
 
 int

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/priv.h b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/priv.h
index e905d44fa1d59a88bf349e8bbcd467f49d7a7a00..afaa68da3221297dd8d539579d502bac1c1570b9 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/fb/priv.h
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/priv.h
@@ -8,7 +8,7 @@ struct nvkm_fb_func {
        void *(*dtor)(struct nvkm_fb *);
        int (*oneinit)(struct nvkm_fb *);
        void (*init)(struct nvkm_fb *);
-       void (*init_page)(struct nvkm_fb *);
+       int (*init_page)(struct nvkm_fb *);
        void (*init_unkn)(struct nvkm_fb *);
        void (*intr)(struct nvkm_fb *);
 
@@ -25,6 +25,8 @@ struct nvkm_fb_func {
        int (*ram_new)(struct nvkm_fb *, struct nvkm_ram **);
 
        bool (*memtype_valid)(struct nvkm_fb *, u32 memtype);
+
+       u8 default_bigpage;
 };
 
 void nvkm_fb_ctor(const struct nvkm_fb_func *, struct nvkm_device *device,
@@ -62,8 +64,8 @@ void nv46_fb_tile_init(struct nvkm_fb *, int i, u32 addr, u32 size,
                       u32 pitch, u32 flags, struct nvkm_fb_tile *);
 
 int gf100_fb_oneinit(struct nvkm_fb *);
-void gf100_fb_init_page(struct nvkm_fb *);
+int gf100_fb_init_page(struct nvkm_fb *);
 bool gf100_fb_memtype_valid(struct nvkm_fb *, u32);
 
-void gm200_fb_init_page(struct nvkm_fb *);
+int gm200_fb_init_page(struct nvkm_fb *);
 #endif