* [PATCH 0/3] fb fixes for gk20a/gm20b
@ 2016-10-27 7:55 Alexandre Courbot
[not found] ` <20161027075554.18791-1-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
0 siblings, 1 reply; 6+ messages in thread
From: Alexandre Courbot @ 2016-10-27 7:55 UTC (permalink / raw)
To: Ben Skeggs; +Cc: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW
After observing random crashes on Tegra devices when patch "fb/gf100: defer DMA
mapping of scratch page to oneinit() hook" was applied, I noticed that moving
the 100c10 page allocation to the oneinit() hook resulted in that page being
now allocated for Tegra as well, and accesses be made to members of gf100_fb
which were not accessible because gk20a_fb_new() called nvkm_fb_new_() directly.
This is fixed by patch 1, but looking at this code made me realize we may just
as well use the regular FB functions for gk20a/gm20b (without the ablity to
allocate VRAM of course), so patches 2 and 3 do that.
Alexandre Courbot (3):
fb/gk20a: fix constructor call
fb/gk20a: use regular gf100's functions
fb: add gm20b device
drm/nouveau/include/nvkm/subdev/fb.h | 1 +
drm/nouveau/nvkm/engine/device/base.c | 2 +-
drm/nouveau/nvkm/subdev/fb/Kbuild | 1 +
drm/nouveau/nvkm/subdev/fb/gf100.h | 4 ++++
drm/nouveau/nvkm/subdev/fb/gk20a.c | 20 ++++++------------
drm/nouveau/nvkm/subdev/fb/gm200.c | 2 +-
drm/nouveau/nvkm/subdev/fb/gm20b.c | 40 +++++++++++++++++++++++++++++++++++
7 files changed, 55 insertions(+), 15 deletions(-)
create mode 100644 drm/nouveau/nvkm/subdev/fb/gm20b.c
--
2.10.0
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau
^ permalink raw reply [flat|nested] 6+ messages in thread[parent not found: <20161027075554.18791-1-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>]
* [PATCH 1/3] fb/gk20a: fix constructor call [not found] ` <20161027075554.18791-1-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> @ 2016-10-27 7:55 ` Alexandre Courbot 2016-10-27 7:55 ` [PATCH 2/3] fb/gk20a: use regular gf100's functions Alexandre Courbot 2016-10-27 7:55 ` [PATCH 3/3] fb: add gm20b device Alexandre Courbot 2 siblings, 0 replies; 6+ messages in thread From: Alexandre Courbot @ 2016-10-27 7:55 UTC (permalink / raw) To: Ben Skeggs; +Cc: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW The gf100 constructor should be called, otherwise we will allocate a smaller object than expected. This was without effect so far because gk20a did not allocate a page, but with gf100's page allocation moved to the oneinit() hook this problem has become apparent. Signed-off-by: Alexandre Courbot <acourbot@nvidia.com> --- drm/nouveau/nvkm/subdev/fb/gk20a.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drm/nouveau/nvkm/subdev/fb/gk20a.c b/drm/nouveau/nvkm/subdev/fb/gk20a.c index f815fe2bbf08..3f516f5adc30 100644 --- a/drm/nouveau/nvkm/subdev/fb/gk20a.c +++ b/drm/nouveau/nvkm/subdev/fb/gk20a.c @@ -20,6 +20,7 @@ * DEALINGS IN THE SOFTWARE. */ #include "priv.h" +#include "gf100.h" #include <core/memory.h> @@ -42,5 +43,5 @@ gk20a_fb = { int gk20a_fb_new(struct nvkm_device *device, int index, struct nvkm_fb **pfb) { - return nvkm_fb_new_(&gk20a_fb, device, index, pfb); + return gf100_fb_new_(&gk20a_fb, device, index, pfb); } -- 2.10.0 _______________________________________________ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/3] fb/gk20a: use regular gf100's functions [not found] ` <20161027075554.18791-1-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-10-27 7:55 ` [PATCH 1/3] fb/gk20a: fix constructor call Alexandre Courbot @ 2016-10-27 7:55 ` Alexandre Courbot [not found] ` <20161027075554.18791-3-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-10-27 7:55 ` [PATCH 3/3] fb: add gm20b device Alexandre Courbot 2 siblings, 1 reply; 6+ messages in thread From: Alexandre Courbot @ 2016-10-27 7:55 UTC (permalink / raw) To: Ben Skeggs; +Cc: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW gk20a's FB is not special compared to other Kepler chips, besides the fact it does not have VRAM. Use the regular gf100 hooks instead of the incomplete versions we rewrote. Signed-off-by: Alexandre Courbot <acourbot@nvidia.com> --- drm/nouveau/nvkm/subdev/fb/gk20a.c | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/drm/nouveau/nvkm/subdev/fb/gk20a.c b/drm/nouveau/nvkm/subdev/fb/gk20a.c index 3f516f5adc30..5d34d6136616 100644 --- a/drm/nouveau/nvkm/subdev/fb/gk20a.c +++ b/drm/nouveau/nvkm/subdev/fb/gk20a.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved. + * Copyright (c) 2014-2016, NVIDIA CORPORATION. All rights reserved. * * Permission is hereby granted, free of charge, to any person obtaining a * copy of this software and associated documentation files (the "Software"), @@ -22,21 +22,14 @@ #include "priv.h" #include "gf100.h" -#include <core/memory.h> - -static void -gk20a_fb_init(struct nvkm_fb *fb) -{ - struct nvkm_device *device = fb->subdev.device; - nvkm_wr32(device, 0x100cc8, nvkm_memory_addr(fb->mmu_wr) >> 8); - nvkm_wr32(device, 0x100ccc, nvkm_memory_addr(fb->mmu_rd) >> 8); -} - +/* GK20A's FB is similar to GF100's, but without the ability to allocate VRAM */ static const struct nvkm_fb_func gk20a_fb = { + .dtor = gf100_fb_dtor, .oneinit = gf100_fb_oneinit, - .init = gk20a_fb_init, + .init = gf100_fb_init, .init_page = gf100_fb_init_page, + .intr = gf100_fb_intr, .memtype_valid = gf100_fb_memtype_valid, }; -- 2.10.0 _______________________________________________ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau ^ permalink raw reply related [flat|nested] 6+ messages in thread
[parent not found: <20161027075554.18791-3-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH 2/3] fb/gk20a: use regular gf100's functions [not found] ` <20161027075554.18791-3-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> @ 2016-10-27 11:52 ` Karol Herbst 0 siblings, 0 replies; 6+ messages in thread From: Karol Herbst @ 2016-10-27 11:52 UTC (permalink / raw) To: Alexandre Courbot; +Cc: ML nouveau, Ben Skeggs looks good Reviewed-By: Karol Herbst <karolherbst@gmail.com> 2016-10-27 9:55 GMT+02:00 Alexandre Courbot <acourbot@nvidia.com>: > gk20a's FB is not special compared to other Kepler chips, besides the > fact it does not have VRAM. Use the regular gf100 hooks instead of the > incomplete versions we rewrote. > > Signed-off-by: Alexandre Courbot <acourbot@nvidia.com> > --- > drm/nouveau/nvkm/subdev/fb/gk20a.c | 17 +++++------------ > 1 file changed, 5 insertions(+), 12 deletions(-) > > diff --git a/drm/nouveau/nvkm/subdev/fb/gk20a.c b/drm/nouveau/nvkm/subdev/fb/gk20a.c > index 3f516f5adc30..5d34d6136616 100644 > --- a/drm/nouveau/nvkm/subdev/fb/gk20a.c > +++ b/drm/nouveau/nvkm/subdev/fb/gk20a.c > @@ -1,5 +1,5 @@ > /* > - * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved. > + * Copyright (c) 2014-2016, NVIDIA CORPORATION. All rights reserved. > * > * Permission is hereby granted, free of charge, to any person obtaining a > * copy of this software and associated documentation files (the "Software"), > @@ -22,21 +22,14 @@ > #include "priv.h" > #include "gf100.h" > > -#include <core/memory.h> > - > -static void > -gk20a_fb_init(struct nvkm_fb *fb) > -{ > - struct nvkm_device *device = fb->subdev.device; > - nvkm_wr32(device, 0x100cc8, nvkm_memory_addr(fb->mmu_wr) >> 8); > - nvkm_wr32(device, 0x100ccc, nvkm_memory_addr(fb->mmu_rd) >> 8); > -} > - > +/* GK20A's FB is similar to GF100's, but without the ability to allocate VRAM */ > static const struct nvkm_fb_func > gk20a_fb = { > + .dtor = gf100_fb_dtor, > .oneinit = gf100_fb_oneinit, > - .init = gk20a_fb_init, > + .init = gf100_fb_init, > .init_page = gf100_fb_init_page, > + .intr = gf100_fb_intr, > .memtype_valid = gf100_fb_memtype_valid, > }; > > -- > 2.10.0 > > _______________________________________________ > Nouveau mailing list > Nouveau@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/nouveau _______________________________________________ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 3/3] fb: add gm20b device [not found] ` <20161027075554.18791-1-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-10-27 7:55 ` [PATCH 1/3] fb/gk20a: fix constructor call Alexandre Courbot 2016-10-27 7:55 ` [PATCH 2/3] fb/gk20a: use regular gf100's functions Alexandre Courbot @ 2016-10-27 7:55 ` Alexandre Courbot [not found] ` <20161027075554.18791-4-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2 siblings, 1 reply; 6+ messages in thread From: Alexandre Courbot @ 2016-10-27 7:55 UTC (permalink / raw) To: Ben Skeggs; +Cc: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW gm20b's FB has the same capabilities as gm200, minus the ability to allocate RAM. Create a device that reflects this instead of re-using the gk20a device which may be incorrect. Signed-off-by: Alexandre Courbot <acourbot@nvidia.com> --- drm/nouveau/include/nvkm/subdev/fb.h | 1 + drm/nouveau/nvkm/engine/device/base.c | 2 +- drm/nouveau/nvkm/subdev/fb/Kbuild | 1 + drm/nouveau/nvkm/subdev/fb/gf100.h | 4 ++++ drm/nouveau/nvkm/subdev/fb/gm200.c | 2 +- drm/nouveau/nvkm/subdev/fb/gm20b.c | 40 +++++++++++++++++++++++++++++++++++ 6 files changed, 48 insertions(+), 2 deletions(-) create mode 100644 drm/nouveau/nvkm/subdev/fb/gm20b.c diff --git a/drm/nouveau/include/nvkm/subdev/fb.h b/drm/nouveau/include/nvkm/subdev/fb.h index 3a410275fa71..cf681065863e 100644 --- a/drm/nouveau/include/nvkm/subdev/fb.h +++ b/drm/nouveau/include/nvkm/subdev/fb.h @@ -93,6 +93,7 @@ int gk104_fb_new(struct nvkm_device *, int, struct nvkm_fb **); int gk20a_fb_new(struct nvkm_device *, int, struct nvkm_fb **); int gm107_fb_new(struct nvkm_device *, int, struct nvkm_fb **); int gm200_fb_new(struct nvkm_device *, int, struct nvkm_fb **); +int gm20b_fb_new(struct nvkm_device *, int, struct nvkm_fb **); int gp100_fb_new(struct nvkm_device *, int, struct nvkm_fb **); int gp104_fb_new(struct nvkm_device *, int, struct nvkm_fb **); diff --git a/drm/nouveau/nvkm/engine/device/base.c b/drm/nouveau/nvkm/engine/device/base.c index 53d171729353..52d932d9df7c 100644 --- a/drm/nouveau/nvkm/engine/device/base.c +++ b/drm/nouveau/nvkm/engine/device/base.c @@ -2131,7 +2131,7 @@ nv12b_chipset = { .bar = gk20a_bar_new, .bus = gf100_bus_new, .clk = gm20b_clk_new, - .fb = gk20a_fb_new, + .fb = gm20b_fb_new, .fuse = gm107_fuse_new, .ibus = gk20a_ibus_new, .imem = gk20a_instmem_new, diff --git a/drm/nouveau/nvkm/subdev/fb/Kbuild b/drm/nouveau/nvkm/subdev/fb/Kbuild index edcc157e6ac8..ef47d57fcb87 100644 --- a/drm/nouveau/nvkm/subdev/fb/Kbuild +++ b/drm/nouveau/nvkm/subdev/fb/Kbuild @@ -24,6 +24,7 @@ nvkm-y += nvkm/subdev/fb/gk104.o nvkm-y += nvkm/subdev/fb/gk20a.o nvkm-y += nvkm/subdev/fb/gm107.o nvkm-y += nvkm/subdev/fb/gm200.o +nvkm-y += nvkm/subdev/fb/gm20b.o nvkm-y += nvkm/subdev/fb/gp100.o nvkm-y += nvkm/subdev/fb/gp104.o diff --git a/drm/nouveau/nvkm/subdev/fb/gf100.h b/drm/nouveau/nvkm/subdev/fb/gf100.h index 449f431644b3..412eb89834e8 100644 --- a/drm/nouveau/nvkm/subdev/fb/gf100.h +++ b/drm/nouveau/nvkm/subdev/fb/gf100.h @@ -16,4 +16,8 @@ void gf100_fb_init(struct nvkm_fb *); 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/drm/nouveau/nvkm/subdev/fb/gm200.c b/drm/nouveau/nvkm/subdev/fb/gm200.c index 62f653240be3..fe5886013ac0 100644 --- a/drm/nouveau/nvkm/subdev/fb/gm200.c +++ b/drm/nouveau/nvkm/subdev/fb/gm200.c @@ -44,7 +44,7 @@ gm200_fb_init_page(struct nvkm_fb *fb) } } -static void +void gm200_fb_init(struct nvkm_fb *base) { struct gf100_fb *fb = gf100_fb(base); diff --git a/drm/nouveau/nvkm/subdev/fb/gm20b.c b/drm/nouveau/nvkm/subdev/fb/gm20b.c new file mode 100644 index 000000000000..b87c233bcd6d --- /dev/null +++ b/drm/nouveau/nvkm/subdev/fb/gm20b.c @@ -0,0 +1,40 @@ +/* + * Copyright (c) 2016, NVIDIA CORPORATION. All rights reserved. + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER + * DEALINGS IN THE SOFTWARE. + */ +#include "priv.h" +#include "gf100.h" + +/* GM20B's FB is similar to GM200, but without the ability to allocate VRAM */ +static const struct nvkm_fb_func +gm20b_fb = { + .dtor = gf100_fb_dtor, + .oneinit = gf100_fb_oneinit, + .init = gm200_fb_init, + .init_page = gm200_fb_init_page, + .intr = gf100_fb_intr, + .memtype_valid = gf100_fb_memtype_valid, +}; + +int +gm20b_fb_new(struct nvkm_device *device, int index, struct nvkm_fb **pfb) +{ + return gf100_fb_new_(&gm20b_fb, device, index, pfb); +} -- 2.10.0 _______________________________________________ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau ^ permalink raw reply related [flat|nested] 6+ messages in thread
[parent not found: <20161027075554.18791-4-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH 3/3] fb: add gm20b device [not found] ` <20161027075554.18791-4-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> @ 2016-10-27 11:52 ` Karol Herbst 0 siblings, 0 replies; 6+ messages in thread From: Karol Herbst @ 2016-10-27 11:52 UTC (permalink / raw) To: Alexandre Courbot; +Cc: ML nouveau, Ben Skeggs Reviewed-By: Karol Herbst <karolherbst@gmail.com> 2016-10-27 9:55 GMT+02:00 Alexandre Courbot <acourbot@nvidia.com>: > gm20b's FB has the same capabilities as gm200, minus the ability to > allocate RAM. Create a device that reflects this instead of re-using the > gk20a device which may be incorrect. > > Signed-off-by: Alexandre Courbot <acourbot@nvidia.com> > --- > drm/nouveau/include/nvkm/subdev/fb.h | 1 + > drm/nouveau/nvkm/engine/device/base.c | 2 +- > drm/nouveau/nvkm/subdev/fb/Kbuild | 1 + > drm/nouveau/nvkm/subdev/fb/gf100.h | 4 ++++ > drm/nouveau/nvkm/subdev/fb/gm200.c | 2 +- > drm/nouveau/nvkm/subdev/fb/gm20b.c | 40 +++++++++++++++++++++++++++++++++++ > 6 files changed, 48 insertions(+), 2 deletions(-) > create mode 100644 drm/nouveau/nvkm/subdev/fb/gm20b.c > > diff --git a/drm/nouveau/include/nvkm/subdev/fb.h b/drm/nouveau/include/nvkm/subdev/fb.h > index 3a410275fa71..cf681065863e 100644 > --- a/drm/nouveau/include/nvkm/subdev/fb.h > +++ b/drm/nouveau/include/nvkm/subdev/fb.h > @@ -93,6 +93,7 @@ int gk104_fb_new(struct nvkm_device *, int, struct nvkm_fb **); > int gk20a_fb_new(struct nvkm_device *, int, struct nvkm_fb **); > int gm107_fb_new(struct nvkm_device *, int, struct nvkm_fb **); > int gm200_fb_new(struct nvkm_device *, int, struct nvkm_fb **); > +int gm20b_fb_new(struct nvkm_device *, int, struct nvkm_fb **); > int gp100_fb_new(struct nvkm_device *, int, struct nvkm_fb **); > int gp104_fb_new(struct nvkm_device *, int, struct nvkm_fb **); > > diff --git a/drm/nouveau/nvkm/engine/device/base.c b/drm/nouveau/nvkm/engine/device/base.c > index 53d171729353..52d932d9df7c 100644 > --- a/drm/nouveau/nvkm/engine/device/base.c > +++ b/drm/nouveau/nvkm/engine/device/base.c > @@ -2131,7 +2131,7 @@ nv12b_chipset = { > .bar = gk20a_bar_new, > .bus = gf100_bus_new, > .clk = gm20b_clk_new, > - .fb = gk20a_fb_new, > + .fb = gm20b_fb_new, > .fuse = gm107_fuse_new, > .ibus = gk20a_ibus_new, > .imem = gk20a_instmem_new, > diff --git a/drm/nouveau/nvkm/subdev/fb/Kbuild b/drm/nouveau/nvkm/subdev/fb/Kbuild > index edcc157e6ac8..ef47d57fcb87 100644 > --- a/drm/nouveau/nvkm/subdev/fb/Kbuild > +++ b/drm/nouveau/nvkm/subdev/fb/Kbuild > @@ -24,6 +24,7 @@ nvkm-y += nvkm/subdev/fb/gk104.o > nvkm-y += nvkm/subdev/fb/gk20a.o > nvkm-y += nvkm/subdev/fb/gm107.o > nvkm-y += nvkm/subdev/fb/gm200.o > +nvkm-y += nvkm/subdev/fb/gm20b.o > nvkm-y += nvkm/subdev/fb/gp100.o > nvkm-y += nvkm/subdev/fb/gp104.o > > diff --git a/drm/nouveau/nvkm/subdev/fb/gf100.h b/drm/nouveau/nvkm/subdev/fb/gf100.h > index 449f431644b3..412eb89834e8 100644 > --- a/drm/nouveau/nvkm/subdev/fb/gf100.h > +++ b/drm/nouveau/nvkm/subdev/fb/gf100.h > @@ -16,4 +16,8 @@ void gf100_fb_init(struct nvkm_fb *); > 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/drm/nouveau/nvkm/subdev/fb/gm200.c b/drm/nouveau/nvkm/subdev/fb/gm200.c > index 62f653240be3..fe5886013ac0 100644 > --- a/drm/nouveau/nvkm/subdev/fb/gm200.c > +++ b/drm/nouveau/nvkm/subdev/fb/gm200.c > @@ -44,7 +44,7 @@ gm200_fb_init_page(struct nvkm_fb *fb) > } > } > > -static void > +void > gm200_fb_init(struct nvkm_fb *base) > { > struct gf100_fb *fb = gf100_fb(base); > diff --git a/drm/nouveau/nvkm/subdev/fb/gm20b.c b/drm/nouveau/nvkm/subdev/fb/gm20b.c > new file mode 100644 > index 000000000000..b87c233bcd6d > --- /dev/null > +++ b/drm/nouveau/nvkm/subdev/fb/gm20b.c > @@ -0,0 +1,40 @@ > +/* > + * Copyright (c) 2016, NVIDIA CORPORATION. All rights reserved. > + * > + * Permission is hereby granted, free of charge, to any person obtaining a > + * copy of this software and associated documentation files (the "Software"), > + * to deal in the Software without restriction, including without limitation > + * the rights to use, copy, modify, merge, publish, distribute, sublicense, > + * and/or sell copies of the Software, and to permit persons to whom the > + * Software is furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice shall be included in > + * all copies or substantial portions of the Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER > + * DEALINGS IN THE SOFTWARE. > + */ > +#include "priv.h" > +#include "gf100.h" > + > +/* GM20B's FB is similar to GM200, but without the ability to allocate VRAM */ > +static const struct nvkm_fb_func > +gm20b_fb = { > + .dtor = gf100_fb_dtor, > + .oneinit = gf100_fb_oneinit, > + .init = gm200_fb_init, > + .init_page = gm200_fb_init_page, > + .intr = gf100_fb_intr, > + .memtype_valid = gf100_fb_memtype_valid, > +}; > + > +int > +gm20b_fb_new(struct nvkm_device *device, int index, struct nvkm_fb **pfb) > +{ > + return gf100_fb_new_(&gm20b_fb, device, index, pfb); > +} > -- > 2.10.0 > > _______________________________________________ > Nouveau mailing list > Nouveau@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/nouveau _______________________________________________ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-10-27 11:52 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-27 7:55 [PATCH 0/3] fb fixes for gk20a/gm20b Alexandre Courbot
[not found] ` <20161027075554.18791-1-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-10-27 7:55 ` [PATCH 1/3] fb/gk20a: fix constructor call Alexandre Courbot
2016-10-27 7:55 ` [PATCH 2/3] fb/gk20a: use regular gf100's functions Alexandre Courbot
[not found] ` <20161027075554.18791-3-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-10-27 11:52 ` Karol Herbst
2016-10-27 7:55 ` [PATCH 3/3] fb: add gm20b device Alexandre Courbot
[not found] ` <20161027075554.18791-4-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-10-27 11:52 ` Karol Herbst
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox