Merge pull request #569 from gameblabla/gte_stalling_libretro

GTE stalls/timings

Merge pull request #571 from gameblabla/cdrom_audio_fix_attempt

Fix stutter before speech samples in Castlevania Symphony of the Night

Hack fix for CDROM timings regression (Castlevania, Megaman X4)

See https://github.com/libretro/pcsx_rearmed/issues/557.

The CDROM timings changes from Redux fixed a bunch of games
(Crash Team Racing's intro no longer cuts off too early,
Worms Pinball gets further instead of just crashing,
FF8 Lunar Cry FMV no longer crashes etc...)
but it broke the other games that relied on ADPCM samples,
which is most noticeable in games like Castlevania or Megaman X4.

According to nicolasnoble, we should not cause a delay if seek destination is close to the current location.
However, this would require a lot more work so in the meantime,
let's do just that and add a comment.

Merge pull request #572 from gameblabla/cd_volume_fix_lib

Fix CD Volume issue in Star Wars Dark Forces

Merge pull request #564 from jdorigao/rpi1

Added rpi1 to makefile

Fix CD Volume issue in Star Wars Dark Forces

CD Volume is 16-bits signed, not unsigned.
Otherwise in Star Wars - Dark Forces :
if you lower the music volume slider all the way down,
the volume will wrap around and instead be set at the highest volume.

GTE stalls/timings

This is to fix an issue with Battle Arena Toshinden 1 going too fast.
Zero Divide is also said to be affected by this as well.

Note that this doesn't fix it for gte_arm.S :
We would still need to increase the cycle count here.

Co-authored-by: firnis <firnis@ya.ru>

Merge pull request #570 from Sanaki/drop_ps3

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

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

This reverts commit c5c20552af08fd4158cc0e7894c1798d95f0d35b.

Merge pull request #567 from gameblabla/fix_minor_close_iso

Fix minor issue when closing ISO.

Fix minor issue when closing ISO.

Merge pull request #566 from gameblabla/neon_fix_libretro

Always look up verify_dirty literals from offsets by neonloop

Always look up verify_dirty literals from offsets by neonloop

Literals are deduplicated, so there's no guarantee they will be stored
next to each other, even if they're written sequentially. verify_dirty
and get_bounds must use the offsets on each instruction, instead of
assuming values are stored sequentially.

Thanks neonloop for the fix

Co-authored-by: neonloop <neonloop@pm.me>

Added rpi1 to makefile

drflac - assume DRFLAC_NO_STDIO by default

Merge pull request #563 from gameblabla/remove_junk

Remove junk files and fix lack of clearing icache when loading new games

Remove junk files and fix regression in icache mode with Armored Core.

Merge pull request #562 from gameblabla/cdrom_fixes_libretro_precautions

Revert some changes as they were not tested or are not required

Revert some changes as they were not tested or are not required

Per feedback from notaz, thanks to him.

Merge pull request #545 from gameblabla/icache_emulation

ICache emulation (Interpreter) from PCSX redux

Fix comment to be more precise about lightrec's status.

Culling off cache bits from the hardware addresses.

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

Interpreter fixes for BREAK and a few other instructions.

This should fix F1 2000. (but it's possible that game
needs some tweaks to the CDROM timings as it's quite sensitive to them)

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

[Interpreter] Link even if branch is not taken in BGEZAL/BLTZAL

Source :
grumpycoders/pcsx-redux@c1a0569

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

OpenBus changes from PCSX-Redux

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

Merge Icache emulation from PCSX Redux + Senquack changes from PCSX4ALL

See (Redux) :
https://github.com/grumpycoders/pcsx-redux/commit/1923ce5

See original icache implementation (mirror of PCSX Reloaded):
https://github.com/gameblabla/pcsxr/commit/6916151

Without icache emulation, F1 2001 will greatly misbehave :
if you accelerate, the car will go around like crazy.
With icache emulation, it works as intended.

Note that our code is slightly different from theirs as i found out that
having the icache arrays in psxregs would cause crashes so instead
what i'm doing is to taking them out of there and only allocating them
on the heap (due to their great size).

Known issue(s) : Instruction cache emulation code fails to run
Spyro 2 PAL version. (or is it because of SBI?)

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

Merge pull request #552 from libretro/fp_fix_libretro

Implement fix from Mednafen for Fantastic Pinball Kyuutenkai.

Merge pull request #561 from gameblabla/sony_armored_fix_libretro

Fix for Armored Core misdetecting a Link cable being detected

Fix for Armored Core misdetecting a Link cable being detected

For some reason, the game detects that a link cable is plugged in
and disables the local multiplayer as a result.

Thanks @sony for fixing the issue in their PS Classic branch, a
simpler fix is done here instead.

Co-authored-by: Sony

Update custom-issue-report.md

Point out that people should try on PCSX-Redux, PCSX Reloaded... as well in case we suffer from a regression or forgot to merge a fix.

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.

Implement fix from Mednafen for Fantastic Pinball Kyuutenkai.

This is taken from the way Mednafen implemented it.
https://github.com/libretro-mirrors/mednafen-git/blob/563b72e377fa8284559df4df0271108d4935c9f1/src/psx/cdc.cpp#L941

This properly fixes the freeze issue in Fantastic Pinball Kyuutenkai.
Thanks notaz for assisting me with the code.

It seems not to work properly on ARM though ?

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)