diff options
Diffstat (limited to 'drivers/edac/edac_mc.c')
-rw-r--r-- | drivers/edac/edac_mc.c | 31 |
1 files changed, 7 insertions, 24 deletions
diff --git a/drivers/edac/edac_mc.c b/drivers/edac/edac_mc.c index d37a4c4a3125..e6ecc7da38a5 100644 --- a/drivers/edac/edac_mc.c +++ b/drivers/edac/edac_mc.c | |||
@@ -278,8 +278,6 @@ static int edac_sysfs_memctrl_setup(void) | |||
278 | if (!err) { | 278 | if (!err) { |
279 | /* Init the MC's kobject */ | 279 | /* Init the MC's kobject */ |
280 | memset(&edac_memctrl_kobj, 0, sizeof (edac_memctrl_kobj)); | 280 | memset(&edac_memctrl_kobj, 0, sizeof (edac_memctrl_kobj)); |
281 | kobject_init(&edac_memctrl_kobj); | ||
282 | |||
283 | edac_memctrl_kobj.parent = &edac_class.kset.kobj; | 281 | edac_memctrl_kobj.parent = &edac_class.kset.kobj; |
284 | edac_memctrl_kobj.ktype = &ktype_memctrl; | 282 | edac_memctrl_kobj.ktype = &ktype_memctrl; |
285 | 283 | ||
@@ -314,9 +312,6 @@ static void edac_sysfs_memctrl_teardown(void) | |||
314 | /* Unregister the MC's kobject */ | 312 | /* Unregister the MC's kobject */ |
315 | kobject_unregister(&edac_memctrl_kobj); | 313 | kobject_unregister(&edac_memctrl_kobj); |
316 | 314 | ||
317 | /* release the master edac mc kobject */ | ||
318 | kobject_put(&edac_memctrl_kobj); | ||
319 | |||
320 | /* Unregister the 'edac' object */ | 315 | /* Unregister the 'edac' object */ |
321 | sysdev_class_unregister(&edac_class); | 316 | sysdev_class_unregister(&edac_class); |
322 | #endif /* DISABLE_EDAC_SYSFS */ | 317 | #endif /* DISABLE_EDAC_SYSFS */ |
@@ -594,8 +589,6 @@ static int edac_sysfs_pci_setup(void) | |||
594 | debugf1("%s()\n", __func__); | 589 | debugf1("%s()\n", __func__); |
595 | 590 | ||
596 | memset(&edac_pci_kobj, 0, sizeof(edac_pci_kobj)); | 591 | memset(&edac_pci_kobj, 0, sizeof(edac_pci_kobj)); |
597 | |||
598 | kobject_init(&edac_pci_kobj); | ||
599 | edac_pci_kobj.parent = &edac_class.kset.kobj; | 592 | edac_pci_kobj.parent = &edac_class.kset.kobj; |
600 | edac_pci_kobj.ktype = &ktype_edac_pci; | 593 | edac_pci_kobj.ktype = &ktype_edac_pci; |
601 | 594 | ||
@@ -619,7 +612,6 @@ static void edac_sysfs_pci_teardown(void) | |||
619 | debugf0("%s()\n", __func__); | 612 | debugf0("%s()\n", __func__); |
620 | 613 | ||
621 | kobject_unregister(&edac_pci_kobj); | 614 | kobject_unregister(&edac_pci_kobj); |
622 | kobject_put(&edac_pci_kobj); | ||
623 | #endif | 615 | #endif |
624 | } | 616 | } |
625 | 617 | ||
@@ -829,7 +821,6 @@ static int edac_create_csrow_object(struct kobject *edac_mci_kobj, | |||
829 | 821 | ||
830 | /* generate ..../edac/mc/mc<id>/csrow<index> */ | 822 | /* generate ..../edac/mc/mc<id>/csrow<index> */ |
831 | 823 | ||
832 | kobject_init(&csrow->kobj); | ||
833 | csrow->kobj.parent = edac_mci_kobj; | 824 | csrow->kobj.parent = edac_mci_kobj; |
834 | csrow->kobj.ktype = &ktype_csrow; | 825 | csrow->kobj.ktype = &ktype_csrow; |
835 | 826 | ||
@@ -1104,7 +1095,6 @@ static int edac_create_sysfs_mci_device(struct mem_ctl_info *mci) | |||
1104 | debugf0("%s() idx=%d\n", __func__, mci->mc_idx); | 1095 | debugf0("%s() idx=%d\n", __func__, mci->mc_idx); |
1105 | 1096 | ||
1106 | memset(edac_mci_kobj, 0, sizeof(*edac_mci_kobj)); | 1097 | memset(edac_mci_kobj, 0, sizeof(*edac_mci_kobj)); |
1107 | kobject_init(edac_mci_kobj); | ||
1108 | 1098 | ||
1109 | /* set the name of the mc<id> object */ | 1099 | /* set the name of the mc<id> object */ |
1110 | err = kobject_set_name(edac_mci_kobj,"mc%d",mci->mc_idx); | 1100 | err = kobject_set_name(edac_mci_kobj,"mc%d",mci->mc_idx); |
@@ -1123,10 +1113,8 @@ static int edac_create_sysfs_mci_device(struct mem_ctl_info *mci) | |||
1123 | /* create a symlink for the device */ | 1113 | /* create a symlink for the device */ |
1124 | err = sysfs_create_link(edac_mci_kobj, &mci->pdev->dev.kobj, | 1114 | err = sysfs_create_link(edac_mci_kobj, &mci->pdev->dev.kobj, |
1125 | EDAC_DEVICE_SYMLINK); | 1115 | EDAC_DEVICE_SYMLINK); |
1126 | if (err) { | 1116 | if (err) |
1127 | kobject_unregister(edac_mci_kobj); | 1117 | goto fail0; |
1128 | return err; | ||
1129 | } | ||
1130 | 1118 | ||
1131 | /* Make directories for each CSROW object | 1119 | /* Make directories for each CSROW object |
1132 | * under the mc<id> kobject | 1120 | * under the mc<id> kobject |
@@ -1139,7 +1127,7 @@ static int edac_create_sysfs_mci_device(struct mem_ctl_info *mci) | |||
1139 | if (csrow->nr_pages > 0) { | 1127 | if (csrow->nr_pages > 0) { |
1140 | err = edac_create_csrow_object(edac_mci_kobj,csrow,i); | 1128 | err = edac_create_csrow_object(edac_mci_kobj,csrow,i); |
1141 | if (err) | 1129 | if (err) |
1142 | goto fail; | 1130 | goto fail1; |
1143 | } | 1131 | } |
1144 | } | 1132 | } |
1145 | 1133 | ||
@@ -1150,16 +1138,14 @@ static int edac_create_sysfs_mci_device(struct mem_ctl_info *mci) | |||
1150 | 1138 | ||
1151 | 1139 | ||
1152 | /* CSROW error: backout what has already been registered, */ | 1140 | /* CSROW error: backout what has already been registered, */ |
1153 | fail: | 1141 | fail1: |
1154 | for ( i--; i >= 0; i--) { | 1142 | for ( i--; i >= 0; i--) { |
1155 | if (csrow->nr_pages > 0) { | 1143 | if (csrow->nr_pages > 0) |
1156 | kobject_unregister(&mci->csrows[i].kobj); | 1144 | kobject_unregister(&mci->csrows[i].kobj); |
1157 | kobject_put(&mci->csrows[i].kobj); | ||
1158 | } | ||
1159 | } | 1145 | } |
1160 | 1146 | ||
1147 | fail0: | ||
1161 | kobject_unregister(edac_mci_kobj); | 1148 | kobject_unregister(edac_mci_kobj); |
1162 | kobject_put(edac_mci_kobj); | ||
1163 | 1149 | ||
1164 | return err; | 1150 | return err; |
1165 | } | 1151 | } |
@@ -1177,16 +1163,13 @@ static void edac_remove_sysfs_mci_device(struct mem_ctl_info *mci) | |||
1177 | 1163 | ||
1178 | /* remove all csrow kobjects */ | 1164 | /* remove all csrow kobjects */ |
1179 | for (i = 0; i < mci->nr_csrows; i++) { | 1165 | for (i = 0; i < mci->nr_csrows; i++) { |
1180 | if (mci->csrows[i].nr_pages > 0) { | 1166 | if (mci->csrows[i].nr_pages > 0) |
1181 | kobject_unregister(&mci->csrows[i].kobj); | 1167 | kobject_unregister(&mci->csrows[i].kobj); |
1182 | kobject_put(&mci->csrows[i].kobj); | ||
1183 | } | ||
1184 | } | 1168 | } |
1185 | 1169 | ||
1186 | sysfs_remove_link(&mci->edac_mci_kobj, EDAC_DEVICE_SYMLINK); | 1170 | sysfs_remove_link(&mci->edac_mci_kobj, EDAC_DEVICE_SYMLINK); |
1187 | 1171 | ||
1188 | kobject_unregister(&mci->edac_mci_kobj); | 1172 | kobject_unregister(&mci->edac_mci_kobj); |
1189 | kobject_put(&mci->edac_mci_kobj); | ||
1190 | #endif /* DISABLE_EDAC_SYSFS */ | 1173 | #endif /* DISABLE_EDAC_SYSFS */ |
1191 | } | 1174 | } |
1192 | 1175 | ||