Re: [PATCH] powerpc: Sort the selects under CONFIG_PPC
From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2017-03-07 03:59:04
Stephen Rothwell [off-list ref] writes:
On Mon, 6 Mar 2017 23:05:17 +1100 Michael Ellerman [off-list ref] wrote:quoted
We have a big list of selects under CONFIG_PPC, and currently they're completely unsorted. This means people tend to add new selects at the bottom of the list, and so two commits which both add a new select will often conflict. Instead sort it alphabetically. This is nicer in and of itself, but also means two commits that add a new select will have a greater chance of not conflicting. Add a note at the top and bottom asking people to keep it sorted. And while we're here pad out the 'if' expressions to make them stand out. Suggested-by: Stephen Rothwell <redacted> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>Looks good.quoted
+ select ARCH_USE_CMPXCHG_LOCKREF if PPC64Why not just move these "if PPC64" selects to "config PPC64" in arch/powerpc/platforms/Kconfig.cputype?
I'd rather they were all in one place.
Secondary question: why not move "config PPC64" to this file?
Fine by me. cheers