Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 155f1127 authored by David Rokhvarg's avatar David Rokhvarg Committed by Alex Deucher
Browse files

drm/amd/powerplay: Make declarations of functions exposed to DAL type-safe. 



Signed-off-by: default avatarDavid Rokhvarg <David.Rokhvarg@amd.com>
Reviewed-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent f1232c61

drivers/gpu/drm/amd/powerplay/amd_powerplay.c

+6 −9
Original line number Original line Diff line number Diff line
@@ -750,10 +750,10 @@ int amd_powerplay_fini(void *handle) 




/* export this function to DAL */

/* 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;

	struct pp_hwmgr  *hwmgr;

	const struct amd_pp_display_configuration *display_config = input;





	PP_CHECK((struct pp_instance *)handle);

	PP_CHECK((struct pp_instance *)handle);
@@ -780,14 +780,13 @@ int amd_powerplay_get_display_power_level(void *handle, 
}

}





int amd_powerplay_get_current_clocks(void *handle,

int amd_powerplay_get_current_clocks(void *handle,

		void *output)

		struct amd_pp_clock_info *clocks)

{

{

	struct pp_hwmgr  *hwmgr;

	struct pp_hwmgr  *hwmgr;

	struct amd_pp_simple_clock_info simple_clocks;

	struct amd_pp_simple_clock_info simple_clocks;

	struct pp_clock_info hw_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;

		return -EINVAL;





	hwmgr = ((struct pp_instance *)handle)->hwmgr;

	hwmgr = ((struct pp_instance *)handle)->hwmgr;
@@ -839,12 +838,10 @@ int amd_powerplay_get_clock_by_type(void *handle, enum amd_pp_clock_type type, s 
	return result;

	return result;

}

}





int amd_powerplay_get_display_mode_validation_clocks(void *handle, const void *input,

int amd_powerplay_get_display_mode_validation_clocks(void *handle,

							void *output)

		struct amd_pp_simple_clock_info *clocks)

{

{

	int result = -1;

	int result = -1;



	struct amd_pp_simple_clock_info *clocks = output;

	struct pp_hwmgr  *hwmgr;

	struct pp_hwmgr  *hwmgr;





	if (handle == NULL || clocks == NULL)

	if (handle == NULL || clocks == NULL)

drivers/gpu/drm/amd/powerplay/inc/amd_powerplay.h

+10 −5
Original line number Original line Diff line number Diff line
@@ -349,18 +349,23 @@ struct amd_powerplay { 




int amd_powerplay_init(struct amd_pp_init *pp_init,

int amd_powerplay_init(struct amd_pp_init *pp_init,

		       struct amd_powerplay *amd_pp);

		       struct amd_powerplay *amd_pp);



int amd_powerplay_fini(void *handle);

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,

int amd_powerplay_get_display_power_level(void *handle,

		struct amd_pp_simple_clock_info *output);

		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,

int amd_powerplay_get_display_mode_validation_clocks(void *handle,

							void *output);

		struct amd_pp_simple_clock_info *output);





#endif /* _AMD_POWERPLAY_H_ */
 
#endif /* _AMD_POWERPLAY_H_ */