From: Michael Auchter <michael.auchter@ni.com>
Date: Mon, 9 Dec 2019 20:27:31 +0000 (+0000)
Subject: cros_ec: use uint instead of uint8_t for cmd param
X-Git-Tag: v2025.01-rc5-pxa1908~2624^2~4^2
X-Git-Url: http://git.dujemihanovic.xyz/repo?a=commitdiff_plain;h=b4f98b3b16ec513f7fa6b97ec49792a5e99ec165;p=u-boot.git

cros_ec: use uint instead of uint8_t for cmd param

Chromium EC commands can be up to 16-bits, so using a uint8_t here can
cause truncation. Update to use a uint instead.

It looks like this should likely have been done as a part of
9fea76f5d30264dc08ac591a7a89427b8441555b, but this function was skipped
for some reason.

Signed-off-by: Michael Auchter <michael.auchter@ni.com>
Cc: Simon Glass <sjg@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
---

diff --git a/drivers/misc/cros_ec.c b/drivers/misc/cros_ec.c
index 382f826286..fa9984f6bd 100644
--- a/drivers/misc/cros_ec.c
+++ b/drivers/misc/cros_ec.c
@@ -313,7 +313,7 @@ static int send_command(struct cros_ec_dev *dev, uint cmd, int cmd_version,
  * @param din_len       Maximum size of response in bytes
  * @return number of bytes in response, or -ve on error
  */
-static int ec_command_inptr(struct udevice *dev, uint8_t cmd,
+static int ec_command_inptr(struct udevice *dev, uint cmd,
 			    int cmd_version, const void *dout, int dout_len,
 			    uint8_t **dinp, int din_len)
 {