drm: omapdrm: Lower indentation level in omap_gem_dma_sync_buffer()
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Thu, 20 Apr 2017 21:33:53 +0000 (00:33 +0300)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Fri, 2 Jun 2017 07:57:06 +0000 (10:57 +0300)
This makes the function more readable.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
drivers/gpu/drm/omapdrm/omap_gem.c

index c9ad18ea31c81b7e70d440315da197b04d7527c8..f4e0daede6e2d3fa9a5fe0ddbfbe999b4e57a400 100644 (file)
@@ -752,36 +752,35 @@ void omap_gem_dma_sync(struct drm_gem_object *obj,
 {
        struct drm_device *dev = obj->dev;
        struct omap_gem_object *omap_obj = to_omap_bo(obj);
+       int i, npages = obj->size >> PAGE_SHIFT;
+       struct page **pages = omap_obj->pages;
+       bool dirty = false;
 
-       if (is_cached_coherent(obj)) {
-               int i, npages = obj->size >> PAGE_SHIFT;
-               struct page **pages = omap_obj->pages;
-               bool dirty = false;
-
-               for (i = 0; i < npages; i++) {
-                       if (!omap_obj->addrs[i]) {
-                               dma_addr_t addr;
+       if (!is_cached_coherent(obj))
+               return;
 
-                               addr = dma_map_page(dev->dev, pages[i], 0,
-                                               PAGE_SIZE, DMA_BIDIRECTIONAL);
+       for (i = 0; i < npages; i++) {
+               if (!omap_obj->addrs[i]) {
+                       dma_addr_t addr;
 
-                               if (dma_mapping_error(dev->dev, addr)) {
-                                       dev_warn(dev->dev,
-                                               "%s: failed to map page\n",
-                                               __func__);
-                                       break;
-                               }
+                       addr = dma_map_page(dev->dev, pages[i], 0,
+                                           PAGE_SIZE, DMA_BIDIRECTIONAL);
 
-                               dirty = true;
-                               omap_obj->addrs[i] = addr;
+                       if (dma_mapping_error(dev->dev, addr)) {
+                               dev_warn(dev->dev, "%s: failed to map page\n",
+                                       __func__);
+                               break;
                        }
-               }
 
-               if (dirty) {
-                       unmap_mapping_range(obj->filp->f_mapping, 0,
-                                       omap_gem_mmap_size(obj), 1);
+                       dirty = true;
+                       omap_obj->addrs[i] = addr;
                }
        }
+
+       if (dirty) {
+               unmap_mapping_range(obj->filp->f_mapping, 0,
+                                   omap_gem_mmap_size(obj), 1);
+       }
 }
 
 /**