Skip to contentSkip to conversation

gr: fixup for legacy paths firmware lookup

PATCH New Alexandre Courbot 1 participant 1 message Nov 7, 2016

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 gr: fixup for legacy paths firmware lookup

The code would print a NULL string if no legacy firmware was found.

---
Hi Ben,

Sorry about the obvious mistake (and thanks Peter for reporting it!). You may want to squash this one into the previous patch if there still is time.

 drm/nouveau/nvkm/engine/gr/gf100.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)
drm/nouveau/nvkm/engine/gr/gf100.c +2 -5
index ed45f92..60a1b5c 100644
@@ -1773,11 +1773,8 @@ gf100_gr_ctor_fw_legacy(struct gf100_gr *gr, const char *fwname,
 		fwname = "fuc41ac";
 	else if (!strcmp(fwname, "gpccs_data"))
 		fwname = "fuc41ad";
-	else
-		fwname = NULL;
-
-	/* nope, let's just return the error we got */
-	if (!fwname) {
+	else {
+		/* nope, let's just return the error we got */
 		nvkm_error(subdev, "failed to load %s\n", fwname);
 		return ret;
 	}
Signature · 6 lines
2.10.1

_______________________________________________
Nouveau mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/nouveau