Commit 246bcae1 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Jani Nikula
Browse files

drm/i915/dsi: Replace check with a (missing) MIPI sequence name 



Names of the MIPI sequence steps are sequential and defined, no
need to check for the gaps. However in seq_name the MIPI_SEQ_END
is missing. Add it there, and drop unneeded NULL check in
sequence_name().

Reviewed-by: default avatarAndi Shyti <andi.shyti@linux.intel.com>
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: default avatarJani Nikula <jani.nikula@intel.com>
Tested-by: default avatarHans de Goede <hdegoede@redhat.com>
Reviewed-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20231103201831.1037416-9-andriy.shevchenko@linux.intel.com
parent a1f763fe

drivers/gpu/drm/i915/display/intel_dsi_vbt.c

+2 −1
Original line number Diff line number Diff line
@@ -653,6 +653,7 @@ static const fn_mipi_elem_exec exec_elem[] = { 
 */



static const char * const seq_name[] = {

	[MIPI_SEQ_END] = "MIPI_SEQ_END",

	[MIPI_SEQ_DEASSERT_RESET] = "MIPI_SEQ_DEASSERT_RESET",

	[MIPI_SEQ_INIT_OTP] = "MIPI_SEQ_INIT_OTP",

	[MIPI_SEQ_DISPLAY_ON] = "MIPI_SEQ_DISPLAY_ON",
@@ -668,7 +669,7 @@ static const char * const seq_name[] = { 


static const char *sequence_name(enum mipi_seq seq_id)

{

	if (seq_id < ARRAY_SIZE(seq_name) && seq_name[seq_id])

	if (seq_id < ARRAY_SIZE(seq_name))

		return seq_name[seq_id];



	return "(unknown)";