drm/client: Remove match on mode name
Commit 3aeeb13d89
("drm/modes: Support modes names on the command
line") initially introduced the named modes support by essentially
matching the name passed on the command-line to the mode names defined
by the drivers.
This proved to be difficult to work with, since all drivers had to
provide properly named modes. This was also needed because we weren't
passing a full blown-mode to the drivers, but were only filling its
name.
Thanks to the previous patches, we now generate a proper mode, and we
thus can use the usual matching algo on timings, and can simply drop the
name match.
Reviewed-by: Noralf Trønnes <noralf@tronnes.org>
Suggested-by: Noralf Trønnes <noralf@tronnes.org>
Acked-in-principle-or-something-like-that-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://lore.kernel.org/r/20220728-rpi-analog-tv-properties-v10-10-256dad125326@cerno.tech
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
This commit is contained in:
parent
cb1cfd781c
commit
eab2161609
|
@ -188,10 +188,6 @@ static struct drm_display_mode *drm_connector_pick_cmdline_mode(struct drm_conne
|
||||||
prefer_non_interlace = !cmdline_mode->interlace;
|
prefer_non_interlace = !cmdline_mode->interlace;
|
||||||
again:
|
again:
|
||||||
list_for_each_entry(mode, &connector->modes, head) {
|
list_for_each_entry(mode, &connector->modes, head) {
|
||||||
/* Check (optional) mode name first */
|
|
||||||
if (!strcmp(mode->name, cmdline_mode->name))
|
|
||||||
return mode;
|
|
||||||
|
|
||||||
/* check width/height */
|
/* check width/height */
|
||||||
if (mode->hdisplay != cmdline_mode->xres ||
|
if (mode->hdisplay != cmdline_mode->xres ||
|
||||||
mode->vdisplay != cmdline_mode->yres)
|
mode->vdisplay != cmdline_mode->yres)
|
||||||
|
|
Loading…
Reference in New Issue