Several braces were misplaced unintentionally. That caused the msi handling became part of the default case of the first switch statement. So add the missing ones. Signed-off-by: Vince Hsu <vinceh at nvidia.com> --- nvkm/subdev/mc/base.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/nvkm/subdev/mc/base.c b/nvkm/subdev/mc/base.c index ca7cee3a314a..25e3b9644a3f 100644 --- a/nvkm/subdev/mc/base.c +++ b/nvkm/subdev/mc/base.c @@ -124,7 +124,7 @@ nouveau_mc_create_(struct nouveau_object *parent, struct nouveau_object *engine, pmc->unk260 = nouveau_mc_unk260; - if (nv_device_is_pci(device)) + if (nv_device_is_pci(device)) { switch (device->pdev->device & 0x0ff0) { case 0x00f0: case 0x02e0: @@ -138,6 +138,7 @@ nouveau_mc_create_(struct nouveau_object *parent, struct nouveau_object *engine, default: pmc->use_msi = true; break; + } } pmc->use_msi = nouveau_boolopt(device->cfgopt, "NvMSI", -- 1.9.1