diff options
| author | Alexander Usyskin <alexander.usyskin@intel.com> | 2014-03-31 17:59:24 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-05-03 19:21:23 -0400 |
| commit | 86113500c060bccb0f08bdcadcecc0bd267fd25a (patch) | |
| tree | f195af9c5646f13d87bb0c4541dd3ac9c66da91c | |
| parent | 04dd36619564c3fcf590c2bf2619b14c09cd0749 (diff) | |
| download | linux-86113500c060bccb0f08bdcadcecc0bd267fd25a.tar.bz2 | |
mei: make return values consistent across the driver
Follow-up for bits missed in
commit 7ca96aa278f8b9983184e318b06a0ed9ad0297b8
mei: make return values consistent across the driver
Signed-off-by: Alexander Usyskin <alexander.usyskin@intel.com>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/misc/mei/bus.c | 2 | ||||
| -rw-r--r-- | drivers/misc/mei/client.c | 5 |
2 files changed, 3 insertions, 4 deletions
diff --git a/drivers/misc/mei/bus.c b/drivers/misc/mei/bus.c index ddc5ac92a200..d5b9ed4452ef 100644 --- a/drivers/misc/mei/bus.c +++ b/drivers/misc/mei/bus.c @@ -247,7 +247,7 @@ static int ___mei_cl_send(struct mei_cl *cl, u8 *buf, size_t length, return id; if (length > dev->me_clients[id].props.max_msg_length) - return -EINVAL; + return -EFBIG; cb = mei_io_cb_init(cl, NULL); if (!cb) diff --git a/drivers/misc/mei/client.c b/drivers/misc/mei/client.c index 9273e89e6a15..3349f626aa88 100644 --- a/drivers/misc/mei/client.c +++ b/drivers/misc/mei/client.c @@ -768,10 +768,9 @@ int mei_cl_read_start(struct mei_cl *cl, size_t length) cb->fop_type = MEI_FOP_READ; if (mei_hbuf_acquire(dev)) { - if (mei_hbm_cl_flow_control_req(dev, cl)) { - rets = -ENODEV; + rets = mei_hbm_cl_flow_control_req(dev, cl); + if (rets < 0) goto out; - } list_add_tail(&cb->list, &dev->read_list.list); } else { |