diff options
author | Geliang Tang <geliangtang@163.com> | 2015-12-18 10:33:25 -0500 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2015-12-20 02:11:10 -0500 |
commit | 7eb7404f7ee4bf59cb034897ab678aba2755c5e0 (patch) | |
tree | 8a8d4ed999773f5b208bb0ca0771971e900fc5e9 /net/bluetooth/cmtp | |
parent | 07f6f4a31e5a8dee67960fc07bb0b37c5f879d4d (diff) |
Bluetooth: use list_for_each_entry*
Use list_for_each_entry*() instead of list_for_each*() to simplify
the code.
Signed-off-by: Geliang Tang <geliangtang@163.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'net/bluetooth/cmtp')
-rw-r--r-- | net/bluetooth/cmtp/capi.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/net/bluetooth/cmtp/capi.c b/net/bluetooth/cmtp/capi.c index 9a50338772f3..46ac686c8911 100644 --- a/net/bluetooth/cmtp/capi.c +++ b/net/bluetooth/cmtp/capi.c | |||
@@ -100,10 +100,8 @@ static void cmtp_application_del(struct cmtp_session *session, struct cmtp_appli | |||
100 | static struct cmtp_application *cmtp_application_get(struct cmtp_session *session, int pattern, __u16 value) | 100 | static struct cmtp_application *cmtp_application_get(struct cmtp_session *session, int pattern, __u16 value) |
101 | { | 101 | { |
102 | struct cmtp_application *app; | 102 | struct cmtp_application *app; |
103 | struct list_head *p; | ||
104 | 103 | ||
105 | list_for_each(p, &session->applications) { | 104 | list_for_each_entry(app, &session->applications, list) { |
106 | app = list_entry(p, struct cmtp_application, list); | ||
107 | switch (pattern) { | 105 | switch (pattern) { |
108 | case CMTP_MSGNUM: | 106 | case CMTP_MSGNUM: |
109 | if (app->msgnum == value) | 107 | if (app->msgnum == value) |
@@ -511,14 +509,12 @@ static int cmtp_proc_show(struct seq_file *m, void *v) | |||
511 | struct capi_ctr *ctrl = m->private; | 509 | struct capi_ctr *ctrl = m->private; |
512 | struct cmtp_session *session = ctrl->driverdata; | 510 | struct cmtp_session *session = ctrl->driverdata; |
513 | struct cmtp_application *app; | 511 | struct cmtp_application *app; |
514 | struct list_head *p; | ||
515 | 512 | ||
516 | seq_printf(m, "%s\n\n", cmtp_procinfo(ctrl)); | 513 | seq_printf(m, "%s\n\n", cmtp_procinfo(ctrl)); |
517 | seq_printf(m, "addr %s\n", session->name); | 514 | seq_printf(m, "addr %s\n", session->name); |
518 | seq_printf(m, "ctrl %d\n", session->num); | 515 | seq_printf(m, "ctrl %d\n", session->num); |
519 | 516 | ||
520 | list_for_each(p, &session->applications) { | 517 | list_for_each_entry(app, &session->applications, list) { |
521 | app = list_entry(p, struct cmtp_application, list); | ||
522 | seq_printf(m, "appl %d -> %d\n", app->appl, app->mapping); | 518 | seq_printf(m, "appl %d -> %d\n", app->appl, app->mapping); |
523 | } | 519 | } |
524 | 520 | ||