[PATCH 5/5] drm/amd/powerplay: add avfs fuse overdriver

Alex Deucher alexdeucher at gmail.com
Thu Apr 20 13:00:25 UTC 2017


On Thu, Apr 20, 2017 at 4:44 AM, Rex Zhu <Rex.Zhu at amd.com> wrote:
> Change-Id: I0f98553985ddbda1473f5313ac803fb9a38ca43a
> ---
>  drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 42 ++++++++++++++++++++++
>  drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.h |  1 +
>  2 files changed, 43 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> index 278def7..c8689ce 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> @@ -2304,6 +2304,48 @@ static int vega10_avfs_enable(struct pp_hwmgr *hwmgr, bool enable)
>         return 0;
>  }
>
> +static int vega10_populate_and_upload_avfs_fuse_override(struct pp_hwmgr *hwmgr)
> +{
> +       int result = 0;
> +
> +       /* Keep Commented until SMC is ready.*/
> +  /*  ULONGLONG SerialNumber = 0;
> +    ULONG TOP32, BOTTOM32;

Fix the types here.  With that fixed, the series is:
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>


> +    PHM_FusesDefault Fuses;
> +    PhwVega10_PrivateData *pPrivate = (PhwVega10_PrivateData *)(pHwMgr->pBackEndPrivateData);
> +    AvfsFuseOverride_t * AVFSFuseTable = &(pPrivate->smcStateTable.AvfsFuseOverrideTable);
> +
> +    Vega10_SendMsgToSmc(pHwMgr->pSmuMgr, PPSMC_MSG_ReadSerialNumTop32);
> +    Vega10_ReadArgFromSmc(pHwMgr->pSmuMgr, &TOP32);
> +
> +    Vega10_SendMsgToSmc(pHwMgr->pSmuMgr, PPSMC_MSG_ReadSerialNumBottom32);
> +    Vega10_ReadArgFromSmc(pHwMgr->pSmuMgr, &BOTTOM32);
> +
> +    SerialNumber = ((ULONGLONG)BOTTOM32 << 32)| TOP32;
> +
> +    if (PP_Override_GetDefaultFuseValue(SerialNumber, PHM_Vega10FusesDefault, &Fuses) == PP_Result_OK)
> +    {
> +        AVFSFuseTable->VFT0_b  = Fuses.VFT0_b;
> +        AVFSFuseTable->VFT0_m1 = Fuses.VFT0_m1;
> +        AVFSFuseTable->VFT0_m2 = Fuses.VFT0_m2;
> +        AVFSFuseTable->VFT1_b  = Fuses.VFT1_b;
> +        AVFSFuseTable->VFT1_m1 = Fuses.VFT2_m1;
> +        AVFSFuseTable->VFT1_m2 = Fuses.VFT1_m2;
> +        AVFSFuseTable->VFT2_b  = Fuses.VFT2_b;
> +        AVFSFuseTable->VFT2_m1 = Fuses.VFT2_m1;
> +        AVFSFuseTable->VFT2_m2 = Fuses.VFT2_m2;
> +
> +        result = PhwVega10_SMCTableManager(pHwMgr, SMUTABLE_WRITE, AVFSFUSETABLE_BIT_MASK);
> +        PP_ASSERT_WITH_CODE(pHwMgr->pPECI, PP_Result_OK == result, "[PhwVega10_PopulateAndUploadAVFSFuseOverride] Failed to update FuseOVerride!", return result;);
> +    }
> +    else
> +    {
> +        result = PP_Result_OK;
> +    }
> +*/
> +       return result;
> +}
> +
>  /**
>  * Initializes the SMC table and uploads it
>  *
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.h b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.h
> index 83c67b9..6070896 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.h
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.h
> @@ -207,6 +207,7 @@ struct vega10_smc_state_table {
>         PPTable_t       pp_table;
>         Watermarks_t    water_marks_table;
>         AvfsTable_t     avfs_table;
> +       AvfsFuseOverride_t AvfsFuseOverrideTable;
>  };
>
>  struct vega10_mclk_latency_entries {
> --
> 1.9.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the amd-gfx mailing list