Message ID | 1fff2054e5dfc00329804dbd3f2a91667c9a8aff.1623438544.git.christophe.leroy@csgroup.eu (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
Series | [1/2] powerpc/64: drop redundant defination of spin_until_cond | expand |
Context | Check | Description |
---|---|---|
snowpatch_ozlabs/apply_patch | success | Successfully applied on branch powerpc/merge (c53db722ec7ab3ebf29ecf61e922820f31e5284b) |
snowpatch_ozlabs/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 17 lines checked |
snowpatch_ozlabs/needsstable | success | Patch has no Fixes tags |
On Fri, Jun 11, 2021 at 07:10:57PM +0000, Christophe Leroy wrote: > From: Sudeep Holla <sudeep.holla@arm.com> > > linux/processor.h has exactly same defination for spin_until_cond. > Drop the redundant defination in asm/processor.h > Wow you must be real good at ML archaeology, this must have been at-least 3+ years old. I found this when I wanted to you spin_until_cond. Thanks anyways for digging the original patch, nobody would have remembered even if you posted fresh 😉.
On Fri, 11 Jun 2021 19:10:57 +0000 (UTC), Christophe Leroy wrote: > linux/processor.h has exactly same defination for spin_until_cond. > Drop the redundant defination in asm/processor.h Applied to powerpc/next. [1/2] powerpc/64: drop redundant defination of spin_until_cond https://git.kernel.org/powerpc/c/db8f7066dc498acf9074ed3c11a7a24f318d8d4f [2/2] powerpc/watchdog: include linux/processor.h for spin_until_cond https://git.kernel.org/powerpc/c/2400c13c437debc99d3399a7100d4e8c3fe20a08 cheers
diff --git a/arch/powerpc/include/asm/processor.h b/arch/powerpc/include/asm/processor.h index 7bf8a15af224..0819854eeab9 100644 --- a/arch/powerpc/include/asm/processor.h +++ b/arch/powerpc/include/asm/processor.h @@ -339,17 +339,6 @@ static inline unsigned long __pack_fe01(unsigned int fpmode) #define spin_end() HMT_medium() -#define spin_until_cond(cond) \ -do { \ - if (unlikely(!(cond))) { \ - spin_begin(); \ - do { \ - spin_cpu_relax(); \ - } while (!(cond)); \ - spin_end(); \ - } \ -} while (0) - #endif /* Check that a certain kernel stack pointer is valid in task_struct p */