On 5/12/23 09:44, Geert Uytterhoeven wrote:

Hi Helge,

On Sat, Apr 22, 2023 at 11:27 PM Helge Deller [off-list ref] wrote:

quoted

Add typical resolution for Full-HD monitors.

Signed-off-by: Helge Deller <deller@gmx.de>

quoted

--- a/drivers/video/fbdev/core/modedb.c
+++ b/drivers/video/fbdev/core/modedb.c

@@ -257,6 +257,10 @@ static const struct fb_videomode modedb[] = {
         { NULL, 72, 480, 300, 33386, 40, 24, 11, 19, 80, 3, 0,
                 FB_VMODE_DOUBLE },

+       /* 1920x1080 @ 60 Hz, 67.3 kHz hsync */
+       { NULL, 60, 1920, 1080, 6734, 148, 88, 63, 4, 44, 5, 0,
+               FB_SYNC_HOR_HIGH_ACT | FB_SYNC_VERT_HIGH_ACT },

This matches [1], except for the "63", which looks like a typo for "36"
(total number of lines should be 1125, not 1152).

Thanks Geert!

I fixed it in the git tree.

Helge

quoted

+
         /* 1920x1200 @ 60 Hz, 74.5 Khz hsync */
         { NULL, 60, 1920, 1200, 5177, 128, 336, 1, 38, 208, 3,
                 FB_SYNC_HOR_HIGH_ACT | FB_SYNC_VERT_HIGH_ACT,

[1] https://projectf.io/posts/video-timings-vga-720p-1080p/#hd-1920x1080-60-hz

Gr{oetje,eeting}s,

                         Geert