On Tue, Sep 27, 2016 at 09:51:40AM -0500, Babu Moger wrote:Sure. thanks. Here it is below. I will re-post v3.
On 9/27/2016 6:40 AM, Peter Zijlstra wrote:The idea is to have the SPAC specific stuff in arch/sparc/Kconfig,
On Tue, Sep 27, 2016 at 06:46:25AM +0200, Sam Ravnborg wrote:How about this?
Since this is only relevant for sparc, and for sparc this is "select"ed,Yes, this should not be user selectable. I don't mind the help being
then there is limited/no gain having this as a visible menu config option.
How about adding just a simple non-visible config symbol:
The nice help text can be added to the H file, and the select
can be move to the sparc/Kconfig file where it really belongs.
Moved everything to arch/sparc/Kconfig.debug. It may be not useful
to have help in
config file as it is not visible. Lets have some explanation in .h file.
I will send v3 version if you all agree.
diff --git a/arch/sparc/Kconfig.debug b/arch/sparc/Kconfig.debug
index 6db35fb..67e58a1 100644
@@ -21,4 +21,9 @@ config FRAME_POINTER
depends on MCOUNT
+ depends on PROVE_LOCKING && SPARC
+ default y
and not scattered in Kconfig files all over the tree.
Therefore drop the "depends".
In sparc/Kconfig you then just do:
select PROVE_LOCKING_SMALL if PROVE_LOCKING
The if part is likely not needed as PROVE_LOCKING_SMALL will be ignored
unless PROVE_LOCKING is enabled.