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>

diff --git a/drivers/misc/mei/bus.c b/drivers/misc/mei/bus.c
index ddc5ac92a200a85d7f83762c7df04b4da90fc15d..d5b9ed4452ef5d7b7feed24b4772d2269fd542ed 100644 (file)
--- 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 9273e89e6a15bf08344f7d54098753907f54a9a4..3349f626aa88eae8d4037a5051a32e1f19c9589a 100644 (file)
--- 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 {