Skip to content
Permalink
Browse files

kconfig: Use 'default' instead of 'def_bool' in Kconfig.defconfig files

Same deal as in commit 4638652 ("Kconfig: Use 'default' instead of
'def_bool' in Kconfig.defconfig files"), fixing new stuff that got
introduced since then.

Some symbols, like ALTERA_AVALON_PIO, are only defined in
Kconfig.defconfig files, and so need the def_bool.

Motivation (from the note at the end of
guides/kconfig/index.html#common-shorthands):

For a symbol defined in multiple locations (e.g., in a Kconfig.defconfig
file in Zephyr), it is best to only give the symbol type for the "base"
definition of the symbol, and to use 'default' (instead of 'def_<type>'
value) for the remaining definitions. That way, if the base definition
of the symbol is removed, the symbol ends up without a type, which
generates a warning that points to the other definitions. That makes the
extra definitions easier to discover and remove.

Signed-off-by: Ulf Magnusson <Ulf.Magnusson@nordicsemi.no>
  • Loading branch information...
ulfalizer authored and nashif committed Mar 18, 2019
1 parent f6784ed commit 7fdb52575479e02f66e49d4278793b5a6f45d8ca
@@ -6,6 +6,6 @@ config BOARD
default "iotdk"

config CPU_ARCEM4
def_bool y
default y

endif # BOARD_IOTDK
@@ -11,8 +11,8 @@ config BOARD
if UART_NS16550

config UART_NS16550_PORT_0
def_bool y if UART_CONSOLE
default y if UART_CONSOLE

endif

endif #BOARD_MEC2016EVB_ASSY6797
endif # BOARD_MEC2016EVB_ASSY6797
@@ -51,10 +51,10 @@ endif # UART_MCUX_LPUART
if NETWORKING

config NET_L2_ETHERNET
def_bool y
default y

config ETH_MCUX_0
def_bool y if NET_L2_ETHERNET
default y if NET_L2_ETHERNET

endif # NETWORKING

@@ -64,10 +64,10 @@ endif # UART_MCUX_LPUART
if NETWORKING

config NET_L2_ETHERNET
def_bool y
default y

config ETH_MCUX_0
def_bool y if NET_L2_ETHERNET
default y if NET_L2_ETHERNET

endif # NETWORKING

@@ -21,27 +21,27 @@ endchoice
if GPIO_MCUX_IGPIO

config GPIO_MCUX_IGPIO_1
def_bool y
default y

config GPIO_MCUX_IGPIO_5
def_bool y
default y

endif # GPIO_MCUX_IGPIO

if UART_MCUX_LPUART

config UART_MCUX_LPUART_1
def_bool y
default y

endif # UART_MCUX_LPUART

if NETWORKING

config NET_L2_ETHERNET
def_bool y
default y

config ETH_MCUX_0
def_bool y if NET_L2_ETHERNET
default y if NET_L2_ETHERNET

endif # NETWORKING

@@ -58,7 +58,7 @@ endif # COUNTER
if I2C

config I2C_SBCON
def_bool y
default y

endif # I2C

@@ -12,7 +12,7 @@ config BOARD
if IEEE802154

config IEEE802154_NRF5
def_bool y
default y

endif # IEEE802154

@@ -71,7 +71,7 @@ if SPI
if SPI_SAM_PORT_0

config SPI_SAME70_PORT_0_PIN_CS3
def_bool y
default y

endif # SPI_SAM_PORT_0

@@ -44,10 +44,10 @@ endif # SPI
if NETWORKING

config NET_L2_ETHERNET
def_bool y
default y

config ETH_STM32_HAL
def_bool y
default y

endif # NETWORKING

@@ -25,26 +25,26 @@ endif
if SERIAL

config UART_PL011
def_bool y
default y

config UART_INTERRUPT_DRIVEN
def_bool y
default y

config UART_PL011_PORT0
def_bool y
default y

config UART_PL011_PORT1
def_bool y
default y

endif # SERIAL

if COUNTER

config TIMER_TMR_CMSDK_APB
def_bool y
default y

config TIMER_DTMR_CMSDK_APB
def_bool y
default y

endif # COUNTER

@@ -92,6 +92,6 @@ endif # BOARD_NATIVE_POSIX
if USB

config USB_NATIVE_POSIX
def_bool y
default y

endif # USB
@@ -8,7 +8,7 @@ config BOARD
if PWM

config PWM_SIFIVE
def_bool y
default y

endif

@@ -9,52 +9,52 @@ config BOARD
if UART_RV32M1_LPUART

config UART_RV32M1_LPUART_0
def_bool y if UART_CONSOLE
default y if UART_CONSOLE

endif # UART_RV32M1

if PINMUX_RV32M1

config PINMUX_RV32M1_PORTA
def_bool y
default y

config PINMUX_RV32M1_PORTB
def_bool y
default y

config PINMUX_RV32M1_PORTC
def_bool y
default y

config PINMUX_RV32M1_PORTD
def_bool y
default y

config PINMUX_RV32M1_PORTE
def_bool y if RV32M1_INTMUX
default y if RV32M1_INTMUX

endif # PINMUX_RV32M1

if GPIO_RV32M1

config GPIO_RV32M1_PORTA
def_bool y
default y

config GPIO_RV32M1_PORTB
def_bool y if RV32M1_INTMUX
default y if RV32M1_INTMUX

config GPIO_RV32M1_PORTC
def_bool y if RV32M1_INTMUX
default y if RV32M1_INTMUX

config GPIO_RV32M1_PORTD
def_bool y if RV32M1_INTMUX
default y if RV32M1_INTMUX

config GPIO_RV32M1_PORTE
def_bool y if RV32M1_INTMUX
default y if RV32M1_INTMUX

endif # GPIO_RV32M1

if SERIAL

config UART_RV32M1_LPUART
def_bool y
default y

endif # SERIAL

@@ -174,10 +174,10 @@ endif
if FLASH

config SPI
def_bool y
default y

config SPI_NOR
def_bool y
default y

config SPI_NOR_PAGE_SIZE
default 256
@@ -186,27 +186,27 @@ config SPI_NOR_SECTOR_SIZE
default 4096

config FLASH_HAS_PAGE_LAYOUT
def_bool y
default y

config FLASH_PAGE_LAYOUT
def_bool y
default y

endif

if SPI

config SPI_DW
def_bool y
default y

config SPI_0
def_bool y
default y

endif

if PINMUX

config PINMUX_INTEL_S1000
def_bool y
default y

endif

@@ -30,20 +30,20 @@ config SYS_CLOCK_HW_CYCLES_PER_SEC
default 144000000

config HARVARD
def_bool y
default y

if SERIAL

config UART_NS16550
def_bool y
default y

endif # SERIAL

if UART_CONSOLE

config UART_NS16550_PORT_0
def_bool y
default y

endif # UART_CONSOLE

endif #ARC_IOT
endif # ARC_IOT
@@ -18,7 +18,7 @@ config SYS_CLOCK_HW_CYCLES_PER_SEC
if SERIAL

config UART_NS16550
def_bool y
default y

endif # SERIAL

@@ -35,7 +35,7 @@ endif # SERIAL
if IPM

config IPM_IMX
def_bool y
default y

endif # IPM

@@ -24,7 +24,7 @@ config IPG_DIV
default 3

config GPIO
def_bool y
default y

if NET_L2_ETHERNET

@@ -53,7 +53,7 @@ endif # I2C
if NET_L2_ETHERNET

config ETH_MCUX
def_bool y if HAS_MCUX_ENET
default y if HAS_MCUX_ENET

endif # NET_L2_ETHERNET

@@ -39,7 +39,7 @@ endif # CLOCK_CONTROL
if COUNTER

config COUNTER_MCUX_RTC
def_bool y
default y

endif # COUNTER

@@ -26,7 +26,7 @@ endif # SERIAL
if I2C

config I2C_GECKO
def_bool y
default y

endif # I2C

@@ -18,7 +18,7 @@ config NUM_IRQS
if GPIO_STM32

config GPIO_STM32_PORTD
def_bool y
default y

config GPIO_STM32_PORTE
default y

0 comments on commit 7fdb525

Please sign in to comment.
You can’t perform that action at this time.