diff options
| author | Paul Sokolovsky <paul.sokolovsky@linaro.org> | 2016-10-21 14:58:43 +0300 |
|---|---|---|
| committer | Anas Nashif <nashif@linux.intel.com> | 2016-10-22 10:04:00 +0000 |
| commit | fb9185cdb8014a445430d1d1740270455e36fe38 (patch) | |
| tree | a2c6346b67362e33552eeb3649922fdfb3e5c69f /include | |
| parent | 3ef74d4fea8cc265dfbbccdb17601401ae60c131 (diff) | |
kernel: Allow SYS_KERNEL_VER_MAJOR(), etc. return full range of values.
Macros SYS_KERNEL_VER_MAJOR(), SYS_KERNEL_VER_MINOR(),
SYS_KERNEL_VER_PATCHLEVEL() capped their return values to 0-15 range,
even though documentation says "Each of these elements must therefore be
in the range 0 to 255, inclusive". Fix to corresponds to the docs. This
issue especially affected SYS_KERNEL_VER_PATCHLEVEL(), which could be
set to a high value to represent WIP code in-between releases.
Change-Id: I0b72fb68f3f0f8d3d3b321a5ba2c48671879dfbc
Signed-off-by: Paul Sokolovsky <paul.sokolovsky@linaro.org>
Diffstat (limited to 'include')
| -rw-r--r-- | include/kernel_version.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/include/kernel_version.h b/include/kernel_version.h index 9fdfec055..9e2ed5e67 100644 --- a/include/kernel_version.h +++ b/include/kernel_version.h @@ -32,13 +32,13 @@ extern "C" { * x -- major release * y -- minor release * z -- patchlevel release - * Each of these elements must therefore be in the range 0 to 256, inclusive. + * Each of these elements must therefore be in the range 0 to 255, inclusive. * * Part 2: The least significant byte is reserved for future use. */ -#define SYS_KERNEL_VER_MAJOR(ver) ((ver >> 24) & 0x0F) -#define SYS_KERNEL_VER_MINOR(ver) ((ver >> 16) & 0x0F) -#define SYS_KERNEL_VER_PATCHLEVEL(ver) ((ver >> 8) & 0x0F) +#define SYS_KERNEL_VER_MAJOR(ver) ((ver >> 24) & 0xFF) +#define SYS_KERNEL_VER_MINOR(ver) ((ver >> 16) & 0xFF) +#define SYS_KERNEL_VER_PATCHLEVEL(ver) ((ver >> 8) & 0xFF) /* kernel version routines */ |