diff options
author | Dan Carpenter <error27@gmail.com> | 2010-10-08 12:56:04 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-10-08 14:29:55 (GMT) |
commit | 055a1eda5fe143fdb8137cfd6eee39c9b58fe72d (patch) | |
tree | bdc4021878e019cb5b6da226d6c377540e2f58e3 /drivers/staging/bcm | |
parent | 785698e38dcd01cd15dacbf0cf1309a04b1e3208 (diff) | |
download | linux-fsl-qoriq-055a1eda5fe143fdb8137cfd6eee39c9b58fe72d.tar.xz |
Staging: bcm: mocro expansion bug
The WIMAX_MAX_MTU macro is used in drivers/staging/bcm/CmHost.c like
this:
if (Adapter->PackInfo[uiSearchRuleIndex].uiMaxBucketSize < WIMAX_MAX_MTU * 8)
The multiplication by eight has precedence over the addition so the
macro needs parenthesis to work.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/bcm')
-rw-r--r-- | drivers/staging/bcm/Macros.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/bcm/Macros.h b/drivers/staging/bcm/Macros.h index 6ee1480..0241234 100644 --- a/drivers/staging/bcm/Macros.h +++ b/drivers/staging/bcm/Macros.h @@ -231,7 +231,7 @@ typedef enum _E_PHS_DSC_ACTION #define MAX_PENDING_CTRL_PACKET (MAX_CTRL_QUEUE_LEN-10) -#define WIMAX_MAX_MTU MTU_SIZE + ETH_HLEN +#define WIMAX_MAX_MTU (MTU_SIZE + ETH_HLEN) #define AUTO_LINKUP_ENABLE 0x2 #define AUTO_SYNC_DISABLE 0x1 #define AUTO_FIRM_DOWNLOAD 0x1 |