On Thu, Oct 28, 2010 at 03:54:34PM +0900, Isaku Yamahata wrote:
Changes v2 -> v3:
- use [first, last] instead of [start, end)
Changes v1 -> v2:
- add comment.
Patch description: This patch set fixes PCI bar allocation when bar overflow occured. I checked if pmm_alloc facility can be used, but it doesn't suit for pci bar allocation. So I resulted in new API, pci_region which encapsulates region allocation and overflow checks. The first patch introduces pci_region, and the second patch fixes the overflow case with pci_region.
Looks okay to me. If there are no further comments, I'll commit.
BTW, as a minor nit - I'd prefer to put function descriptions in the .c file next to the code instead of in the .h file - no need to resend the patch though.