Commit 58cfaf25 authored by Lee Jones's avatar Lee Jones Committed by Alex Deucher

drm/amd/pm/powerplay/hwmgr/ppatomctrl: Fix a myriad of kernel-doc issues

Fixes the following W=1 kernel build warning(s):

 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:104: warning: Function parameter or member 'reg_block' not described in 'atomctrl_set_mc_reg_address_table'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:104: warning: Function parameter or member 'table' not described in 'atomctrl_set_mc_reg_address_table'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:213: warning: Function parameter or member 'hwmgr' not described in 'atomctrl_set_engine_dram_timings_rv770'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:213: warning: Function parameter or member 'engine_clock' not described in 'atomctrl_set_engine_dram_timings_rv770'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:213: warning: Function parameter or member 'memory_clock' not described in 'atomctrl_set_engine_dram_timings_rv770'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:239: warning: Function parameter or member 'device' not described in 'get_voltage_info_table'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:519: warning: Function parameter or member 'hwmgr' not described in 'atomctrl_get_reference_clock'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:548: warning: Function parameter or member 'hwmgr' not described in 'atomctrl_is_voltage_controlled_by_gpio_v3'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:548: warning: Function parameter or member 'voltage_type' not described in 'atomctrl_is_voltage_controlled_by_gpio_v3'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:548: warning: Function parameter or member 'voltage_mode' not described in 'atomctrl_is_voltage_controlled_by_gpio_v3'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:640: warning: Function parameter or member 'device' not described in 'get_gpio_lookup_table'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:663: warning: Function parameter or member 'hwmgr' not described in 'atomctrl_get_pp_assign_pin'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:663: warning: Function parameter or member 'pinId' not described in 'atomctrl_get_pp_assign_pin'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:663: warning: Function parameter or member 'gpio_pin_assignment' not described in 'atomctrl_get_pp_assign_pin'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1152: warning: Function parameter or member 'hwmgr' not described in 'atomctrl_get_voltage_evv'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1152: warning: Function parameter or member 'virtual_voltage_id' not described in 'atomctrl_get_voltage_evv'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1152: warning: Function parameter or member 'voltage' not described in 'atomctrl_get_voltage_evv'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1194: warning: Function parameter or member 'hwmgr' not described in 'atomctrl_get_mpll_reference_clock'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1227: warning: Function parameter or member 'device' not described in 'asic_internal_ss_get_ss_table'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1258: warning: Function parameter or member 'hwmgr' not described in 'asic_internal_ss_get_ss_asignment'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1258: warning: Function parameter or member 'clockSource' not described in 'asic_internal_ss_get_ss_asignment'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1258: warning: Function parameter or member 'clockSpeed' not described in 'asic_internal_ss_get_ss_asignment'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1258: warning: Function parameter or member 'ssEntry' not described in 'asic_internal_ss_get_ss_asignment'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1321: warning: Function parameter or member 'hwmgr' not described in 'atomctrl_get_memory_clock_spread_spectrum'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1321: warning: Function parameter or member 'memory_clock' not described in 'atomctrl_get_memory_clock_spread_spectrum'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1321: warning: Function parameter or member 'ssInfo' not described in 'atomctrl_get_memory_clock_spread_spectrum'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1332: warning: Function parameter or member 'hwmgr' not described in 'atomctrl_get_engine_clock_spread_spectrum'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1332: warning: Function parameter or member 'engine_clock' not described in 'atomctrl_get_engine_clock_spread_spectrum'
 drivers/gpu/drm/amd/amdgpu/../pm/powerplay/hwmgr/ppatomctrl.c:1332: warning: Function parameter or member 'ssInfo' not described in 'atomctrl_get_engine_clock_spread_spectrum'

Cc: Evan Quan <evan.quan@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: "Christian König" <christian.koenig@amd.com>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: amd-gfx@lists.freedesktop.org
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 9795f4eb
...@@ -92,11 +92,11 @@ static int atomctrl_retrieve_ac_timing( ...@@ -92,11 +92,11 @@ static int atomctrl_retrieve_ac_timing(
} }
/** /**
* Get memory clock AC timing registers index from VBIOS table * atomctrl_set_mc_reg_address_table - Get memory clock AC timing registers index from VBIOS table
* VBIOS set end of memory clock AC timing registers by ucPreRegDataLength bit6 = 1 * VBIOS set end of memory clock AC timing registers by ucPreRegDataLength bit6 = 1
* @param reg_block the address ATOM_INIT_REG_BLOCK * @reg_block: the address ATOM_INIT_REG_BLOCK
* @param table the address of MCRegTable * @table: the address of MCRegTable
* @return 0 * Return: 0
*/ */
static int atomctrl_set_mc_reg_address_table( static int atomctrl_set_mc_reg_address_table(
ATOM_INIT_REG_BLOCK *reg_block, ATOM_INIT_REG_BLOCK *reg_block,
...@@ -203,7 +203,7 @@ int atomctrl_initialize_mc_reg_table_v2_2( ...@@ -203,7 +203,7 @@ int atomctrl_initialize_mc_reg_table_v2_2(
return result; return result;
} }
/** /*
* Set DRAM timings based on engine clock and memory clock. * Set DRAM timings based on engine clock and memory clock.
*/ */
int atomctrl_set_engine_dram_timings_rv770( int atomctrl_set_engine_dram_timings_rv770(
...@@ -229,7 +229,7 @@ int atomctrl_set_engine_dram_timings_rv770( ...@@ -229,7 +229,7 @@ int atomctrl_set_engine_dram_timings_rv770(
(uint32_t *)&engine_clock_parameters); (uint32_t *)&engine_clock_parameters);
} }
/** /*
* Private Function to get the PowerPlay Table Address. * Private Function to get the PowerPlay Table Address.
* WARNING: The tabled returned by this function is in * WARNING: The tabled returned by this function is in
* dynamically allocated memory. * dynamically allocated memory.
...@@ -274,12 +274,13 @@ static const ATOM_VOLTAGE_OBJECT_V3 *atomctrl_lookup_voltage_type_v3( ...@@ -274,12 +274,13 @@ static const ATOM_VOLTAGE_OBJECT_V3 *atomctrl_lookup_voltage_type_v3(
return NULL; return NULL;
} }
/** atomctrl_get_memory_pll_dividers_si(). /**
* atomctrl_get_memory_pll_dividers_si().
* *
* @param hwmgr input parameter: pointer to HwMgr * @hwmgr: input parameter: pointer to HwMgr
* @param clock_value input parameter: memory clock * @clock_value: input parameter: memory clock
* @param dividers output parameter: memory PLL dividers * @dividers: output parameter: memory PLL dividers
* @param strobe_mode input parameter: 1 for strobe mode, 0 for performance mode * @strobe_mode: input parameter: 1 for strobe mode, 0 for performance mode
*/ */
int atomctrl_get_memory_pll_dividers_si( int atomctrl_get_memory_pll_dividers_si(
struct pp_hwmgr *hwmgr, struct pp_hwmgr *hwmgr,
...@@ -326,11 +327,12 @@ int atomctrl_get_memory_pll_dividers_si( ...@@ -326,11 +327,12 @@ int atomctrl_get_memory_pll_dividers_si(
return result; return result;
} }
/** atomctrl_get_memory_pll_dividers_vi(). /**
* atomctrl_get_memory_pll_dividers_vi().
* *
* @param hwmgr input parameter: pointer to HwMgr * @hwmgr: input parameter: pointer to HwMgr
* @param clock_value input parameter: memory clock * @clock_value: input parameter: memory clock
* @param dividers output parameter: memory PLL dividers * @dividers: output parameter: memory PLL dividers
*/ */
int atomctrl_get_memory_pll_dividers_vi(struct pp_hwmgr *hwmgr, int atomctrl_get_memory_pll_dividers_vi(struct pp_hwmgr *hwmgr,
uint32_t clock_value, pp_atomctrl_memory_clock_param *mpll_param) uint32_t clock_value, pp_atomctrl_memory_clock_param *mpll_param)
...@@ -512,7 +514,7 @@ int atomctrl_get_dfs_pll_dividers_vi( ...@@ -512,7 +514,7 @@ int atomctrl_get_dfs_pll_dividers_vi(
return result; return result;
} }
/** /*
* Get the reference clock in 10KHz * Get the reference clock in 10KHz
*/ */
uint32_t atomctrl_get_reference_clock(struct pp_hwmgr *hwmgr) uint32_t atomctrl_get_reference_clock(struct pp_hwmgr *hwmgr)
...@@ -535,7 +537,7 @@ uint32_t atomctrl_get_reference_clock(struct pp_hwmgr *hwmgr) ...@@ -535,7 +537,7 @@ uint32_t atomctrl_get_reference_clock(struct pp_hwmgr *hwmgr)
return clock; return clock;
} }
/** /*
* Returns true if the given voltage type is controlled by GPIO pins. * Returns true if the given voltage type is controlled by GPIO pins.
* voltage_type is one of SET_VOLTAGE_TYPE_ASIC_VDDC, * voltage_type is one of SET_VOLTAGE_TYPE_ASIC_VDDC,
* SET_VOLTAGE_TYPE_ASIC_MVDDC, SET_VOLTAGE_TYPE_ASIC_MVDDQ. * SET_VOLTAGE_TYPE_ASIC_MVDDC, SET_VOLTAGE_TYPE_ASIC_MVDDQ.
...@@ -630,7 +632,7 @@ static bool atomctrl_lookup_gpio_pin( ...@@ -630,7 +632,7 @@ static bool atomctrl_lookup_gpio_pin(
return false; return false;
} }
/** /*
* Private Function to get the PowerPlay Table Address. * Private Function to get the PowerPlay Table Address.
* WARNING: The tabled returned by this function is in * WARNING: The tabled returned by this function is in
* dynamically allocated memory. * dynamically allocated memory.
...@@ -653,7 +655,7 @@ static ATOM_GPIO_PIN_LUT *get_gpio_lookup_table(void *device) ...@@ -653,7 +655,7 @@ static ATOM_GPIO_PIN_LUT *get_gpio_lookup_table(void *device)
return (ATOM_GPIO_PIN_LUT *)table_address; return (ATOM_GPIO_PIN_LUT *)table_address;
} }
/** /*
* Returns 1 if the given pin id find in lookup table. * Returns 1 if the given pin id find in lookup table.
*/ */
bool atomctrl_get_pp_assign_pin( bool atomctrl_get_pp_assign_pin(
...@@ -1101,14 +1103,15 @@ int atomctrl_calculate_voltage_evv_on_sclk( ...@@ -1101,14 +1103,15 @@ int atomctrl_calculate_voltage_evv_on_sclk(
return result; return result;
} }
/** atomctrl_get_voltage_evv_on_sclk gets voltage via call to ATOM COMMAND table. /**
* @param hwmgr input: pointer to hwManager * atomctrl_get_voltage_evv_on_sclk gets voltage via call to ATOM COMMAND table.
* @param voltage_type input: type of EVV voltage VDDC or VDDGFX * @hwmgr: input: pointer to hwManager
* @param sclk input: in 10Khz unit. DPM state SCLK frequency * @voltage_type: input: type of EVV voltage VDDC or VDDGFX
* which is define in PPTable SCLK/VDDC dependence * @sclk: input: in 10Khz unit. DPM state SCLK frequency
* table associated with this virtual_voltage_Id * which is define in PPTable SCLK/VDDC dependence
* @param virtual_voltage_Id input: voltage id which match per voltage DPM state: 0xff01, 0xff02.. 0xff08 * table associated with this virtual_voltage_Id
* @param voltage output: real voltage level in unit of mv * @virtual_voltage_Id: input: voltage id which match per voltage DPM state: 0xff01, 0xff02.. 0xff08
* @voltage: output: real voltage level in unit of mv
*/ */
int atomctrl_get_voltage_evv_on_sclk( int atomctrl_get_voltage_evv_on_sclk(
struct pp_hwmgr *hwmgr, struct pp_hwmgr *hwmgr,
...@@ -1142,9 +1145,9 @@ int atomctrl_get_voltage_evv_on_sclk( ...@@ -1142,9 +1145,9 @@ int atomctrl_get_voltage_evv_on_sclk(
/** /**
* atomctrl_get_voltage_evv gets voltage via call to ATOM COMMAND table. * atomctrl_get_voltage_evv gets voltage via call to ATOM COMMAND table.
* @param hwmgr input: pointer to hwManager * @hwmgr: input: pointer to hwManager
* @param virtual_voltage_id input: voltage id which match per voltage DPM state: 0xff01, 0xff02.. 0xff08 * @virtual_voltage_id: input: voltage id which match per voltage DPM state: 0xff01, 0xff02.. 0xff08
* @param voltage output: real voltage level in unit of mv * @voltage: output: real voltage level in unit of mv
*/ */
int atomctrl_get_voltage_evv(struct pp_hwmgr *hwmgr, int atomctrl_get_voltage_evv(struct pp_hwmgr *hwmgr,
uint16_t virtual_voltage_id, uint16_t virtual_voltage_id,
...@@ -1187,7 +1190,7 @@ int atomctrl_get_voltage_evv(struct pp_hwmgr *hwmgr, ...@@ -1187,7 +1190,7 @@ int atomctrl_get_voltage_evv(struct pp_hwmgr *hwmgr,
return result; return result;
} }
/** /*
* Get the mpll reference clock in 10KHz * Get the mpll reference clock in 10KHz
*/ */
uint32_t atomctrl_get_mpll_reference_clock(struct pp_hwmgr *hwmgr) uint32_t atomctrl_get_mpll_reference_clock(struct pp_hwmgr *hwmgr)
...@@ -1220,7 +1223,7 @@ uint32_t atomctrl_get_mpll_reference_clock(struct pp_hwmgr *hwmgr) ...@@ -1220,7 +1223,7 @@ uint32_t atomctrl_get_mpll_reference_clock(struct pp_hwmgr *hwmgr)
return clock; return clock;
} }
/** /*
* Get the asic internal spread spectrum table * Get the asic internal spread spectrum table
*/ */
static ATOM_ASIC_INTERNAL_SS_INFO *asic_internal_ss_get_ss_table(void *device) static ATOM_ASIC_INTERNAL_SS_INFO *asic_internal_ss_get_ss_table(void *device)
...@@ -1248,7 +1251,7 @@ bool atomctrl_is_asic_internal_ss_supported(struct pp_hwmgr *hwmgr) ...@@ -1248,7 +1251,7 @@ bool atomctrl_is_asic_internal_ss_supported(struct pp_hwmgr *hwmgr)
return false; return false;
} }
/** /*
* Get the asic internal spread spectrum assignment * Get the asic internal spread spectrum assignment
*/ */
static int asic_internal_ss_get_ss_asignment(struct pp_hwmgr *hwmgr, static int asic_internal_ss_get_ss_asignment(struct pp_hwmgr *hwmgr,
...@@ -1311,7 +1314,7 @@ static int asic_internal_ss_get_ss_asignment(struct pp_hwmgr *hwmgr, ...@@ -1311,7 +1314,7 @@ static int asic_internal_ss_get_ss_asignment(struct pp_hwmgr *hwmgr,
return entry_found ? 0 : 1; return entry_found ? 0 : 1;
} }
/** /*
* Get the memory clock spread spectrum info * Get the memory clock spread spectrum info
*/ */
int atomctrl_get_memory_clock_spread_spectrum( int atomctrl_get_memory_clock_spread_spectrum(
...@@ -1322,7 +1325,8 @@ int atomctrl_get_memory_clock_spread_spectrum( ...@@ -1322,7 +1325,8 @@ int atomctrl_get_memory_clock_spread_spectrum(
return asic_internal_ss_get_ss_asignment(hwmgr, return asic_internal_ss_get_ss_asignment(hwmgr,
ASIC_INTERNAL_MEMORY_SS, memory_clock, ssInfo); ASIC_INTERNAL_MEMORY_SS, memory_clock, ssInfo);
} }
/**
/*
* Get the engine clock spread spectrum info * Get the engine clock spread spectrum info
*/ */
int atomctrl_get_engine_clock_spread_spectrum( int atomctrl_get_engine_clock_spread_spectrum(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment