diff options
author | Michael Ellerman <mpe@ellerman.id.au> | 2017-04-04 13:22:19 +1000 |
---|---|---|
committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2017-04-05 12:30:44 +1000 |
commit | 036d2e8d4f90e7828177f37ce330768370b0e845 (patch) | |
tree | 3ae2cf5abd1666be7c31dd9043bda01bf0a831c6 /core/opal.c | |
parent | c2bfa357ad3b8f8d56bbd6ef895795e1d2f1dc6b (diff) | |
download | skiboot-036d2e8d4f90e7828177f37ce330768370b0e845.zip skiboot-036d2e8d4f90e7828177f37ce330768370b0e845.tar.gz skiboot-036d2e8d4f90e7828177f37ce330768370b0e845.tar.bz2 |
core/init: Use '_' as separator in names of "exports" properties
The names of the properties under /ibm,opal/firmware/exports are used
directly by Linux to create files in sysfs. To remain consistent with
the existing naming of OPAL sysfs files, use '_' as the separator.
In particular for the symbol map which is already exported separately,
it's cleaner for the two files to have the same name, eg:
/sys/firmware/opal/exports/symbol_map
/sys/firmware/opal/symbol_map
Fixes: 9ffbfe269ec6 ("core/init: Add hdat-map property to OPAL node")
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'core/opal.c')
-rw-r--r-- | core/opal.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/core/opal.c b/core/opal.c index 7bc403c..73223b1 100644 --- a/core/opal.c +++ b/core/opal.c @@ -135,8 +135,13 @@ static void add_opal_firmware_exports_node(struct dt_node *node) uint64_t sym_start = (uint64_t)__sym_map_start; uint64_t sym_size = (uint64_t)__sym_map_end - sym_start; - dt_add_property_u64s(exports, "symbol-map", sym_start, sym_size); - dt_add_property_u64s(exports, "hdat-map", SPIRA_HEAP_BASE, + /* + * These property names will be used by Linux as the user-visible file + * name, so make them meaningful if possible. We use _ as the separator + * here to remain consistent with existing file names in /sys/opal. + */ + dt_add_property_u64s(exports, "symbol_map", sym_start, sym_size); + dt_add_property_u64s(exports, "hdat_map", SPIRA_HEAP_BASE, SPIRA_HEAP_SIZE); } |