[PATCH 1/4] Remove path attribute from sgi_hotplug

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Alex Chiang
Date: Thursday, February 28, 2008 - 5:26 pm

Rename the slot to be the contents of the 'path' sysfs attribute, and
delete the attribute.  The mapping from pci address to slot name is
supposed to be done through the 'address' file, which will be provided
automatically later in this series of patches.

Signed-off-by: Alex Chiang <achiang@hp.com>
Signed-off-by: Matthew Wilcox <matthew@wil.cx>
---
 drivers/pci/hotplug/sgi_hotplug.c |   32 +-------------------------------
 1 files changed, 1 insertions(+), 31 deletions(-)

diff --git a/drivers/pci/hotplug/sgi_hotplug.c b/drivers/pci/hotplug/sgi_hotplug.c
index ef07c36..693519e 100644
--- a/drivers/pci/hotplug/sgi_hotplug.c
+++ b/drivers/pci/hotplug/sgi_hotplug.c
@@ -91,21 +91,6 @@ static struct hotplug_slot_ops sn_hotplug_slot_ops = {
 
 static DEFINE_MUTEX(sn_hotplug_mutex);
 
-static ssize_t path_show (struct hotplug_slot *bss_hotplug_slot,
-	       		  char *buf)
-{
-	int retval = -ENOENT;
-	struct slot *slot = bss_hotplug_slot->private;
-
-	if (!slot)
-		return retval;
-
-	retval = sprintf (buf, "%s\n", slot->physical_path);
-	return retval;
-}
-
-static struct hotplug_slot_attribute sn_slot_path_attr = __ATTR_RO(path);
-
 static int sn_pci_slot_valid(struct pci_bus *pci_bus, int device)
 {
 	struct pcibus_info *pcibus_info;
@@ -173,18 +158,10 @@ static int sn_hp_slot_private_alloc(struct hotplug_slot *bss_hotplug_slot,
 		return -ENOMEM;
 	bss_hotplug_slot->private = slot;
 
-	bss_hotplug_slot->name = kmalloc(SN_SLOT_NAME_SIZE, GFP_KERNEL);
-	if (!bss_hotplug_slot->name) {
-		kfree(bss_hotplug_slot->private);
-		return -ENOMEM;
-	}
+	bss_hotplug_slot->name = slot->physical_path;
 
 	slot->device_num = device;
 	slot->pci_bus = pci_bus;
-	sprintf(bss_hotplug_slot->name, "%04x:%02x:%02x",
-		pci_domain_nr(pci_bus),
-		((u16)pcibus_info->pbi_buscommon.bs_persist_busnum),
-		device + 1);
 
 	sn_generate_path(pci_bus, slot->physical_path);
 
@@ -203,8 +180,6 @@ static struct hotplug_slot * sn_hp_destroy(void)
 		bss_hotplug_slot = slot->hotplug_slot;
 		list_del(&((struct slot *)bss_hotplug_slot->private)->
 			 hp_list);
-		sysfs_remove_file(&bss_hotplug_slot->kobj,
-				  &sn_slot_path_attr.attr);
 		break;
 	}
 	return bss_hotplug_slot;
@@ -653,11 +628,6 @@ static int sn_hotplug_slot_register(struct pci_bus *pci_bus)
 		rc = pci_hp_register(bss_hotplug_slot);
 		if (rc)
 			goto register_err;
-
-		rc = sysfs_create_file(&bss_hotplug_slot->kobj,
-				       &sn_slot_path_attr.attr);
-		if (rc)
-			goto register_err;
 	}
 	dev_dbg(&pci_bus->self->dev, "Registered bus with hotplug\n");
 	return rc;
-- 
1.5.3.1.g1e61

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/4, v7] PCI, ACPI: Physical PCI slot objects, Alex Chiang, (Thu Feb 28, 5:23 pm)
[PATCH 1/4] Remove path attribute from sgi_hotplug, Alex Chiang, (Thu Feb 28, 5:26 pm)
[PATCH 2/4] Construct one fakephp slot per pci slot, Alex Chiang, (Thu Feb 28, 5:27 pm)
[PATCH 3/4] Introduce pci_slot, Alex Chiang, (Thu Feb 28, 5:28 pm)
[PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Thu Feb 28, 5:29 pm)
Re: [PATCH 3/4] Introduce pci_slot, Greg KH, (Fri Feb 29, 10:24 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Greg KH, (Fri Feb 29, 10:25 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Matthew Wilcox, (Sat Mar 1, 7:43 am)
Re: [PATCH 1/4] Remove path attribute from sgi_hotplug, Jesse Barnes, (Mon Mar 3, 11:48 am)
Re: [PATCH 1/4] Remove path attribute from sgi_hotplug, Prarit Bhargava, (Mon Mar 3, 11:54 am)
Re: [PATCH 3/4] Introduce pci_slot, Alex Chiang, (Mon Mar 3, 1:56 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Greg KH, (Mon Mar 3, 10:49 pm)
Re: [PATCH 3/4] Introduce pci_slot, Greg KH, (Mon Mar 3, 10:58 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Jesse Barnes, (Tue Mar 4, 11:18 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Greg KH, (Tue Mar 4, 12:30 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Jesse Barnes, (Tue Mar 4, 1:02 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kristen Carlson Accardi, (Tue Mar 4, 1:12 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Tue Mar 4, 3:58 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Tue Mar 4, 4:09 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Greg KH, (Tue Mar 4, 4:15 pm)
[PATCH 3/4, v8] Introduce pci_slot, Alex Chiang, (Tue Mar 4, 4:30 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Tue Mar 4, 4:46 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Tue Mar 4, 6:11 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Wed Mar 5, 1:20 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Matthew Wilcox, (Wed Mar 5, 1:34 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Wed Mar 5, 7:07 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Tue Mar 11, 6:10 am)
[PATCH 3/(3+1)] ACPI PCI slot detection driver, Kenji Kaneshige, (Tue Mar 11, 6:13 am)
Re: [PATCH 3/(3+1)] ACPI PCI slot detection driver, Kenji Kaneshige, (Tue Mar 11, 6:17 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Matthew Wilcox, (Tue Mar 11, 6:28 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Jesse Barnes, (Tue Mar 11, 9:56 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kristen Carlson Accardi, (Tue Mar 11, 11:04 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Tue Mar 11, 12:14 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kristen Carlson Accardi, (Tue Mar 11, 4:34 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Tue Mar 11, 9:08 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Tue Mar 11, 10:51 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Wed Mar 12, 3:50 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Wed Mar 12, 4:33 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Wed Mar 12, 5:59 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Wed Mar 12, 8:24 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Gary Hade, (Thu Mar 13, 7:16 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Thu Mar 13, 10:34 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Tue Mar 18, 1:49 pm)