PCI: acpiphp: remove 'name' parameter
We do not need to manage our own name parameter, especially since the PCI core can change it on our behalf, in the case of duplicate slot names. Remove 'name' from acpiphp's version of struct slot. Cc: kristen.c.accardi@intel.com Acked-by: Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com> Signed-off-by: Alex Chiang <achiang@hp.com> Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
This commit is contained in:
parent
5fe6cc6068
commit
df77cd1007
2 changed files with 21 additions and 19 deletions
|
@ -50,9 +50,6 @@
|
||||||
#define info(format, arg...) printk(KERN_INFO "%s: " format, MY_NAME , ## arg)
|
#define info(format, arg...) printk(KERN_INFO "%s: " format, MY_NAME , ## arg)
|
||||||
#define warn(format, arg...) printk(KERN_WARNING "%s: " format, MY_NAME , ## arg)
|
#define warn(format, arg...) printk(KERN_WARNING "%s: " format, MY_NAME , ## arg)
|
||||||
|
|
||||||
/* name size which is used for entries in pcihpfs */
|
|
||||||
#define SLOT_NAME_SIZE 20 /* {_SUN} */
|
|
||||||
|
|
||||||
struct acpiphp_bridge;
|
struct acpiphp_bridge;
|
||||||
struct acpiphp_slot;
|
struct acpiphp_slot;
|
||||||
|
|
||||||
|
@ -63,9 +60,13 @@ struct slot {
|
||||||
struct hotplug_slot *hotplug_slot;
|
struct hotplug_slot *hotplug_slot;
|
||||||
struct acpiphp_slot *acpi_slot;
|
struct acpiphp_slot *acpi_slot;
|
||||||
struct hotplug_slot_info info;
|
struct hotplug_slot_info info;
|
||||||
char name[SLOT_NAME_SIZE];
|
|
||||||
};
|
};
|
||||||
|
|
||||||
|
static inline const char *slot_name(struct slot *slot)
|
||||||
|
{
|
||||||
|
return hotplug_slot_name(slot->hotplug_slot);
|
||||||
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* struct acpiphp_bridge - PCI bridge information
|
* struct acpiphp_bridge - PCI bridge information
|
||||||
*
|
*
|
||||||
|
|
|
@ -44,6 +44,9 @@
|
||||||
|
|
||||||
#define MY_NAME "acpiphp"
|
#define MY_NAME "acpiphp"
|
||||||
|
|
||||||
|
/* name size which is used for entries in pcihpfs */
|
||||||
|
#define SLOT_NAME_SIZE 21 /* {_SUN} */
|
||||||
|
|
||||||
static int debug;
|
static int debug;
|
||||||
int acpiphp_debug;
|
int acpiphp_debug;
|
||||||
|
|
||||||
|
@ -84,7 +87,6 @@ static struct hotplug_slot_ops acpi_hotplug_slot_ops = {
|
||||||
.get_adapter_status = get_adapter_status,
|
.get_adapter_status = get_adapter_status,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* acpiphp_register_attention - set attention LED callback
|
* acpiphp_register_attention - set attention LED callback
|
||||||
* @info: must be completely filled with LED callbacks
|
* @info: must be completely filled with LED callbacks
|
||||||
|
@ -136,7 +138,7 @@ static int enable_slot(struct hotplug_slot *hotplug_slot)
|
||||||
{
|
{
|
||||||
struct slot *slot = hotplug_slot->private;
|
struct slot *slot = hotplug_slot->private;
|
||||||
|
|
||||||
dbg("%s - physical_slot = %s\n", __func__, hotplug_slot->name);
|
dbg("%s - physical_slot = %s\n", __func__, slot_name(slot));
|
||||||
|
|
||||||
/* enable the specified slot */
|
/* enable the specified slot */
|
||||||
return acpiphp_enable_slot(slot->acpi_slot);
|
return acpiphp_enable_slot(slot->acpi_slot);
|
||||||
|
@ -154,7 +156,7 @@ static int disable_slot(struct hotplug_slot *hotplug_slot)
|
||||||
struct slot *slot = hotplug_slot->private;
|
struct slot *slot = hotplug_slot->private;
|
||||||
int retval;
|
int retval;
|
||||||
|
|
||||||
dbg("%s - physical_slot = %s\n", __func__, hotplug_slot->name);
|
dbg("%s - physical_slot = %s\n", __func__, slot_name(slot));
|
||||||
|
|
||||||
/* disable the specified slot */
|
/* disable the specified slot */
|
||||||
retval = acpiphp_disable_slot(slot->acpi_slot);
|
retval = acpiphp_disable_slot(slot->acpi_slot);
|
||||||
|
@ -177,7 +179,7 @@ static int disable_slot(struct hotplug_slot *hotplug_slot)
|
||||||
{
|
{
|
||||||
int retval = -ENODEV;
|
int retval = -ENODEV;
|
||||||
|
|
||||||
dbg("%s - physical_slot = %s\n", __func__, hotplug_slot->name);
|
dbg("%s - physical_slot = %s\n", __func__, hotplug_slot_name(hotplug_slot));
|
||||||
|
|
||||||
if (attention_info && try_module_get(attention_info->owner)) {
|
if (attention_info && try_module_get(attention_info->owner)) {
|
||||||
retval = attention_info->set_attn(hotplug_slot, status);
|
retval = attention_info->set_attn(hotplug_slot, status);
|
||||||
|
@ -200,7 +202,7 @@ static int get_power_status(struct hotplug_slot *hotplug_slot, u8 *value)
|
||||||
{
|
{
|
||||||
struct slot *slot = hotplug_slot->private;
|
struct slot *slot = hotplug_slot->private;
|
||||||
|
|
||||||
dbg("%s - physical_slot = %s\n", __func__, hotplug_slot->name);
|
dbg("%s - physical_slot = %s\n", __func__, slot_name(slot));
|
||||||
|
|
||||||
*value = acpiphp_get_power_status(slot->acpi_slot);
|
*value = acpiphp_get_power_status(slot->acpi_slot);
|
||||||
|
|
||||||
|
@ -222,7 +224,7 @@ static int get_attention_status(struct hotplug_slot *hotplug_slot, u8 *value)
|
||||||
{
|
{
|
||||||
int retval = -EINVAL;
|
int retval = -EINVAL;
|
||||||
|
|
||||||
dbg("%s - physical_slot = %s\n", __func__, hotplug_slot->name);
|
dbg("%s - physical_slot = %s\n", __func__, hotplug_slot_name(hotplug_slot));
|
||||||
|
|
||||||
if (attention_info && try_module_get(attention_info->owner)) {
|
if (attention_info && try_module_get(attention_info->owner)) {
|
||||||
retval = attention_info->get_attn(hotplug_slot, value);
|
retval = attention_info->get_attn(hotplug_slot, value);
|
||||||
|
@ -245,7 +247,7 @@ static int get_latch_status(struct hotplug_slot *hotplug_slot, u8 *value)
|
||||||
{
|
{
|
||||||
struct slot *slot = hotplug_slot->private;
|
struct slot *slot = hotplug_slot->private;
|
||||||
|
|
||||||
dbg("%s - physical_slot = %s\n", __func__, hotplug_slot->name);
|
dbg("%s - physical_slot = %s\n", __func__, slot_name(slot));
|
||||||
|
|
||||||
*value = acpiphp_get_latch_status(slot->acpi_slot);
|
*value = acpiphp_get_latch_status(slot->acpi_slot);
|
||||||
|
|
||||||
|
@ -265,7 +267,7 @@ static int get_adapter_status(struct hotplug_slot *hotplug_slot, u8 *value)
|
||||||
{
|
{
|
||||||
struct slot *slot = hotplug_slot->private;
|
struct slot *slot = hotplug_slot->private;
|
||||||
|
|
||||||
dbg("%s - physical_slot = %s\n", __func__, hotplug_slot->name);
|
dbg("%s - physical_slot = %s\n", __func__, slot_name(slot));
|
||||||
|
|
||||||
*value = acpiphp_get_adapter_status(slot->acpi_slot);
|
*value = acpiphp_get_adapter_status(slot->acpi_slot);
|
||||||
|
|
||||||
|
@ -299,7 +301,7 @@ static void release_slot(struct hotplug_slot *hotplug_slot)
|
||||||
{
|
{
|
||||||
struct slot *slot = hotplug_slot->private;
|
struct slot *slot = hotplug_slot->private;
|
||||||
|
|
||||||
dbg("%s - physical_slot = %s\n", __func__, hotplug_slot->name);
|
dbg("%s - physical_slot = %s\n", __func__, slot_name(slot));
|
||||||
|
|
||||||
kfree(slot->hotplug_slot);
|
kfree(slot->hotplug_slot);
|
||||||
kfree(slot);
|
kfree(slot);
|
||||||
|
@ -310,6 +312,7 @@ int acpiphp_register_hotplug_slot(struct acpiphp_slot *acpiphp_slot)
|
||||||
{
|
{
|
||||||
struct slot *slot;
|
struct slot *slot;
|
||||||
int retval = -ENOMEM;
|
int retval = -ENOMEM;
|
||||||
|
char name[SLOT_NAME_SIZE];
|
||||||
|
|
||||||
slot = kzalloc(sizeof(*slot), GFP_KERNEL);
|
slot = kzalloc(sizeof(*slot), GFP_KERNEL);
|
||||||
if (!slot)
|
if (!slot)
|
||||||
|
@ -321,8 +324,6 @@ int acpiphp_register_hotplug_slot(struct acpiphp_slot *acpiphp_slot)
|
||||||
|
|
||||||
slot->hotplug_slot->info = &slot->info;
|
slot->hotplug_slot->info = &slot->info;
|
||||||
|
|
||||||
slot->hotplug_slot->name = slot->name;
|
|
||||||
|
|
||||||
slot->hotplug_slot->private = slot;
|
slot->hotplug_slot->private = slot;
|
||||||
slot->hotplug_slot->release = &release_slot;
|
slot->hotplug_slot->release = &release_slot;
|
||||||
slot->hotplug_slot->ops = &acpi_hotplug_slot_ops;
|
slot->hotplug_slot->ops = &acpi_hotplug_slot_ops;
|
||||||
|
@ -336,12 +337,12 @@ int acpiphp_register_hotplug_slot(struct acpiphp_slot *acpiphp_slot)
|
||||||
slot->hotplug_slot->info->cur_bus_speed = PCI_SPEED_UNKNOWN;
|
slot->hotplug_slot->info->cur_bus_speed = PCI_SPEED_UNKNOWN;
|
||||||
|
|
||||||
acpiphp_slot->slot = slot;
|
acpiphp_slot->slot = slot;
|
||||||
snprintf(slot->name, sizeof(slot->name), "%u", slot->acpi_slot->sun);
|
snprintf(name, SLOT_NAME_SIZE, "%u", slot->acpi_slot->sun);
|
||||||
|
|
||||||
retval = pci_hp_register(slot->hotplug_slot,
|
retval = pci_hp_register(slot->hotplug_slot,
|
||||||
acpiphp_slot->bridge->pci_bus,
|
acpiphp_slot->bridge->pci_bus,
|
||||||
acpiphp_slot->device,
|
acpiphp_slot->device,
|
||||||
slot->name);
|
name);
|
||||||
if (retval == -EBUSY)
|
if (retval == -EBUSY)
|
||||||
goto error_hpslot;
|
goto error_hpslot;
|
||||||
if (retval) {
|
if (retval) {
|
||||||
|
@ -349,7 +350,7 @@ int acpiphp_register_hotplug_slot(struct acpiphp_slot *acpiphp_slot)
|
||||||
goto error_hpslot;
|
goto error_hpslot;
|
||||||
}
|
}
|
||||||
|
|
||||||
info("Slot [%s] registered\n", slot->hotplug_slot->name);
|
info("Slot [%s] registered\n", slot_name(slot));
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
error_hpslot:
|
error_hpslot:
|
||||||
|
@ -366,7 +367,7 @@ void acpiphp_unregister_hotplug_slot(struct acpiphp_slot *acpiphp_slot)
|
||||||
struct slot *slot = acpiphp_slot->slot;
|
struct slot *slot = acpiphp_slot->slot;
|
||||||
int retval = 0;
|
int retval = 0;
|
||||||
|
|
||||||
info ("Slot [%s] unregistered\n", slot->hotplug_slot->name);
|
info("Slot [%s] unregistered\n", slot_name(slot));
|
||||||
|
|
||||||
retval = pci_hp_deregister(slot->hotplug_slot);
|
retval = pci_hp_deregister(slot->hotplug_slot);
|
||||||
if (retval)
|
if (retval)
|
||||||
|
|
Reference in a new issue