[PATCH 18/20] iwlwifi: fix malformed CONFIG_IWLWIFI_PCIE_RTPM default

2018-02-04 Thread Ulf Magnusson
'default false' should be 'default n', though they happen to have the
same effect here, due to undefined symbols ('false' in this case)
evaluating to n in a tristate sense.

Remove the default instead of changing it. bool and tristate symbols
implicitly default to n.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson 
---
 drivers/net/wireless/intel/iwlwifi/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/net/wireless/intel/iwlwifi/Kconfig 
b/drivers/net/wireless/intel/iwlwifi/Kconfig
index c5f2ddf9b0fe..e5a2fc738ac3 100644
--- a/drivers/net/wireless/intel/iwlwifi/Kconfig
+++ b/drivers/net/wireless/intel/iwlwifi/Kconfig
@@ -91,7 +91,6 @@ config IWLWIFI_BCAST_FILTERING
 config IWLWIFI_PCIE_RTPM
bool "Enable runtime power management mode for PCIe devices"
depends on IWLMVM && PM && EXPERT
-   default false
help
  Say Y here to enable runtime power management for PCIe
  devices.  If enabled, the device will go into low power mode
-- 
2.14.1



Re: [PATCH 01/10] net/sched: kconfig: Remove empty help texts

2018-01-31 Thread Ulf Magnusson
On Wed, Jan 31, 2018 at 4:43 PM, David Miller  wrote:
> From: Masahiro Yamada 
> Date: Thu, 1 Feb 2018 00:37:27 +0900
>
>> 2018-02-01 0:32 GMT+09:00 David Miller :
>>> From: Ulf Magnusson 
>>> Date: Tue, 30 Jan 2018 20:05:23 +0100
>>>
>>>> In preparation for adding a warning ("kconfig: Warn if help text is
>>>> blank"): https://lkml.org/lkml/2018/1/30/516
>>>>
>>>> Signed-off-by: Ulf Magnusson 
>>>
>>> Applied.
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
>>> the body of a message to majord...@vger.kernel.org
>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
>>
>> I had applied the whole series into linux-kbuild.
>>
>>
>> Do you want to apply it to your tree?
>
> I already did :-)
>
> It won't cause any problems for it to be in both of our trees so let's
> just leave it this way.

Yeah, sorry about that mess. I had assumed this would require
coordination from more people. :)

Masahiro: Thanks for merging it so quickly!

Cheers,
Ulf


[PATCH 01/11] net/sched: kconfig: Remove blank help texts

2018-01-31 Thread Ulf Magnusson
Blank help texts are probably either a typo, a Kconfig misunderstanding,
or some kind of half-committing to adding a help text (in which case a
TODO comment would be clearer, if the help text really can't be added
right away).

Best to remove them, IMO.

Signed-off-by: Ulf Magnusson 
---
 net/sched/Kconfig | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/net/sched/Kconfig b/net/sched/Kconfig
index c03d86a7775e..f24a6ae6819a 100644
--- a/net/sched/Kconfig
+++ b/net/sched/Kconfig
@@ -857,17 +857,14 @@ config NET_ACT_TUNNEL_KEY
 config NET_IFE_SKBMARK
 tristate "Support to encoding decoding skb mark on IFE action"
 depends on NET_ACT_IFE
----help---
 
 config NET_IFE_SKBPRIO
 tristate "Support to encoding decoding skb prio on IFE action"
 depends on NET_ACT_IFE
----help---
 
 config NET_IFE_SKBTCINDEX
 tristate "Support to encoding decoding skb tcindex on IFE action"
 depends on NET_ACT_IFE
----help---
 
 config NET_CLS_IND
bool "Incoming device classification"
-- 
2.14.1



[PATCH v2 01/10] net/sched: kconfig: Remove empty help texts

2018-01-30 Thread Ulf Magnusson
In preparation for adding a warning ("kconfig: Warn if help text is
blank"): https://lkml.org/lkml/2018/1/30/516

Signed-off-by: Ulf Magnusson 
---
 net/sched/Kconfig | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/net/sched/Kconfig b/net/sched/Kconfig
index c03d86a7775e..f24a6ae6819a 100644
--- a/net/sched/Kconfig
+++ b/net/sched/Kconfig
@@ -857,17 +857,14 @@ config NET_ACT_TUNNEL_KEY
 config NET_IFE_SKBMARK
 tristate "Support to encoding decoding skb mark on IFE action"
 depends on NET_ACT_IFE
----help---
 
 config NET_IFE_SKBPRIO
 tristate "Support to encoding decoding skb prio on IFE action"
 depends on NET_ACT_IFE
----help---
 
 config NET_IFE_SKBTCINDEX
 tristate "Support to encoding decoding skb tcindex on IFE action"
 depends on NET_ACT_IFE
----help---
 
 config NET_CLS_IND
bool "Incoming device classification"
-- 
2.14.1



[PATCH 01/10] net/sched: kconfig: Remove empty help texts

2018-01-30 Thread Ulf Magnusson
In preparation for adding a warning ("kconfig: Warn if help text is
blank"): https://lkml.org/lkml/2018/1/30/516

Signed-off-by: Ulf Magnusson 
---
 net/sched/Kconfig | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/net/sched/Kconfig b/net/sched/Kconfig
index c03d86a7775e..f24a6ae6819a 100644
--- a/net/sched/Kconfig
+++ b/net/sched/Kconfig
@@ -857,17 +857,14 @@ config NET_ACT_TUNNEL_KEY
 config NET_IFE_SKBMARK
 tristate "Support to encoding decoding skb mark on IFE action"
 depends on NET_ACT_IFE
----help---
 
 config NET_IFE_SKBPRIO
 tristate "Support to encoding decoding skb prio on IFE action"
 depends on NET_ACT_IFE
----help---
 
 config NET_IFE_SKBTCINDEX
 tristate "Support to encoding decoding skb tcindex on IFE action"
 depends on NET_ACT_IFE
----help---
 
 config NET_CLS_IND
bool "Incoming device classification"
-- 
2.14.1