On Monday 23 October 2017 04:54 PM, Tom Rini wrote: > On Mon, Oct 23, 2017 at 10:50:15AM +0530, Faiz Abbas wrote: >> Hi Tom, >> >> On Monday 23 October 2017 08:30 AM, Tom Rini wrote: >>> On Mon, Oct 23, 2017 at 08:07:09AM +0530, Lokesh Vutla wrote: >>>> 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_ab...@ti.com> >>>>> Reviewed-by: Simon Glass <s...@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? >>> >>> How important is it that this be enabled? Is this 100% within the SoC? >>> Thanks! >>> >> >> There is an upcoming errata which requires temperature support to >> workaround. This errata will apply to the HS boards as well. Will send a >> new patch with imply added to TARGET_DRA7XX_EVM for these configs. > > Well, can you _not_ have this support enabled, on the dra7xx family? If > it must be always there, please select it. Otherwise imply is OK. > Thanks! >
Its not enabled in the latest v4. Have added imply in TARGET_DRA7XX_EVM and TARGET_AM57XX_EVM instead. Thanks, Faiz _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
