]> git.proxmox.com Git - mirror_ubuntu-artful-kernel.git/commitdiff
Staging: most: Use list_first_entry instead of list_entry
authorBhumika Goyal <bhumirks@gmail.com>
Sun, 6 Mar 2016 15:36:47 +0000 (21:06 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 11 Mar 2016 03:26:05 +0000 (19:26 -0800)
This patch replaces list_entry with list_first_entry as it makes the
code more clear.
Done using coccinelle:

@@
expression e;
@@
(
- list_entry(e->next,
+ list_first_entry(e,
  ...)
|
- list_entry(e->prev,
+ list_last_entry(e,
  ...)
)

Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/most/hdm-dim2/dim2_hdm.c

index caff5b209c9b3d48190f16fa3098b6ca99466708..0dc86add71612ddb962eec4b8c3a08d2e6be614f 100644 (file)
@@ -251,7 +251,7 @@ static int try_start_dim_transfer(struct hdm_channel *hdm_ch)
                return -EAGAIN;
        }
 
-       mbo = list_entry(head->next, struct mbo, list);
+       mbo = list_first_entry(head, struct mbo, list);
        buf_size = mbo->buffer_length;
 
        BUG_ON(mbo->bus_address == 0);
@@ -362,7 +362,7 @@ static void service_done_flag(struct dim2_hdm *dev, int ch_idx)
                        break;
                }
 
-               mbo = list_entry(head->next, struct mbo, list);
+               mbo = list_first_entry(head, struct mbo, list);
                list_del(head->next);
                spin_unlock_irqrestore(&dim_lock, flags);
 
@@ -495,7 +495,7 @@ static void complete_all_mbos(struct list_head *head)
                        break;
                }
 
-               mbo = list_entry(head->next, struct mbo, list);
+               mbo = list_first_entry(head, struct mbo, list);
                list_del(head->next);
                spin_unlock_irqrestore(&dim_lock, flags);