ecore-drm: Don't always check fb size against dumb buffer
authorChris Michael <cp.michael@samsung.com>
Wed, 4 Nov 2015 14:40:54 +0000 (09:40 -0500)
committerChris Michael <cp.michael@samsung.com>
Wed, 4 Nov 2015 14:40:54 +0000 (09:40 -0500)
Summary: If we are running using gl_drm, then we don't need to create
software dumb buffers on the drm device. Since we may not have the
dumb buffers (only used in software rendering), then we should not
always be checking the framebuffer size against the dumb buffer size.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>
src/lib/ecore_drm/ecore_drm_fb.c

index 69c610761749141c2a1ce39419d23c924dbd06ce..780516795bef6a19f0648e2cb6f4737ce9c508bf 100644 (file)
@@ -176,11 +176,14 @@ ecore_drm_fb_set(Ecore_Drm_Device *dev, Ecore_Drm_Fb *fb)
    EINA_SAFETY_ON_NULL_RETURN(dev);
    EINA_SAFETY_ON_NULL_RETURN(fb);
 
-   if ((fb->w != dev->dumb[0]->w) || (fb->h != dev->dumb[0]->h))
+   if (dev->dumb[0])
      {
-        /* we need to copy from fb to dev->dumb */
-        WRN("Trying to set a Framebuffer of improper size !!");
-        return;
+        if ((fb->w != dev->dumb[0]->w) || (fb->h != dev->dumb[0]->h))
+          {
+             /* we need to copy from fb to dev->dumb */
+             WRN("Trying to set a Framebuffer of improper size !!");
+             return;
+          }
      }
 
    if (!dev->next) dev->next = fb;