usb: ulpi: kconfig: Remove meaningless choice default
authorUlf Magnusson <ulfalizer@gmail.com>
Tue, 30 Jan 2018 13:07:05 +0000 (14:07 +0100)
committerTom Rini <trini@konsulko.com>
Tue, 6 Feb 2018 01:58:12 +0000 (20:58 -0500)
'default' on a choice refers to the symbol selected by default, not to
the choice mode, so 'default n' is meaningless.

No functional changes. Optional choices implicitly default to n mode
(and there is no way to make them default to another mode).

Discovered in Kconfiglib (https://github.com/ulfalizer/Kconfiglib),
which prints the following warning:

warning: the default selection n (undefined) of <choice> (defined at drivers/usb/ulpi/Kconfig:3) is not contained in the choice

I've added a corresponding warning to the C tools too, which is
currently in linux-next: https://patchwork.kernel.org/patch/9983667/

Signed-off-by: Ulf Magnusson <ulfalizer@gmail.com>
drivers/usb/ulpi/Kconfig

index 329d2df3ed729b818283bfc0bce48f1e67b84921..001564d40c63360eeed714e95fab1ee3c74dff14 100644 (file)
@@ -3,7 +3,6 @@ comment "ULPI drivers"
 choice
        prompt "ULPI Viewport type"
        optional
-       default n
        help
          Select ULPI viewport (SoC-side interface to ULPI) implementation
          appropriate for the device if you want to communicate with