ChangeSet 1.1832.55.29, 2004/09/05 05:22:50+02:00, [EMAIL PROTECTED]

[PATCH] list_for_each_entry: drivers-usb-class-usb-midi.c

Make code more readable with list_for_each_entry.
Compile tested.


Signed-off-by: Domen Puncer <[EMAIL PROTECTED]>
Signed-off-by: Maximilian Attems <[EMAIL PROTECTED]>
Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]>


 drivers/usb/class/usb-midi.c |   11 +++--------
 1 files changed, 3 insertions(+), 8 deletions(-)


diff -Nru a/drivers/usb/class/usb-midi.c b/drivers/usb/class/usb-midi.c
--- a/drivers/usb/class/usb-midi.c      2004-10-19 08:17:56 -07:00
+++ b/drivers/usb/class/usb-midi.c      2004-10-19 08:17:56 -07:00
@@ -805,7 +805,6 @@
 {
        int minor = iminor(inode);
        DECLARE_WAITQUEUE(wait, current);
-       struct list_head      *devs, *mdevs;
        struct usb_midi_state *s;
        struct usb_mididev    *m;
        unsigned long flags;
@@ -817,10 +816,8 @@
 
        for(;;) {
                down(&open_sem);
-               list_for_each(devs, &mididevs) {
-                       s = list_entry(devs, struct usb_midi_state, mididev);
-                       list_for_each(mdevs, &s->midiDevList) {
-                               m = list_entry(mdevs, struct usb_mididev, list);
+               list_for_each_entry(s, &mididevs, mididev) {
+                       list_for_each_entry(m, &s->midiDevList, list) {
                                if ( !((m->dev_midi ^ minor) & ~0xf) )
                                        goto device_found;
                        }
@@ -1994,7 +1991,6 @@
 static void usb_midi_disconnect(struct usb_interface *intf)
 {
        struct usb_midi_state *s = usb_get_intfdata (intf);
-       struct list_head      *list;
        struct usb_mididev    *m;
 
        if ( !s )
@@ -2012,8 +2008,7 @@
        s->usbdev = NULL;
        usb_set_intfdata (intf, NULL);
 
-       list_for_each(list, &s->midiDevList) {
-               m = list_entry(list, struct usb_mididev, list);
+       list_for_each_entry(m, &s->midiDevList, list) {
                wake_up(&(m->min.ep->wait));
                wake_up(&(m->mout.ep->wait));
                if ( m->dev_midi >= 0 ) {



-------------------------------------------------------
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to