kernel/scheduler: Use std::mutex instead of spin lock

Remove bad and useless packages from LGTM build

cmake: Update to Qt 5.12.8

bsd: Resolve a few warnings

audio_core: Apollo Part 1, AudioRenderer refactor

It still fails due to CMake version being 3.13.4, but at
least we are not ones to blame now.

Project Mjölnir: Part 2 - Controller Applet

configure_input_player: Resolve sign conversion warnings in UpdateMappingWithDefaults()

video_core: Remove all Core::System references in renderer

gc_adapter: Make DeviceConnected() a const member function

This doesn't modify instance state, so it can be made const.

Previously the address provided to SendToImpl would never be propagated
to SendTo(). This fixes that.

Same behavior, less typing.

We can simplify this a little by explicitly specifying the typename for
the lambda function.

game_list_p: Mark some constants as constexpr

Now that the GPU is initialized when video backends are initialized,
it's no longer needed to query components once the game is running: it
can be done when yuzu is booting.

This allows us to pass components between constructors and in the
process remove all Core::System references in the video backend.

 services: Implement most of bsd:s and GetCurrentIpAddress from nifm

hid: Implement MergeSingleJoyasDualJoy

- Used in multiple games such as Super Mario Odyssey.

Resolves -Wsign-compare and -Wunused-variable

- Some games like Shipped have a minimum requirement of 0 connected players and is undesired behavior. We must require a minimum of 1 player connected regardless of what games may ask.

Now left and right joycons have the same priority (meaning both needs to be supported by the game).

Explanation of the new heuristic:
Assign left joycons to even player indices and right joycons to odd player indices.
We do this since Captain Toad Treasure Tracker expects a left joycon for Player 1 and a right Joycon for Player 2 in 2 Player Assist mode.

Implement the fallback applet for the SDL frontend, connecting only the minimum amount of players required.

This avoids unintentionally changing the states of elements while loading them in.

Avoids repetitive usages of the int literal '8' or calls to player_widgets.size()

"Explain Text" is additional text that is shown for each player in the controller applet.

Co-authored-by: Its-Rei <kupfel@gmail.com>

Port citra-emu/citra#5495: "Add LGTM static analyzer config file"

externals: Update Xbyak to 5.96

gc_poller: Resolve compilation warnings on MSVC

common_funcs: Add missing XOR operators to DECLARE_ENUM_FLAG_OPERATORS

hle/scheduler: Fix data race in is_context_switch_pending

async_shaders: Mark getters as const member functions