From: Enric Balletbo i Serra <enric.balletbo@collabora.com>
To: lee.jones@linaro.org, bleung@chromium.org
Cc: groeck@chromium.org, andy.shevchenko@gmail.com,
kernel@collabora.com, gwendal@chromium.org,
linux-kernel@vger.kernel.org, Olof Johansson <olof@lixom.net>
Subject: [PATCH v4 0/6] platform/chrome: cros_ec debugfs and sysfs updates
Date: Fri, 23 Mar 2018 18:42:41 +0100 [thread overview]
Message-ID: <20180323174247.6680-1-enric.balletbo@collabora.com> (raw)
Hi,
This is a four patchset that introduces some cros_ec debugfs and sysfs
updates. This version is practically the same as version 3, it only
fixes the issue pointed by Gwendal to only allocate one extra sensor
intead of two.
First patch was already sent [1] but did not receive feedback so I am
resending it. Needs to go through platform/chrome tree and can be
picked without dependencies.
Patches 2 to 4 are cleanups of current code, basically is the
implementation of the changes suggested by Andy Shevchenko on the first
series. Again these must go through platform/chrome tree.
In Patch 5 I added a missing define, so now this patch can be picked
without any dependency, although now needs the ack from mfd and
platform/chrome maintainer as touches both sub-systems.
Finally, patch 6 was already sent [2] but now needed to be reworked due
the split of the cros_ec_devs in two parts [3]. This patch touches
platform/chrome and mfd so we will probably need an immutable branch.
This version also include a fix found by Gwendal.
Best regards,
Enric
Changes in v4:
- [6/6] Allocate only one extra sensor (Gwendal)
Changes in v3:
- [2/6] Add Reviewed-by Andy Shevchenko
- [3/6] Add Reviewed-by Andy Shevchenko
- [4/6] Add Reviewed-by Andy Shevchenko
- [5/6] Add EC_USB_PD_MAX_PORTS define to avoid interdependencies.
- [5/6] Add Reviewed-by Andy Shevchenko
- [6/6] Add Reviewed-by Andy Shevchenko
- [6/6] Fix the code that has_kb_wake_angle in cros_ec_sensors_register().
Changes in v2:
- [2/6] Add a new define to get the cros_ec_dev.
- [3/6] Use DEVICE_ATTR variants.
- [4/6] Use octal permissions.
- [5/6] Drop unrelated changes.
- [6/6] Use DEVICE_ATTR_RW variant.
- [6/6] Use one line when fits in 80 characters.
- [6/6] Use the previous defined to_cros_ec_dev
Enric Balletbo i Serra (3):
platform/chrome: cros_ec_sysfs: introduce to_cros_ec_dev define.
platform/chrome: cros_ec_sysfs: use permission-specific DEVICE_ATTR
variants
platform/chrome: cros_ec_debugfs: Use octal permissions '0444'
Gwendal Grignou (2):
platform/chrome: cros_ec_sysfs: Modify error handling
platform/chrome: mfd/cros_ec_dev: Add sysfs entry to set keyboard wake
lid angle
Shawn Nematbakhsh (1):
platform/chrome: cros_ec_debugfs: Add PD port info to debugfs
drivers/mfd/cros_ec_dev.c | 31 +++----
drivers/platform/chrome/cros_ec_debugfs.c | 76 +++++++++++++++-
drivers/platform/chrome/cros_ec_sysfs.c | 141 +++++++++++++++++++++---------
include/linux/mfd/cros_ec.h | 2 +
include/linux/mfd/cros_ec_commands.h | 3 +
5 files changed, 194 insertions(+), 59 deletions(-)
--
2.16.2
next reply other threads:[~2018-03-23 17:42 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-23 17:42 Enric Balletbo i Serra [this message]
2018-03-23 17:42 ` [PATCH v4 1/6] platform/chrome: cros_ec_sysfs: Modify error handling Enric Balletbo i Serra
2018-03-23 17:42 ` [PATCH v4 2/6] platform/chrome: cros_ec_sysfs: introduce to_cros_ec_dev define Enric Balletbo i Serra
2018-03-23 17:42 ` [PATCH v4 3/6] platform/chrome: cros_ec_sysfs: use permission-specific DEVICE_ATTR variants Enric Balletbo i Serra
2018-03-23 17:42 ` [PATCH v4 4/6] platform/chrome: cros_ec_debugfs: Use octal permissions '0444' Enric Balletbo i Serra
2018-03-23 17:42 ` [PATCH v4 5/6] platform/chrome: cros_ec_debugfs: Add PD port info to debugfs Enric Balletbo i Serra
2018-03-28 11:08 ` Lee Jones
2018-03-23 17:42 ` [PATCH v4 6/6] platform/chrome: mfd/cros_ec_dev: Add sysfs entry to set keyboard wake lid angle Enric Balletbo i Serra
2018-03-28 11:10 ` Lee Jones
2018-03-26 20:26 ` [PATCH v4 0/6] platform/chrome: cros_ec debugfs and sysfs updates Gwendal Grignou
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=20180323174247.6680-1-enric.balletbo@collabora.com \
--to=enric.balletbo@collabora.com \
--cc=andy.shevchenko@gmail.com \
--cc=bleung@chromium.org \
--cc=groeck@chromium.org \
--cc=gwendal@chromium.org \
--cc=kernel@collabora.com \
--cc=lee.jones@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=olof@lixom.net \
/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
all inboxes | Powered by JetHome®