drm/amdgpu/soc15: return proper error codes in baco reset
Rather than just -1. Reviewed-by: JimQu <Jim.Qu@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
@@ -427,7 +427,7 @@ static int soc15_asic_get_baco_capability(struct amdgpu_device *adev, bool *cap)
|
|||||||
|
|
||||||
if (!pp_funcs || !pp_funcs->get_asic_baco_capability) {
|
if (!pp_funcs || !pp_funcs->get_asic_baco_capability) {
|
||||||
*cap = false;
|
*cap = false;
|
||||||
return -1;
|
return -ENOENT;
|
||||||
}
|
}
|
||||||
|
|
||||||
return pp_funcs->get_asic_baco_capability(pp_handle, cap);
|
return pp_funcs->get_asic_baco_capability(pp_handle, cap);
|
||||||
@@ -439,15 +439,15 @@ static int soc15_asic_baco_reset(struct amdgpu_device *adev)
|
|||||||
const struct amd_pm_funcs *pp_funcs = adev->powerplay.pp_funcs;
|
const struct amd_pm_funcs *pp_funcs = adev->powerplay.pp_funcs;
|
||||||
|
|
||||||
if (!pp_funcs ||!pp_funcs->get_asic_baco_state ||!pp_funcs->set_asic_baco_state)
|
if (!pp_funcs ||!pp_funcs->get_asic_baco_state ||!pp_funcs->set_asic_baco_state)
|
||||||
return -1;
|
return -ENOENT;
|
||||||
|
|
||||||
/* enter BACO state */
|
/* enter BACO state */
|
||||||
if (pp_funcs->set_asic_baco_state(pp_handle, 1))
|
if (pp_funcs->set_asic_baco_state(pp_handle, 1))
|
||||||
return -1;
|
return -EIO;
|
||||||
|
|
||||||
/* exit BACO state */
|
/* exit BACO state */
|
||||||
if (pp_funcs->set_asic_baco_state(pp_handle, 0))
|
if (pp_funcs->set_asic_baco_state(pp_handle, 0))
|
||||||
return -1;
|
return -EIO;
|
||||||
|
|
||||||
dev_info(adev->dev, "GPU BACO reset\n");
|
dev_info(adev->dev, "GPU BACO reset\n");
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user