[PATCH v3 2/3] drm/sti: Remove local fbdev emulation Kconfig option

2015-10-30 Thread Archit Taneja
Hi Vincent, On 10/30/2015 6:00 PM, Vincent ABRIOU wrote: > Hi Archit, > > I tested fbdev_emulation module param and it works fine for the sti driver. > I will abandon my patch and integrate yours. And if you don't mind, I > will update your patch to remove the #ifdef CONFIG_DRM_FBDEV_EMULATION. S

[PATCH v3 2/3] drm/sti: Remove local fbdev emulation Kconfig option

2015-10-30 Thread Vincent ABRIOU
Hi Archit, I tested fbdev_emulation module param and it works fine for the sti driver. I will abandon my patch and integrate yours. And if you don't mind, I will update your patch to remove the #ifdef CONFIG_DRM_FBDEV_EMULATION. Thanks Vincent Reviewed-by: Vincent Abriou On 10/30/2015 11:25 A

[PATCH v3 2/3] drm/sti: Remove local fbdev emulation Kconfig option

2015-10-30 Thread Daniel Vetter
On Thu, Oct 29, 2015 at 02:33:02PM +0100, Vincent ABRIOU wrote: > Hi, > > I am fine with your proposal to remove DRM_STI_FBDEV but I would like to > be able to enable it by a module parameter. This is what I propose in > "[PATCH 1/1] drm/sti: enable fbdev compatibility through module param" We

[PATCH v3 2/3] drm/sti: Remove local fbdev emulation Kconfig option

2015-10-29 Thread Vincent ABRIOU
Hi, I am fine with your proposal to remove DRM_STI_FBDEV but I would like to be able to enable it by a module parameter. This is what I propose in "[PATCH 1/1] drm/sti: enable fbdev compatibility through module param" BR Vincent On 10/27/2015 09:10 AM, Archit Taneja wrote: > DRM_STI_FBDEV conf

[PATCH v3 2/3] drm/sti: Remove local fbdev emulation Kconfig option

2015-10-27 Thread Archit Taneja
DRM_STI_FBDEV config is currently used to enable/disable fbdev emulation for the sti kms driver. Remove this local config option and use the top level DRM_FBDEV_EMULATION config option instead where applicable. We replace the #ifdef in sti_drm_load with CONFIG_DRM_FBDEV_EMULATION. It's probably o