diff options
author | Grant Likely <grant.likely@secretlab.ca> | 2012-11-29 11:35:41 -0500 |
---|---|---|
committer | Grant Likely <grant.likely@secretlab.ca> | 2012-11-29 12:27:19 -0500 |
commit | 499b42c3e4ca08f0de27683f31d3b8d5afd2a720 (patch) | |
tree | 0873e1179e801a20385001e6a8e54ce3f8aec9fc /arch/powerpc/sysdev | |
parent | b8fbdc42c5c5df8ab1f358fe90e3a8a1bdc9ae48 (diff) |
powerpc: Fix fallout from device_node->name constification
Commit c22618a1, "drivers/of: Constify device_node->name and
->path_component_name" changes device_node name to a const value, but
the PowerPC scom code still assigns it to a non-void field in
debugfs_blob_wrapper. The /right/ solution might be to change the
debugfs_blob_wrapper->data to also be const, but that is a bit
risky. Instead, cast the value to (void*). It is a bit ugly, but it
is the safest change until it can be investigated where
debugfs_blob_wrapper can be modified.
Reported-by: Michael Neuling <mikey@neuling.org>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
Diffstat (limited to 'arch/powerpc/sysdev')
-rw-r--r-- | arch/powerpc/sysdev/scom.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/powerpc/sysdev/scom.c b/arch/powerpc/sysdev/scom.c index 702256a1ca11..9193e12df695 100644 --- a/arch/powerpc/sysdev/scom.c +++ b/arch/powerpc/sysdev/scom.c | |||
@@ -157,7 +157,7 @@ static int scom_debug_init_one(struct dentry *root, struct device_node *dn, | |||
157 | ent->map = SCOM_MAP_INVALID; | 157 | ent->map = SCOM_MAP_INVALID; |
158 | spin_lock_init(&ent->lock); | 158 | spin_lock_init(&ent->lock); |
159 | snprintf(ent->name, 8, "scom%d", i); | 159 | snprintf(ent->name, 8, "scom%d", i); |
160 | ent->blob.data = dn->full_name; | 160 | ent->blob.data = (void*) dn->full_name; |
161 | ent->blob.size = strlen(dn->full_name); | 161 | ent->blob.size = strlen(dn->full_name); |
162 | 162 | ||
163 | dir = debugfs_create_dir(ent->name, root); | 163 | dir = debugfs_create_dir(ent->name, root); |