From: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
To: Daniel Lezcano <daniel.lezcano@linaro.org>,
jic23@kernel.org, rui.zhang@intel.com, edubezval@gmail.com,
linux-pm@vger.kernel.org, linux-iio@vger.kernel.org
Subject: Re: [PATCH 0/6] thermal: iio bindings
Date: Sun, 27 Sep 2015 07:29:16 -0700 [thread overview]
Message-ID: <1443364156.2907.2.camel@linux.intel.com> (raw)
In-Reply-To: <5607D1F8.5060807@linaro.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
> >
>
>
prev parent reply other threads:[~2015-09-27 14:29 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-26 22:05 [PATCH 0/6] thermal: iio bindings Srinivas Pandruvada
2015-09-26 22:05 ` [PATCH 1/6] thermal: core: enhance thermal_zone_device_update Srinivas Pandruvada
2015-09-26 22:05 ` [PATCH 2/6] thermal: documentation update Srinivas Pandruvada
2015-09-26 22:05 ` [PATCH 3/6] thermal: iio device for thermal sensor Srinivas Pandruvada
2015-09-27 18:18 ` Jonathan Cameron
[not found] ` <1443305111-28272-4-git-send-email-srinivas.pandruvada-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2015-12-31 19:18 ` Eduardo Valentin
[not found] ` <20151231191847.GA14793-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2015-12-31 23:18 ` Srinivas Pandruvada
[not found] ` <1451603888.4047.34.camel-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2015-12-31 23:31 ` Pandruvada, Srinivas
2015-09-26 22:05 ` [PATCH 4/6] thermal: use iio binding calls Srinivas Pandruvada
2015-09-26 22:05 ` [PATCH 5/6] thermal: iio Documentation Srinivas Pandruvada
2015-09-27 18:23 ` Jonathan Cameron
2015-10-02 17:01 ` Srinivas Pandruvada
2015-09-26 22:05 ` [PATCH 6/6] thermal: x86_pkg_temp: Register threshold callbacks Srinivas Pandruvada
2015-09-27 11:24 ` [PATCH 0/6] thermal: iio bindings Daniel Lezcano
2015-09-27 14:29 ` Srinivas Pandruvada [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1443364156.2907.2.camel@linux.intel.com \
--to=srinivas.pandruvada@linux.intel.com \
--cc=daniel.lezcano@linaro.org \
--cc=edubezval@gmail.com \
--cc=jic23@kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=rui.zhang@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).