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

Switch to OpenCore from Clover #23

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

TheNightmanCodeth
Copy link

I took some time to update the instructions and the files in the repo to reflect the necessary changes to use OpenCore for better compatibility with Catalina.

Copy link
Owner

@drasbeck drasbeck left a comment

Choose a reason for hiding this comment

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

Hi @TheNightmanCodeth first of all, thanks A LOT(!) for creating this PR. I have been wanting to look into OC for while, but relying on my Hack I have not taken the plunge just yet. It seems now would be a terrific time to do so. =)

But before I merge your changes into the repo, could you give a quick overview of the contents of the PR?
Also mention any quirks / things that doesn't work, as compared to the standing Clover setup?

(OPTION 1 Difficult - but recommended) Make your own SSDT and DSDT:
-> First you'll need the vanilla .aml generated by your UEFI (BIOS)
When in the clover boot menu press F4 (and Fn+F4 just in case)
When in the *clover* boot menu press F4 (and Fn+F4 just in case)
Copy link
Owner

Choose a reason for hiding this comment

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

How are users reaching this menu, if they do not install Clover?

Copy link
Author

Choose a reason for hiding this comment

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

Fair enough. OpenCore doesn't have this feature, though. Maybe there will need to be a step for booting from a Clover flash drive? Not sure if there's another way to get DSDT/SSDT besides through clover.

Choose a reason for hiding this comment

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

You can grab a copy from UEFI shell or using SSDTime which is recommended from creating the various SSDTs you'll need. You can read more about it in the Dortania ACPI docs.

@Stereo8
Copy link

Stereo8 commented Oct 16, 2019

I can't get OpenCore to recognize any keyboard input with this config. I tried disabling KeySupport in config.plist, tried enabling it and removing UsbKbDxe.efi, none of it worked.

@TheNightmanCodeth
Copy link
Author

@Stereo8 I'm not sure what the problem could be. I just finished following this README step by step, in case I left something out, and it's working fine for me. Are you using a T430? Does it work with Clover? I would recommend sticking with Clover version until OC comes out of beta. UsbKbDxe.efi will not affect keyboard input.
Do you mean keyboard input on the OC screen or in Mac OS? You need to press the number of the system you are wishing to boot, not arrow keys.

@drasbeck
Copy link
Owner

@TheNightmanCodeth, awesome! I will try it out when I get back from my autumn retreat. If most users should wait with using OpenCore, I think it makes sense to keep Clover alongside OpenCore in the README.md. I shall see to that, before merging.

@Stereo8
Copy link

Stereo8 commented Oct 18, 2019

What about CPU power management? I see you're using NullCPUPowerManagement.kext here, I'd fix that before merging this


on /usr/local/bin/clover-genconfig > config.plist
execute and take the xml

#### HOW TO FLASH T430 BIOS (if you want to install an 1300AC Wi-Fi …)

Choose a reason for hiding this comment

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

You should probably add something here about the ivyra1n BIOS mods since that unlocks everything you need to really tune your hack and then some.

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