drm/i915/gsc: Fix error code in intel_gsc_uc_heci_cmd_submit_nonpriv()

Dan Carpenter · Andi Shyti · commit 24335848e543 · 2023-06-08T02:11:04.000+02:00
This should return negative -EAGAIN instead of positive EAGAIN. Fixes: e5e1e6d ("drm/i915/pxp: Add MTL helpers to submit Heci-Cmd-Packet to GSC") Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> Reviewed-by: Alan Previn <alan.previn.teres.alexis@intel.com> Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com> Signed-off-by: Andi Shyti <andi.shyti@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/ZH7sr+Vs4zOQoouU@moroto
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_gsc_uc_heci_cmd_submit.c b/drivers/gpu/drm/i915/gt/uc/intel_gsc_uc_heci_cmd_submit.c
@@ -202,7 +202,7 @@ intel_gsc_uc_heci_cmd_submit_nonpriv(struct intel_gsc_uc *gsc,
 			if (++trials < 10)
 				goto retry;
 			else
-				err = EAGAIN;
+				err = -EAGAIN;
 		}
 	}
 	i915_gem_ww_ctx_fini(&ww);

Original file line number	Diff line number	Diff line change
`@@ -202,7 +202,7 @@ intel_gsc_uc_heci_cmd_submit_nonpriv(struct intel_gsc_uc *gsc,`
`202`	`202`	`if (++trials < 10)`
`203`	`203`	`goto retry;`
`204`	`204`	`else`
`205`		`- err = EAGAIN;`
	`205`	`+ err = -EAGAIN;`
`206`	`206`	`}`
`207`	`207`	`}`
`208`	`208`	`i915_gem_ww_ctx_fini(&ww);`