diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-02-22 22:33:33 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-03-05 14:48:04 -0800 |
| commit | a7df59ece7f845dffb354aae8318a30e7650034c (patch) | |
| tree | a74fdc32455350af9688605872e55b0cc538c03b /drivers/staging/gdm72xx | |
| parent | e0aa762cac39d18aa60938a3d1f7f6b42fcb2cdf (diff) | |
Staging: gdm72xx: remove duplicate condition
We know "len" is not zero because we tested for that at the beginning of
the function so this test can be removed.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/gdm72xx')
| -rw-r--r-- | drivers/staging/gdm72xx/gdm_wimax.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/staging/gdm72xx/gdm_wimax.c b/drivers/staging/gdm72xx/gdm_wimax.c index d29b32efcfd5..617d5f1c97c1 100644 --- a/drivers/staging/gdm72xx/gdm_wimax.c +++ b/drivers/staging/gdm72xx/gdm_wimax.c @@ -661,9 +661,8 @@ static void gdm_wimax_transmit_pkt(struct net_device *dev, char *buf, int len) cmd_len = be16_to_cpup((const __be16 *)&buf[2]); if (len < cmd_len + HCI_HEADER_SIZE) { - if (len) - netdev_err(dev, "%s: invalid length [%d/%d]\n", - __func__, cmd_len + HCI_HEADER_SIZE, len); + netdev_err(dev, "%s: invalid length [%d/%d]\n", + __func__, cmd_len + HCI_HEADER_SIZE, len); return; } |