Re: [U-Boot] [PATCH v3 1/2] thermal: ti-bandgap: Add support for temperature sensor

On Wednesday 18 October 2017 05:11 PM, Faiz Abbas wrote:

The dra7xx series of SOCs contain a temperature sensor and an associated analog-to-digital converter (ADC) which produces an output which is proportional to the SOC temperature. Add support for this temperature sensor.

Signed-off-by: Faiz Abbas faiz_abbas@ti.com Reviewed-by: Simon Glass sjg@chromium.org

---

v3: Added Reviewed-by comment

v2:

1. Removed extra elements from ti_bandgap
2. Changed base address to ulong
3. Changed to udev to dev to maintain consistency
4. minor format changes

common/spl/Kconfig | 8 + configs/dra7xx_evm_defconfig | 3 + .../thermal/ti_soc_thermal.txt | 35 ++++ drivers/Makefile | 2 +- drivers/thermal/Kconfig | 6 + drivers/thermal/Makefile | 1 + drivers/thermal/ti-bandgap.c | 197 +++++++++++++++++++++ 7 files changed, 251 insertions(+), 1 deletion(-) create mode 100644 doc/device-tree-bindings/thermal/ti_soc_thermal.txt create mode 100644 drivers/thermal/ti-bandgap.c

diff --git a/common/spl/Kconfig b/common/spl/Kconfig index 0bd8370..0bf3ee3 100644 --- a/common/spl/Kconfig +++ b/common/spl/Kconfig @@ -604,6 +604,14 @@ config SPL_SPI_SUPPORT enable SPI drivers that are needed for other purposes also, such as a SPI PMIC.

+config SPL_THERMAL

• bool "Driver support for thermal devices"
• help

•  Enable support for temperature-sensing devices. Some SoCs have on-chip
  

•  temperature sensors to permit warnings, speed throttling or even
  

•  automatic power-off when the temperature gets too high or low. Other
  

•  devices may be discrete but connected on a suitable bus.
  

config SPL_USB_HOST_SUPPORT bool "Support USB host drivers" help diff --git a/configs/dra7xx_evm_defconfig b/configs/dra7xx_evm_defconfig index 468c288..447fb89 100644 --- a/configs/dra7xx_evm_defconfig +++ b/configs/dra7xx_evm_defconfig @@ -90,3 +90,6 @@ CONFIG_USB_GADGET=y CONFIG_USB_GADGET_MANUFACTURER="Texas Instruments" CONFIG_USB_GADGET_VENDOR_NUM=0x0451 CONFIG_USB_GADGET_PRODUCT_NUM=0xd022 +CONFIG_SPL_THERMAL=y +CONFIG_DM_THERMAL=y +CONFIG_TI_DRA7_THERMAL=y

Tom, Do you want these options to be implied so that it will take effect for HS board as well. Or enable it in defconfig as is?

Thanks and regards, Lokesh