One was missing and one was poor style.

This matches RISC-V and avoids using the binutils INSN_* namespace.

This reduces the diff to upstream.

It is impossible to have an otype outside of the valid range with the
current compression schemes. Now that we extract the otype from the
pesbt bits it's also impossible for cap_get_otype_unsigned() to return
such a value.

These helpers are not called by the TCG generated for Morello.

This removes some CAP_cc usages from the Morello helpers.

And factor out a new cap_otype_is_reserved() helper function.

After the latest refactoring we can now drop the TARGET_MORELLO branch.

Most uses appear to assume that LAST_NONRESERVED_OTYPE is a valid
non-reserved otype rather than the first reserved one. However, all of
those checks are incorrect for Morello, so we should really remove
this macro.

Performing comparisons against this constant is likely to cause bugs
since Morello allocates otypes from the bottom but MIPS/RISC-V allocate
from the top, so any less than/greater than checks are wrong for at
least one architecture.

This makes it clear that we are referring the the MIN/MAX numerical value.
Previously it was -1 to -16 for MIPS/RISC-V which meant that my
implementation of cap_otype_is_reserved() was broken since the check
`otype > 0x3ffff && otype <= 0x3fff0` is never true.

This does not corrupt other fields, but they will be truncated silently.