Commit eb0c3cd4 authored by Guchun Chen's avatar Guchun Chen Committed by Alex Deucher

drm/amdgpu: add one definition for RAS's sysfs/debugfs name(v2)

Add one definition for the RAS module's FS name. It's used
in both debugfs and sysfs cases.

v2: Use static variable instead of macro definition.
Signed-off-by: default avatarGuchun Chen <guchun.chen@amd.com>
Reviewed-by: default avatarHawking Zhang <Hawking.Zhang@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent bf0b91b7
...@@ -34,6 +34,8 @@ ...@@ -34,6 +34,8 @@
#include "amdgpu_xgmi.h" #include "amdgpu_xgmi.h"
#include "ivsrcid/nbio/irqsrcs_nbif_7_4.h" #include "ivsrcid/nbio/irqsrcs_nbif_7_4.h"
static const char *RAS_FS_NAME = "ras";
const char *ras_error_string[] = { const char *ras_error_string[] = {
"none", "none",
"parity", "parity",
...@@ -1037,7 +1039,7 @@ static int amdgpu_ras_sysfs_create_feature_node(struct amdgpu_device *adev) ...@@ -1037,7 +1039,7 @@ static int amdgpu_ras_sysfs_create_feature_node(struct amdgpu_device *adev)
NULL NULL
}; };
struct attribute_group group = { struct attribute_group group = {
.name = "ras", .name = RAS_FS_NAME,
.attrs = attrs, .attrs = attrs,
.bin_attrs = bin_attrs, .bin_attrs = bin_attrs,
}; };
...@@ -1078,7 +1080,7 @@ static int amdgpu_ras_sysfs_remove_feature_node(struct amdgpu_device *adev) ...@@ -1078,7 +1080,7 @@ static int amdgpu_ras_sysfs_remove_feature_node(struct amdgpu_device *adev)
NULL NULL
}; };
struct attribute_group group = { struct attribute_group group = {
.name = "ras", .name = RAS_FS_NAME,
.attrs = attrs, .attrs = attrs,
.bin_attrs = bin_attrs, .bin_attrs = bin_attrs,
}; };
...@@ -1113,7 +1115,7 @@ int amdgpu_ras_sysfs_create(struct amdgpu_device *adev, ...@@ -1113,7 +1115,7 @@ int amdgpu_ras_sysfs_create(struct amdgpu_device *adev,
if (sysfs_add_file_to_group(&adev->dev->kobj, if (sysfs_add_file_to_group(&adev->dev->kobj,
&obj->sysfs_attr.attr, &obj->sysfs_attr.attr,
"ras")) { RAS_FS_NAME)) {
put_obj(obj); put_obj(obj);
return -EINVAL; return -EINVAL;
} }
...@@ -1133,7 +1135,7 @@ int amdgpu_ras_sysfs_remove(struct amdgpu_device *adev, ...@@ -1133,7 +1135,7 @@ int amdgpu_ras_sysfs_remove(struct amdgpu_device *adev,
sysfs_remove_file_from_group(&adev->dev->kobj, sysfs_remove_file_from_group(&adev->dev->kobj,
&obj->sysfs_attr.attr, &obj->sysfs_attr.attr,
"ras"); RAS_FS_NAME);
obj->attr_inuse = 0; obj->attr_inuse = 0;
put_obj(obj); put_obj(obj);
...@@ -1179,7 +1181,7 @@ static void amdgpu_ras_debugfs_create_ctrl_node(struct amdgpu_device *adev) ...@@ -1179,7 +1181,7 @@ static void amdgpu_ras_debugfs_create_ctrl_node(struct amdgpu_device *adev)
struct amdgpu_ras *con = amdgpu_ras_get_context(adev); struct amdgpu_ras *con = amdgpu_ras_get_context(adev);
struct drm_minor *minor = adev->ddev->primary; struct drm_minor *minor = adev->ddev->primary;
con->dir = debugfs_create_dir("ras", minor->debugfs_root); con->dir = debugfs_create_dir(RAS_FS_NAME, minor->debugfs_root);
debugfs_create_file("ras_ctrl", S_IWUGO | S_IRUGO, con->dir, debugfs_create_file("ras_ctrl", S_IWUGO | S_IRUGO, con->dir,
adev, &amdgpu_ras_debugfs_ctrl_ops); adev, &amdgpu_ras_debugfs_ctrl_ops);
debugfs_create_file("ras_eeprom_reset", S_IWUGO | S_IRUGO, con->dir, debugfs_create_file("ras_eeprom_reset", S_IWUGO | S_IRUGO, con->dir,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment