LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] ARM: imx_v6_v7_defconfig: Select CONFIG_FB_MXS
@ 2015-01-07 23:35 Fabio Estevam
  2015-01-08  3:30 ` Liu Ying
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Fabio Estevam @ 2015-01-07 23:35 UTC (permalink / raw)
  To: shawn.guo; +Cc: linux-kernel, Fabio Estevam

From: Fabio Estevam <fabio.estevam@freescale.com>

CONFIG_FB_MXS is the LCD driver for mx6solo-lite and mx6solox.

Enable it by default.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 arch/arm/configs/imx_v6_v7_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
index 9575af8..1f36977 100644
--- a/arch/arm/configs/imx_v6_v7_defconfig
+++ b/arch/arm/configs/imx_v6_v7_defconfig
@@ -203,6 +203,7 @@ CONFIG_DRM_PANEL_SIMPLE=y
 CONFIG_LCD_CLASS_DEVICE=y
 CONFIG_LCD_L4F00242T03=y
 CONFIG_LCD_PLATFORM=y
+CONFIG_FB_MXS=y
 CONFIG_BACKLIGHT_PWM=y
 CONFIG_BACKLIGHT_GPIO=y
 CONFIG_FRAMEBUFFER_CONSOLE=y
-- 
1.9.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ARM: imx_v6_v7_defconfig: Select CONFIG_FB_MXS
  2015-01-07 23:35 [PATCH] ARM: imx_v6_v7_defconfig: Select CONFIG_FB_MXS Fabio Estevam
@ 2015-01-08  3:30 ` Liu Ying
       [not found] ` <CA+8Hj80oLByoaEWTQw_jchxfha1Gm=HhZp8pgTdeOb06fu=uQQ@mail.gmail.com>
  2015-01-20  6:14 ` Shawn Guo
  2 siblings, 0 replies; 4+ messages in thread
From: Liu Ying @ 2015-01-08  3:30 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: Shawn Guo, linux-kernel, Fabio Estevam

2015-01-08 7:35 GMT+08:00 Fabio Estevam <festevam@gmail.com>:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> CONFIG_FB_MXS is the LCD driver for mx6solo-lite and mx6solox.
>
> Enable it by default.

I'm not sure if it is suitable to enable CONFIG_FB_MXS by default.
At least, I have no strong objections.

I am considering to add a LCDIF CRTC driver to the imx-drm framework
so that we may use the existing LDB drm driver to support the mx6solox
LVDS interface.

I see the i915 and omap2 fb drivers depend on !DRM_XXXX:

grep DRM_ drivers/video/fbdev/ -nr
drivers/video/fbdev/omap2/omapfb/Kconfig:3:        depends on FB &&
OMAP2_DSS && !DRM_OMAP
drivers/video/fbdev/Kconfig:1197:       depends on !DRM_I915

So, similarly, we'll probably have two LCDIF drivers and make the DRM
one enabled by default - finally, CONFIG_FB_MXS is disabled.

Another concern is the compatibility. Since both the framebuffer driver
and the imx-drm driver would export the framebuffer interface to the
user space, we'd better to keep the interface behavior the same.
Maybe, this is not a big problem, because we need to keep the compatibility
anyway...

Regards,
Liu Ying

>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  arch/arm/configs/imx_v6_v7_defconfig | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
> index 9575af8..1f36977 100644
> --- a/arch/arm/configs/imx_v6_v7_defconfig
> +++ b/arch/arm/configs/imx_v6_v7_defconfig
> @@ -203,6 +203,7 @@ CONFIG_DRM_PANEL_SIMPLE=y
>  CONFIG_LCD_CLASS_DEVICE=y
>  CONFIG_LCD_L4F00242T03=y
>  CONFIG_LCD_PLATFORM=y
> +CONFIG_FB_MXS=y
>  CONFIG_BACKLIGHT_PWM=y
>  CONFIG_BACKLIGHT_GPIO=y
>  CONFIG_FRAMEBUFFER_CONSOLE=y
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/



-- 
Best Regards,
Liu Ying

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ARM: imx_v6_v7_defconfig: Select CONFIG_FB_MXS
       [not found] ` <CA+8Hj80oLByoaEWTQw_jchxfha1Gm=HhZp8pgTdeOb06fu=uQQ@mail.gmail.com>
@ 2015-01-14 15:13   ` Fabio Estevam
  0 siblings, 0 replies; 4+ messages in thread
From: Fabio Estevam @ 2015-01-14 15:13 UTC (permalink / raw)
  To: Liu Ying; +Cc: Shawn Guo, linux-kernel, Fabio Estevam

On Thu, Jan 8, 2015 at 1:15 AM, Liu Ying <liu.y.victor@gmail.com> wrote:

> So, similarly, we'll probably have two LCDIF drivers and make the DRM
> one enabled by default - finally, CONFIG_FB_MXS is disabled.

Yes, agree. Most likely it will take some time to get your new DRM
version reviewed/applied, so we could at least have some working
framebuffer on mx6sl and mx6sx in 3.20 by default with my patch.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ARM: imx_v6_v7_defconfig: Select CONFIG_FB_MXS
  2015-01-07 23:35 [PATCH] ARM: imx_v6_v7_defconfig: Select CONFIG_FB_MXS Fabio Estevam
  2015-01-08  3:30 ` Liu Ying
       [not found] ` <CA+8Hj80oLByoaEWTQw_jchxfha1Gm=HhZp8pgTdeOb06fu=uQQ@mail.gmail.com>
@ 2015-01-20  6:14 ` Shawn Guo
  2 siblings, 0 replies; 4+ messages in thread
From: Shawn Guo @ 2015-01-20  6:14 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: linux-kernel, Fabio Estevam

On Wed, Jan 07, 2015 at 09:35:54PM -0200, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> CONFIG_FB_MXS is the LCD driver for mx6solo-lite and mx6solox.
> 
> Enable it by default.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>

Applied, thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-01-20  6:15 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-07 23:35 [PATCH] ARM: imx_v6_v7_defconfig: Select CONFIG_FB_MXS Fabio Estevam
2015-01-08  3:30 ` Liu Ying
     [not found] ` <CA+8Hj80oLByoaEWTQw_jchxfha1Gm=HhZp8pgTdeOb06fu=uQQ@mail.gmail.com>
2015-01-14 15:13   ` Fabio Estevam
2015-01-20  6:14 ` Shawn Guo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).