[PATCH] drm/amd/powerplay: correct LoadLineResistance value in pptable.

[Alex Deucher]

On Tue, May 2, 2017 at 2:49 AM, Rex Zhu <Rex.Zhu at amd.com> wrote:
> this value is used by avfs to adjust inversion voltage.
>
> Change-Id: Ic6520c184ae938384d9f5e1590ecaf864fa9f95a
> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/powerplay/hwmgr/vega10_powertune.c       | 4 ++--
>  drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_powertune.c b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_powertune.c
> index 692f752..3f72268 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_powertune.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_powertune.c
> @@ -48,8 +48,8 @@ void vega10_initialize_power_tune_defaults(struct pp_hwmgr *hwmgr)
>         table->Tliquid1Limit = cpu_to_le16(tdp_table->usTemperatureLimitLiquid1);
>         table->Tliquid2Limit = cpu_to_le16(tdp_table->usTemperatureLimitLiquid2);
>         table->TplxLimit = cpu_to_le16(tdp_table->usTemperatureLimitPlx);
> -       table->LoadLineResistance = cpu_to_le16(
> -                       hwmgr->platform_descriptor.LoadLineSlope);
> +       table->LoadLineResistance =
> +                       hwmgr->platform_descriptor.LoadLineSlope * 256;
>         table->FitLimit = 0; /* Not used for Vega10 */
>
>         table->Liquid1_I2C_address = tdp_table->ucLiquid1_I2C_address;
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c
> index 8b55ae0..00e9551 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c
> @@ -407,7 +407,7 @@ static int get_tdp_table(
>                 tdp_table->ucPlx_I2C_address = power_tune_table->ucPlx_I2C_address;
>                 tdp_table->ucPlx_I2C_Line = power_tune_table->ucPlx_I2C_LineSCL;
>                 tdp_table->ucPlx_I2C_LineSDA = power_tune_table->ucPlx_I2C_LineSDA;
> -               hwmgr->platform_descriptor.LoadLineSlope = power_tune_table->usLoadLineResistance;
> +               hwmgr->platform_descriptor.LoadLineSlope = le16_to_cpu(power_tune_table->usLoadLineResistance);
>         } else {
>                 power_tune_table_v2 = (ATOM_Vega10_PowerTune_Table_V2 *)table;
>                 tdp_table->usMaximumPowerDeliveryLimit = le16_to_cpu(power_tune_table_v2->usSocketPowerLimit);
> @@ -453,7 +453,7 @@ static int get_tdp_table(
>                 tdp_table->ucPlx_I2C_LineSDA = sda;
>
>                 hwmgr->platform_descriptor.LoadLineSlope =
> -                                       power_tune_table_v2->usLoadLineResistance;
> +                                       le16_to_cpu(power_tune_table_v2->usLoadLineResistance);
>         }
>
>         *info_tdp_table = tdp_table;
> --
> 1.9.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx