Commit 2ff427c7 authored by Lee Jones's avatar Lee Jones Committed by Alex Deucher

drm/amd/display/dc/bios/bios_parser2: Fix some formatting issues and missing parameter docs

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

 drivers/gpu/drm/amd/amdgpu/../display/dc/bios/bios_parser2.c:501: warning: Function parameter or member 'dcb' not described in 'bios_parser_get_gpio_pin_info'
 drivers/gpu/drm/amd/amdgpu/../display/dc/bios/bios_parser2.c:501: warning: Function parameter or member 'gpio_id' not described in 'bios_parser_get_gpio_pin_info'
 drivers/gpu/drm/amd/amdgpu/../display/dc/bios/bios_parser2.c:501: warning: Function parameter or member 'info' not described in 'bios_parser_get_gpio_pin_info'
 drivers/gpu/drm/amd/amdgpu/../display/dc/bios/bios_parser2.c:815: warning: Function parameter or member 'dcb' not described in 'bios_parser_get_spread_spectrum_info'
 drivers/gpu/drm/amd/amdgpu/../display/dc/bios/bios_parser2.c:815: warning: Function parameter or member 'signal' not described in 'bios_parser_get_spread_spectrum_info'
 drivers/gpu/drm/amd/amdgpu/../display/dc/bios/bios_parser2.c:815: warning: Function parameter or member 'index' not described in 'bios_parser_get_spread_spectrum_info'
 drivers/gpu/drm/amd/amdgpu/../display/dc/bios/bios_parser2.c:815: warning: Function parameter or member 'ss_info' not described in 'bios_parser_get_spread_spectrum_info'
 drivers/gpu/drm/amd/amdgpu/../display/dc/bios/bios_parser2.c:1210: warning: Function parameter or member 'dcb' not described in 'bios_parser_set_scratch_critical_state'
 drivers/gpu/drm/amd/amdgpu/../display/dc/bios/bios_parser2.c:1210: warning: Function parameter or member 'state' not described in 'bios_parser_set_scratch_critical_state'

Cc: Harry Wentland <harry.wentland@amd.com>
Cc: Leo Li <sunpeng.li@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 b76f0c6c
...@@ -485,10 +485,11 @@ static struct atom_hpd_int_record *get_hpd_record( ...@@ -485,10 +485,11 @@ static struct atom_hpd_int_record *get_hpd_record(
* bios_parser_get_gpio_pin_info * bios_parser_get_gpio_pin_info
* Get GpioPin information of input gpio id * Get GpioPin information of input gpio id
* *
* @param gpio_id, GPIO ID * @dcb: pointer to the DC BIOS
* @param info, GpioPin information structure * @gpio_id: GPIO ID
* @return Bios parser result code * @info: GpioPin information structure
* @note * return: Bios parser result code
* note:
* to get the GPIO PIN INFO, we need: * to get the GPIO PIN INFO, we need:
* 1. get the GPIO_ID from other object table, see GetHPDInfo() * 1. get the GPIO_ID from other object table, see GetHPDInfo()
* 2. in DATA_TABLE.GPIO_Pin_LUT, search all records, * 2. in DATA_TABLE.GPIO_Pin_LUT, search all records,
...@@ -801,11 +802,11 @@ static enum bp_result get_ss_info_v4_2( ...@@ -801,11 +802,11 @@ static enum bp_result get_ss_info_v4_2(
* ver 3.1, * ver 3.1,
* there is only one entry for each signal /ss id. However, there is * there is only one entry for each signal /ss id. However, there is
* no planning of supporting multiple spread Sprectum entry for EverGreen * no planning of supporting multiple spread Sprectum entry for EverGreen
* @param [in] this * @dcb: pointer to the DC BIOS
* @param [in] signal, ASSignalType to be converted to info index * @signal: ASSignalType to be converted to info index
* @param [in] index, number of entries that match the converted info index * @index: number of entries that match the converted info index
* @param [out] ss_info, sprectrum information structure, * @ss_info: sprectrum information structure,
* @return Bios parser result code * return: Bios parser result code
*/ */
static enum bp_result bios_parser_get_spread_spectrum_info( static enum bp_result bios_parser_get_spread_spectrum_info(
struct dc_bios *dcb, struct dc_bios *dcb,
...@@ -1196,13 +1197,11 @@ static bool bios_parser_is_accelerated_mode( ...@@ -1196,13 +1197,11 @@ static bool bios_parser_is_accelerated_mode(
} }
/** /**
* bios_parser_set_scratch_critical_state * bios_parser_set_scratch_critical_state - update critical state bit
* in VBIOS scratch register
* *
* @brief * @dcb: pointer to the DC BIO
* update critical state bit in VBIOS scratch register * @state: set or reset state
*
* @param
* bool - to set or reset state
*/ */
static void bios_parser_set_scratch_critical_state( static void bios_parser_set_scratch_critical_state(
struct dc_bios *dcb, struct dc_bios *dcb,
......
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