svc: Provide more detailed error logs for svc functions

renderer_opengl: Update OpenGL backend version requirement to 4.6

video_core: Delete morton

moron.h & morton.cpp are not used anywhere and are just empty files

lm: Fix ReadLeb128

pl_u: Fix read out of bounds

Fixes assertion on Bloodstained Ritual of the Night.

We would over read sometimes, this is fixed by checking if the top bit is set in the first iteration. We also lock the loop off to be only the max size of the type we can fit. Finally we changed an incorrect print of "DEBUG" to "TRACE" to reflect the proper log severity

ring_buffer: Remove granularity template argument

k_address_arbiter: Minor cleanup

Non-obvious bug in RingBuffer::Push(std::vector<T>&) when granularity != 1

Just remove it altogether because we do not have a use for granularity != 1

HID: Update the HID service to match more closely to switchbrew part 1

Allows for more descriptive error messages and also doesn't hide
control-path exit returns from the reader.

This is a useful function in a generic context or with types that
overload unary operator&. However, primitives and pointers will never do
this, so we can opt for a more straightforward syntax.

This code is never used, so we can remove it. It's in version control,
so it can always be brought back when needed.

Allows SVC calls to have much more informative information during error
cases. This also doesn't hide control flow returns from the reader.

Kernel Rework: Refactor KEvent/KReadableEvent/KWritableEvent

k_priority_queue: Minor cleanup

IApplicationFunctions: Implement GetHealthWarningDisappearedSystemEvent

video_core: host_shaders: Don't pass --quiet to glslangValidator if unavailable

k_affinity_mask: Avoid implicit truncation to bool

This can cause compiler warnings. Instead, we can explicitly add a
boolean expression around it to naturally turn the result into a bool.

Given these are only used as function existence checks, we can simplify
some usages of declval, given they aren't particularly useful here.

Reduces a few template instantiations, which at most reduces compile
times a tiny bit.

We can make use of the _t variants of the templates to cut down on a
little bit of verbosity.

An identifier containing a starting underscore followed by a capital
letter is reserved by the standard. It's trivial to avoid this by moving
the underscore to the end of the identifier.

While the likelihood of clashing here being minimal, we can turn a
"should not break" scenario into a definitive "will not break" one, so
why not?.

key_manager: Create the keys directory if it does not exist

config: Always update handheld config