| Submitter | Tushar Behera |
|---|---|
| Date | Nov. 16, 2012, 6:50 a.m. |
| Message ID | <1353048646-10935-15-git-send-email-tushar.behera@linaro.org> |
| Download | mbox | patch |
| Permalink | /patch/199496/ |
| State | Not Applicable |
| Delegated to: | David Miller |
| Headers | show |
Comments
On Fri, 2012-11-16 at 12:20 +0530, Tushar Behera wrote: > No need to check whether unsigned variable is less than 0. > > CC: Luciano Coelho <coelho@ti.com> > CC: linux-wireless@vger.kernel.org > CC: netdev@vger.kernel.org > Signed-off-by: Tushar Behera <tushar.behera@linaro.org> > --- Applied in the wl12xx.git tree. Thanks! -- Luca. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Patch
diff --git a/drivers/net/wireless/ti/wlcore/debugfs.c b/drivers/net/wireless/ti/wlcore/debugfs.c index c86bb00..93f801d 100644 --- a/drivers/net/wireless/ti/wlcore/debugfs.c +++ b/drivers/net/wireless/ti/wlcore/debugfs.c @@ -993,7 +993,7 @@ static ssize_t sleep_auth_write(struct file *file, return -EINVAL; } - if (value < 0 || value > WL1271_PSM_MAX) { + if (value > WL1271_PSM_MAX) { wl1271_warning("sleep_auth must be between 0 and %d", WL1271_PSM_MAX); return -ERANGE;
No need to check whether unsigned variable is less than 0. CC: Luciano Coelho <coelho@ti.com> CC: linux-wireless@vger.kernel.org CC: netdev@vger.kernel.org Signed-off-by: Tushar Behera <tushar.behera@linaro.org> --- drivers/net/wireless/ti/wlcore/debugfs.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)