Re: [PATCH 3/4] PCI: introduce reset_resource()

From: Jesse Barnes
Date: Fri Feb 25 2011 - 17:56:33 EST


On Mon, 14 Feb 2011 17:43:19 -0800
Ram Pai <linuxram@xxxxxxxxxx> wrote:

> introduced reset_resource() which factors out resource reset logic.
>
> Signed-off-by: Ram Pai <linuxram@xxxxxxxxxx>
> ---
> drivers/pci/setup-bus.c | 11 ++++++++---
> 1 files changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/pci/setup-bus.c b/drivers/pci/setup-bus.c
> index bcf5752..a94ecc1 100644
> --- a/drivers/pci/setup-bus.c
> +++ b/drivers/pci/setup-bus.c
> @@ -88,6 +88,13 @@ static void __dev_sort_resources(struct pci_dev *dev,
> pdev_sort_resources(dev, head);
> }
>
> +static inline void reset_resource(struct resource *res)
> +{
> + res->start = 0;
> + res->end = 0;
> + res->flags = 0;
> +}
> +

Probably belongs in resource.c, and there may be other users that can
be cleaned up as well. Nice improvement to readability.

--
Jesse Barnes, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/