The current named mode parsing relies only the mode name, and doesn't allow
to specify any other parameter.

Let's convert that string list to an array of a custom structure that will
hold the name and some additional parameters in the future.

Signed-off-by: Maxime Ripard <max...@cerno.tech>

diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c
index e85099df0326..1421e5da49e0 100644
--- a/drivers/gpu/drm/drm_modes.c
+++ b/drivers/gpu/drm/drm_modes.c
@@ -1768,9 +1768,13 @@ static int drm_mode_parse_cmdline_options(const char 
*str,
        return 0;
 }
 
-static const char * const drm_named_modes_whitelist[] = {
-       "NTSC",
-       "PAL",
+struct drm_named_mode {
+       const char *name;
+};
+
+static const struct drm_named_mode drm_named_modes[] = {
+       { "NTSC", },
+       { "PAL", },
 };
 
 static bool drm_mode_parse_cmdline_named_mode(const char *name,
@@ -1779,14 +1783,15 @@ static bool drm_mode_parse_cmdline_named_mode(const 
char *name,
 {
        unsigned int i;
 
-       for (i = 0; i < ARRAY_SIZE(drm_named_modes_whitelist); i++) {
+       for (i = 0; i < ARRAY_SIZE(drm_named_modes); i++) {
+               const struct drm_named_mode *mode = &drm_named_modes[i];
                int ret;
 
-               ret = str_has_prefix(name, drm_named_modes_whitelist[i]);
+               ret = str_has_prefix(name, mode->name);
                if (ret != name_end)
                        continue;
 
-               strcpy(cmdline_mode->name, drm_named_modes_whitelist[i]);
+               strcpy(cmdline_mode->name, mode->name);
                cmdline_mode->specified = true;
 
                return true;

-- 
b4 0.10.0-dev-49460

Reply via email to