diff options
| author | Alexandru Gagniuc <mr.nuke.me@gmail.com> | 2013-11-23 18:54:44 -0600 |
|---|---|---|
| committer | Alexandru Gagniuc <mr.nuke.me@gmail.com> | 2013-11-25 23:59:59 +0100 |
| commit | d7134e06b1b1ac21108b23438604b7385e4a43fa (patch) | |
| tree | af710fa02c9dc7a87593aa22627e34fc89eca571 /src/include/types.h | |
| parent | fe9e30d6b74e340f9a97b9e358e20ec8a2785954 (diff) | |
| download | coreboot-d7134e06b1b1ac21108b23438604b7385e4a43fa.tar.gz coreboot-d7134e06b1b1ac21108b23438604b7385e4a43fa.tar.bz2 coreboot-d7134e06b1b1ac21108b23438604b7385e4a43fa.zip | |
pc80/mc146818rtc: Return an error code rather than an integer
Do not return hardcoded numerical values to communicate succes/failure, but
instead use an enumeration.
Change-Id: I742b08796adf136dce5984b702533f91640846dd
Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
Reviewed-on: http://review.coreboot.org/4265
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
Diffstat (limited to 'src/include/types.h')
| -rw-r--r-- | src/include/types.h | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/include/types.h b/src/include/types.h index 180fa3a68a4d..d1f991eb5dae 100644 --- a/src/include/types.h +++ b/src/include/types.h @@ -34,6 +34,13 @@ enum cb_err { CB_SUCCESS = 0, /**< Call completed succesfully */ CB_ERR = -1, /**< Generic error code */ CB_ERR_ARG = -2, /**< Invalid argument */ + + /* NVRAM/CMOS errors */ + CB_CMOS_OTABLE_DISABLED = -100, /**< Option table disabled */ + CB_CMOS_LAYOUT_NOT_FOUND = -101, /**< Layout file not found */ + CB_CMOS_OPTION_NOT_FOUND = -102, /**< Option string not found */ + CB_CMOS_ACCESS_ERROR = -103, /**< CMOS access error */ + CB_CMOS_CHECKSUM_INVALID = -104, /**< CMOS checksum is invalid */ }; #endif /* __TYPES_H */ |