Commit 4984dbb6 authored by Shyam Sundar S K's avatar Shyam Sundar S K Committed by Hans de Goede
Browse files

platform/x86/amd/pmf: Make source_as_str() as non-static 



Add amd_pmf prefix to source_as_str() function, so that the function name
does not look generic. As this is a helper function make it as non-static
so that it can be reused across multiple PMF features.

Reviewed-by: default avatarMario Limonciello <mario.limonciello@amd.com>
Signed-off-by: default avatarShyam Sundar S K <Shyam-sundar.S-k@amd.com>
Link: https://lore.kernel.org/r/20231212014705.2017474-10-Shyam-sundar.S-k@amd.com


Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
parent d0ba7ad4

drivers/platform/x86/amd/pmf/pmf.h

+1 −0
Original line number Diff line number Diff line
@@ -600,6 +600,7 @@ int apmf_get_static_slider_granular(struct amd_pmf_dev *pdev, 
				    struct apmf_static_slider_granular_output *output);

bool is_pprof_balanced(struct amd_pmf_dev *pmf);

int amd_pmf_power_slider_update_event(struct amd_pmf_dev *dev);

const char *amd_pmf_source_as_str(unsigned int state);





int apmf_update_fan_idx(struct amd_pmf_dev *pdev, bool manual, u32 idx);

drivers/platform/x86/amd/pmf/sps.c

+3 −2
Original line number Diff line number Diff line
@@ -27,7 +27,7 @@ static const char *slider_as_str(unsigned int state) 
	}

}



static const char *source_as_str(unsigned int state)

const char *amd_pmf_source_as_str(unsigned int state)

{

	switch (state) {

	case POWER_SOURCE_AC:
@@ -47,7 +47,8 @@ static void amd_pmf_dump_sps_defaults(struct amd_pmf_static_slider_granular *dat 


	for (i = 0; i < POWER_SOURCE_MAX; i++) {

		for (j = 0; j < POWER_MODE_MAX; j++) {

			pr_debug("--- Source:%s Mode:%s ---\n", source_as_str(i), slider_as_str(j));

			pr_debug("--- Source:%s Mode:%s ---\n", amd_pmf_source_as_str(i),

				 slider_as_str(j));

			pr_debug("SPL: %u mW\n", data->prop[i][j].spl);

			pr_debug("SPPT: %u mW\n", data->prop[i][j].sppt);

			pr_debug("SPPT_ApuOnly: %u mW\n", data->prop[i][j].sppt_apu_only);