Prev: [PATCH 2/2] vgacon: use request_region(), not request_resource(), to set BUSY
Next: cpu_stop: implement and use cpu_stop, take#2
From: Bjorn Helgaas on 4 May 2010 15:00

Set the IORESOURCE_TYPE based on the parent. Without this, the type is
never set, which is slightly confusing if we ever print the resource later.

Signed-off-by: Bjorn Helgaas <bjorn.helgaas(a)hp.com>
---

kernel/resource.c | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)


diff --git a/kernel/resource.c b/kernel/resource.c
index aa63746..f0d83a4 100644
--- a/kernel/resource.c
+++ b/kernel/resource.c
@@ -699,8 +699,9 @@ struct resource * __request_region(struct resource *parent,
res->name = name;
res->start = start;
res->end = start + n - 1;
- res->flags = IORESOURCE_BUSY;
- res->flags |= flags;
+ res->flags = parent->flags & IORESOURCE_TYPE_BITS;
+ res->flags |= (flags & ~IORESOURCE_TYPE_BITS);
+ res->flags |= IORESOURCE_BUSY;

write_lock(&resource_lock);


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: [PATCH 2/2] vgacon: use request_region(), not request_resource(), to set BUSY
Next: cpu_stop: implement and use cpu_stop, take#2