accel/habanalabs: fix gaudi2_get_tpc_idle_status() return

Dan Carpenter · ogabbay · commit 9ec7639b5e12 · 2023-06-08T12:35:55.000+03:00
The gaudi2_get_tpc_idle_status() function returned the incorrect variable so it always returned true. Fixes: d85f053 ("accel/habanalabs: break is_idle function into per-engine sub-routines") Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> Reviewed-by: Oded Gabbay <ogabbay@kernel.org> Signed-off-by: Oded Gabbay <ogabbay@kernel.org>
diff --git a/drivers/accel/habanalabs/gaudi2/gaudi2.c b/drivers/accel/habanalabs/gaudi2/gaudi2.c
@@ -7238,7 +7238,7 @@ static bool gaudi2_get_tpc_idle_status(struct hl_device *hdev, u64 *mask_arr, u8
 
 	gaudi2_iterate_tpcs(hdev, &tpc_iter);
 
-	return tpc_idle_data.is_idle;
+	return *tpc_idle_data.is_idle;
 }
 
 static bool gaudi2_get_decoder_idle_status(struct hl_device *hdev, u64 *mask_arr, u8 mask_len,

Original file line number	Diff line number	Diff line change
`@@ -7238,7 +7238,7 @@ static bool gaudi2_get_tpc_idle_status(struct hl_device hdev, u64 mask_arr, u8`
`7238`	`7238`
`7239`	`7239`	`gaudi2_iterate_tpcs(hdev, &tpc_iter);`
`7240`	`7240`
`7241`		`- return tpc_idle_data.is_idle;`
	`7241`	`+ return *tpc_idle_data.is_idle;`
`7242`	`7242`	`}`
`7243`	`7243`
`7244`	`7244`	`static bool gaudi2_get_decoder_idle_status(struct hl_device hdev, u64 mask_arr, u8 mask_len,`