linux-pm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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
> > 
> 
> 

      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).