Please add the followings to the description part. With that added, the patch
is Reviewed-by: Evan Quan
"
This fixes the following static checker warning.
drivers/gpu/drm/amd/amdgpu/../powerplay/smu_v11_0.c:390
smu_v11_0_setup_pptable()
warn: passing casted pointer '' to 'smu_get_atom_data_table()' 32
vs 16.
"
> -Original Message-
> From: amd-gfx On Behalf Of
> Wang, Kevin(Yang)
> Sent: Monday, August 19, 2019 11:47 PM
> To: amd-gfx@lists.freedesktop.org
> Cc: Wang, Kevin(Yang) ;
> dan.carpen...@oracle.com
> Subject: [PATCH] drm/amd/powerplay: fix variable type errors in
> smu_v11_0_setup_pptable
>
> fix size type errors, from uint32_t to uint16_t.
> it will cause only initializes the highest 16 bits in
> smu_get_atom_data_table function.
>
> Signed-off-by: Kevin Wang
> Reported-by: Dan Carpenter
> ---
> drivers/gpu/drm/amd/powerplay/smu_v11_0.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> index b69632fe856b..e9a022447ac1 100644
> --- a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> +++ b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> @@ -368,6 +368,7 @@ static int smu_v11_0_setup_pptable(struct
> smu_context *smu)
> const struct smc_firmware_header_v1_0 *hdr;
> int ret, index;
> uint32_t size;
> + uint16_t atom_table_size;
> uint8_t frev, crev;
> void *table;
> uint16_t version_major, version_minor;
> @@ -395,10 +396,11 @@ static int smu_v11_0_setup_pptable(struct
> smu_context *smu)
> index =
> get_index_into_master_table(atom_master_list_of_data_tables_v2_1,
> powerplayinfo);
>
> - ret = smu_get_atom_data_table(smu, index, (uint16_t
> *), , ,
> + ret = smu_get_atom_data_table(smu, index,
> _table_size, , ,
> (uint8_t **));
> if (ret)
> return ret;
> + size = atom_table_size;
> }
>
> if (!smu->smu_table.power_play_table)
> --
> 2.22.0
>
> ___
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
