Merge pull request #559 from gameblabla/cdrom_various_fixes

Various CDROM fixes from Duckstation and Mednafen

Use the "magic numbers" from Duckstation for CDLPause

We need to make sure not to break Gundam Battle Assault 2 in PAL mode
as otherwise it will lockup.
Amusingly, Mednafen currently cannot run this game in PAL mode.

Fix CdlGetTD according to Mednafen's implementation.

CdlGetmode is GetParam, fix it according to Mednafen's implementation

The result size for GetParam (previously Getmode) should be only 5 and done this way.

Here's Mednafen implementation :
https://github.com/libretro-mirrors/mednafen-git/blob/1899500078169e787f86eb5302a3f35abdaa8764/src/psx/cdc.cpp#L2095

Nocash's documentation also shows that our previous implementation was wrong :
https://problemkaputt.de/psx-spx.htm
(See CDROM Controller Command Summary section or CDROM - Status Commands)

Don't throw an error for CdlGetQ.

I doubt this is used by any game but who knows.

See Mednafen for reference :
https://github.com/libretro-mirrors/mednafen-git/blob/1899500078169e787f86eb5302a3f35abdaa8764/src/psx/cdc.cpp#L2527

Improve SetLoc CDROM command from Duckstation

See Duckstation's CDROM for reference :
https://github.com/stenzek/duckstation/blob/18241978ea056a81ff9847f0ad89658f886153ea/src/core/cdrom.cpp#L1087

CdlSync should be NULL and return an error.

This has been confirmed by both Mednafen and nocash's documentation.

Merge pull request #558 from gameblabla/cdrom_fixes_libretro_popo

Merge several fixes from PCSX Redux and adjust delay for SetLocPending.

Merge several fixes from PCSX Redux and adjust delay for SetLocPending.

There's a game, PoPoLoCrois Monogatari II, that unfortunately locks up
during the intro screen.
I should have known that code was wrong as Mednafen did not have anything
like that in their code either, hence the confusion.

Their fix however still don't include the Driver fix so the game would still
crash if we don't have the "+ Seektime".
To be honest, i'm not sure why the PCSX Reloaded team did it this way...

I noticed that the fastforward and fastbackward code was pretty much unused.
Looked at Mednafen and all they do is just adjust the cursector
and make sure that fastword & backword trigger the AUTO_REPORT code
so i did the latter.

