Thank you for adding this.
12 comments:
File Documentation/device/resource_allocation.md:
Maybe it’s just for review, but it’d be great, if you wrapped the lines.
I learned something new: https://en.wikipedia.org/wiki/Viz. ;-)
Maybe, just use a colon (:) instead?
Patch Set #1, Line 21: independent
independently?
Patch Set #1, Line 24: IORESOURCE_IO
Mark this up as monospace with `…`?
Patch Set #1, Line 27: “bridge resources”
Ditto.
Patch Set #1, Line 27: “fixed resources”
I’d just mark them up as italic with *fixed resources*.
Patch Set #1, Line 32: Both the flags
Both flags …
Patch Set #1, Line 34: (IO / MEM)
I’d remove the spaces.
Patch Set #1, Line 48: Currently, there are two flavors of resource allocators implemented in coreboot - v3 and v4.
Remove the trailing space.
Patch Set #1, Line 70: * Finds a single window for resource allocation towards the top of addressable memory under 4G boundary.
That’s exactly the same item as above.
Please spell it out, or add a link.
Please remove the blank line at the end of the file.
To view, visit change 43603. To unsubscribe, or for help writing mail filters, visit settings.