Skip to content
This repository has been archived by the owner on Apr 18, 2024. It is now read-only.

Commit

Permalink
mptcp: fix Kconfig warning
Browse files Browse the repository at this point in the history
  net/mptcp/Kconfig:62:warning: choice default symbol 'DEFAULT' is not contained in the choice
  net/mptcp/Kconfig:124:warning: choice default symbol 'DEFAULT' is not contained in the choice

Also removed a trailing whitespace introduces in the same commit.

Note that it would be nice to prefix all these "DEFAULT_xxx" with
"MPTCP_" and even "MPTCP_PM_" / "MPTCP_SCHED_" because it is not obvious
to know what they are when you see that in the .config:

  CONFIG_DEFAULT_DUMMY
  CONFIG_DEFAULT_FULLMESH
  CONFIG_DEFAULT_SCHEDULER
  (...)

But I guess it is too late and it should not conflicts with new kconfig
because they should always be prefixed with something.

Fixes: 54d8697 (Modularize the path-manager)
Signed-off-by: Matthieu Baerts <[email protected]>
Signed-off-by: Christoph Paasch <[email protected]>
(cherry picked from commit 1efcfb3)
Signed-off-by: Christoph Paasch <[email protected]>
  • Loading branch information
matttbe authored and cpaasch committed Feb 20, 2020
1 parent b864616 commit b56d4e2
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions net/mptcp/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ config MPTCP_NETLINK

choice
prompt "Default MPTCP Path-Manager"
default DEFAULT
default DEFAULT_DUMMY
help
Select the Path-Manager of your choice

Expand All @@ -75,7 +75,7 @@ endif
config DEFAULT_MPTCP_PM
string
default "default" if DEFAULT_DUMMY
default "fullmesh" if DEFAULT_FULLMESH
default "fullmesh" if DEFAULT_FULLMESH
default "ndiffports" if DEFAULT_NDIFFPORTS
default "binder" if DEFAULT_BINDER
default "default"
Expand Down Expand Up @@ -111,7 +111,7 @@ config MPTCP_REDUNDANT

choice
prompt "Default MPTCP Scheduler"
default DEFAULT
default DEFAULT_SCHEDULER
help
Select the Scheduler of your choice

Expand Down

0 comments on commit b56d4e2

Please sign in to comment.