On Wed, May 27, 2026 at 3:43 PM <[email protected]> wrote: > > From: Ivan Lipski <[email protected]> > > [Why&How] > If a parsed dc_status case is not covered by the dc_status_to_str, the > switch case is skipped, and the function returns > "Unexpected status error". > > This causes build failures when new dc_status enums are introduced. > Changing the 'return "Unexpected status error"' into default resolves it. > > Signed-off-by: Ivan Lipski <[email protected]>
Reviewed-by: Alex Deucher <[email protected]> > --- > drivers/gpu/drm/amd/display/dc/core/dc_debug.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_debug.c > b/drivers/gpu/drm/amd/display/dc/core/dc_debug.c > index bbce751b485f..44028ba88f80 100644 > --- a/drivers/gpu/drm/amd/display/dc/core/dc_debug.c > +++ b/drivers/gpu/drm/amd/display/dc/core/dc_debug.c > @@ -272,9 +272,9 @@ char *dc_status_to_str(enum dc_status status) > return "Fail DP Tunnel BW validation"; > case DC_ERROR_UNEXPECTED: > return "Unexpected error"; > + default: > + return "Unexpected status error"; > } > - > - return "Unexpected status error"; > } > > char *dc_pixel_encoding_to_str(enum dc_pixel_encoding pixel_encoding) > -- > 2.43.0 >
