From 999b288e648da8e21d63bc4371c07d38ab3bcba0 Mon Sep 17 00:00:00 2001 From: David Rokhvarg Date: Mon, 14 Dec 2015 10:51:39 -0500 Subject: [PATCH 0289/1110] drm/amd/powerplay: Make declarations of functions exposed to DAL type-safe. Signed-off-by: David Rokhvarg Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 15 ++++++--------- drivers/gpu/drm/amd/powerplay/inc/amd_powerplay.h | 15 ++++++++++----- 2 files changed, 16 insertions(+), 14 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c index 7a5baeb..86e86e2 100644 --- a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c +++ b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c @@ -753,10 +753,10 @@ int amd_powerplay_fini(void *handle) /* export this function to DAL */ -int amd_powerplay_display_configuration_change(void *handle, const void *input) +int amd_powerplay_display_configuration_change(void *handle, + const struct amd_pp_display_configuration *display_config) { struct pp_hwmgr *hwmgr; - const struct amd_pp_display_configuration *display_config = input; PP_CHECK((struct pp_instance *)handle); @@ -783,14 +783,13 @@ int amd_powerplay_get_display_power_level(void *handle, } int amd_powerplay_get_current_clocks(void *handle, - void *output) + struct amd_pp_clock_info *clocks) { struct pp_hwmgr *hwmgr; struct amd_pp_simple_clock_info simple_clocks; struct pp_clock_info hw_clocks; - struct amd_pp_clock_info *clocks = (struct amd_pp_clock_info *)output; - if (handle == NULL || output == NULL) + if (handle == NULL || clocks == NULL) return -EINVAL; hwmgr = ((struct pp_instance *)handle)->hwmgr; @@ -842,12 +841,10 @@ int amd_powerplay_get_clock_by_type(void *handle, enum amd_pp_clock_type type, s return result; } -int amd_powerplay_get_display_mode_validation_clocks(void *handle, const void *input, - void *output) +int amd_powerplay_get_display_mode_validation_clocks(void *handle, + struct amd_pp_simple_clock_info *clocks) { int result = -1; - - struct amd_pp_simple_clock_info *clocks = output; struct pp_hwmgr *hwmgr; if (handle == NULL || clocks == NULL) diff --git a/drivers/gpu/drm/amd/powerplay/inc/amd_powerplay.h b/drivers/gpu/drm/amd/powerplay/inc/amd_powerplay.h index 235f2fb..bbe02ec 100644 --- a/drivers/gpu/drm/amd/powerplay/inc/amd_powerplay.h +++ b/drivers/gpu/drm/amd/powerplay/inc/amd_powerplay.h @@ -348,18 +348,23 @@ struct amd_powerplay { int amd_powerplay_init(struct amd_pp_init *pp_init, struct amd_powerplay *amd_pp); + int amd_powerplay_fini(void *handle); -int amd_powerplay_display_configuration_change(void *handle, const void *input); +int amd_powerplay_display_configuration_change(void *handle, + const struct amd_pp_display_configuration *input); int amd_powerplay_get_display_power_level(void *handle, struct amd_pp_simple_clock_info *output); -int amd_powerplay_get_current_clocks(void *handle, void *output); +int amd_powerplay_get_current_clocks(void *handle, + struct amd_pp_clock_info *output); -int amd_powerplay_get_clock_by_type(void *handle, enum amd_pp_clock_type type, struct amd_pp_clocks *clocks); +int amd_powerplay_get_clock_by_type(void *handle, + enum amd_pp_clock_type type, + struct amd_pp_clocks *clocks); -int amd_powerplay_get_display_mode_validation_clocks(void *handle, const void *input, - void *output); +int amd_powerplay_get_display_mode_validation_clocks(void *handle, + struct amd_pp_simple_clock_info *output); #endif /* _AMD_POWERPLAY_H_ */ -- 2.7.4