From: Michael Ellerman <michael@ellerman.id.au>
Date: Wed, 3 Aug 2005 10:21:23 +0000 (+1000)
Subject: [PATCH] ppc64: Consolidate some macros
X-Git-Tag: accepted/tizen/common/20141203.182822~44026^2~12
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ce21795275ab469b97384faa36462350af17eca0;p=platform%2Fkernel%2Flinux-arm64.git

[PATCH] ppc64: Consolidate some macros

The only caller of chunk_offset() and abs_chunk() is phys_to_abs(), so
fold the former two into the latter.

Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Paul Mackerras <paulus@samba.org>
---

diff --git a/include/asm-ppc64/abs_addr.h b/include/asm-ppc64/abs_addr.h
index 05414a9..0255277 100644
--- a/include/asm-ppc64/abs_addr.h
+++ b/include/asm-ppc64/abs_addr.h
@@ -44,24 +44,17 @@ static inline unsigned long addr_to_chunk(unsigned long addr)
 	return addr >> MSCHUNKS_CHUNK_SHIFT;
 }
 
-static inline unsigned long chunk_offset(unsigned long addr)
+static inline unsigned long phys_to_abs(unsigned long pa)
 {
-	return addr & MSCHUNKS_OFFSET_MASK;
-}
+	unsigned long chunk;
 
-static inline unsigned long abs_chunk(unsigned long pchunk)
-{
-	if (pchunk >= mschunks_map.num_chunks)
-		return pchunk;
+	chunk = addr_to_chunk(pa);
 
-	return mschunks_map.mapping[pchunk];
-}
+	if (chunk < mschunks_map.num_chunks)
+		chunk = mschunks_map.mapping[chunk];
 
-/* A macro so it can take pointers or unsigned long. */
-#define phys_to_abs(pa)						     \
-	({ unsigned long _pa = (unsigned long)(pa);			     \
-	   chunk_to_addr(abs_chunk(addr_to_chunk(_pa))) + chunk_offset(_pa); \
-	})
+	return chunk_to_addr(chunk) + (pa & MSCHUNKS_OFFSET_MASK);
+}
 
 static inline unsigned long
 physRpn_to_absRpn(unsigned long rpn)