From mboxrd@z Thu Jan 1 00:00:00 1970 From: Srinivas Pandruvada Subject: Re: [PATCH 0/6] thermal: iio bindings Date: Sun, 27 Sep 2015 07:29:16 -0700 Message-ID: <1443364156.2907.2.camel@linux.intel.com> References: <1443305111-28272-1-git-send-email-srinivas.pandruvada@linux.intel.com> <5607D1F8.5060807@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from mga11.intel.com ([192.55.52.93]:38377 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755900AbbI0O3h (ORCPT ); Sun, 27 Sep 2015 10:29:37 -0400 In-Reply-To: <5607D1F8.5060807@linaro.org> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Daniel Lezcano , jic23@kernel.org, rui.zhang@intel.com, edubezval@gmail.com, linux-pm@vger.kernel.org, linux-iio@vger.kernel.org On Sun, 2015-09-27 at 04:24 -0700, Daniel Lezcano wrote: > On 09/26/2015 03:05 PM, Srinivas Pandruvada wrote: > > v0 > > Since this was discussed during LPC in Seattle and three versions > > of RFC are > > published and reviwed, this is posted as non RFC version. This has > > changes in iio interface file as suggested by Jonathan. We may add > > threshold > > event direction flag later if required, without much impact by > > adding new > > event type. > > Not everyone was at LPC. > > You should fix the changelog(s) and explain what you are trying to > solve > and why it is important. Sure. The very first RFC had this. I hope you did get chance to look at this. > > > > RFC v2 > > - Rebase to 4.3.rc1 > > - Additional parameter to thermal_zone_device_update for event type > > - Moved the dummy IIO callbacks to local include file, in this > > way "not used" warnings can be avoided > > - Remove changes from user space governor, rather notify from core > > - Validate the trigger, so that when thermal device supports async > > notifications, this will be exclusive trigger > > - Use new thermal core event type to differentiate events for iio > > threshold notification > > > > RFC v1 > > - Change commit message to be more clear > > - Split in three patches (First for thermal_iio.c, second to > > integrate > > to thermal_core and third for user space governor). > > - Removed IIO defines in thermal.h > > - Removed direct push to buffers > > - Other comments from Jonathan except introduce validate_trigger > > > > RFC v0 > > Base version for first review > > > > Srinivas Pandruvada (6): > > thermal: core: enhance thermal_zone_device_update > > thermal: documentation update > > thermal: iio device for thermal sensor > > thermal: use iio binding calls > > thermal: iio Documentation > > thermal: x86_pkg_temp: Register threshold callbacks > > > > Documentation/thermal/sysfs-api.txt | 10 + > > Documentation/thermal/thermal_iio_binding | 88 +++++ > > drivers/acpi/thermal.c | 2 +- > > drivers/platform/x86/acerhdf.c | 2 +- > > drivers/thermal/Kconfig | 12 + > > drivers/thermal/Makefile | 1 + > > drivers/thermal/db8500_thermal.c | 2 +- > > drivers/thermal/hisi_thermal.c | 3 +- > > drivers/thermal/imx_thermal.c | 4 +- > > .../thermal/int340x_thermal/int340x_thermal_zone.h | 2 +- > > drivers/thermal/intel_soc_dts_iosf.c | 3 +- > > drivers/thermal/of-thermal.c | 2 +- > > drivers/thermal/qcom-spmi-temp-alarm.c | 2 +- > > drivers/thermal/rcar_thermal.c | 3 +- > > drivers/thermal/rockchip_thermal.c | 3 +- > > drivers/thermal/samsung/exynos_tmu.c | 2 +- > > drivers/thermal/st/st_thermal_memmap.c | 3 +- > > drivers/thermal/thermal_core.c | 21 +- > > drivers/thermal/thermal_iio.c | 383 > > +++++++++++++++++++++ > > drivers/thermal/thermal_iio.h | 45 +++ > > drivers/thermal/ti-soc-thermal/ti-thermal-common.c | 4 +- > > drivers/thermal/x86_pkg_temp_thermal.c | 11 +- > > include/linux/thermal.h | 26 +- > > 23 files changed, 609 insertions(+), 25 deletions(-) > > create mode 100644 Documentation/thermal/thermal_iio_binding > > create mode 100644 drivers/thermal/thermal_iio.c > > create mode 100644 drivers/thermal/thermal_iio.h > > > >