Re: [PATCH 1/6] mfd: cros_ec: fail early if we cannot identify the EC

From: Lee Jones
Date: Wed Mar 07 2018 - 10:13:04 EST


On Mon, 19 Feb 2018, Enric Balletbo i Serra wrote:

> From: Vincent Palatin <vpalatin@xxxxxxxxxxxx>
>
> If we cannot communicate with the EC chip to detect the protocol version
> and its features, it's very likely useless to continue. Else we will
> commit all kind of uninformed mistakes (using the wrong protocol, the
> wrong buffer size, mixing the EC with other chips).
>
> Signed-off-by: Vincent Palatin <vpalatin@xxxxxxxxxxxx>
> Acked-by: Benson Leung <bleung@xxxxxxxxxxxx>
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@xxxxxxxxxxxxx>
> ---
> drivers/mfd/cros_ec.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)

Unless it is my mailer that is playing up, it looks as though you have
sent this set un-threaded, which means that when others or myself
start replying to them one-by-one, they will be sprawled throughout my
inbox. It also seems as though there is no 0th (cover letter) patch,
which means I am lacking potentially important information regarding
the history of the set and what you are trying to achieve.

Anyway, the patch has been applied this time (hopefully there aren't
any inter-dependencies.

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