From: Prashant Malani <pmalani@chromium.org> To: linux-kernel@vger.kernel.org Cc: "Prashant Malani" <pmalani@chromium.org>, "Akshu Agrawal" <akshu.agrawal@amd.com>, "Alexandre Belloni" <alexandre.belloni@bootlin.com>, alsa-devel@alsa-project.org (moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM...), "Benjamin Tissoires" <benjamin.tissoires@redhat.com>, "Dmitry Torokhov" <dmitry.torokhov@gmail.com>, "Enric Balletbo i Serra" <enric.balletbo@collabora.com>, "Evan Green" <evgreen@chromium.org>, "Fabien Lahoudere" <fabien.lahoudere@collabora.com>, "Guenter Roeck" <groeck@chromium.org>, "Gwendal Grignou" <gwendal@chromium.org>, "Hans Verkuil" <hverkuil-cisco@xs4all.nl>, "Hartmut Knaack" <knaack.h@gmx.de>, "Jonathan Cameron" <Jonathan.Cameron@huawei.com>, "Lars-Peter Clausen" <lars@metafoo.de>, "Lee Jones" <lee.jones@linaro.org>, linux-i2c@vger.kernel.org (open list:I2C SUBSYSTEM HOST DRIVERS), linux-iio@vger.kernel.org (open list:IIO SUBSYSTEM AND DRIVERS), linux-input@vger.kernel.org (open list:HID CORE LAYER), linux-media@vger.kernel.org (open list:MEDIA INPUT INFRASTRUCTURE (V4L/DVB)), linux-pm@vger.kernel.org (open list:POWER SUPPLY CLASS/SUBSYSTEM and DRIVERS), linux-pwm@vger.kernel.org (open list:PWM SUBSYSTEM), linux-rtc@vger.kernel.org (open list:REAL TIME CLOCK (RTC) SUBSYSTEM), "Mauro Carvalho Chehab" <mchehab+samsung@kernel.org>, "Neil Armstrong" <narmstrong@baylibre.com>, "Nick Vaccaro" <nvaccaro@chromium.org>, "Peter Meerwald-Stadler" <pmeerw@pmeerw.net>, "Raul E Rangel" <rrangel@chromium.org>, "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>, "Wolfram Sang" <wsa@the-dreams.de> Subject: [PATCH 00/17] platform/chrome: Replace cros_ec_cmd_xfer_status Date: Thu, 30 Jan 2020 12:30:31 -0800 [thread overview] Message-ID: <20200130203106.201894-1-pmalani@chromium.org> (raw) Many callers of cros_ec_cmd_xfer_status() use similar setup and cleanup code, including setting up the cros_ec_command message struct and copying the received buffer. This series introduces a replacement function cros_ec_send_cmd_msg() that performs this setup and teardown, and then updates all call sites that used xfer_status() to use the new function instead. The final patch in the series drops cros_ec_cmd_xfer_status() altogether. Prashant Malani (17): platform/chrome: Add EC command msg wrapper platform/chrome: chardev: Use send_cmd_msg() platform/chrome: proto: Use send_cmd_msg platform/chrome: usbpd_logger: Use cmd_send_msg() platform/chrome: sensorhub: Use send_cmd_msg() platform/chrome: debugfs: Use send_cmd_msg() platform/chrome: sysfs: Use send_cmd_msg() extcon: cros_ec: Use cros_ec_send_cmd_msg() hid: google-hammer: Use cros_ec_send_cmd_msg() iio: cros_ec: Use cros_ec_send_cmd_msg() ASoC: cros_ec_codec: Use cros_ec_send_cmd_msg() power: supply: cros: Use cros_ec_send_cmd_msg() pwm: cros-ec: Remove cros_ec_cmd_xfer_status() rtc: cros-ec: Use cros_ec_send_cmd_msg() media: cros-ec-cec: Use cros_ec_send_cmd_msg() i2c: cros-ec-tunnel: Use cros_ec_send_cmd_msg() platform/chrome: Drop cros_ec_cmd_xfer_status() drivers/extcon/extcon-usbc-cros-ec.c | 62 ++------ drivers/hid/hid-google-hammer.c | 23 +-- drivers/i2c/busses/i2c-cros-ec-tunnel.c | 23 ++- .../cros_ec_sensors/cros_ec_sensors_core.c | 43 +++--- .../media/platform/cros-ec-cec/cros-ec-cec.c | 39 ++--- drivers/platform/chrome/cros_ec_chardev.c | 18 +-- drivers/platform/chrome/cros_ec_debugfs.c | 135 ++++++------------ drivers/platform/chrome/cros_ec_proto.c | 75 ++++++---- drivers/platform/chrome/cros_ec_sensorhub.c | 29 ++-- drivers/platform/chrome/cros_ec_sysfs.c | 106 ++++++-------- drivers/platform/chrome/cros_usbpd_logger.c | 13 +- drivers/power/supply/cros_usbpd-charger.c | 63 ++------ drivers/pwm/pwm-cros-ec.c | 27 ++-- drivers/rtc/rtc-cros-ec.c | 27 ++-- include/linux/platform_data/cros_ec_proto.h | 6 +- sound/soc/codecs/cros_ec_codec.c | 71 +++------ 16 files changed, 276 insertions(+), 484 deletions(-) -- 2.25.0.341.g760bfbb309-goog
WARNING: multiple messages have this Message-ID (diff)
From: Prashant Malani <pmalani@chromium.org> To: linux-kernel@vger.kernel.org Cc: Prashant Malani <pmalani@chromium.org>, Akshu Agrawal <akshu.agrawal@amd.com>, Alexandre Belloni <alexandre.belloni@bootlin.com>, "moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM..." <alsa-devel@alsa-project.org>, Benjamin Tissoires <benjamin.tissoires@redhat.com>, Dmitry Torokhov <dmitry.torokhov@gmail.com>, Enric Balletbo i Serra <enric.balletbo@collabora.com>, Evan Green <evgreen@chromium.org>, Fabien Lahoudere <fabien.lahoudere@collabora.com>, Guenter Roeck <groeck@chromium.org>, Gwendal Grignou <gwendal@chromium.org>, Hans Verkuil <hverkuil-cisco@xs4all.nl>, Hartmut Knaack <knaack.h@gmx.de>, Jonathan Cameron <Jonathan.Cameron@huawei.com>, Lars-Peter Clausen <lars@metafoo.de>, Lee Jones <lee.jones@linaro.org>, linux-i2c@vger.kernel.o Subject: [PATCH 00/17] platform/chrome: Replace cros_ec_cmd_xfer_status Date: Thu, 30 Jan 2020 12:30:31 -0800 [thread overview] Message-ID: <20200130203106.201894-1-pmalani@chromium.org> (raw) Many callers of cros_ec_cmd_xfer_status() use similar setup and cleanup code, including setting up the cros_ec_command message struct and copying the received buffer. This series introduces a replacement function cros_ec_send_cmd_msg() that performs this setup and teardown, and then updates all call sites that used xfer_status() to use the new function instead. The final patch in the series drops cros_ec_cmd_xfer_status() altogether. Prashant Malani (17): platform/chrome: Add EC command msg wrapper platform/chrome: chardev: Use send_cmd_msg() platform/chrome: proto: Use send_cmd_msg platform/chrome: usbpd_logger: Use cmd_send_msg() platform/chrome: sensorhub: Use send_cmd_msg() platform/chrome: debugfs: Use send_cmd_msg() platform/chrome: sysfs: Use send_cmd_msg() extcon: cros_ec: Use cros_ec_send_cmd_msg() hid: google-hammer: Use cros_ec_send_cmd_msg() iio: cros_ec: Use cros_ec_send_cmd_msg() ASoC: cros_ec_codec: Use cros_ec_send_cmd_msg() power: supply: cros: Use cros_ec_send_cmd_msg() pwm: cros-ec: Remove cros_ec_cmd_xfer_status() rtc: cros-ec: Use cros_ec_send_cmd_msg() media: cros-ec-cec: Use cros_ec_send_cmd_msg() i2c: cros-ec-tunnel: Use cros_ec_send_cmd_msg() platform/chrome: Drop cros_ec_cmd_xfer_status() drivers/extcon/extcon-usbc-cros-ec.c | 62 ++------ drivers/hid/hid-google-hammer.c | 23 +-- drivers/i2c/busses/i2c-cros-ec-tunnel.c | 23 ++- .../cros_ec_sensors/cros_ec_sensors_core.c | 43 +++--- .../media/platform/cros-ec-cec/cros-ec-cec.c | 39 ++--- drivers/platform/chrome/cros_ec_chardev.c | 18 +-- drivers/platform/chrome/cros_ec_debugfs.c | 135 ++++++------------ drivers/platform/chrome/cros_ec_proto.c | 75 ++++++---- drivers/platform/chrome/cros_ec_sensorhub.c | 29 ++-- drivers/platform/chrome/cros_ec_sysfs.c | 106 ++++++-------- drivers/platform/chrome/cros_usbpd_logger.c | 13 +- drivers/power/supply/cros_usbpd-charger.c | 63 ++------ drivers/pwm/pwm-cros-ec.c | 27 ++-- drivers/rtc/rtc-cros-ec.c | 27 ++-- include/linux/platform_data/cros_ec_proto.h | 6 +- sound/soc/codecs/cros_ec_codec.c | 71 +++------ 16 files changed, 276 insertions(+), 484 deletions(-) -- 2.25.0.341.g760bfbb309-goog
next reply other threads:[~2020-01-30 20:31 UTC|newest] Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-01-30 20:30 Prashant Malani [this message] 2020-01-30 20:30 ` [PATCH 00/17] platform/chrome: Replace cros_ec_cmd_xfer_status Prashant Malani 2020-01-30 20:30 ` [PATCH 01/17] platform/chrome: Add EC command msg wrapper Prashant Malani 2020-02-03 15:27 ` Enric Balletbo i Serra 2020-02-03 18:24 ` Prashant Malani 2020-01-30 20:30 ` [PATCH 02/17] platform/chrome: chardev: Use send_cmd_msg() Prashant Malani 2020-01-30 20:30 ` [PATCH 03/17] platform/chrome: proto: Use send_cmd_msg Prashant Malani 2020-01-30 20:30 ` [PATCH 04/17] platform/chrome: usbpd_logger: Use cmd_send_msg() Prashant Malani 2020-01-30 20:30 ` [PATCH 05/17] platform/chrome: sensorhub: Use send_cmd_msg() Prashant Malani 2020-01-30 20:30 ` [PATCH 06/17] platform/chrome: debugfs: " Prashant Malani 2020-01-30 20:30 ` [PATCH 07/17] platform/chrome: sysfs: " Prashant Malani 2020-01-30 20:30 ` [PATCH 08/17] extcon: cros_ec: Use cros_ec_send_cmd_msg() Prashant Malani 2020-01-30 20:30 ` [PATCH 09/17] hid: google-hammer: " Prashant Malani 2020-01-30 20:30 ` [PATCH 10/17] iio: cros_ec: " Prashant Malani 2020-02-02 9:43 ` Jonathan Cameron 2020-02-03 18:31 ` Prashant Malani 2020-01-30 20:30 ` [PATCH 11/17] ASoC: cros_ec_codec: " Prashant Malani 2020-01-30 20:30 ` [alsa-devel] " Prashant Malani 2020-02-01 11:03 ` Mark Brown 2020-02-01 11:03 ` [alsa-devel] " Mark Brown 2020-02-03 18:42 ` Prashant Malani 2020-02-03 18:42 ` [alsa-devel] " Prashant Malani 2020-01-30 20:30 ` [PATCH 12/17] power: supply: cros: " Prashant Malani 2020-01-30 20:31 ` [PATCH 13/17] pwm: cros-ec: Remove cros_ec_cmd_xfer_status() Prashant Malani 2020-01-30 20:31 ` Prashant Malani 2020-02-03 15:33 ` Enric Balletbo i Serra 2020-02-03 18:26 ` Prashant Malani 2020-02-03 18:39 ` Prashant Malani 2020-01-30 20:31 ` [PATCH 14/17] rtc: cros-ec: Use cros_ec_send_cmd_msg() Prashant Malani 2020-01-30 20:31 ` [PATCH 15/17] media: cros-ec-cec: " Prashant Malani 2020-02-02 22:08 ` kbuild test robot 2020-02-02 22:08 ` kbuild test robot 2020-02-03 5:35 ` kbuild test robot 2020-02-03 5:35 ` kbuild test robot 2020-01-30 20:31 ` [PATCH 16/17] i2c: cros-ec-tunnel: " Prashant Malani 2020-01-30 20:31 ` Prashant Malani 2020-01-30 20:31 ` [PATCH 17/17] platform/chrome: Drop cros_ec_cmd_xfer_status() Prashant Malani 2020-02-03 15:35 ` Enric Balletbo i Serra
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=20200130203106.201894-1-pmalani@chromium.org \ --to=pmalani@chromium.org \ --cc=Jonathan.Cameron@huawei.com \ --cc=akshu.agrawal@amd.com \ --cc=alexandre.belloni@bootlin.com \ --cc=alsa-devel@alsa-project.org \ --cc=benjamin.tissoires@redhat.com \ --cc=dmitry.torokhov@gmail.com \ --cc=enric.balletbo@collabora.com \ --cc=evgreen@chromium.org \ --cc=fabien.lahoudere@collabora.com \ --cc=groeck@chromium.org \ --cc=gwendal@chromium.org \ --cc=hverkuil-cisco@xs4all.nl \ --cc=knaack.h@gmx.de \ --cc=lars@metafoo.de \ --cc=lee.jones@linaro.org \ --cc=linux-i2c@vger.kernel.org \ --cc=linux-iio@vger.kernel.org \ --cc=linux-input@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-media@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=linux-pwm@vger.kernel.org \ --cc=linux-rtc@vger.kernel.org \ --cc=mchehab+samsung@kernel.org \ --cc=narmstrong@baylibre.com \ --cc=nvaccaro@chromium.org \ --cc=pmeerw@pmeerw.net \ --cc=rrangel@chromium.org \ --cc=u.kleine-koenig@pengutronix.de \ --cc=wsa@the-dreams.de \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.