[ALSA] alsa core: convert to list_for_each_entry*

This patch converts most uses of list_for_each to list_for_each_entry all
across alsa. In some place apparently an item can be on a list with
different pointers so of course that isn't compatible with list_for_each, I
therefore didn't touch those places.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jaroslav Kysela <perex@suse.cz>

1 files changed, 3 insertions, 7 deletions

diff --git a/sound/core/memalloc.c b/sound/core/memalloc.c
index bc0bd0910a62..f057430db0d0 100644
--- a/sound/core/memalloc.c
+++ b/sound/core/memalloc.c
@@ -406,19 +406,17 @@ void snd_dma_free_pages(struct snd_dma_buffer *dmab)
  */
 size_t snd_dma_get_reserved_buf(struct snd_dma_buffer *dmab, unsigned int id)
 {
-        struct list_head *p;
         struct snd_mem_list *mem;
 
         snd_assert(dmab, return 0);
 
         mutex_lock(&list_mutex);
-        list_for_each(p, &mem_list_head) {
-                mem = list_entry(p, struct snd_mem_list, list);
+        list_for_each_entry(mem, &mem_list_head, list) {
                 if (mem->id == id &&
                     (mem->buffer.dev.dev == NULL || dmab->dev.dev == NULL ||
                      ! memcmp(&mem->buffer.dev, &dmab->dev, sizeof(dmab->dev)))) {
                         struct device *dev = dmab->dev.dev;
-                        list_del(p);
+                        list_del(&mem->list);
                         *dmab = mem->buffer;
                         if (dmab->dev.dev == NULL)
                                 dmab->dev.dev = dev;
@@ -488,7 +486,6 @@ static int snd_mem_proc_read(char *page, char **start, off_t off,
 {
         int len = 0;
         long pages = snd_allocated_pages >> (PAGE_SHIFT-12);
-        struct list_head *p;
         struct snd_mem_list *mem;
         int devno;
         static char *types[] = { "UNKNOWN", "CONT", "DEV", "DEV-SG", "SBUS" };
@@ -498,8 +495,7 @@ static int snd_mem_proc_read(char *page, char **start, off_t off,
                         "pages  : %li bytes (%li pages per %likB)\n",
                         pages * PAGE_SIZE, pages, PAGE_SIZE / 1024);
         devno = 0;
-        list_for_each(p, &mem_list_head) {
-                mem = list_entry(p, struct snd_mem_list, list);
+        list_for_each_entry(mem, &mem_list_head, list) {
                 devno++;
                 len += snprintf(page + len, count - len,
                                 "buffer %d : ID %08x : type %s\n",