Add openbios.bin to mednafen_psx_libretro.info, and mednafen_psx_hw_libretro.info

[davidhedlund]

Users should be notified whether openbios.bin file has been loaded from the System/BIOS directory under Main Menu → Information → Core Information the under Firmware section.

Fixes:

• Feature Request: Add OpenBIOS status to Core Information beetle-psx-libretro#979

Main Menu → Information → Core Information results

With openbios.bin added to the System/BIOS directory

Indicates that external OpenBIOS is used:

Without openbios.bin added to the System/BIOS directory

Indicates that the inbuilt version of OpenBIOS is used:

Additional information

If these commits are approved, openbios.bin should also be added to:

• https://github.com/libretro/libretro-super/blob/master/dist/info/pcsx_rearmed_interpreter_libretro.info
• https://github.com/libretro/libretro-super/blob/master/dist/info/pcsx_rearmed_libretro.info
• https://github.com/libretro/libretro-super/blob/master/dist/info/pcsx_rearmed_neon_libretro.info
• https://github.com/libretro/libretro-super/blob/master/dist/info/swanstation_libretro.info

Reviewers

@hizzlekizzle @ZachCook

[davidhedlund]

Considerations

Region

Currently, it displays "(PS1 OpenBIOS)". But it is a multi-region BIOS that even supports Asian regions (corresponding to scph5003.bin) so should it instead display "(PS1 AS/EU/JP/US OpenBIOS)"?

MD5sum

It displays md5: varies because the BIOS constantly updates, which is inherent to how it is downloaded and built.[1] Alternatively, the description line can be omitted entirely, resulting in this format:

References

1: How do download openbios.bin

• As noted on https://github.com/grumpycoders/pcsx-redux, the latest AppImage is available at https://distrib.app/pub/org/pcsx-redux/project/dev-linux-x64.
• Extract the Appimage: ./PCSX-Redux-HEAD-x86_64.AppImage --appimage-extract.
• Copy the openbios.bin file from the generated directory squashfs-root/usr/share/pcsx-redux/resources.
• Place openbios.bin in your RetroArch System/BIOS directory.

[ZachCook]

Another somewhat related consideration, is if the region specific bios files really still need to be called required now that the builtin openbios is available as a fallback.

[davidhedlund]

Another somewhat related consideration, is if the region specific bios files really still need to be called required now that the builtin openbios is available as a fallback.

Yeah! OpenBIOS is inbuilt and supports multiple regions. But can you please give feedback to this before I change firmware*_opt values from false to true:

Region

Currently, it displays "(PS1 OpenBIOS)". But it is a multi-region BIOS that even supports Asian regions (corresponding to scph5003.bin) so should it instead display "(PS1 AS/EU/JP/US OpenBIOS)"?

[ZachCook]

I don't know the conventions for the descriptions, but it could be made to match PSP and PS3 and be (OpenBIOS PS1 BIOS)

[davidhedlund]

I don't know the conventions for the descriptions, but it could be made to match PSP and PS3 and be (OpenBIOS PS1 BIOS)

Yeah, let's keep the current revision for that.

[ZachCook]

typo in PR description openbiox.bin -> openbios.bin

[davidhedlund]

Another somewhat related consideration, is if the region specific bios files really still need to be called required now that the builtin openbios is available as a fallback.

Fixed:

• 61cb08a
• 6960fb4

[davidhedlund]

typo in PR description openbiox.bin -> openbios.bin

Fixed. Thanks for all the helpful feedback!

[ZachCook]

I also need your feedback on this @ZachCook

MD5sum

It displays md5: varies because the BIOS constantly updates, which is inherent to how it is downloaded and built.[1] Alternatively, the description line can be omitted entirely, resulting in this format:

See the main post for the screenshot with md5: varies to compare them.

I think md5: varies is fine, it hints to the user that there is no single openbios.bin version, another option that kinda fits the spirit of this is mentioning the source file location the user can use to compare checksums.

[davidhedlund]

I think md5: varies is fine, it hints to the user that there is no single openbios.bin version

Thank you!

another option that kinda fits the spirit of this is mentioning the source file location the user can use to compare checksums.

The problem is that the text is static in the info files. But it is not a problem because system directory is displayed at the top. See Looking in:

[davidhedlund]

I think the PR is ready to get merged now but I wait for @hizzlekizzle to review it too.

[ZachCook]

I was thinking about a link to pcsx-redux or https://docs.libretro.com/library/beetle_psx/#bios, maybe just "see bios section in the docs"

Then a future improvement is some instructions can be added to https://github.com/libretro/docs/blob/master/docs/library/beetle_psx.md and beetle_psx_hw.md to help users extract the openbios.bin

[davidhedlund]

Then a future improvement is some instructions can be added to https://github.com/libretro/docs/blob/master/docs/library/beetle_psx.md and beetle_psx_hw.md to help users extract the openbios.bin

Yeah, I've mentioned how to do that in multiple issues over the years (including #2029 (comment)) but I never documented it. So I just submitted this PR:

• Update beetle_psx.md, beetle_psx_hw.md, and pcsx_rearmed.md: Added a How to obtain OpenBIOS section docs#1170

Thanks!