Re: [PATCH 05/11] fbdev: Read video= option with fb_get_option() in modedb

2023-02-17 Thread Javier Martinez Canillas
Thomas Zimmermann  writes:

> Get the kernel's global video= parameter with fb_get_option(). Done
> to unexport the internal fbdev state fb_mode_config. No functional
> changes.
>
> Signed-off-by: Thomas Zimmermann 
> ---

Reviewed-by: Javier Martinez Canillas 

Best regards,
Javier



[PATCH 05/11] fbdev: Read video= option with fb_get_option() in modedb

2023-02-09 Thread Thomas Zimmermann
Get the kernel's global video= parameter with fb_get_option(). Done
to unexport the internal fbdev state fb_mode_config. No functional
changes.

Signed-off-by: Thomas Zimmermann 
---
 drivers/video/fbdev/core/modedb.c | 8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/drivers/video/fbdev/core/modedb.c 
b/drivers/video/fbdev/core/modedb.c
index 6473e0dfe146..23cf8eba785d 100644
--- a/drivers/video/fbdev/core/modedb.c
+++ b/drivers/video/fbdev/core/modedb.c
@@ -620,6 +620,7 @@ int fb_find_mode(struct fb_var_screeninfo *var,
 const struct fb_videomode *default_mode,
 unsigned int default_bpp)
 {
+   char *mode_option_buf = NULL;
int i;
 
/* Set up defaults */
@@ -635,8 +636,10 @@ int fb_find_mode(struct fb_var_screeninfo *var,
default_bpp = 8;
 
/* Did the user specify a video mode? */
-   if (!mode_option)
-   mode_option = fb_mode_option;
+   if (!mode_option) {
+   fb_get_options(NULL, _option_buf);
+   mode_option = mode_option_buf;
+   }
if (mode_option) {
const char *name = mode_option;
unsigned int namelen = strlen(name);
@@ -715,6 +718,7 @@ int fb_find_mode(struct fb_var_screeninfo *var,
res_specified = 1;
}
 done:
+   kfree(mode_option_buf);
if (cvt) {
struct fb_videomode cvt_mode;
int ret;
-- 
2.39.1