nouveau: fix devinit paths to only handle display on GSP.This reverts:nouveau/gsp: don't check devinit disable on GSP.and applies a further fix.It turns out the open gpu driver, checks this reg
nouveau: fix devinit paths to only handle display on GSP.This reverts:nouveau/gsp: don't check devinit disable on GSP.and applies a further fix.It turns out the open gpu driver, checks this register,but only for display.Match that behaviour and in the turing path only disablethe display block. (ampere already only does displays).Fixes: 5d4e8ae6e57b ("nouveau/gsp: don't check devinit disable on GSP.")Reviewed-by: Danilo Krummrich <[email protected]>Signed-off-by: Dave Airlie <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
show more ...
nouveau/gsp: don't check devinit disable on GSP.GSP should be handling this and I can see no evidence in opengpudriver that this register should be touched.Fixed acceleration on 2080 Ti GPUs.F
nouveau/gsp: don't check devinit disable on GSP.GSP should be handling this and I can see no evidence in opengpudriver that this register should be touched.Fixed acceleration on 2080 Ti GPUs.Fixes: 15740541e8f0 ("drm/nouveau/devinit/tu102-: prepare for GSP-RM")Signed-off-by: Dave Airlie <[email protected]>Signed-off-by: Danilo Krummrich <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/nouveau/devinit/tu102-: prepare for GSP-RM- add R535 implementation of DEVINIT, we need some of this for boot- add display disable fuse for ga100-Signed-off-by: Ben Skeggs <[email protected]
drm/nouveau/devinit/tu102-: prepare for GSP-RM- add R535 implementation of DEVINIT, we need some of this for boot- add display disable fuse for ga100-Signed-off-by: Ben Skeggs <[email protected]>Signed-off-by: Dave Airlie <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/nouveau/devinit/tu102-: remove attempt at loading PreOSFrom Turing, HW will already have handled this and locked-down thefalcon before we get control. So this *should* be a no-op.Signed-off
drm/nouveau/devinit/tu102-: remove attempt at loading PreOSFrom Turing, HW will already have handled this and locked-down thefalcon before we get control. So this *should* be a no-op.Signed-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>Acked-by: Danilo Krummrich <[email protected]>Signed-off-by: Lyude Paul <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/nouveau/devinit/tu102-: wait for GFW_BOOT_PROGRESS == COMPLETEDStarting from Turing, the driver is no longer responsible for initiatingDEVINIT when required as the GPU started loading a FW ima
drm/nouveau/devinit/tu102-: wait for GFW_BOOT_PROGRESS == COMPLETEDStarting from Turing, the driver is no longer responsible for initiatingDEVINIT when required as the GPU started loading a FW image from ROM andexecuting DEVINIT itself after power-on.However - we apparently still need to wait for it to complete.This should correct some issues with runpm on some systems, where we getcontrol of the HW before it's been fully reinitialised after resume fromsuspend.Signed-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>Signed-off-by: Lyude Paul <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/nouveau/devinit: Convert function disable() to be voidThe current design of callback function disable() of structnvkm_devinit_func is defined to return a u64 value. In its implementationin th
drm/nouveau/devinit: Convert function disable() to be voidThe current design of callback function disable() of structnvkm_devinit_func is defined to return a u64 value. In its implementationin the driver modules, the function always returns a fixed value 0. Hencethe design and implementation of this function should be enhanced to returnvoid instead of a fixed value. This change also eliminates untouchedreturn variables.The change is identified using the returnvar.cocci Coccinelle semanticpatch script.Signed-off-by: Deepak R Varma <[email protected]>Reviewed-by: Lyude Paul <[email protected]>Signed-off-by: Lyude Paul <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/nouveau/pmu: move preinit() falcon reset to devinitCleanup before falcon changes.Signed-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/devinit: fix returnvar.cocci warningsFix the following coccicheck warnings:drivers/gpu/drm/nouveau/nvkm/subdev/devinit/gf100.c:71:5-12:Unneeded variable: "disable". Return "0ULL" on
drm/nouveau/devinit: fix returnvar.cocci warningsFix the following coccicheck warnings:drivers/gpu/drm/nouveau/nvkm/subdev/devinit/gf100.c:71:5-12:Unneeded variable: "disable". Return "0ULL" on line 90.drivers/gpu/drm/nouveau/nvkm/subdev/devinit/gm107.c:35:5-12:Unneeded variable: "disable". Return "0ULL" on line 44.drivers/gpu/drm/nouveau/nvkm/subdev/devinit/g98.c:35:5-12:Unneeded variable: "disable". Return "0ULL" on line 50.Signed-off-by: Guo Zhengkui <[email protected]>Reviewed-by: Lyude Paul <[email protected]>Signed-off-by: Lyude Paul <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/nouveau/devinit/nva3-: fix returnvar.cocci warningFix the following coccicheck warning:drivers/gpu/drm/nouveau/nvkm/subdev/devinit/gt215.c:71:5-12:Unneeded variable: "disable". Return "0ULL"
drm/nouveau/devinit/nva3-: fix returnvar.cocci warningFix the following coccicheck warning:drivers/gpu/drm/nouveau/nvkm/subdev/devinit/gt215.c:71:5-12:Unneeded variable: "disable". Return "0ULL" on line 85.Signed-off-by: Guo Zhengkui <[email protected]>Reviewed-by: Lyude Paul <[email protected]>Signed-off-by: Lyude Paul <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
nouveau/nvkm/subdev/devinit/mcp89.c:Unneeded variabledisable,delete disable and return 0Signed-off-by: tangchunyou <[email protected]>Signed-off-by: Ben Skeggs <[email protected]>Reviewed
nouveau/nvkm/subdev/devinit/mcp89.c:Unneeded variabledisable,delete disable and return 0Signed-off-by: tangchunyou <[email protected]>Signed-off-by: Ben Skeggs <[email protected]>Reviewed-by: Karol Herbst <[email protected]>Signed-off-by: Karol Herbst <[email protected]>Link: https://gitlab.freedesktop.org/drm/nouveau/-/merge_requests/10
drm/nouveau/vic: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/sec: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/msvld: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/msppp: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/mspdec: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/msenc: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/mpeg: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/disp: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/cipher: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/ce: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/bsp,vp: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/devinit: switch to instanced constructorSigned-off-by: Ben Skeggs <[email protected]>Reviewed-by: Lyude Paul <[email protected]>
drm/nouveau/devinit/ga10[024]: initial supportVPLL regs changed a bit. There's more stuff to do around these, but it'sless invasive to stick those changes into disp for now.None of that belongs
drm/nouveau/devinit/ga10[024]: initial supportVPLL regs changed a bit. There's more stuff to do around these, but it'sless invasive to stick those changes into disp for now.None of that belongs here anymore anyhow - fix that someday.Signed-off-by: Ben Skeggs <[email protected]>
drm/nouveau: Use fallthrough pseudo-keywordReplace the existing /* fall through */ comments and its variants withthe new pseudo-keyword macro fallthrough[1]. Also, remove unnecessaryfall-through
drm/nouveau: Use fallthrough pseudo-keywordReplace the existing /* fall through */ comments and its variants withthe new pseudo-keyword macro fallthrough[1]. Also, remove unnecessaryfall-through markings when it is the case.[1] https://www.kernel.org/doc/html/latest/process/deprecated.html?highlight=fallthrough#implicit-switch-case-fall-throughSigned-off-by: Gustavo A. R. Silva <[email protected]>Signed-off-by: Ben Skeggs <[email protected]>
drm/nouveau: fix bogus GPL-2 license headerThe bulk SPDX addition made all these files into GPL-2.0 licensed files.However the remainder of the project is MIT-licensed, these fileswere simply mis
drm/nouveau: fix bogus GPL-2 license headerThe bulk SPDX addition made all these files into GPL-2.0 licensed files.However the remainder of the project is MIT-licensed, these fileswere simply missing the boiler plate and got caught up in the global update.Fixes: 96ac6d4351004 (treewide: Add SPDX license identifier - Kbuild)Signed-off-by: Ben Skeggs <[email protected]>
123