Skip to contentSkip to conversation

usb/kaweth: use wait_event_timeout()

PATCH Greg KH 2 participants 3 messages Jan 31, 2005 Kernel janitors

lore
Share & tools
lore

Greg KH

author
#0 lore raw
Quoted 7 lines · Nishanth Aravamudan

Hi,

Please consider applying.

Description: Use wait_event_timeout() instead of custom wait-queue code. Remove now unused variables. I changed the code to only add to the wait-queue if necessary, but I'm not sure if this is correct.

Please at least compile your patches to make sure they build. This one does not :(

greg k-h

_______________________________________________
Kernel-janitors mailing list
[email protected]
http://lists.osdl.org/mailman/listinfo/kernel-janitors

#1 lore raw
Quoted 11 lines · Greg KH
Quoted 7 lines · Nishanth Aravamudan

Hi,

Please consider applying.

Description: Use wait_event_timeout() instead of custom wait-queue code. Remove now unused variables. I changed the code to only add to the wait-queue if necessary, but I'm not sure if this is correct.

Please at least compile your patches to make sure they build. This one does not :(

I'm not seeing a build error here; what errors are coming up for you?

Thanks,
Nish

_______________________________________________
Kernel-janitors mailing list
[email protected]
http://lists.osdl.org/mailman/listinfo/kernel-janitors

author
#2 lore raw
In reply to Greg KH
Quoted 6 lines · Nishanth Aravamudan
Quoted 5 lines · Greg KH
Quoted 4 lines · Nishanth Aravamudan

Hi,

Please consider applying.

Quoted 8 lines
Quoted 6 lines
Quoted 3 lines

Description: Use wait_event_timeout() instead of custom wait-queue code. Remove now unused variables. I changed the code to only add to the wait-queue if necessary, but I'm not sure if this is correct.

Please at least compile your patches to make sure they build. This one does not :(

I'm not seeing a build error here; what errors are coming up for you?

Don't remember anymore, as I reverted the patch already...

I think it was a {} matching issue like your other patches.

thanks,

greg k-h

_______________________________________________
Kernel-janitors mailing list
[email protected]
http://lists.osdl.org/mailman/listinfo/kernel-janitors