diff options
| author | Juan Castillo <jcastillo.devel@gmail.com> | 2015-10-01 17:55:11 +0100 |
|---|---|---|
| committer | Juan Castillo <juan.castillo@arm.com> | 2015-10-23 16:57:52 +0100 |
| commit | 7e26fe1f05bb05ce6e0f95b9a44ada2a21d53dd3 (patch) | |
| tree | 8aa2b46736442be975e7f83cb689082018903d98 /include/drivers/io | |
| parent | 84ab33e1e95500a3dfc776fe416b9d1a51288057 (diff) | |
IO Framework: use standard errno codes as return values
This patch redefines the values of IO_FAIL, IO_NOT_SUPPORTED and
IO_RESOURCES_EXHAUSTED to match the corresponding definitions in
errno.h:
#define IO_FAIL (-ENOENT)
#define IO_NOT_SUPPORTED (-ENODEV)
#define IO_RESOURCES_EXHAUSTED (-ENOMEM)
NOTE: please note that the IO_FAIL, IO_NOT_SUPPORTED and
IO_RESOURCES_EXHAUSTED definitions are considered deprecated
and their usage should be avoided. Callers should rely on errno.h
definitions when checking the return values of IO functions.
Change-Id: Ic8491aa43384b6ee44951ebfc053a3ded16a80be
Diffstat (limited to 'include/drivers/io')
| -rw-r--r-- | include/drivers/io/io_storage.h | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/include/drivers/io/io_storage.h b/include/drivers/io/io_storage.h index e98dcd04..4c3526ef 100644 --- a/include/drivers/io/io_storage.h +++ b/include/drivers/io/io_storage.h @@ -31,6 +31,7 @@ #ifndef __IO_H__ #define __IO_H__ +#include <errno.h> #include <stdint.h> #include <stdio.h> /* For ssize_t */ #include <uuid.h> @@ -88,11 +89,13 @@ typedef struct io_block_spec { #define IO_MODE_RW (1 << 1) -/* Return codes reported by 'io_*' APIs */ +/* Return codes reported by 'io_*' APIs. + * IMPORTANT: these definitions are deprecated. Callers should use standard + * errno definitions when checking the return value of io_* APIs. */ #define IO_SUCCESS (0) -#define IO_FAIL (-1) -#define IO_NOT_SUPPORTED (-2) -#define IO_RESOURCES_EXHAUSTED (-3) +#define IO_FAIL (-ENOENT) +#define IO_NOT_SUPPORTED (-ENODEV) +#define IO_RESOURCES_EXHAUSTED (-ENOMEM) /* Open a connection to a device */ |