Remove enum elaborated type specifier

...from orientation params. Elaborated type specifiers are not needed
for C++ code and removing them makes doxygen happy. Verified working
on both gcc2h and gcc4h builds.
Fix rest of warnings, doxygen now runs cleanly.

The BTextView::SetText() methods were included twice, update the first and
remove the second.
Debugger: ElfFile::_CheckRange(): fix check
RadeonHD: Fix some marketing names

* Finally got Sea Islands sorted out
  (unless they add new chipsets to the line)
RadeonHD: Drop marketing names

* They are all over the place.. I give up
* Going off of engineering names and DCE is more accurate
* A lot of this info came from the x.org wiki
* I'd like to transition some of the engineering
  name checks to use DCE versions.. they tend to be more
  accurate and exact. (in some cases we can't, but most of
  the time we can)
Debugger: fix value reading problem.

When reading a variable value from a register, if the value's size is
smaller than the size of the register and the architecture is little
endian, we need to adjust the bit offset we read from in addition to
byte swapping, otherwise we wind up reading the wrong bytes.

This was mainly noticeable in the case of functions that returned
boolean values, which would consequently sometimes show up incorrectly.
kernel/int.cpp: typo fix
PCI: added MSI-X and HyperTransport definitions

* renamed PCI_cap_id_ldt to PCI_cap_id_ht
PCI: implemented reading, configuring and enabling MSI-X.
PCI: the x86 module now exports an MSI-X API.
Debugger: ValueLoader: Add TODO note.
sysinfo: Fixed incorrect memory stats on PAE systems.

Memory statistics were calculated as 32-bit values, so they were incorrect
on machines with more than 4GB RAM (and on x86_64).
PCI: added definitions for NVM Express storage controller subclass.
PCI: added FindHTCapability() for HyperTransport capabilities.

* untested.
* as it's possible to have several HyperTransport capability blocks, the offset parameter
is used as the start pointer for the search in case it's non zero.
PCI: added some missing subclasses definitions

* also renamed the HT capability mask
PCI: added a HyperTransport mapping info for x86

* fill it with HT MSI mapping capability information.
* enable/disable mapping on the device accordingly in enable_msi(),
disable_msi(), enable_msix(), disable_msix().
* untested. The mapping could instead be enabled/disabled on the HT PCI bus,
aka the parent device of the device passed to enable_msi/disable_msix().
Debugger: Cleanups for BreakConditionConfigWindow.

- When switching between breaking on all images vs a custom list, rather
than enabling/disabling the controls individually, which wasn't
necessarily obvious from a visibility standpoint, simply show/hide the
entire group of controls as needed.
- Update state of add/remove buttons properly.
- Sanitize input for leading/trailing whitespace.
Debugger: minor tweak.

Also ensure the custom image list controls are appropriately
shown/hidden when enabling the top-level stop on image load setting.
Debugger: SourceView: Fix several drawing bugs.

- MarkerView: If the view was resized, and the newly revealed region was
after the last source line, it wouldn't be redrawn properly.
- TextView: Reset low color for painting the region after the
source.  Otherwise, if either the instruction pointer or a breakpoint
were at the last line of the current source, the empty space after would
be filled in that color rather than the background color.
Terminal: work-around missing/out-of-order modifiers events

... more aggressively than before:
* Only use modifiers().
* Before forwarding any relevant event to the active state, first check
  whether the modifiers have changed.

While the issues should really be fixed where they originate (app
server?), this hopefully fixes all situations where the hyperlink mode
gets stuck.