Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BDM: Make loading block device modules other than USB optional #444

Merged
merged 1 commit into from
Jun 25, 2021
Merged

BDM: Make loading block device modules other than USB optional #444

merged 1 commit into from
Jun 25, 2021

Conversation

KrahJohlito
Copy link
Member

@KrahJohlito KrahJohlito commented Jun 24, 2021

Pull Request checklist

Note: these are not necessarily requirements

  • I reformatted the code with clang-format
  • I checked to make sure my submission worked
  • I am the author of submission or have permission from the original author
  • Requires update of the PS2SDK
  • Requires update of the gsKit
  • Others (please specify below)

Pull Request description

This is just to make loading of additional block device modules optional, I tested and it works fine.

I did originally have an option for USB on/off also but it was more hassle than its worth as it appears one set of modules must be loaded otherwise I would just get BSOD when loading BDM modules..so the obvious choice is to have USB as the default block device and all others IEEE 1394, MX4SIO & UDP optional.

I don't see any point in adding language strings for names of things, devices etc since..they would always be the same so FireWire is hardcoded..as seen below.
bdm

@AKuHAK
Copy link
Member

AKuHAK commented Jun 24, 2021

I still like to see that any new GUI option is mentioned somewhere in some readme file :)

Copy link
Member

@uyjulian uyjulian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks OK codewise

@rickgaiser
Copy link
Member

Thank you @KrahJohlito , merging.

@rickgaiser rickgaiser merged commit 57d7477 into ps2homebrew:master Jun 25, 2021
@KrahJohlito KrahJohlito deleted the bdm branch July 3, 2021 02:13
AKuHAK pushed a commit that referenced this pull request Sep 30, 2021
BDM: Make loading block device modules other than USB optional
citronalco pushed a commit to citronalco/OPL-Daily-Builds that referenced this pull request Sep 10, 2023
BDM: Make loading block device modules other than USB optional
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants