2012-02-11 08:19:03

by Yinghai Lu

[permalink] [raw]
Subject: [PATCH] PCI: move pci_find_saved_cap out of linux/pci.h

Only have user in driver/pci/pci.c

Don't need to put it in global pci.h

Signed-off-by: Yinghai Lu <[email protected].

---
drivers/pci/pci.c | 19 +++++++++++++++++++
include/linux/pci.h | 19 -------------------
2 files changed, 19 insertions(+), 19 deletions(-)

Index: linux-2.6/drivers/pci/pci.c
===================================================================
--- linux-2.6.orig/drivers/pci/pci.c
+++ linux-2.6/drivers/pci/pci.c
@@ -825,6 +825,19 @@ EXPORT_SYMBOL(pci_choose_state);
#define pcie_cap_has_sltctl2(type, flags) \
((flags & PCI_EXP_FLAGS_VERS) > 1)

+static struct pci_cap_saved_state *pci_find_saved_cap(
+ struct pci_dev *pci_dev, char cap)
+{
+ struct pci_cap_saved_state *tmp;
+ struct hlist_node *pos;
+
+ hlist_for_each_entry(tmp, pos, &pci_dev->saved_cap_space, next) {
+ if (tmp->cap.cap_nr == cap)
+ return tmp;
+ }
+ return NULL;
+}
+
static int pci_save_pcie_state(struct pci_dev *dev)
{
int pos, i = 0;
@@ -1869,6 +1882,12 @@ void platform_pci_wakeup_init(struct pci
platform_pci_sleep_wake(dev, false);
}

+static void pci_add_saved_cap(struct pci_dev *pci_dev,
+ struct pci_cap_saved_state *new_cap)
+{
+ hlist_add_head(&new_cap->next, &pci_dev->saved_cap_space);
+}
+
/**
* pci_add_save_buffer - allocate buffer for saving given capability registers
* @dev: the PCI device
Index: linux-2.6/include/linux/pci.h
===================================================================
--- linux-2.6.orig/include/linux/pci.h
+++ linux-2.6/include/linux/pci.h
@@ -368,25 +368,6 @@ static inline int pci_channel_offline(st
return (pdev->error_state != pci_channel_io_normal);
}

-static inline struct pci_cap_saved_state *pci_find_saved_cap(
- struct pci_dev *pci_dev, char cap)
-{
- struct pci_cap_saved_state *tmp;
- struct hlist_node *pos;
-
- hlist_for_each_entry(tmp, pos, &pci_dev->saved_cap_space, next) {
- if (tmp->cap.cap_nr == cap)
- return tmp;
- }
- return NULL;
-}
-
-static inline void pci_add_saved_cap(struct pci_dev *pci_dev,
- struct pci_cap_saved_state *new_cap)
-{
- hlist_add_head(&new_cap->next, &pci_dev->saved_cap_space);
-}
-
/*
* The first PCI_BRIDGE_RESOURCE_NUM PCI bus resources (those that correspond
* to P2P or CardBus bridge windows) go in a table. Additional ones (for


2012-02-22 21:04:09

by Bjorn Helgaas

[permalink] [raw]
Subject: Re: [PATCH] PCI: move pci_find_saved_cap out of linux/pci.h

On Sat, Feb 11, 2012 at 12:18 AM, Yinghai Lu <[email protected]> wrote:
> Only have user in driver/pci/pci.c
>
> Don't need to put it in global pci.h

Reviewed-by: Bjorn Helgaas <[email protected]>

> Signed-off-by: Yinghai Lu <[email protected].
>
> ---
> ?drivers/pci/pci.c ? | ? 19 +++++++++++++++++++
> ?include/linux/pci.h | ? 19 -------------------
> ?2 files changed, 19 insertions(+), 19 deletions(-)
>
> Index: linux-2.6/drivers/pci/pci.c
> ===================================================================
> --- linux-2.6.orig/drivers/pci/pci.c
> +++ linux-2.6/drivers/pci/pci.c
> @@ -825,6 +825,19 @@ EXPORT_SYMBOL(pci_choose_state);
> ?#define pcie_cap_has_sltctl2(type, flags) ? ? ? ? ? ? ?\
> ? ? ? ? ? ? ? ?((flags & PCI_EXP_FLAGS_VERS) > 1)
>
> +static struct pci_cap_saved_state *pci_find_saved_cap(
> + ? ? ? struct pci_dev *pci_dev, char cap)
> +{
> + ? ? ? struct pci_cap_saved_state *tmp;
> + ? ? ? struct hlist_node *pos;
> +
> + ? ? ? hlist_for_each_entry(tmp, pos, &pci_dev->saved_cap_space, next) {
> + ? ? ? ? ? ? ? if (tmp->cap.cap_nr == cap)
> + ? ? ? ? ? ? ? ? ? ? ? return tmp;
> + ? ? ? }
> + ? ? ? return NULL;
> +}
> +
> ?static int pci_save_pcie_state(struct pci_dev *dev)
> ?{
> ? ? ? ?int pos, i = 0;
> @@ -1869,6 +1882,12 @@ void platform_pci_wakeup_init(struct pci
> ? ? ? ?platform_pci_sleep_wake(dev, false);
> ?}
>
> +static void pci_add_saved_cap(struct pci_dev *pci_dev,
> + ? ? ? struct pci_cap_saved_state *new_cap)
> +{
> + ? ? ? hlist_add_head(&new_cap->next, &pci_dev->saved_cap_space);
> +}
> +
> ?/**
> ?* pci_add_save_buffer - allocate buffer for saving given capability registers
> ?* @dev: the PCI device
> Index: linux-2.6/include/linux/pci.h
> ===================================================================
> --- linux-2.6.orig/include/linux/pci.h
> +++ linux-2.6/include/linux/pci.h
> @@ -368,25 +368,6 @@ static inline int pci_channel_offline(st
> ? ? ? ?return (pdev->error_state != pci_channel_io_normal);
> ?}
>
> -static inline struct pci_cap_saved_state *pci_find_saved_cap(
> - ? ? ? struct pci_dev *pci_dev, char cap)
> -{
> - ? ? ? struct pci_cap_saved_state *tmp;
> - ? ? ? struct hlist_node *pos;
> -
> - ? ? ? hlist_for_each_entry(tmp, pos, &pci_dev->saved_cap_space, next) {
> - ? ? ? ? ? ? ? if (tmp->cap.cap_nr == cap)
> - ? ? ? ? ? ? ? ? ? ? ? return tmp;
> - ? ? ? }
> - ? ? ? return NULL;
> -}
> -
> -static inline void pci_add_saved_cap(struct pci_dev *pci_dev,
> - ? ? ? struct pci_cap_saved_state *new_cap)
> -{
> - ? ? ? hlist_add_head(&new_cap->next, &pci_dev->saved_cap_space);
> -}
> -
> ?/*
> ?* The first PCI_BRIDGE_RESOURCE_NUM PCI bus resources (those that correspond
> ?* to P2P or CardBus bridge windows) go in a table. ?Additional ones (for
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to [email protected]
> More majordomo info at ?http://vger.kernel.org/majordomo-info.html

2012-02-23 20:27:12

by Jesse Barnes

[permalink] [raw]
Subject: Re: [PATCH] PCI: move pci_find_saved_cap out of linux/pci.h

On Sat, 11 Feb 2012 00:18:41 -0800
Yinghai Lu <[email protected]> wrote:

> Only have user in driver/pci/pci.c
>
> Don't need to put it in global pci.h
>
> Signed-off-by: Yinghai Lu <[email protected].

Applied, thanks.

--
Jesse Barnes, Intel Open Source Technology Center


Attachments:
signature.asc (836.00 B)