On Mon, Jun 17, 2019 at 9:15 AM Andrey Smirnov <[email protected]> wrote: > > Everyone: > > This series contains patches adding support for HWMON integration, bug > fixes and general improvements (hopefully) for TMU driver I made while > working on it on i.MX8MQ.n > > Feedback is welcome! > > Thanks, > Andrey Smirnov >
Eduardo: Do you have any comment on this series? Thanks, Andrey Smirnov > Changes since [v5] > > - Rebased on recent linux-next, dropped "thermal: qoriq: Remove > unnecessary DT node is NULL check" since it is already in the > tree > > - Dropped dependency on [rfc] > > Changes since [v4] > > - Collected Tested-by from Lucas > > - Collected Reviewed-by from Daniel > > - Converted "thermal: qoriq: Enable all sensors before registering > them" to use if instead of switch statement for error checking > > Changes since [v3] > > - Series reabse on top of [rfc] > > - Fixed incorrect goto label in "thermal: qoriq: Pass data to > qoriq_tmu_calibration()" > > - Added REGS_TRITSR() register description to "thermal: qoriq: Do > not report invalid temperature reading" > > - Reworded commit message of "thermal: qoriq: Remove unnecessary > DT node is NULL check" > > Changes since [v2] > > - Patches rebased on v5.1-rc1 > > Changes since [v1] > > - Rebased on "linus" branch of > git.kernel.org/pub/scm/linux/kernel/git/evalenti/linux-soc-thermal.git > that included latest chagnes adding multi-sensors support > > - Dropped > > thermal: qoriq: Add support for multiple thremal sites > thermal: qoriq: Be more strict when parsing > thermal: qoriq: Simplify error handling in qoriq_tmu_get_sensor_id() > > since they are no longer relevant > > - Added > > thermal: qoriq: Don't store struct thermal_zone_device reference > thermal: qoriq: Add local struct qoriq_sensor pointer > thermal: qoriq: Embed per-sensor data into struct qoriq_tmu_data > thermal: qoriq: Pass data to qoriq_tmu_register_tmu_zone() directly > > to simplify latest codebase > > - Changed "thermal: qoriq: Do not report invalid temperature > reading" to use regmap_read_poll_timeout() to make sure that > tmu_get_temp() waits for fist sample to be ready before > reporting it. This case is triggered on my setup if > qoriq_thermal is compiled as a module > > [v1] lore.kernel.org/lkml/[email protected] > [v2] lore.kernel.org/lkml/[email protected] > [v3] lore.kernel.org/lkml/[email protected] > [v4] lore.kernel.org/lkml/[email protected] > [v5] lore.kernel.org/lkml/[email protected] > [rfc] lore.kernel.org/lkml/[email protected] > > > Andrey Smirnov (12): > thermal: qoriq: Add local struct device pointer > thermal: qoriq: Don't store struct thermal_zone_device reference > thermal: qoriq: Add local struct qoriq_sensor pointer > thermal: qoriq: Embed per-sensor data into struct qoriq_tmu_data > thermal: qoriq: Pass data to qoriq_tmu_register_tmu_zone() directly > thermal: qoriq: Pass data to qoriq_tmu_calibration() directly > thermal: qoriq: Convert driver to use devm_ioremap() > thermal: qoriq: Convert driver to use regmap API > thermal: qoriq: Enable all sensors before registering them > thermal: qoriq: Do not report invalid temperature reading > thermal_hwmon: Add devres wrapper for thermal_add_hwmon_sysfs() > thermal: qoriq: Add hwmon support > > drivers/thermal/qoriq_thermal.c | 272 ++++++++++++++++---------------- > drivers/thermal/thermal_hwmon.c | 28 ++++ > drivers/thermal/thermal_hwmon.h | 7 + > 3 files changed, 175 insertions(+), 132 deletions(-) > > -- > 2.21.0 >
