Applied.
Alex
On Mon, Jun 30, 2025 at 4:33 PM Fedor Pchelkin <pchel...@ispras.ru> wrote:
>
> There is a small typo in phm_wait_on_indirect_register().
>
> Swap mask and value arguments provided to phm_wait_on_register() so that
> they satisfy the function signature and actual usage scheme.
>
> Found by Linux Verification Center (linuxtesting.org) with Svace static
> analysis tool.
>
> Fixes: 3bace3591493 ("drm/amd/powerplay: add hardware manager sub-component")
> Signed-off-by: Fedor Pchelkin <pchel...@ispras.ru>
> ---
> drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu_helper.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu_helper.c
> b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu_helper.c
> index 79a566f3564a..c305ea4ec17d 100644
> --- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu_helper.c
> +++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu_helper.c
> @@ -149,7 +149,7 @@ int phm_wait_on_indirect_register(struct pp_hwmgr *hwmgr,
> }
>
> cgs_write_register(hwmgr->device, indirect_port, index);
> - return phm_wait_on_register(hwmgr, indirect_port + 1, mask, value);
> + return phm_wait_on_register(hwmgr, indirect_port + 1, value, mask);
> }
>
> int phm_wait_for_register_unequal(struct pp_hwmgr *hwmgr,
> --
> 2.50.0
>
