[PATCH 2/2] drm/amd/powerplay: bug fixs for getsmuclockinfo

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, May 28, 2018 at 09:11:04AM +0800, Evan Quan wrote:
> The .syspll_id and .dfsdid are not initialzed correctly. And
> le32_to_cpu transfer is needed on the output.
> 
> Signed-off-by: Evan Quan <evan.quan at amd.com>
> 

Reviewed-by: Huang Rui <ray.huang at amd.com>

> Change-Id: I491e70b276554f38e1a296970892347474814e75
> ---
>  drivers/gpu/drm/amd/powerplay/hwmgr/ppatomfwctrl.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/ppatomfwctrl.c b/drivers/gpu/drm/amd/powerplay/hwmgr/ppatomfwctrl.c
> index 60d591e..f9b5815c 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/ppatomfwctrl.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/ppatomfwctrl.c
> @@ -496,7 +496,9 @@ int pp_atomfwctrl__get_clk_information_by_clkid(struct pp_hwmgr *hwmgr, BIOS_CLK
>  	uint32_t ix;
>  
>  	parameters.clk_id = id;
> +	parameters.syspll_id = 0;
>  	parameters.command = GET_SMU_CLOCK_INFO_V3_1_GET_CLOCK_FREQ;
> +	parameters.dfsdid = 0;
>  
>  	ix = GetIndexIntoMasterCmdTable(getsmuclockinfo);
>  
> @@ -505,7 +507,7 @@ int pp_atomfwctrl__get_clk_information_by_clkid(struct pp_hwmgr *hwmgr, BIOS_CLK
>  		return -EINVAL;
>  
>  	output = (struct atom_get_smu_clock_info_output_parameters_v3_1 *)&parameters;
> -	*frequency = output->atom_smu_outputclkfreq.smu_clock_freq_hz / 10000;
> +	*frequency = le32_to_cpu(output->atom_smu_outputclkfreq.smu_clock_freq_hz) / 10000;
>  
>  	return 0;
>  }
> -- 
> 2.7.4
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux