Given we use braces all the time conditional indents do not make code
cleaner.

Add attach-return-type to astyle

Run astyle with align-pointer=name

Use astyle's unpad-paren to unpad all parentheses that are not included
by pad-header, pad-oper, and pad-comma.

Instead style.sh from seL4_tools should be used

This produces minor changes

Clang does not support the rex.w instruction prefix and instead requires
sys[ret/exit] mnemonics.

Clang doesn't provide support for __attribute__((optimize ...)). There
are alternatives to provide the same functionality but due to how rarely
the kernel is compiled without optimisations, this can be added on
demand.

This change alters RISC-V fault handling to pass all non-VM faults as
user exceptions. As we cannot guarantee what the hardware will send us
in the scause register, we must handle all possible values.

It also makes fault handling more consistent with other architectures in
c_traps.c.

putConsoleChar was needless indirection around putDebugChar. Remove
this.

module_paddr_region_valid is redundant since region_size returned
by find_load_paddr is guaranteed greater than user image size.

GCC accepts MCR/MRC instructions to modify fpu control registers. Clang
does not. VMSR/VMRS are instructions that the ARM specification suggests
to use when accessing fpu control registers.

This is to allow other modules to use it as an input without rebuilding.

Add input file dependencies to CMAKE_CONFIGURE_DEPENDS for the source
directory. This will cause CMake to be rerun by ninja if any of them are
modified.

Check if output header file is older than inputs before rerunning
execute_process as this step can sometimes be long running.

configure_file only updates the target file if the source file has
changed. This will detect changes due to changing the target platform.
check_outfile_stale will check that the dtb is newer than the
dts, otherwise rebuild it.

This checks if a file is older than other files during CMake configure
phase. This is to prevent unnecessary long running execute_process
calls.

Fail with an error if the script returns an error. Additionally hook
stdio up to execute process to allow easier use of python debugging
tools if required.

This reduces chances of other parts of the script printing to stdout and
allows the compatibility strings file to be inspected or manually
changed for debugging purposes.

Updated YAML validation error message to use logging.warning,
forwarding the output to stderr. This would otherwise end up
in the CMake dts compatibility string.

This change moves all arm platform timers to drivers/timer
and renames them to the compatibility string specified in the dts.
Now the correct timer is included at build time based on the dts,
this follows the same pattern as the serial-refactor.
Have moved the arm generic_timer and priv_timer into the timer
folder and updated cmake configs appropriately.

Modify hardware_gen script to only include chosen
serial path. Added newlines to serial drivers to fix
concat issues when compiling. Move cmake macro RegisterDriver
 up a level to support timer refactor. Modify arm cmake to reflect
this.

This was causing errors with a reproducible build check in the binary
verification toolchains.

This is done because while GCC can infer the correct size of the operand
from the inline assembly, Clang cannot. Using the operand constraint by
providing "%w0" does not work with MRS and the simplest solution to use
word_t instead.

Moved imx6q-uart.c and imx31-uart.c -> imx.c and updated config.cmake
appropriately.