- 14 5月, 2019 40 次提交
-
-
由 Talel Shenhar 提交于
The devres.o gets linked if HAS_IOMEM is present so on ARCH=um allyesconfig (COMPILE_TEST) failed on many files with: drivers/thermal/thermal_mmio.o: In function 'thermal_mmio_probe':thermal_mmio.c:(.text+0xe1): undefined reference to `devm_ioremap_resource' The users of devm_ioremap_resource() which are compile-testable should depend on HAS_IOMEM. Reported-by: Nkbuild test robot <lkp@intel.com> Signed-off-by: NTalel Shenhar <talel@amazon.com> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Daniel Lezcano 提交于
The structure cpufreq_cooling_device provides a backpointer to the thermal device but this one is used for a trace and to unregister. For the trace, we don't really need this field and the unregister function as the same pointer passed as parameter. Remove it. Acked-by: NViresh Kumar <viresh.kumar@linaro.org> Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Daniel Lezcano 提交于
For license auditing purpose, let's add the SPDX tag. Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org> Acked-by: NViresh Kumar <viresh.kumar@linaro.org> Acked-by: NPhilippe Ombredanne <pombredanne@nexb.com> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Daniel Lezcano 提交于
The copyright format does not conform to the format requested by Linaro: https://wiki.linaro.org/Copyright Fix it. Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org> Acked-by: NViresh Kumar <viresh.kumar@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Daniel Lezcano 提交于
When the static power computation was removed, the test with the power being negative was not removed. However, the substraction which was responsible of the negative value was removed and the variable is now an u32. A double reason to remove the test which does not make sense. Fixes: 84fe2cab ("cpu_cooling: Drop static-power related stuff") Cc: Viresh Kumar <viresh.kumar@linaro.org> Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org> Acked-by: NViresh Kumar <viresh.kumar@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Jiada Wang 提交于
Currently IRQ remains enabled after .remove, later if device is probed, IRQ is requested before .thermal_init, this may cause IRQ function be called before device is initialized. this patch disables interrupt in .remove, to ensure irq function only be called after device is fully initialized. Signed-off-by: NJiada Wang <jiada_wang@mentor.com> Reviewed-by: NSimon Horman <horms+renesas@verge.net.au> Reviewed-by: NDaniel Lezcano <daniel.lezcano@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Jiada Wang 提交于
Currently IRQF_SHARED type interrupt line is allocated, but it is not appropriate, as the interrupt line isn't shared between different devices, instead IRQF_ONESHOT is the proper type. By changing interrupt type to IRQF_ONESHOT, now irq handler is no longer needed, as clear of interrupt status can be done in threaded interrupt context. Because IRQF_ONESHOT type interrupt line is kept disabled until the threaded handler has been run, so there is no need to protect read/write of REG_GEN3_IRQSTR with lock. Fixes: 7d4b2697 ("enable hardware interrupts for trip points") Signed-off-by: NJiada Wang <jiada_wang@mentor.com> Reviewed-by: NSimon Horman <horms+renesas@verge.net.au> Tested-by: NSimon Horman <horms+renesas@verge.net.au> Reviewed-by: NDaniel Lezcano <daniel.lezcano@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Guenter Roeck 提交于
thermal_of_cooling_device_register() and thermal_cooling_device_register() are typically called from driver probe functions, and thermal_cooling_device_unregister() is called from remove functions. This makes both a perfect candidate for device managed functions. Introduce devm_thermal_of_cooling_device_register(). This function can also be used to replace thermal_cooling_device_register() by passing a NULL pointer as device node. The new function requires both struct device * and struct device_node * as parameters since the struct device_node * parameter is not always identical to dev->of_node. Don't introduce a device managed remove function since it is not needed at this point. Signed-off-by: NGuenter Roeck <linux@roeck-us.net> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Jean-Francois Dagenais 提交于
Update binding description making lookup-table optional. Signed-off-by: NJean-Francois Dagenais <jeff.dagenais@gmail.com> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Jean-Francois Dagenais 提交于
Certain ADC channels, such as the xilinx-ams temperature channels, give milliCelcius already when read with iio_read_channel_processed. Rather than having to provide a 1:1 dummy lookup table, simply allow to bypass the mechanism. Signed-off-by: NJean-Francois Dagenais <jeff.dagenais@gmail.com> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Andrey Smirnov 提交于
It's impossible to use this driver outside of Device Tree, so if the probe function is called, the dev.of_node is guaranteed to not be NULL and guarding against that is pointless. Drop it. Signed-off-by: NAndrey Smirnov <andrew.smirnov@gmail.com> Acked-by: NDaniel Lezcano <daniel.lezcano@linaro.org> Cc: Chris Healy <cphealy@gmail.com> Cc: Lucas Stach <l.stach@pengutronix.de> Cc: Eduardo Valentin <edubezval@gmail.com> Cc: Daniel Lezcano <daniel.lezcano@linaro.org> Cc: Angus Ainslie (Purism) <angus@akkea.ca> Cc: linux-imx@nxp.com Cc: linux-pm@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Talel Shenhar 提交于
This is a generic thermal driver for simple MMIO sensors, of which amazon,al-thermal is one. This device uses a single MMIO transaction to read the temperature and report it to the thermal subsystem. Signed-off-by: NTalel Shenhar <talel@amazon.com> Reviewed-by: NDavid Woodhouse <dwmw@amazon.co.uk> Reviewed-by: NDaniel Lezcano <daniel.lezcano@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Talel Shenhar 提交于
Add thermal binding documentation for Amazon's Annapurna Labs Thermal Sensor. Signed-off-by: NTalel Shenhar <talel@amazon.com> Reviewed-by: NDavid Woodhouse <dwmw@amazon.co.uk> Reviewed-by: NRob Herring <robh@kernel.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Hoan Nguyen An 提交于
Fix setting value for IRQCTL register. We are setting the last 6 bits of (IRQCTL) to be 1 (0x3f), this is only suitable for H3ES1.*, according to Hardware manual values 1 are "setting prohibited" for Gen3. Signed-off-by: NHoan Nguyen An <na-hoan@jinso.co.jp> Acked-by: NWolfram Sang <wsa+renesas@sang-engineering.com> Reviewed-by: NGeert Uytterhoeven <geert+renesas@glider.be> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
This will allow calibration routines to correctly include the constants from anywhere and allow more code sharing. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
qcs404 has a single TSENS IP block with 10 sensors. It uses version 1.4 of the TSENS IP, functionality for which is encapsulated inside the qcom,tsens-v1 compatible. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
qcs404 uses v1 of the TSENS IP block. Create a fallback DT property "qcom,tsens-v1" to gather common code Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Reviewed-by: NRob Herring <robh@kernel.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
get_temp() learns to return temperature regardless of whether it is returned as ADC code or direct temperature. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
Just rename the function and move it to allow code sharing with future versions of TSENS IP Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
The current implementation is based on an algorithm published in the docs. Instead of reading the temperature thrice w/o any explanation, improve the algorithm. This will become the basis for a common get_temp routine in the future. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
The IP can support 'm' sensors while the platform can enable 'n' sensors of the 'm' where n <= m. Track maximum sensors supported by the IP so that we can correctly track what subset of the sensors are supported on the platform. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
The IDs cannot be negative, fix the data type. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
is_sensor_enabled() checks if the sensors are enabled on this platform. It is possible that the SoC might choose not to enable all the sensors that the IP block is capable of supporting. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
We print a calibration failure message on -EPROBE_DEFER from nvmem/qfprom as follows: [ 3.003090] qcom-tsens 4a9000.thermal-sensor: version: 1.4 [ 3.005376] qcom-tsens 4a9000.thermal-sensor: tsens calibration failed [ 3.113248] qcom-tsens 4a9000.thermal-sensor: version: 1.4 This confuses people when, in fact, calibration succeeds later when nvmem/qfprom device is available. Don't print this message on a -EPROBE_DEFER. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
Code cleanup making it easier to read Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
As we add support for newer versions of the TSENS IP, the current approach isn't scaling because registers and bitfields get moved around, requiring platform-specific hacks in the code. By moving to regmap, we can hide the register level differences away from the code. Define a common set of registers and bit-fields that we care about across the various tsens IP versions. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
8974 and 8916 have the same version of the TSENS IP. Merge the files to allow for better code reuse. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
Some #defines in tsens-v_0_1.c clash with those in tsens-8974.c. Prefix them with 8916 to avoid the clash so we can merge the two files. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
8916 and 8974 use v0.1.0 of the TSENS IP. Rename tsens-8916 to prepare it for merging with tsens-8974 in a later commit. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
check_patch complains a lot as follows: WARNING: function definition argument 'struct tsens_priv *' should also have an identifier name + int (*init)(struct tsens_priv *); Fix it. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
tsens_get_temp() uses the name 'data' for the void pointer, use the same in tsens_get_trend() for consistency. Remove a stray space while we're at it. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
tmdev seems to imply that this is a device pointer when in fact it is just private platform data for each tsens device. Rename it to priv improve code readability. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
Rename to tsens_priv to denote that it is private data for each tsens instance. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
Rename to tsens_plat_data to denote that it is platform-data passed in at compile-time. Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Amit Kucheria 提交于
Describe how the TSENS device and the various sensors connected to it are described in the driver Signed-off-by: NAmit Kucheria <amit.kucheria@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Wolfram Sang 提交于
We should get 'driver_data' from 'struct device' directly. Going via platform_device is an unneeded step back and forth. Signed-off-by: NWolfram Sang <wsa+renesas@sang-engineering.com> Reviewed-by: NSimon Horman <horms+renesas@verge.net.au> Reviewed-by: NDaniel Lezcano <daniel.lezcano@linaro.org> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
Formatting of Kconfig files doesn't look so pretty, so just take damp cloth and clean it up. Signed-off-by: NEnrico Weigelt, metux IT consult <info@metux.net> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Yangtao Li 提交于
Define pr_fmt macro to add a prefix to the message, this can make the thermal log better recognized. Before: [ 0.602672] nfc: nfc_init: NFC Core ver 0.1 [ 0.602828] NET: Registered protocol family 39 [ 0.603435] clocksource: Switched to clocksource mct-frc [ 0.746216] failed to build thermal zone cpu-thermal: -22 [ 0.746451] NET: Registered protocol family 2 After: [ 0.602804] NET: Registered protocol family 39 [ 0.603463] clocksource: Switched to clocksource mct-frc [ 0.746309] thermal_sys: failed to build thermal zone cpu-thermal: -22 [ 0.746545] NET: Registered protocol family 2 Signed-off-by: NYangtao Li <tiny.windzz@gmail.com> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Wei Ni 提交于
Parse Over Current settings from DT and program them to generate interrupts. Also enable hw throttling whenever there are OC events. Log the OC events as debug messages. Signed-off-by: NWei Ni <wni@nvidia.com> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-
由 Wei Ni 提交于
Add OC HW throttle configuration for soctherm in DT. It is used to describe the OCx throttle events. Signed-off-by: NWei Ni <wni@nvidia.com> Signed-off-by: NEduardo Valentin <edubezval@gmail.com>
-