Commit 5d6b1fa3 authored by Bjorn Helgaas's avatar Bjorn Helgaas Committed by Jesse Barnes

resources: factor out resource_clip() to simplify find_resource()

This factors out the min/max clipping to simplify find_resource().
No functional change.
Signed-off-by: default avatarBjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: default avatarJesse Barnes <jbarnes@virtuousgeek.org>
parent a9cea017
...@@ -365,6 +365,15 @@ static resource_size_t simple_align_resource(void *data, ...@@ -365,6 +365,15 @@ static resource_size_t simple_align_resource(void *data,
return avail->start; return avail->start;
} }
static void resource_clip(struct resource *res, resource_size_t min,
resource_size_t max)
{
if (res->start < min)
res->start = min;
if (res->end > max)
res->end = max;
}
/* /*
* Find empty slot in the resource tree given range and alignment. * Find empty slot in the resource tree given range and alignment.
*/ */
...@@ -394,10 +403,8 @@ static int find_resource(struct resource *root, struct resource *new, ...@@ -394,10 +403,8 @@ static int find_resource(struct resource *root, struct resource *new,
tmp.end = this->start - 1; tmp.end = this->start - 1;
else else
tmp.end = root->end; tmp.end = root->end;
if (tmp.start < min)
tmp.start = min; resource_clip(&tmp, min, max);
if (tmp.end > max)
tmp.end = max;
tmp.start = ALIGN(tmp.start, align); tmp.start = ALIGN(tmp.start, align);
tmp.start = alignf(alignf_data, &tmp, size, align); tmp.start = alignf(alignf_data, &tmp, size, align);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment