Make code more readable with list_for_each_entry.
Compile tested.


Signed-off-by: Domen Puncer <[EMAIL PROTECTED]>

Patch incremental on previous list_for_each().
Signed-off-by: Maximilian Attems <[EMAIL PROTECTED]>



---

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

diff -puN drivers/usb/class/usb-midi.c~list-for-each-entry-drivers_usb_class_usb-midi 
drivers/usb/class/usb-midi.c
--- 
linux-2.6.9-rc1-bk7/drivers/usb/class/usb-midi.c~list-for-each-entry-drivers_usb_class_usb-midi
     2004-09-01 19:34:26.000000000 +0200
+++ linux-2.6.9-rc1-bk7-max/drivers/usb/class/usb-midi.c        2004-09-01 
19:34:26.000000000 +0200
@@ -805,7 +805,6 @@ static int usb_midi_open(struct inode *i
 {
        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 @@ static int usb_midi_open(struct inode *i
 
        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 int usb_midi_probe(struct usb_int
 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 @@ static void usb_midi_disconnect(struct u
        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 BEA Weblogic Workshop
FREE Java Enterprise J2EE developer tools!
Get your free copy of BEA WebLogic Workshop 8.1 today.
http://ads.osdn.com/?ad_id=5047&alloc_id=10808&op=click
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to