Skip to content

Commit c966dc0

Browse files
arndbalexdeucher
authored andcommitted
drm/amd/display: avoid stringop-overflow warnings for dp_decide_lane_settings()
gcc prints a warning about a possible array overflow for a couple of callers of dp_decide_lane_settings() after commit 1b56c90 ("Makefile: Enable -Wstringop-overflow globally"): drivers/gpu/drm/amd/amdgpu/../display/dc/link/protocols/link_dp_training_fixed_vs_pe_retimer.c: In function 'dp_perform_fixed_vs_pe_training_sequence_legacy': drivers/gpu/drm/amd/amdgpu/../display/dc/link/protocols/link_dp_training_fixed_vs_pe_retimer.c:426:25: error: 'dp_decide_lane_settings' accessing 4 bytes in a region of size 1 [-Werror=stringop-overflow=] 426 | dp_decide_lane_settings(lt_settings, dpcd_lane_adjust, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 427 | lt_settings->hw_lane_settings, lt_settings->dpcd_lane_settings); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/amd/amdgpu/../display/dc/link/protocols/link_dp_training_fixed_vs_pe_retimer.c:426:25: note: referencing argument 4 of type 'union dpcd_training_lane[4]' I'm not entirely sure what caused this, but changing the prototype to expect a pointer instead of an array avoids the warnings. Fixes: 7727e7b ("drm/amd/display: Improve robustness of FIXED_VS link training at DP1 rates") Acked-by: Randy Dunlap <rdunlap@infradead.org> Tested-by: Randy Dunlap <rdunlap@infradead.org> # build-tested Signed-off-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
1 parent 1ac725b commit c966dc0

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_training.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -807,7 +807,7 @@ void dp_decide_lane_settings(
807807
const struct link_training_settings *lt_settings,
808808
const union lane_adjust ln_adjust[LANE_COUNT_DP_MAX],
809809
struct dc_lane_settings hw_lane_settings[LANE_COUNT_DP_MAX],
810-
union dpcd_training_lane dpcd_lane_settings[LANE_COUNT_DP_MAX])
810+
union dpcd_training_lane *dpcd_lane_settings)
811811
{
812812
uint32_t lane;
813813

drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_training.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ void dp_decide_lane_settings(
111111
const struct link_training_settings *lt_settings,
112112
const union lane_adjust ln_adjust[LANE_COUNT_DP_MAX],
113113
struct dc_lane_settings hw_lane_settings[LANE_COUNT_DP_MAX],
114-
union dpcd_training_lane dpcd_lane_settings[LANE_COUNT_DP_MAX]);
114+
union dpcd_training_lane *dpcd_lane_settings);
115115

116116
enum dc_dp_training_pattern decide_cr_training_pattern(
117117
const struct dc_link_settings *link_settings);

0 commit comments

Comments
 (0)