From: Tomeu Vizoso <tomeu.vizoso@collabora.com>
To: Brian Norris <briannorris@chromium.org>,
Lee Jones <lee.jones@linaro.org>,
Thierry Reding <thierry.reding@gmail.com>,
Olof Johansson <olof@lixom.net>
Cc: linux-kernel@vger.kernel.org,
Doug Anderson <dianders@chromium.org>,
Brian Norris <computersforpeace@gmail.com>,
linux-pwm@vger.kernel.org, devicetree@vger.kernel.org,
Boris Brezillon <boris.brezillon@free-electrons.com>,
Stephen Barber <smbarber@chromium.org>,
Sameer Nanda <snanda@chromium.org>,
Javier Martinez Canillas <javier@osg.samsung.com>,
Benson Leung <bleung@chromium.org>,
Enric Balletbo <enric.balletbo@collabora.co.uk>,
Randall Spangler <rspangler@chromium.org>,
Shawn Nematbakhsh <shawnn@chromium.org>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Todd Broch <tbroch@chromium.org>,
Gwendal Grignou <gwendal@chromium.org>
Subject: Re: [PATCH 0/4] pwm: add support for ChromeOS EC PWM
Date: Mon, 30 May 2016 08:44:47 +0200 [thread overview]
Message-ID: <8682e810-dd86-71c7-a454-35bfbab8afa8@collabora.com> (raw)
In-Reply-To: <1464399556-17222-1-git-send-email-briannorris@chromium.org>
On 05/28/2016 03:39 AM, Brian Norris wrote:
> Hi,
>
> This series adds support for the new ChromeOS EC PWM API, so we can control,
> e.g., the backlight when it's attached to the EC. It uses Boris's latest
> "atomic" hooks for the PWM API (i.e., the ->apply() callback), which were
> recently merged.
>
> It seems nice to have the cros_ec_cmd_xfer_status() helper, which we have
> locally in the ChromeOS kernel, and which has been part of another larger patch
> series:
>
> https://lkml.org/lkml/2016/4/12/342
>
> So I've picked it into this series as well. Obviously, I don't care which one
> is taken, but AFAICT, Tomeu's USB PD series isn't extremely active right now.
Hi Brian,
that's on hold during the ongoing discussion about type-c userspace API,
because right now the value of that patchset is on how userspace can
decide from what port to charge (if any), and it was badly abusing a
power supply property for that.
Regards,
Tomeu
> As this touches MFD (sort of), drivers/platform/chrome/, and drivers/pwm/, I'm
> not sure who it should all go through: Lee, Thierry, or Olof?
>
> Anyway, please review.
>
> Regards,
> Brian
>
> Brian Norris (3):
> mfd: cros_ec: add EC_PWM function definitions
> doc: dt: pwm: add binding for ChromeOS EC PWM
> pwm: add ChromeOS EC PWM driver
>
> Tomeu Vizoso (1):
> mfd: cros_ec: Add cros_ec_cmd_xfer_status helper
>
> .../devicetree/bindings/pwm/google,cros-ec-pwm.txt | 25 +++
> drivers/platform/chrome/cros_ec_proto.c | 15 ++
> drivers/pwm/Kconfig | 7 +
> drivers/pwm/Makefile | 1 +
> drivers/pwm/pwm-cros-ec.c | 230 +++++++++++++++++++++
> include/linux/mfd/cros_ec.h | 18 ++
> include/linux/mfd/cros_ec_commands.h | 31 +++
> 7 files changed, 327 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.txt
> create mode 100644 drivers/pwm/pwm-cros-ec.c
>
prev parent reply other threads:[~2016-05-30 6:44 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-28 1:39 Brian Norris
2016-05-28 1:39 ` [PATCH 1/4] mfd: cros_ec: Add cros_ec_cmd_xfer_status helper Brian Norris
2016-05-28 1:39 ` [PATCH 2/4] mfd: cros_ec: add EC_PWM function definitions Brian Norris
2016-05-28 1:39 ` [PATCH 3/4] doc: dt: pwm: add binding for ChromeOS EC PWM Brian Norris
2016-05-29 5:00 ` Gwendal Grignou
2016-06-01 1:10 ` Brian Norris
2016-06-03 1:17 ` Brian Norris
2016-05-28 1:39 ` [PATCH 4/4] pwm: add ChromeOS EC PWM driver Brian Norris
2016-05-29 5:02 ` Gwendal Grignou
2016-05-31 23:55 ` Brian Norris
2016-05-30 6:44 ` Tomeu Vizoso [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=8682e810-dd86-71c7-a454-35bfbab8afa8@collabora.com \
--to=tomeu.vizoso@collabora.com \
--cc=bleung@chromium.org \
--cc=boris.brezillon@free-electrons.com \
--cc=briannorris@chromium.org \
--cc=computersforpeace@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=dianders@chromium.org \
--cc=dmitry.torokhov@gmail.com \
--cc=enric.balletbo@collabora.co.uk \
--cc=gwendal@chromium.org \
--cc=javier@osg.samsung.com \
--cc=lee.jones@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pwm@vger.kernel.org \
--cc=olof@lixom.net \
--cc=rspangler@chromium.org \
--cc=shawnn@chromium.org \
--cc=smbarber@chromium.org \
--cc=snanda@chromium.org \
--cc=tbroch@chromium.org \
--cc=thierry.reding@gmail.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
Powered by JetHome