On Fri, Oct 12, 2012 at 12:01 AM, Dan Carpenter <[email protected]> wrote: > On Thu, Oct 11, 2012 at 09:02:04PM -0400, Kevin McKinney wrote: >> @@ -193,7 +193,7 @@ VOID Bcm_kill_all_URBs(PS_INTERFACE_ADAPTER >> psIntfAdapter) >> */ >> >> /* Cancel submitted Interrupt-URB's */ >> - if (psIntfAdapter->psInterruptUrb != NULL) { >> + if (!psIntfAdapter->psInterruptUrb) { > > This test is reversed. Btw, if you run Smatch on your code it would > have caught this. >
Yeah, this is a stupid mistake on my part. I usually verify with Smatch before submitting, I just did not last night; but I will from now on. I will resubmit this entire patch set. Thanks, Kevin _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