There was also a mysterious leftover code from PCSX Reloaded in Cdlplay:
that had a condition that forced it to SEEK_DONE
(with a comment even saying that it should be set to SEEK instead)
so i set it to SEEK_PENDING instead.
(Mednafen's code seems to suggest so as well)

Fix regression caused by #9361a5aa4fef73e06bc78b7c462eb28a7d12e3d0

Not sure why it caused that but it causes the function to fail.
I don't want to wait months for him to fix it properly
so let's just do that for now.

Merge pull request #553 from gameblabla/football_fix_libretro

CDROM: Rename Reset+Init commands, fix "This is Football 2" lockup

CDROM: Rename Reset+Init commands, fix "This is Football 2" lockup

Upon the reset command being issued, we need to set the Mode to 0x20
(as it was confirmed in Mednafen) and also disable the muting.

We also change the seektime for the reset command appropriately.
Please see Mednafen's implementation as reference :
https://github.com/libretro-mirrors/mednafen-git/blob/1899500078169e787f86eb5302a3f35abdaa8764/src/psx/cdc.cpp#L2007

This fixes the black screen issue in "This is Football 2".
This also fixes the black screen issue in Pooh's Party
after you selected a character.

Properly fix the extern to const void instead of just void.

Merge pull request #551 from gameblabla/hle_compil

Fix broken HLE PR

Fix broken HLE PR.

Merge pull request #543 from gameblabla/hle_fies_corruption_libretro

Properly protect the HLE instructions against corrupted memory.

Merge pull request #547 from gameblabla/blues_clues_fix_audio_fmv

CDROM: Ignore sectors with channel number 255

CDROM: Ignore sectors with channel number 255

This was inspired by the fix in Duckstation :
https://github.com/stenzek/duckstation/commit/0710e3b6d384526ed939f742f8f657623bb354bb

Some games have junk audio sectors with a channel number of 255.
If these are not skipped, then they will play wrong file.

This was tested on "Blue's Clues : Blue's Big Musical" and
it fixed the missing audio there.

Taxi 2 is also said to be affected by this.

Merge pull request #544 from gameblabla/gpu_hle_rewrite_libretro

Rewrite GPU bios functions according to OpenBIOS.

Merge pull request #542 from gameblabla/mdec_fix

mdec.cpp: Dare to set MDEC_BIAS to 10

Merge pull request #541 from gameblabla/spu_fix_libretro

Fixes Diablo 1 SPU bug properly and remove Diablo hack.

Rewrite GPU bios functions according to OpenBIOS.

See PCSX-Redux :
https://github.com/grumpycoders/pcsx-redux/blob/93653ba5281487d3bed57371d7b64c32dfc669f0/src/mips/openbios/gpu/gpu.c

Properly protect the HLE instructions against corrupted memory.

Fix is from PCSX-redux :
grumpycoders/pcsx-redux@99c9508

This should hopefully fix HDHOSHY's experimental patch properly.

Co-authored-by: Nicolas Noble <nicolasnoble@users.noreply.github.com>

Fixes Diablo 1 SPU bug properly and remove Diablo hack.

(Mostly) inspired by the fixes done in PCSX Redux :
https://github.com/grumpycoders/pcsx-redux/blob/93653ba5281487d3bed57371d7b64c32dfc669f0/src/spu/registers.cc#L504

It seems that there was an attempt to initially implement it
in PCSX Reloaded (hence the bIgnoreLoop in freeze.c) but it was never
implemented properly.

Co-authored-by: Nicolas Noble <nicolasnoble@users.noreply.github.com>

Merge pull request #539 from gameblabla/spustate_register_fix_libretro

[SPU] Emulate SPUSTAT[5:0] as a mirror of SPUCNT[5:0]

Merge pull request #540 from gameblabla/gte_senquack_fix_libretro

gte: Fix gteH division and sign extension (from PCSX4ALL)

gte: Fix gteH division and sign extension (from PCSX4ALL)

gteH register is u16, not s16. DIVIDE macro/func assumed it was
s16 for some reason. Behavior now matches Mednafen.

Co-authored-by: senquack <dansilsby@gmail.com>

[SPU] Emulate SPUSTAT[5:0] as a mirror of SPUCNT[5:0]

Fix is from PCSX-Redux :
grumpycoders/pcsx-redux@4e905d7

I have tested the fix against Loonies 8192 (a PSn00bSDK made homebrew game) and it no longer
locks up during loading.

Co-authored-by: wheremyfoodat <44909372+wheremyfoodat@users.noreply.github.com>

Merge pull request #538 from gameblabla/fix_cdom_savestate

Fix save state compatibility that was broken by my CDrom timing changes.

mdec.cpp: Dare to set MDEC_BIAS to 10

This fixes graphic artifacts during cinematics in :
- Vandal Hearts
- R-Types
- Galerians

Dmitrysmagin had a similar fix in PCSX4ALL but he set it to 6.
It turns out MDEC_BIAS set to 6 was still not enough for Galerians
to look properly so it was set to 10.
It seems so far that other games are unaffected.

Co-authored-by: Dmitry Smagin <exmortis@yandex.ru>

Fix save state compatibility that was broken by my CDrom timing changes.

Merge pull request #537 from embercold/warningless-build

Sort out compiler warnings, all of them

Sort out compiler warnings, all of them

Merge pull request #536 from gameblabla/cdrom_fixes

CDRom timings changes

CDRom timings changes

Upstreamed changes from PCSX Redux as seen here :
https://github.com/grumpycoders/pcsx-redux/commit/5730e04f0183f37038bc1d133cf9f9092425b90a

This apparently fixes Deus Fight in Xenogears.

Co-authored-by: Nicolas Noble <nicolasnoble@users.noreply.github.com>

Merge pull request #532 from davidgfnet/master

Fix initialization on Frontends without variable support.

Fix initialization on Frontends without variable support.

Frontends that do not support RETRO_ENVIRONMENT_GET_VARIABLE (like
miniretro & nanoarch) cause a segfault in the CPU emulation path unless
the BIOS file is present. This is due to the emulator not supporting
running without a BIOS in dynarec mode and the code not being able to
swap CPU types (unless the callback returns true).

Merge pull request #530 from embercold/fix-div-insn

Handle DIV overflow in lightrec and the interpreter

Handle DIV overflow in lightrec and the interpreter

Attempt to fix Android build

Merge pull request #529 from negativeExponent/nodebugger

Do not compile debugger codes by default

Do not compile debugger codes by default

- Reduces binary to about half the size
- Compile with DEBUG=1 to build core with debug code

Merge pull request #522 from negativeExponent/patch-2

Set inputs to use "standard" by default

Set inputs to use "standard" by default

Merge pull request #520 from negativeExponent/multitap_option_description

Update multitap option description

These are now unnecessary

Update multitap option description

Merge pull request #517 from negativeExponent/input

Fix reporting for connected or unconnected inputs

Merge pull request #516 from negativeExponent/vfs

start adding libretro vfs

Fix reporting for connected or unconnected inputs

Current implementation assumes all inputs are plugged, regardless of
input type settings, so input reporting in games also says its
connected.

This PR fix this minor issue, by sending appropriate input register
values when no controllers are attached.

Related issue (what i used as well for testing):
https://github.com/libretro/duckstation/issues/35

Fix fgets param

Add libretro VFS and use VFS for windows target

- Reverts the use of fopen_utf8 from previous commit and use
  "file_stream_transforms.h" instead: \7fhttps://github.com/libretro/pcsx_rearmed/commit/270c6dd14f876b8a67929aa22abefc47f4588324#diff-2065d1414bac1198fb423574b4874d5bf92a2114a2f3e235c469d8fd6288dd10

- Enable the use of VFS for windows by default.

- compile with USE_LIBRETRO_VFS=1 to enable vfs when compiling for other
  platforms (including android if needed)

Merge pull request #514 from negativeExponent/fopen_utf8

libchdr: Use also fopen_utf8 here

Merge pull request #513 from negativeExponent/simplity_multitap_core_option

Simplify multitap core options

Buildfix for Apple

Merge pull request #512 from negativeExponent/update_libchdr

Update libchdr

implify multitap core options

- Combines multitap1 and multitap2 core options
- Removes "automatic" mode

libchdr: Use also fopen_utf8 here

RE: https://github.com/libretro/libretro-fceumm/issues/448

Android: Update jni

3ds: Buildfix

Restore deps/mman

Accidentally purged in last commit, needed for lightrec/lightning

Update Makefile

git subrepo clone (merge) https://github.com/rtissera/libchdr deps/libchdr

subrepo:
  subdir:   "deps/libchdr"
  merged:   "a17c0da"
upstream:
  origin:   "https://github.com/rtissera/libchdr"
  branch:   "master"
  commit:   "a17c0da"
git-subrepo:
  version:  "0.4.3"
  origin:   "???"
  commit:   "???"

Cleanup deps

Merge pull request #511 from negativeExponent/updates

Selectively replace fopen() with fopen_utf8()

Add missing files

Selectively replace fopen() with fopen_utf8()

This selectively replace fopen calls with fopen_utf8, to handle paths
that uses special characters in Windows.

The replacement covers these:
- opening of bios file
- opening, writting and creating memcard2
- opening and handling of disk images

Related: https://github.com/libretro/libretro-fceumm/issues/448

Get rid of interpreter name

Add ios-arm64/tvos-arm64, ios9, and osx-arm64. osx-arm64 is interpreter
for now (lightrec crashes at startup)

Merge pull request #506 from jdgleaver/libchdr-update

Update libchdr (replace libflac with dr_flac)

Update libchdr (replace libflac with dr_flac)

Merge pull request #502 from justinweiss/restrict-threaded-rendering-drivers

Restrict threaded rendering to unai / peops

Restrict threaded rendering to unai / peops

Merge pull request #500 from bslenul/master

Change HLE BIOS message. Please review first!

Change HLE BIOS message.

Merge pull request #499 from negativeExponent/fix_input

multitap option fix

Do not hide multitap core options

Set multitap options disabled by default, prevent previous defaults from being accidentally loaded instead.

Merge pull request #498 from glebm/patch-1

libretro: Add support for ps1_rom.bin (PS3 BIOS)

libretro: Add support for ps1_rom.bin (PS3 BIOS)

Disable lightrec for libnx

Requires the impl. of the rw/rx patches first

Merge pull request #487 from liberodark/patch-1

add H5 + N2 support

add H5 + N2 support

Update .gitlab-ci.yml

Update .gitlab-ci.yml

Update .gitlab-ci.yml

Merge pull request #483 from webgeek1234/master

Android Updates

libretro: android: remove flag allowing text relocations

This is not allowed on api 23+

libretro: android: Fix non-armv7 builds

Create .gitlab-ci.yml

Update

Merge pull request #479 from dev-0x7C6/master

Removed conditional include on non-existent file.

Removed conditional include on non-existent file.

Signed-off-by: Bartłomiej Burdukiewicz <bartlomiej.burdukiewicz@gmail.com>

Merge pull request #476 from vanfanel/master

Add Rpi4 platform in 64bit mode.

Add Rpi4 platform in 64bit mode.

Merge pull request #474 from negativeExponent/ios_interpreter

iOS: Another try at interpreter override

Merge pull request #475 from negativeExponent/multitap

Set multitap options to "disabled" by default

Set multitap options to "disabled" by default

Due to moving input selector to frontend instead from the core options,
ports 3-8 can be always set to standard causing multitap to be auto-activated.
With multitaps enabled, some games would misbehave or inputs not working
at all for those that are not multiplayer capable.

Setting core option multitap1 and multitap2 to disable for now.

iOS: Another try at interpreter override

Merge pull request #473 from negativeExponent/chd_image_cache

CHD: Support for precache mode CD access method