zephyr/drivers/sensor/amg88xx/Kconfig
Kumar Gala df81fef944 drivers: sensor: Convert Kconfig bus 'depends on' to 'select'
This change in pattern is meant to address a misconfiguration issue
that can occur for sensors that support being on multiple busses
like I2C & SPI.

For example, you can have a configuration in which such a sensor is
on the I2C bus in the devicetree and the sensor is enabled.  However
the application configuration enables CONFIG_SPI=y and CONFIG_I2C=n
and this will cause the sensor driver to be built by default, however
since we don't have the I2C bus enabled the driver will not compile
correctly.

Previously we had been adding to board Kconfig.defconfig something
like:

	config I2C
		default y if SENSOR

This pattern doesn't scale well and may differ from what an application
specific need/use is.

So instead move to a pattern in which we leave the default enablement
up to the devicetree "status" property for the sensor.  We then have
the Kconfig move from 'depends on <BUS>' to 'select <BUS>' and in
the case of drivers that support multiple busses we have the Kconfig
be: 'select <BUS> if $(dt_compat_on_bus,$(<DT_COMPAT>),<BUS>) for
each bus type the sensor supports.

This removes the need to add Kconfig logic to each board and enables
the bus subsystem and bus controller driver if the sensor requires
it by default in the build system.

Fixes: #48518

Signed-off-by: Kumar Gala <galak@kernel.org>
2022-08-08 06:58:18 +01:00

55 lines
1.1 KiB
Plaintext

# AMG88XX infrared thermopile sensor configuration options
# Copyright (c) 2017 Phytec Messtechnik GmbH
# SPDX-License-Identifier: Apache-2.0
menuconfig AMG88XX
bool "AMG88XX Infrared Thermopile Sensor"
default y
depends on DT_HAS_PANASONIC_AMG88XX_ENABLED
select I2C
help
Enable driver for AMG88XX infrared thermopile sensor.
if AMG88XX
choice
prompt "Trigger mode"
default AMG88XX_TRIGGER_NONE
help
Specify the type of triggering used by the driver.
config AMG88XX_TRIGGER_NONE
bool "No trigger"
config AMG88XX_TRIGGER_GLOBAL_THREAD
bool "Use global thread"
depends on GPIO
select AMG88XX_TRIGGER
config AMG88XX_TRIGGER_OWN_THREAD
bool "Use own thread"
depends on GPIO
select AMG88XX_TRIGGER
endchoice
config AMG88XX_TRIGGER
bool
config AMG88XX_THREAD_PRIORITY
int "Thread priority"
depends on AMG88XX_TRIGGER_OWN_THREAD
default 10
help
Priority of thread used by the driver to handle interrupts.
config AMG88XX_THREAD_STACK_SIZE
int "Thread stack size"
depends on AMG88XX_TRIGGER_OWN_THREAD
default 1024
help
Stack size of thread used by the driver to handle interrupts.
endif # AMG88XX