Reviewed-by: Yang Wang <kevinyang.w...@amd.com>
Best Regards,
Kevin
-----Original Message-----
From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of Samasth
Norway Ananda
Sent: Tuesday, January 23, 2024 5:32 AM
To: amd-gfx@lists.freedesktop.org
Cc: Deucher, Alexander <alexander.deuc...@amd.com>;
samasth.norway.ana...@oracle.com
Subject: [PATCH 1/1] drm/amdgpu: fix wrong sizeof argument
voltage_parameters is a point to a struct of type SET_VOLTAGE_PARAMETERS_V1_3.
Passing just voltage_parameters would not print the right size of the struct
variable. So we need to pass *voltage_parameters to sizeof().
Fixes: 4630d5031cd8 ("drm/amdgpu: check PS, WS index")
Signed-off-by: Samasth Norway Ananda <samasth.norway.ana...@oracle.com>
---
drivers/gpu/drm/amd/pm/powerplay/hwmgr/ppatomctrl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/ppatomctrl.c
b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/ppatomctrl.c
index 8f58954739e0..b1b4c09c3467 100644
--- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/ppatomctrl.c
+++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/ppatomctrl.c
@@ -1526,7 +1526,7 @@ int atomctrl_get_leakage_id_from_efuse(struct pp_hwmgr
*hwmgr, uint16_t *virtual
result = amdgpu_atom_execute_table(adev->mode_info.atom_context,
GetIndexIntoMasterTable(COMMAND, SetVoltage),
- (uint32_t *)voltage_parameters,
sizeof(voltage_parameters));
+ (uint32_t *)voltage_parameters,
sizeof(*voltage_parameters));
*virtual_voltage_id = voltage_parameters->usVoltageLevel;
--
2.42.0
