Bluetooth: replace list_for_each with list_for_each_entry whenever possible

When all items in the list have the same type there is no much of a point
to use list_for_each except if you want to use the list pointer itself.

Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>

1 files changed, 4 insertions, 9 deletions

diff --git a/net/bluetooth/cmtp/core.c b/net/bluetooth/cmtp/core.c
index 7d00ddf9e9dc..9e8940b24bba 100644
--- a/net/bluetooth/cmtp/core.c
+++ b/net/bluetooth/cmtp/core.c
@@ -53,15 +53,13 @@ static LIST_HEAD(cmtp_session_list);
 static struct cmtp_session *__cmtp_get_session(bdaddr_t *bdaddr)
 {
         struct cmtp_session *session;
-        struct list_head *p;
 
         BT_DBG("");
 
-        list_for_each(p, &cmtp_session_list) {
-                session = list_entry(p, struct cmtp_session, list);
+        list_for_each_entry(session, &cmtp_session_list, list)
                 if (!bacmp(bdaddr, &session->bdaddr))
                         return session;
-        }
+
         return NULL;
 }
 
@@ -431,19 +429,16 @@ int cmtp_del_connection(struct cmtp_conndel_req *req)
 
 int cmtp_get_connlist(struct cmtp_connlist_req *req)
 {
-        struct list_head *p;
+        struct cmtp_session *session;
         int err = 0, n = 0;
 
         BT_DBG("");
 
         down_read(&cmtp_session_sem);
 
-        list_for_each(p, &cmtp_session_list) {
-                struct cmtp_session *session;
+        list_for_each_entry(session, &cmtp_session_list, list) {
                 struct cmtp_conninfo ci;
 
-                session = list_entry(p, struct cmtp_session, list);
-
                 __cmtp_copy_session(session, &ci);
 
                 if (copy_to_user(req->ci, &ci, sizeof(ci))) {