Commit be3286f5 authored by Andrew Morton's avatar Andrew Morton Committed by Linus Torvalds

[PATCH] Fix sign handling bugs in mpt fusion

From: Ravikiran G Thirumalai <kiran@in.ibm.com>

Don't compare unsigned value for < 0.
parent f52e7cca
...@@ -1819,6 +1819,7 @@ mpt_adapter_disable(MPT_ADAPTER *this, int freeup) ...@@ -1819,6 +1819,7 @@ mpt_adapter_disable(MPT_ADAPTER *this, int freeup)
if (this != NULL) { if (this != NULL) {
int sz; int sz;
u32 state; u32 state;
int ret;
/* Disable the FW */ /* Disable the FW */
state = mpt_GetIocState(this, 1); state = mpt_GetIocState(this, 1);
...@@ -1829,9 +1830,9 @@ mpt_adapter_disable(MPT_ADAPTER *this, int freeup) ...@@ -1829,9 +1830,9 @@ mpt_adapter_disable(MPT_ADAPTER *this, int freeup)
if (this->cached_fw != NULL) { if (this->cached_fw != NULL) {
ddlprintk((KERN_INFO MYNAM ": Pushing FW onto adapter\n")); ddlprintk((KERN_INFO MYNAM ": Pushing FW onto adapter\n"));
if ((state = mpt_downloadboot(this, NO_SLEEP)) < 0) { if ((ret = mpt_downloadboot(this, NO_SLEEP)) < 0) {
printk(KERN_WARNING MYNAM printk(KERN_WARNING MYNAM
": firmware downloadboot failure (%d)!\n", state); ": firmware downloadboot failure (%d)!\n", ret);
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment