diff options
| author | Russell King <rmk@dyn-67.arm.linux.org.uk> | 2005-10-29 08:18:10 -0400 |
|---|---|---|
| committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2005-10-29 08:18:10 -0400 |
| commit | 8c18fe2562c45180c407872d05857c55c1e5e37b (patch) | |
| tree | 52b4b8e5b501aaf614504e85f3a458704906a55f | |
| parent | fb31690fbac6f4055c1765522752e4746759f5de (diff) | |
[ARM] Fix buggy __phys_to_pfn / __pfn_to_phys
Macro arguments should _always_ be surrounded by parentheses
when used to prevent unexpected problems with operator precedence.
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
| -rw-r--r-- | include/asm-arm/mach/map.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/asm-arm/mach/map.h b/include/asm-arm/mach/map.h index 0619522bd926..b338936bde4f 100644 --- a/include/asm-arm/mach/map.h +++ b/include/asm-arm/mach/map.h | |||
| @@ -27,8 +27,8 @@ struct meminfo; | |||
| 27 | #define MT_ROM 6 | 27 | #define MT_ROM 6 |
| 28 | #define MT_IXP2000_DEVICE 7 | 28 | #define MT_IXP2000_DEVICE 7 |
| 29 | 29 | ||
| 30 | #define __phys_to_pfn(paddr) (paddr >> PAGE_SHIFT) | 30 | #define __phys_to_pfn(paddr) ((paddr) >> PAGE_SHIFT) |
| 31 | #define __pfn_to_phys(pfn) (pfn << PAGE_SHIFT) | 31 | #define __pfn_to_phys(pfn) ((pfn) << PAGE_SHIFT) |
| 32 | 32 | ||
| 33 | extern void create_memmap_holes(struct meminfo *); | 33 | extern void create_memmap_holes(struct meminfo *); |
| 34 | extern void memtable_init(struct meminfo *); | 34 | extern void memtable_init(struct meminfo *); |