Re: [PATCH] fbdev: mxsfb: implement FB_PRE_INIT_FB option

2019-04-01 Thread Bartlomiej Zolnierkiewicz

On 02/06/2019 11:14 AM, Martin Kepplinger wrote:
> From: Melchior Franz 
> 
> The FB_PRE_INIT_FB option keeps the kernel from reinitializing the display
> and prevents flickering during the transition from a bootloader splash
> screen to the kernel logo screen.
> 
> Make this option available for the mxsfb driver.
> 
> Signed-off-by: Melchior Franz 
> Signed-off-by: Martin Kepplinger 
> Signed-off-by: Manfred Schlaegl 

Patch queued for v5.2, thanks.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH] fbdev: mxsfb: implement FB_PRE_INIT_FB option

2019-02-07 Thread Martin Kepplinger
From: Melchior Franz 

The FB_PRE_INIT_FB option keeps the kernel from reinitializing the display
and prevents flickering during the transition from a bootloader splash
screen to the kernel logo screen.

Make this option available for the mxsfb driver.

Signed-off-by: Melchior Franz 
Signed-off-by: Martin Kepplinger 
Signed-off-by: Manfred Schlaegl 
---
 drivers/video/fbdev/Kconfig |  2 +-
 drivers/video/fbdev/mxsfb.c | 12 
 2 files changed, 13 insertions(+), 1 deletion(-)

diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig
index 58a9590c9db6..0e7ab29c9c70 100644
--- a/drivers/video/fbdev/Kconfig
+++ b/drivers/video/fbdev/Kconfig
@@ -2183,7 +2183,7 @@ config FB_EP93XX
 
 config FB_PRE_INIT_FB
bool "Don't reinitialize, use bootloader's GDC/Display configuration"
-   depends on FB && FB_MB862XX_LIME
+   depends on FB && (FB_MB862XX_LIME || FB_MXS)
---help---
  Select this option if display contents should be inherited as set by
  the bootloader.
diff --git a/drivers/video/fbdev/mxsfb.c b/drivers/video/fbdev/mxsfb.c
index 12c8bd1d24d5..a017200a16b3 100644
--- a/drivers/video/fbdev/mxsfb.c
+++ b/drivers/video/fbdev/mxsfb.c
@@ -181,6 +181,7 @@ struct mxsfb_info {
const struct mxsfb_devdata *devdata;
u32 sync;
struct regulator *reg_lcd;
+   int pre_init;
 };
 
 #define mxsfb_is_v3(host) (host->devdata->ipversion == 3)
@@ -419,6 +420,12 @@ static int mxsfb_set_par(struct fb_info *fb_info)
 
fb_info->fix.line_length = line_size;
 
+   if (host->pre_init) {
+   mxsfb_enable_controller(fb_info);
+   host->pre_init = 0;
+   return 0;
+   }
+
/*
 * It seems, you can't re-program the controller if it is still running.
 * This may lead into shifted pictures (FIFO issue?).
@@ -931,6 +938,10 @@ static int mxsfb_probe(struct platform_device *pdev)
if (IS_ERR(host->reg_lcd))
host->reg_lcd = NULL;
 
+#if defined(CONFIG_FB_PRE_INIT_FB)
+   host->pre_init = 1;
+#endif
+
fb_info->pseudo_palette = devm_kcalloc(>dev, 16, sizeof(u32),
   GFP_KERNEL);
if (!fb_info->pseudo_palette) {
@@ -963,6 +974,7 @@ static int mxsfb_probe(struct platform_device *pdev)
mxsfb_enable_controller(fb_info);
}
 
+   host->pre_init = 0;
dev_info(>dev, "initialized\n");
 
return 0;
-- 
2.20.1



smime.p7s
Description: S/MIME cryptographic signature
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel