2014-06-05 22:09:51

by Pranith Kumar

[permalink] [raw]
Subject: [PATCH 1/1] define generic version of cmpxchg{64} only if not, previously defined

define generic versions of cmpxchg{64} only if not previously defined.

This makes these definition in-line to other definitions of generic versions


Signed-off-by: Pranith Kumar <[email protected]>
---
include/asm-generic/cmpxchg.h | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/include/asm-generic/cmpxchg.h b/include/asm-generic/cmpxchg.h
index 811fb1e..7154b79 100644
--- a/include/asm-generic/cmpxchg.h
+++ b/include/asm-generic/cmpxchg.h
@@ -102,7 +102,12 @@ unsigned long __xchg(unsigned long x, volatile void *ptr, int size)
#define cmpxchg64_local(ptr, o, n) __cmpxchg64_local_generic((ptr), (o), (n))
#endif

+#ifndef cmpxchg
#define cmpxchg(ptr, o, n) cmpxchg_local((ptr), (o), (n))
+#endif
+
+#ifndef cmpxchg64
#define cmpxchg64(ptr, o, n) cmpxchg64_local((ptr), (o), (n))
+#endif

#endif /* __ASM_GENERIC_CMPXCHG_H */
--
1.7.9.5


2014-06-11 12:55:27

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH 1/1] define generic version of cmpxchg{64} only if not, previously defined

On Friday 06 June 2014, Pranith Kumar wrote:
>
> define generic versions of cmpxchg{64} only if not previously defined.
>
> This makes these definition in-line to other definitions of generic versions
>
>
> Signed-off-by: Pranith Kumar <[email protected]>

Acked-by: Arnd Bergmann <[email protected]>

I assume you have a patch that depends on this. Please queue up this first
patch in the same series as the one that needs it.

Arnd

2014-06-11 16:07:54

by Pranith Kumar

[permalink] [raw]
Subject: Re: [PATCH 1/1] define generic version of cmpxchg{64} only if not, previously defined

On 06/11/2014 08:55 AM, Arnd Bergmann wrote:
> On Friday 06 June 2014, Pranith Kumar wrote:
>>
>> define generic versions of cmpxchg{64} only if not previously defined.
>>
>> This makes these definition in-line to other definitions of generic versions
>>
>
> Acked-by: Arnd Bergmann <[email protected]>
>
> I assume you have a patch that depends on this. Please queue up this first
> patch in the same series as the one that needs it.
>

This is just a clean up patch and there are no other patches depending on this. I will mark them as such from now on :)

--
Pranith