diff options
author | Tomas Winkler <tomas.winkler@intel.com> | 2012-12-25 12:05:59 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-01-07 13:31:27 -0500 |
commit | 438763f37eb9664b6372bdfee990f8c33acdc63c (patch) | |
tree | 71f2feab0d7b91cfd28c538c2943a8d8fe000c96 /drivers/misc/mei/amthif.c | |
parent | d1c3ed669a2d452cacfb48c2d171a1f364dae2ed (diff) |
mei: drop redundant length parameter from mei_write_message function
The length is already part of the message header and it is validated
before the function call
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/misc/mei/amthif.c')
-rw-r--r-- | drivers/misc/mei/amthif.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/misc/mei/amthif.c b/drivers/misc/mei/amthif.c index 18794aea6062..8a9313a1ee7b 100644 --- a/drivers/misc/mei/amthif.c +++ b/drivers/misc/mei/amthif.c | |||
@@ -300,8 +300,7 @@ static int mei_amthif_send_cmd(struct mei_device *dev, struct mei_cl_cb *cb) | |||
300 | mei_hdr.reserved = 0; | 300 | mei_hdr.reserved = 0; |
301 | dev->iamthif_msg_buf_index += mei_hdr.length; | 301 | dev->iamthif_msg_buf_index += mei_hdr.length; |
302 | if (mei_write_message(dev, &mei_hdr, | 302 | if (mei_write_message(dev, &mei_hdr, |
303 | (unsigned char *)(dev->iamthif_msg_buf), | 303 | (unsigned char *)dev->iamthif_msg_buf)) |
304 | mei_hdr.length)) | ||
305 | return -ENODEV; | 304 | return -ENODEV; |
306 | 305 | ||
307 | if (mei_hdr.msg_complete) { | 306 | if (mei_hdr.msg_complete) { |
@@ -463,8 +462,7 @@ int mei_amthif_irq_write_complete(struct mei_device *dev, s32 *slots, | |||
463 | 462 | ||
464 | *slots -= msg_slots; | 463 | *slots -= msg_slots; |
465 | if (mei_write_message(dev, mei_hdr, | 464 | if (mei_write_message(dev, mei_hdr, |
466 | dev->iamthif_msg_buf + dev->iamthif_msg_buf_index, | 465 | dev->iamthif_msg_buf + dev->iamthif_msg_buf_index)) { |
467 | mei_hdr->length)) { | ||
468 | dev->iamthif_state = MEI_IAMTHIF_IDLE; | 466 | dev->iamthif_state = MEI_IAMTHIF_IDLE; |
469 | cl->status = -ENODEV; | 467 | cl->status = -ENODEV; |
470 | list_del(&cb->list); | 468 | list_del(&cb->list); |