Re: [PATCH v6] mfd: cros_ec_dev: Register cros_ec_accel_legacy driver as a subdevice

From: Lee Jones
Date: Mon Jun 24 2019 - 09:58:28 EST


On Tue, 11 Jun 2019, Gwendal Grignou wrote:

> With this patch, the cros_ec_ctl driver will register the legacy
> accelerometer driver (named cros_ec_accel_legacy) if it fails to
> register sensors through the usual path cros_ec_sensors_register().
> This legacy device is present on Chromebook devices with older EC
> firmware only supporting deprecated EC commands:
> - Glimmer based devices [Intel SOC using LPC transport]
> - Veyron minnie devices [ARM SOC using SPI transport]
>
> Tested-by: Gwendal Grignou <gwendal@xxxxxxxxxxxx>
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@xxxxxxxxxxxxx>
> Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
> Signed-off-by: Gwendal Grignou <gwendal@xxxxxxxxxxxx>
> ---
> Changes in v6:
> - Remove .id field in mfd_cell, PLATFORM_DEVID_AUTO is set to auto
> assign ids.
> - Add support for ARM devices with older EC.
>
> Changes in v5:
> - Remove unnecessary white lines.
>
> Changes in v4:
> - [5/8] Nit: EC -> ECs (Lee Jones)
> - [5/8] Statically define cros_ec_accel_legacy_cells (Lee Jones)
>
> Changes in v3:
> - [5/8] Add the Reviewed-by Andy Shevchenko.
>
> Changes in v2:
> - [5/8] Add the Reviewed-by Gwendal.
>
> drivers/mfd/cros_ec_dev.c | 69 +++++++++++++++++++++++++++++++++++++++
> 1 file changed, 69 insertions(+)

Applied, thanks.

--
Lee Jones [æçæ]
Linaro Services Technical Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog