[PATCH 8/8] drm/amd/powerplay: add pp_tables_get_response_times function in process pptables

[Alex Deucher]

On Fri, Jul 15, 2016 at 1:20 AM, Huang Rui <ray.huang at amd.com> wrote:
> The pp_tables_get_response_times function will be used on iceland HW
> mananger.
>
> Signed-off-by: Huang Rui <ray.huang at amd.com>

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

> ---
>  drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.c | 13 +++++++++++++
>  drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.h | 17 ++++++++++-------
>  2 files changed, 23 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.c b/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.c
> index 2f1a14f..947c4de 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.c
> @@ -807,6 +807,19 @@ static const ATOM_PPLIB_POWERPLAYTABLE *get_powerplay_table(
>         return (const ATOM_PPLIB_POWERPLAYTABLE *)table_addr;
>  }
>
> +int pp_tables_get_response_times(struct pp_hwmgr *hwmgr,
> +                               uint32_t *vol_rep_time, uint32_t *bb_rep_time)
> +{
> +       const ATOM_PPLIB_POWERPLAYTABLE *powerplay_tab = get_powerplay_table(hwmgr);
> +
> +       PP_ASSERT_WITH_CODE(NULL != powerplay_tab,
> +                           "Missing PowerPlay Table!", return -EINVAL);
> +
> +       *vol_rep_time = (uint32_t)le16_to_cpu(powerplay_tab->usVoltageTime);
> +       *bb_rep_time = (uint32_t)le16_to_cpu(powerplay_tab->usBackbiasTime);
> +
> +       return 0;
> +}
>
>  int pp_tables_get_num_of_entries(struct pp_hwmgr *hwmgr,
>                                      unsigned long *num_of_entries)
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.h b/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.h
> index 3043480..baddaa7 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.h
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.h
> @@ -32,16 +32,19 @@ struct pp_hw_power_state;
>  extern const struct pp_table_func pptable_funcs;
>
>  typedef int (*pp_tables_hw_clock_info_callback)(struct pp_hwmgr *hwmgr,
> -                                       struct pp_hw_power_state *hw_ps,
> -                                                       unsigned int index,
> -                                                const void *clock_info);
> +                                               struct pp_hw_power_state *hw_ps,
> +                                               unsigned int index,
> +                                               const void *clock_info);
>
>  int pp_tables_get_num_of_entries(struct pp_hwmgr *hwmgr,
> -                                    unsigned long *num_of_entries);
> +                                unsigned long *num_of_entries);
>
>  int pp_tables_get_entry(struct pp_hwmgr *hwmgr,
> -                                               unsigned long entry_index,
> -                                               struct pp_power_state *ps,
> -                               pp_tables_hw_clock_info_callback func);
> +                       unsigned long entry_index,
> +                       struct pp_power_state *ps,
> +                       pp_tables_hw_clock_info_callback func);
> +
> +int pp_tables_get_response_times(struct pp_hwmgr *hwmgr,
> +                                uint32_t *vol_rep_time, uint32_t *bb_rep_time);
>
>  #endif
> --
> 2.7.4
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx