diff options
author | Bin Meng <bmeng.cn@gmail.com> | 2019-09-09 06:00:01 -0700 |
---|---|---|
committer | Jagan Teki <jagan@amarulasolutions.com> | 2019-10-25 00:48:31 +0530 |
commit | 4b060003957db07fa0e35beafe6559f9cc91954c (patch) | |
tree | ba4a762a7dbe6ed0c16a2a93e6b8a46c2f74876e /drivers/spi/bcm63xx_spi.c | |
parent | 4dd520b36bce9b35f7a12f3ffd9a95269d425170 (diff) |
dm: spi: Change cs_info op to return -EINVAL for invalid cs num
We need distinguish the following two situations in various SPI APIs:
- given chip select num is invalid
- given chip select num is valid, but no device is attached
Currently -ENODEV is returned for both cases.
For the first case, it's more reasonable to return -EINVAL instead of
-ENODEV for invalid chip select numbers.
Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Tested-by: Jagan Teki <jagan@amarulasolutions.com> # SoPine
Reviewed-by: Jagan Teki <jagan@amarulasolutions.com>
Diffstat (limited to 'drivers/spi/bcm63xx_spi.c')
-rw-r--r-- | drivers/spi/bcm63xx_spi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/spi/bcm63xx_spi.c b/drivers/spi/bcm63xx_spi.c index 4d19e03523..69f88c9e08 100644 --- a/drivers/spi/bcm63xx_spi.c +++ b/drivers/spi/bcm63xx_spi.c @@ -130,7 +130,7 @@ static int bcm63xx_spi_cs_info(struct udevice *bus, uint cs, if (cs >= priv->num_cs) { printf("no cs %u\n", cs); - return -ENODEV; + return -EINVAL; } return 0; |