diff options
| author | Andy Lutomirski <luto@kernel.org> | 2017-06-15 14:46:00 +0300 |
|---|---|---|
| committer | Jean Delvare <jdelvare@suse.de> | 2017-06-15 14:46:00 +0300 |
| commit | c926820085437a27b27e78996b2c7a5ad94e8055 (patch) | |
| tree | b792ab228b685848a0f8cd8880bcf8ca0876ceb5 /include/linux | |
| parent | c9aba14362a6eec583819ec8f4b872c1816f5cbe (diff) | |
| download | linux-c926820085437a27b27e78996b2c7a5ad94e8055.tar.xz | |
firmware: dmi_scan: Make dmi_walk and dmi_walk_early return real error codes
Currently they return -1 on error, which will confuse callers if
they try to interpret it as a normal negative error code.
Signed-off-by: Andy Lutomirski <luto@kernel.org>
Signed-off-by: Darren Hart (VMware) <dvhart@infradead.org>
Signed-off-by: Jean Delvare <jdelvare@suse.de>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/dmi.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/dmi.h b/include/linux/dmi.h index 5e9c74cf8894..9bbf21a516e4 100644 --- a/include/linux/dmi.h +++ b/include/linux/dmi.h @@ -136,7 +136,7 @@ static inline int dmi_name_in_vendors(const char *s) { return 0; } static inline int dmi_name_in_serial(const char *s) { return 0; } #define dmi_available 0 static inline int dmi_walk(void (*decode)(const struct dmi_header *, void *), - void *private_data) { return -1; } + void *private_data) { return -ENXIO; } static inline bool dmi_match(enum dmi_field f, const char *str) { return false; } static inline void dmi_memdev_name(u16 handle, const char **bank, |