[RFC PATCH] drm/nouveau: split off nvc0 compilation
Ilia Mirkin
imirkin at alum.mit.edu
Fri Feb 14 20:14:52 PST 2014
On Fri, Feb 14, 2014 at 9:32 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> On Fri, Feb 14, 2014 at 7:38 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
>> So... I was wondering what the impact of splitting up the card compilation by
>> e.g. generation would be. Depending on the split things would get fairly
>> intertwined, so I thought I'd start small. This just splits NVC0 from
>> everything else. I figure that for the people this matters the most to, NVC0
>> is the least relevant card -- people with sub-1GB of RAM, older hardware.
>>
>> With my config options and everything compiled, the total size of nouveau.ko
>> is 1343324 bytes (incl symbols), which is split up
>>
>> text data bss dec hex filename
>> 579171 184264 480 763915 ba80b ./drivers/gpu/drm/nouveau/nouveau.ko
>>
>> Turning off nvc0+ gives us 995667 bytes, split up
>>
>> text data bss dec hex filename
>> 472244 59528 480 532252 81f1c ./drivers/gpu/drm/nouveau/nouveau.ko
>>
>> So we lost 100K of text and 130K of data (probably all that pgraph fw), and
>> the remaining 100K must be symbols. (I'm not stripping because nobody in their
>> right mind strips these kernel modules.)
>
> In a local patch that also lops off pre-nv50 cards (with one small
> problem that involves moving a function somewhere -- seems like
> nouveau_crtc_set_config is defined in dispnv04 but used outside of it
> -- which I haven't done yet) the module size goes down to 721366,
> split up as:
>
> text data bss dec hex filename
> 346212 34727 480 381419 5d1eb ./drivers/gpu/drm/nouveau/nouveau.ko
>
> So another 130K of text, 26K of data, and 130K in symbols (?). I
> didn't have to do anything too dirty, with the mild exception of the
> device/base.c file which had that giant if statement, which will have
> to be done a bit more cleverly now to not end up as a maze. My final
> exercise will be to try to lop off nv50 as well, making the driver
> ~useless, but still interesting as it would allow one to only compile
> one of the 3 categories (or any combo).
Here's some further size info:
Nothing: 416118 bytes
text data bss dec hex filename
198669 9525 80 208274 32d92 drivers/gpu/drm/nouveau/nouveau.ko
NV04: 710270 bytes
text data bss dec hex filename
333755 35094 160 369009 5a171 drivers/gpu/drm/nouveau/nouveau.ko
NV50: 722079 bytes
text data bss dec hex filename
346476 34727 480 381683 5d2f3 drivers/gpu/drm/nouveau/nouveau.ko
NVC0: 878726 bytes
text data bss dec hex filename
360593 140327 80 501000 7a508 drivers/gpu/drm/nouveau/nouveau.ko
NV04 + NV50: 995817 bytes
text data bss dec hex filename
472244 59528 480 532252 81f1c drivers/gpu/drm/nouveau/nouveau.ko
NV04 + NVC0: 1167727 bytes
text data bss dec hex filename
493567 165704 160 659431 a0fe7 drivers/gpu/drm/nouveau/nouveau.ko
NV50 + NVC0: 1070110 bytes
text data bss dec hex filename
453459 159463 480 613402 95c1a drivers/gpu/drm/nouveau/nouveau.ko
NV04 + NV50 + NVC0: 1343482 bytes
text data bss dec hex filename
579171 184264 480 763915 ba80b drivers/gpu/drm/nouveau/nouveau.ko
The final patch that does all this is not much dirtier than this
patch, just more changes to the Makefile. I'll hold off on sending it
out though, awaiting some feedback on whether people think this is
useful and/or should be done differently.
>
>>
>> The impact to the code is pretty minor. Depending on the situation it's easier
>> to either ifdef out code or to provide stubs for functions.
>>
>> What do people think about this? It shouldn't be too difficult for me to
>> extend this to categories like <pre-nv50> <nv50-nvc0> <nvc0-nve0> <nve0+>, or
>> maybe even more fine-grained, esp since nvd0 is when a lot of changes
>> happened. Or maybe less-fine-grained and keep the nvc0/nve0 cards under one
>> option.
>>
>> Feedback appreciated.
>>
>> Not-Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
>>
>> ---
>>
>> drivers/gpu/drm/nouveau/Kconfig | 11 +-
>> drivers/gpu/drm/nouveau/Makefile | 136 +++++++++++-----------
>> drivers/gpu/drm/nouveau/core/engine/device/base.c | 2 +
>> drivers/gpu/drm/nouveau/nouveau_drm.c | 4 +-
>> drivers/gpu/drm/nouveau/nouveau_fbcon.h | 16 +++
>> 5 files changed, 98 insertions(+), 71 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/nouveau/Kconfig b/drivers/gpu/drm/nouveau/Kconfig
>> index 7cf787d..927fc38 100644
>> --- a/drivers/gpu/drm/nouveau/Kconfig
>> +++ b/drivers/gpu/drm/nouveau/Kconfig
>> @@ -26,9 +26,14 @@ config DRM_NOUVEAU
>> help
>> Choose this option for open-source nVidia support.
>>
>> +if DRM_NOUVEAU
>> +
>> +config DRM_NOUVEAU_NVC0
>> + bool "NVC0+ (Fermi / Kepler)"
>> + default y
>> +
>> config NOUVEAU_DEBUG
>> int "Maximum debug level"
>> - depends on DRM_NOUVEAU
>> range 0 7
>> default 5
>> help
>> @@ -48,7 +53,6 @@ config NOUVEAU_DEBUG
>>
>> config NOUVEAU_DEBUG_DEFAULT
>> int "Default debug level"
>> - depends on DRM_NOUVEAU
>> range 0 7
>> default 3
>> help
>> @@ -56,8 +60,9 @@ config NOUVEAU_DEBUG_DEFAULT
>>
>> config DRM_NOUVEAU_BACKLIGHT
>> bool "Support for backlight control"
>> - depends on DRM_NOUVEAU
>> default y
>> help
>> Say Y here if you want to control the backlight of your display
>> (e.g. a laptop panel).
>> +
>> +endif
>> diff --git a/drivers/gpu/drm/nouveau/Makefile b/drivers/gpu/drm/nouveau/Makefile
>> index d310c19..a1e8513 100644
>> --- a/drivers/gpu/drm/nouveau/Makefile
>> +++ b/drivers/gpu/drm/nouveau/Makefile
>> @@ -25,7 +25,7 @@ nouveau-y += core/core/subdev.o
>>
>> nouveau-y += core/subdev/bar/base.o
>> nouveau-y += core/subdev/bar/nv50.o
>> -nouveau-y += core/subdev/bar/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/bar/nvc0.o
>> nouveau-y += core/subdev/bios/base.o
>> nouveau-y += core/subdev/bios/bit.o
>> nouveau-y += core/subdev/bios/boost.o
>> @@ -53,7 +53,7 @@ nouveau-y += core/subdev/bus/nv04.o
>> nouveau-y += core/subdev/bus/nv31.o
>> nouveau-y += core/subdev/bus/nv50.o
>> nouveau-y += core/subdev/bus/nv94.o
>> -nouveau-y += core/subdev/bus/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/bus/nvc0.o
>> nouveau-y += core/subdev/clock/base.o
>> nouveau-y += core/subdev/clock/nv04.o
>> nouveau-y += core/subdev/clock/nv40.o
>> @@ -61,8 +61,8 @@ nouveau-y += core/subdev/clock/nv50.o
>> nouveau-y += core/subdev/clock/nv84.o
>> nouveau-y += core/subdev/clock/nva3.o
>> nouveau-y += core/subdev/clock/nvaa.o
>> -nouveau-y += core/subdev/clock/nvc0.o
>> -nouveau-y += core/subdev/clock/nve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/clock/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/clock/nve0.o
>> nouveau-y += core/subdev/clock/pllnv04.o
>> nouveau-y += core/subdev/clock/pllnva3.o
>> nouveau-y += core/subdev/devinit/base.o
>> @@ -76,7 +76,7 @@ nouveau-y += core/subdev/devinit/nv84.o
>> nouveau-y += core/subdev/devinit/nv98.o
>> nouveau-y += core/subdev/devinit/nva3.o
>> nouveau-y += core/subdev/devinit/nvaf.o
>> -nouveau-y += core/subdev/devinit/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/devinit/nvc0.o
>> nouveau-y += core/subdev/fb/base.o
>> nouveau-y += core/subdev/fb/nv04.o
>> nouveau-y += core/subdev/fb/nv10.o
>> @@ -98,8 +98,8 @@ nouveau-y += core/subdev/fb/nv84.o
>> nouveau-y += core/subdev/fb/nva3.o
>> nouveau-y += core/subdev/fb/nvaa.o
>> nouveau-y += core/subdev/fb/nvaf.o
>> -nouveau-y += core/subdev/fb/nvc0.o
>> -nouveau-y += core/subdev/fb/nve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/fb/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/fb/nve0.o
>> nouveau-y += core/subdev/fb/ramnv04.o
>> nouveau-y += core/subdev/fb/ramnv10.o
>> nouveau-y += core/subdev/fb/ramnv1a.o
>> @@ -112,15 +112,15 @@ nouveau-y += core/subdev/fb/ramnv4e.o
>> nouveau-y += core/subdev/fb/ramnv50.o
>> nouveau-y += core/subdev/fb/ramnva3.o
>> nouveau-y += core/subdev/fb/ramnvaa.o
>> -nouveau-y += core/subdev/fb/ramnvc0.o
>> -nouveau-y += core/subdev/fb/ramnve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/fb/ramnvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/fb/ramnve0.o
>> nouveau-y += core/subdev/fb/sddr3.o
>> nouveau-y += core/subdev/fb/gddr5.o
>> nouveau-y += core/subdev/gpio/base.o
>> nouveau-y += core/subdev/gpio/nv10.o
>> nouveau-y += core/subdev/gpio/nv50.o
>> -nouveau-y += core/subdev/gpio/nvd0.o
>> -nouveau-y += core/subdev/gpio/nve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/gpio/nvd0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/gpio/nve0.o
>> nouveau-y += core/subdev/i2c/base.o
>> nouveau-y += core/subdev/i2c/anx9805.o
>> nouveau-y += core/subdev/i2c/aux.o
>> @@ -129,14 +129,14 @@ nouveau-y += core/subdev/i2c/nv04.o
>> nouveau-y += core/subdev/i2c/nv4e.o
>> nouveau-y += core/subdev/i2c/nv50.o
>> nouveau-y += core/subdev/i2c/nv94.o
>> -nouveau-y += core/subdev/i2c/nvd0.o
>> -nouveau-y += core/subdev/ibus/nvc0.o
>> -nouveau-y += core/subdev/ibus/nve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/i2c/nvd0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/ibus/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/ibus/nve0.o
>> nouveau-y += core/subdev/instmem/base.o
>> nouveau-y += core/subdev/instmem/nv04.o
>> nouveau-y += core/subdev/instmem/nv40.o
>> nouveau-y += core/subdev/instmem/nv50.o
>> -nouveau-y += core/subdev/ltcg/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/ltcg/nvc0.o
>> nouveau-y += core/subdev/mc/base.o
>> nouveau-y += core/subdev/mc/nv04.o
>> nouveau-y += core/subdev/mc/nv40.o
>> @@ -145,17 +145,17 @@ nouveau-y += core/subdev/mc/nv4c.o
>> nouveau-y += core/subdev/mc/nv50.o
>> nouveau-y += core/subdev/mc/nv94.o
>> nouveau-y += core/subdev/mc/nv98.o
>> -nouveau-y += core/subdev/mc/nvc0.o
>> -nouveau-y += core/subdev/mc/nvc3.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/mc/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/mc/nvc3.o
>> nouveau-y += core/subdev/mxm/base.o
>> nouveau-y += core/subdev/mxm/mxms.o
>> nouveau-y += core/subdev/mxm/nv50.o
>> nouveau-y += core/subdev/pwr/base.o
>> nouveau-y += core/subdev/pwr/memx.o
>> nouveau-y += core/subdev/pwr/nva3.o
>> -nouveau-y += core/subdev/pwr/nvc0.o
>> -nouveau-y += core/subdev/pwr/nvd0.o
>> -nouveau-y += core/subdev/pwr/nv108.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/pwr/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/pwr/nvd0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/pwr/nv108.o
>> nouveau-y += core/subdev/therm/base.o
>> nouveau-y += core/subdev/therm/fan.o
>> nouveau-y += core/subdev/therm/fannil.o
>> @@ -167,7 +167,7 @@ nouveau-y += core/subdev/therm/nv40.o
>> nouveau-y += core/subdev/therm/nv50.o
>> nouveau-y += core/subdev/therm/nv84.o
>> nouveau-y += core/subdev/therm/nva3.o
>> -nouveau-y += core/subdev/therm/nvd0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/therm/nvd0.o
>> nouveau-y += core/subdev/timer/base.o
>> nouveau-y += core/subdev/timer/nv04.o
>> nouveau-y += core/subdev/vm/base.o
>> @@ -175,7 +175,7 @@ nouveau-y += core/subdev/vm/nv04.o
>> nouveau-y += core/subdev/vm/nv41.o
>> nouveau-y += core/subdev/vm/nv44.o
>> nouveau-y += core/subdev/vm/nv50.o
>> -nouveau-y += core/subdev/vm/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/subdev/vm/nvc0.o
>> nouveau-y += core/subdev/volt/base.o
>> nouveau-y += core/subdev/volt/gpio.o
>> nouveau-y += core/subdev/volt/nv40.o
>> @@ -185,15 +185,15 @@ nouveau-y += core/engine/xtensa.o
>> nouveau-y += core/engine/dmaobj/base.o
>> nouveau-y += core/engine/dmaobj/nv04.o
>> nouveau-y += core/engine/dmaobj/nv50.o
>> -nouveau-y += core/engine/dmaobj/nvc0.o
>> -nouveau-y += core/engine/dmaobj/nvd0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/dmaobj/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/dmaobj/nvd0.o
>> nouveau-y += core/engine/bsp/nv84.o
>> nouveau-y += core/engine/bsp/nv98.o
>> -nouveau-y += core/engine/bsp/nvc0.o
>> -nouveau-y += core/engine/bsp/nve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/bsp/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/bsp/nve0.o
>> nouveau-y += core/engine/copy/nva3.o
>> -nouveau-y += core/engine/copy/nvc0.o
>> -nouveau-y += core/engine/copy/nve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/copy/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/copy/nve0.o
>> nouveau-y += core/engine/crypt/nv84.o
>> nouveau-y += core/engine/crypt/nv98.o
>> nouveau-y += core/engine/device/base.o
>> @@ -204,8 +204,8 @@ nouveau-y += core/engine/device/nv20.o
>> nouveau-y += core/engine/device/nv30.o
>> nouveau-y += core/engine/device/nv40.o
>> nouveau-y += core/engine/device/nv50.o
>> -nouveau-y += core/engine/device/nvc0.o
>> -nouveau-y += core/engine/device/nve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/device/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/device/nve0.o
>> nouveau-y += core/engine/disp/base.o
>> nouveau-y += core/engine/disp/nv04.o
>> nouveau-y += core/engine/disp/nv50.o
>> @@ -213,20 +213,20 @@ nouveau-y += core/engine/disp/nv84.o
>> nouveau-y += core/engine/disp/nv94.o
>> nouveau-y += core/engine/disp/nva0.o
>> nouveau-y += core/engine/disp/nva3.o
>> -nouveau-y += core/engine/disp/nvd0.o
>> -nouveau-y += core/engine/disp/nve0.o
>> -nouveau-y += core/engine/disp/nvf0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/disp/nvd0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/disp/nve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/disp/nvf0.o
>> nouveau-y += core/engine/disp/dacnv50.o
>> nouveau-y += core/engine/disp/dport.o
>> nouveau-y += core/engine/disp/hdanva3.o
>> -nouveau-y += core/engine/disp/hdanvd0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/disp/hdanvd0.o
>> nouveau-y += core/engine/disp/hdminv84.o
>> nouveau-y += core/engine/disp/hdminva3.o
>> -nouveau-y += core/engine/disp/hdminvd0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/disp/hdminvd0.o
>> nouveau-y += core/engine/disp/piornv50.o
>> nouveau-y += core/engine/disp/sornv50.o
>> nouveau-y += core/engine/disp/sornv94.o
>> -nouveau-y += core/engine/disp/sornvd0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/disp/sornvd0.o
>> nouveau-y += core/engine/disp/vga.o
>> nouveau-y += core/engine/fifo/base.o
>> nouveau-y += core/engine/fifo/nv04.o
>> @@ -235,20 +235,20 @@ nouveau-y += core/engine/fifo/nv17.o
>> nouveau-y += core/engine/fifo/nv40.o
>> nouveau-y += core/engine/fifo/nv50.o
>> nouveau-y += core/engine/fifo/nv84.o
>> -nouveau-y += core/engine/fifo/nvc0.o
>> -nouveau-y += core/engine/fifo/nve0.o
>> -nouveau-y += core/engine/fifo/nv108.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/fifo/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/fifo/nve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/fifo/nv108.o
>> nouveau-y += core/engine/graph/ctxnv40.o
>> nouveau-y += core/engine/graph/ctxnv50.o
>> -nouveau-y += core/engine/graph/ctxnvc0.o
>> -nouveau-y += core/engine/graph/ctxnvc1.o
>> -nouveau-y += core/engine/graph/ctxnvc3.o
>> -nouveau-y += core/engine/graph/ctxnvc8.o
>> -nouveau-y += core/engine/graph/ctxnvd7.o
>> -nouveau-y += core/engine/graph/ctxnvd9.o
>> -nouveau-y += core/engine/graph/ctxnve4.o
>> -nouveau-y += core/engine/graph/ctxnvf0.o
>> -nouveau-y += core/engine/graph/ctxnv108.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/ctxnvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/ctxnvc1.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/ctxnvc3.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/ctxnvc8.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/ctxnvd7.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/ctxnvd9.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/ctxnve4.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/ctxnvf0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/ctxnv108.o
>> nouveau-y += core/engine/graph/nv04.o
>> nouveau-y += core/engine/graph/nv10.o
>> nouveau-y += core/engine/graph/nv20.o
>> @@ -259,15 +259,15 @@ nouveau-y += core/engine/graph/nv34.o
>> nouveau-y += core/engine/graph/nv35.o
>> nouveau-y += core/engine/graph/nv40.o
>> nouveau-y += core/engine/graph/nv50.o
>> -nouveau-y += core/engine/graph/nvc0.o
>> -nouveau-y += core/engine/graph/nvc1.o
>> -nouveau-y += core/engine/graph/nvc3.o
>> -nouveau-y += core/engine/graph/nvc8.o
>> -nouveau-y += core/engine/graph/nvd7.o
>> -nouveau-y += core/engine/graph/nvd9.o
>> -nouveau-y += core/engine/graph/nve4.o
>> -nouveau-y += core/engine/graph/nvf0.o
>> -nouveau-y += core/engine/graph/nv108.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/nvc1.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/nvc3.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/nvc8.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/nvd7.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/nvd9.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/nve4.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/nvf0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/graph/nv108.o
>> nouveau-y += core/engine/mpeg/nv31.o
>> nouveau-y += core/engine/mpeg/nv40.o
>> nouveau-y += core/engine/mpeg/nv44.o
>> @@ -279,19 +279,19 @@ nouveau-y += core/engine/perfmon/nv40.o
>> nouveau-y += core/engine/perfmon/nv50.o
>> nouveau-y += core/engine/perfmon/nv84.o
>> nouveau-y += core/engine/perfmon/nva3.o
>> -nouveau-y += core/engine/perfmon/nvc0.o
>> -nouveau-y += core/engine/perfmon/nve0.o
>> -nouveau-y += core/engine/perfmon/nvf0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/perfmon/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/perfmon/nve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/perfmon/nvf0.o
>> nouveau-y += core/engine/ppp/nv98.o
>> -nouveau-y += core/engine/ppp/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/ppp/nvc0.o
>> nouveau-y += core/engine/software/nv04.o
>> nouveau-y += core/engine/software/nv10.o
>> nouveau-y += core/engine/software/nv50.o
>> -nouveau-y += core/engine/software/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/software/nvc0.o
>> nouveau-y += core/engine/vp/nv84.o
>> nouveau-y += core/engine/vp/nv98.o
>> -nouveau-y += core/engine/vp/nvc0.o
>> -nouveau-y += core/engine/vp/nve0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/vp/nvc0.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += core/engine/vp/nve0.o
>>
>> # drm/core
>> nouveau-y += nouveau_drm.o nouveau_chan.o nouveau_dma.o nouveau_fence.o
>> @@ -299,12 +299,14 @@ nouveau-y += nouveau_vga.o nouveau_agp.o
>> nouveau-y += nouveau_ttm.o nouveau_sgdma.o nouveau_bo.o nouveau_gem.o
>> nouveau-y += nouveau_prime.o nouveau_abi16.o
>> nouveau-y += nv04_fence.o nv10_fence.o nv17_fence.o
>> -nouveau-y += nv50_fence.o nv84_fence.o nvc0_fence.o
>> +nouveau-y += nv50_fence.o nv84_fence.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += nvc0_fence.o
>>
>> # drm/kms
>> nouveau-y += nouveau_bios.o nouveau_fbcon.o nouveau_display.o
>> nouveau-y += nouveau_connector.o nouveau_dp.o
>> -nouveau-y += nv04_fbcon.o nv50_fbcon.o nvc0_fbcon.o
>> +nouveau-y += nv04_fbcon.o nv50_fbcon.o
>> +nouveau-$(CONFIG_DRM_NOUVEAU_NVC0) += nvc0_fbcon.o
>>
>> # drm/kms/nv04:nv50
>> include $(src)/dispnv04/Makefile
>> @@ -323,4 +325,4 @@ endif
>> nouveau-$(CONFIG_DRM_NOUVEAU_BACKLIGHT) += nouveau_backlight.o
>> nouveau-$(CONFIG_DEBUG_FS) += nouveau_debugfs.o
>>
>> -obj-$(CONFIG_DRM_NOUVEAU)+= nouveau.o
>> +obj-$(CONFIG_DRM_NOUVEAU) += nouveau.o
>> diff --git a/drivers/gpu/drm/nouveau/core/engine/device/base.c b/drivers/gpu/drm/nouveau/core/engine/device/base.c
>> index dd01c6c..c66953b 100644
>> --- a/drivers/gpu/drm/nouveau/core/engine/device/base.c
>> +++ b/drivers/gpu/drm/nouveau/core/engine/device/base.c
>> @@ -205,9 +205,11 @@ nouveau_devobj_ctor(struct nouveau_object *parent,
>> case NV_30: ret = nv30_identify(device); break;
>> case NV_40: ret = nv40_identify(device); break;
>> case NV_50: ret = nv50_identify(device); break;
>> +#ifdef CONFIG_DRM_NOUVEAU_NVC0
>> case NV_C0:
>> case NV_D0: ret = nvc0_identify(device); break;
>> case NV_E0: ret = nve0_identify(device); break;
>> +#endif
>> default:
>> ret = -EINVAL;
>> break;
>> diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c b/drivers/gpu/drm/nouveau/nouveau_drm.c
>> index 9c9ce4d..875c6a0 100644
>> --- a/drivers/gpu/drm/nouveau/nouveau_drm.c
>> +++ b/drivers/gpu/drm/nouveau/nouveau_drm.c
>> @@ -138,7 +138,7 @@ nouveau_accel_init(struct nouveau_drm *drm)
>> struct nouveau_device *device = nv_device(drm->device);
>> struct nouveau_object *object;
>> u32 arg0, arg1;
>> - int ret;
>> + int ret = -ENODEV;
>>
>> if (nouveau_noaccel || !nouveau_fifo(device) /*XXX*/)
>> return;
>> @@ -150,7 +150,9 @@ nouveau_accel_init(struct nouveau_drm *drm)
>> else if (device->card_type < NV_50) ret = nv17_fence_create(drm);
>> else if (device->chipset < 0x84) ret = nv50_fence_create(drm);
>> else if (device->card_type < NV_C0) ret = nv84_fence_create(drm);
>> +#ifdef CONFIG_DRM_NOUVEAU_NVC0
>> else ret = nvc0_fence_create(drm);
>> +#endif
>> if (ret) {
>> NV_ERROR(drm, "failed to initialise sync subsystem, %d\n", ret);
>> nouveau_accel_fini(drm);
>> diff --git a/drivers/gpu/drm/nouveau/nouveau_fbcon.h b/drivers/gpu/drm/nouveau/nouveau_fbcon.h
>> index fdfc0c9..b4ff8d5 100644
>> --- a/drivers/gpu/drm/nouveau/nouveau_fbcon.h
>> +++ b/drivers/gpu/drm/nouveau/nouveau_fbcon.h
>> @@ -51,10 +51,26 @@ int nv50_fbcon_copyarea(struct fb_info *info, const struct fb_copyarea *region);
>> int nv50_fbcon_imageblit(struct fb_info *info, const struct fb_image *image);
>> int nv50_fbcon_accel_init(struct fb_info *info);
>>
>> +#ifdef CONFIG_DRM_NOUVEAU_NVC0
>> int nvc0_fbcon_fillrect(struct fb_info *info, const struct fb_fillrect *rect);
>> int nvc0_fbcon_copyarea(struct fb_info *info, const struct fb_copyarea *region);
>> int nvc0_fbcon_imageblit(struct fb_info *info, const struct fb_image *image);
>> int nvc0_fbcon_accel_init(struct fb_info *info);
>> +#else
>> +#define _STUB(name, type) \
>> +static inline int name(struct fb_info *info, const struct type *t) \
>> +{ \
>> + return -ENODEV; \
>> +}
>> +_STUB(nvc0_fbcon_fillrect, fb_fillrect)
>> +_STUB(nvc0_fbcon_copyarea, fb_copyarea)
>> +_STUB(nvc0_fbcon_imageblit, fb_image)
>> +#undef _STUB
>> +static inline int nvc0_fbcon_accel_init(struct fb_info *info)
>> +{
>> + return -ENODEV;
>> +}
>> +#endif
>>
>> void nouveau_fbcon_gpu_lockup(struct fb_info *info);
>>
>> --
>> 1.8.3.2
>>
More information about the dri-devel
mailing list