drivers: spmi: msm: fix return values for errors in parameters

If an out-of-range usid or pid is passed to the
msm_spmi_{read|write}() functions, those currently return -EIO.
Since no actual I/O has taken place at this point, the error code is
misleading. Return the more appropriate -EINVAL in this case.

diff --git a/drivers/spmi/spmi-msm.c b/drivers/spmi/spmi-msm.c
index 48bc15759645835760ee7730e0e458372aae9201..41541875de71dad4325c10e3d24307170064b26e 100644 (file)
--- a/drivers/spmi/spmi-msm.c
+++ b/drivers/spmi/spmi-msm.c
@@ -57,9 +57,9 @@ static int msm_spmi_write(struct udevice *dev, int usid, int pid, int off,
        uint32_t reg = 0;
 
        if (usid >= SPMI_MAX_SLAVES)
-               return -EIO;
+               return -EINVAL;
        if (pid >= SPMI_MAX_PERIPH)
-               return -EIO;
+               return -EINVAL;
 
        channel = priv->channel_map[usid][pid];
 
@@ -102,9 +102,9 @@ static int msm_spmi_read(struct udevice *dev, int usid, int pid, int off)
        uint32_t reg = 0;
 
        if (usid >= SPMI_MAX_SLAVES)
-               return -EIO;
+               return -EINVAL;
        if (pid >= SPMI_MAX_PERIPH)
-               return -EIO;
+               return -EINVAL;
 
        channel = priv->channel_map[usid][pid];