[PATCH] ARM: Use list_for_each_entry() for dmabounce
authorRussell King <rmk@dyn-67.arm.linux.org.uk>
Wed, 22 Jun 2005 20:25:58 +0000 (21:25 +0100)
committerRussell King <rmk@dyn-67.arm.linux.org.uk>
Wed, 22 Jun 2005 20:25:58 +0000 (21:25 +0100)
Convert dmabounce.c to use list_for_each_entry() instead of
list_for_each() + list_entry().

Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
arch/arm/common/dmabounce.c

index 9d63a01..cbf2165 100644 (file)
@@ -93,15 +93,12 @@ static void print_alloc_stats(struct dmabounce_device_info *device_info)
 static inline struct dmabounce_device_info *
 find_dmabounce_dev(struct device *dev)
 {
-       struct list_head *entry;
-
-       list_for_each(entry, &dmabounce_devs) {
-               struct dmabounce_device_info *d =
-                       list_entry(entry, struct dmabounce_device_info, node);
+       struct dmabounce_device_info *d;
 
+       list_for_each_entry(d, &dmabounce_devs, node)
                if (d->dev == dev)
                        return d;
-       }
+
        return NULL;
 }
 
@@ -172,15 +169,11 @@ alloc_safe_buffer(struct dmabounce_device_info *device_info, void *ptr,
 static inline struct safe_buffer *
 find_safe_buffer(struct dmabounce_device_info *device_info, dma_addr_t safe_dma_addr)
 {
-       struct list_head *entry;
-
-       list_for_each(entry, &device_info->safe_buffers) {
-               struct safe_buffer *b =
-                       list_entry(entry, struct safe_buffer, node);
+       struct safe_buffer *b;
 
+       list_for_each_entry(b, &device_info->safe_buffers, node)
                if (b->safe_dma_addr == safe_dma_addr)
                        return b;
-       }
 
        return NULL;
 }
@@ -301,7 +294,6 @@ unmap_single(struct device *dev, dma_addr_t dma_addr, size_t size,
                        __func__, buf->ptr, (void *) virt_to_dma(dev, buf->ptr),
                        buf->safe, (void *) buf->safe_dma_addr);
 
-
                DO_STATS ( device_info->bounce_count++ );
 
                if (dir == DMA_FROM_DEVICE || dir == DMA_BIDIRECTIONAL) {