diff options
| author | Michael Auchter <michael.auchter@ni.com> | 2019-12-09 20:27:31 +0000 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2019-12-10 21:11:31 -0700 |
| commit | b4f98b3b16ec513f7fa6b97ec49792a5e99ec165 (patch) | |
| tree | 788b127af58613bb3f0671f88a40bf4e627729bd /drivers | |
| parent | d08c38c32f64498946230059bd4ec1431c23476a (diff) | |
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>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/misc/cros_ec.c | 2 |
1 files changed, 1 insertions, 1 deletions
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) { |