diff mbox series

[2/2] drm/exynos/fbdev: do not skip fbdev init if there are no connectors

Message ID 20181026101328.26242-2-a.hajda@samsung.com (mailing list archive)
State Not Applicable
Headers show
Series [1/2] drm/exynos/dsi: register connector if it is created after drm bind | expand

Commit Message

Andrzej Hajda Oct. 26, 2018, 10:13 a.m. UTC
Since connectors can be created dynamically, fbdev should be initialized
even if there are no connectors at the moment. Otherwise fbdev will
not be created even after connector's appearance.
The patch fixes lack of fbdev on rinato and trats boards.

Fixes: 6afb7721e2a0 ("drm/exynos: move connector creation to attach callback")
Reported-by: Marek Szyprowski <m.szyprowski@samsung.com>
Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
---
 drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Marek Szyprowski Oct. 31, 2018, 7:33 a.m. UTC | #1
Hi,


On 2018-10-26 12:13, Andrzej Hajda wrote:
> Since connectors can be created dynamically, fbdev should be initialized
> even if there are no connectors at the moment. Otherwise fbdev will
> not be created even after connector's appearance.
> The patch fixes lack of fbdev on rinato and trats boards.
>
> Fixes: 6afb7721e2a0 ("drm/exynos: move connector creation to attach callback")
> Reported-by: Marek Szyprowski <m.szyprowski@samsung.com>
> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>

Together with v2 of "drm/exynos/dsi: register connector if it is created
after
drm bind" patch this finally restores /dev/fb0 emulation on Rinato and Trats
boards.

Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>

> ---
>  drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_fbdev.c b/drivers/gpu/drm/exynos/exynos_drm_fbdev.c
> index 918dd2c82209..01d182289efa 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_fbdev.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_fbdev.c
> @@ -192,7 +192,7 @@ int exynos_drm_fbdev_init(struct drm_device *dev)
>  	struct drm_fb_helper *helper;
>  	int ret;
>  
> -	if (!dev->mode_config.num_crtc || !dev->mode_config.num_connector)
> +	if (!dev->mode_config.num_crtc)
>  		return 0;
>  
>  	fbdev = kzalloc(sizeof(*fbdev), GFP_KERNEL);

Best regards
diff mbox series

Patch

diff --git a/drivers/gpu/drm/exynos/exynos_drm_fbdev.c b/drivers/gpu/drm/exynos/exynos_drm_fbdev.c
index 918dd2c82209..01d182289efa 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_fbdev.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_fbdev.c
@@ -192,7 +192,7 @@  int exynos_drm_fbdev_init(struct drm_device *dev)
 	struct drm_fb_helper *helper;
 	int ret;
 
-	if (!dev->mode_config.num_crtc || !dev->mode_config.num_connector)
+	if (!dev->mode_config.num_crtc)
 		return 0;
 
 	fbdev = kzalloc(sizeof(*fbdev), GFP_KERNEL);