lkml.org 
[lkml]   [2008]   [Dec]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[patch 042/104] PCI: Hotplug core: remove name
    2.6.27-stable review patch.  If anyone has any objections, please let us know.

    ------------------
    From: Alex Chiang <achiang@hp.com>

    commit 58319b802a614f10f1b5238fbde7a4b2e9a60069 upstream.

    Now that the PCI core manages the 'name' for each individual
    hotplug driver, and all drivers (except rpaphp) have been converted
    to use hotplug_slot_name(), there is no need for the PCI hotplug
    core to drag around its own copy of name either.

    Cc: kristen.c.accardi@intel.com
    Cc: matthew@wil.cx
    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>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    drivers/pci/hotplug/pci_hotplug_core.c | 6 +++---
    include/linux/pci_hotplug.h | 3 ---
    2 files changed, 3 insertions(+), 6 deletions(-)

    --- a/drivers/pci/hotplug/pci_hotplug_core.c
    +++ b/drivers/pci/hotplug/pci_hotplug_core.c
    @@ -533,7 +533,7 @@ static struct hotplug_slot *get_slot_fro

    list_for_each (tmp, &pci_hotplug_slot_list) {
    slot = list_entry (tmp, struct hotplug_slot, slot_list);
    - if (strcmp(slot->name, name) == 0)
    + if (strcmp(hotplug_slot_name(slot), name) == 0)
    return slot;
    }
    return NULL;
    @@ -611,7 +611,7 @@ int pci_hp_deregister(struct hotplug_slo
    return -ENODEV;

    mutex_lock(&pci_hp_mutex);
    - temp = get_slot_from_name(hotplug->name);
    + temp = get_slot_from_name(hotplug_slot_name(hotplug));
    if (temp != hotplug) {
    mutex_unlock(&pci_hp_mutex);
    return -ENODEV;
    @@ -621,7 +621,7 @@ int pci_hp_deregister(struct hotplug_slo

    slot = hotplug->pci_slot;
    fs_remove_slot(slot);
    - dbg("Removed slot %s from the list\n", hotplug->name);
    + dbg("Removed slot %s from the list\n", hotplug_slot_name(hotplug));

    hotplug->release(hotplug);
    slot->hotplug = NULL;
    --- a/include/linux/pci_hotplug.h
    +++ b/include/linux/pci_hotplug.h
    @@ -142,8 +142,6 @@ struct hotplug_slot_info {

    /**
    * struct hotplug_slot - used to register a physical slot with the hotplug pci core
    - * @name: the name of the slot being registered. This string must
    - * be unique amoung slots registered on this system.
    * @ops: pointer to the &struct hotplug_slot_ops to be used for this slot
    * @info: pointer to the &struct hotplug_slot_info for the initial values for
    * this slot.
    @@ -153,7 +151,6 @@ struct hotplug_slot_info {
    * needs.
    */
    struct hotplug_slot {
    - char *name;
    struct hotplug_slot_ops *ops;
    struct hotplug_slot_info *info;
    void (*release) (struct hotplug_slot *slot);


    \
     
     \ /
      Last update: 2008-12-03 21:09    [W:4.036 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site