[Guide] Acer S3-391 Ivy Bridge with Clover

[Metalcored00d]

Note: Moved to guides subforum.

Thanks, and thank you especially for everything that you share & represent in this community!

 

[shinzon]

It is indeed an older kext, but I will try to implement the kext patching in Clover (once I figure that out)

I used this guide to use clover to patch AppleHDA in such a way that it would survive OSX upgrades. Bear in mind that this method requires a working patched AppleHDA as a prerequisite, and I'm not sure you can use ML files with this method on Yosemite.

http://www.insanelymac.com/forum/topic/298027-guide-aio-guides-for-hackintosh/page-2#entry2030060

True, I haven't added anything and this works without problem.

Would you mind sharing your DSDT.dsl? I keep pressing this point because while at first glance RehabMan's battery kext worked without the DSDT patch, I ended up getting random Null errors and such over the course of using it. Your Acer may be different than mine in this regard, however.

Thank you so much for your help here! I never tried to patch a kext on the go with clover before! Will definitely have to look more into this matter! For now I applied the patch, but I still have to replace the kext with the original one.

This is the general procedure for Clover kext patches:
After placing the original, untouched AppleFramebufferCapri in /System/Library/Extensions, reboot your machine without caches, use Kext Utility to rebuild the kext cache, reboot the computer again (this time with caches), reboot once more from the login screen (again with caches), and you should be good to go.

Edit: Looking at your original guide again, trim support can be done with a Clover kext patch too:

http://clover-wiki.zetam.org/Configuration/KernelAndKextPatches

            <dict>
                <key>Comment</key>
                <string>Trim Enabler</string>
                <key>Find</key>
                <data>
                AEFQUExFIFNTRAA=
                </data>
                <key>Name</key>
                <string>IOAHCIBlockStorage</string>
                <key>Replace</key>
                <data>
                AAAAAAAAAAAAAAA=
                </data>
            </dict>

 

[RehabMan]

...
Would you mind sharing your DSDT.dsl? I keep pressing this point because while at first glance RehabMan's battery kext worked without the DSDT patch, I ended up getting random Null errors and such over the course of using it. Your Acer may be different than mine in this regard, however.
....

What is a "Null error"?

 

[shinzon]

What is a "Null error"?

Instead of giving me a battery percentage, it'd read "null". Seemed to often occur after waking from sleep, or after plugging or unplugging the AC adapter, though not always. Patching EC to use 8bit fields fixed the issue.

 

[RehabMan]

Instead of giving me a battery percentage, it'd read "null". Seemed to often occur after waking from sleep, or after plugging or unplugging the AC adapter, though not always. Patching EC to use 8bit fields fixed the issue.

I'd have to see a screen image.

Patching EC for 8-bit is a requirement... never optional.

 

[shinzon]

Patching EC for 8-bit is a requirement... never optional.

That's strange. Why would my battery stats (mostly) work without the patch then? I will say that my machine needed only light EC edits, ERIB and SMD0 if I remember correctly.

https://github.com/RehabMan/Laptop-DSDT-Patch/blob/master/battery/battery_Acer-Aspire-E1-571.txt

 

[RehabMan]

That's strange. Why would my battery stats (mostly) work without the patch then? I will say that my machine needed only light EC edits, ERIB and SMD0 if I remember correctly.

https://github.com/RehabMan/Laptop-DSDT-Patch/blob/master/battery/battery_Acer-Aspire-E1-571.txt

Simple. It mostly worked because it was only partially broken.

 

[shinzon]

Simple. It mostly worked because it was only partially broken.

Haha, makes sense.

 

[Metalcored00d]

I used this guide to use clover to patch AppleHDA in such a way that it would survive OSX upgrades. Bear in mind that this method requires a working patched AppleHDA as a prerequisite, and I'm not sure you can use ML files with this method on Yosemite.

http://www.insanelymac.com/forum/topic/298027-guide-aio-guides-for-hackintosh/page-2#entry2030060

Tried this and can confirm this works!

This is what I added to my config.plist:

<dict>
                <key>Name</key>
                <string>AppleHDA</string>
                <key>Comment</key>
                <string>Expect 4 matches</string>
                <key>Find</key>
                <data>hBnUEQ==</data>
                <key>Replace</key>
                <data>aQLsEA==</data>
            </dict>
            <dict>
                <key>Name</key>
                <string>AppleHDA</string>
                <key>Comment</key>
                <string>Expect 4 matches</string>
                <key>Find</key>
                <data>hAjsEA==</data>
                <key>Replace</key>
                <data>AAAAAA==</data>
            </dict>

Would you mind sharing your DSDT.dsl? I keep pressing this point because while at first glance RehabMan's battery kext worked without the DSDT patch, I ended up getting random Null errors and such over the course of using it. Your Acer may be different than mine in this regard, however.

No problem. It's the .aml file, hope that's fine too?
View attachment DSDT.aml

Looking at your original guide again, trim support can be done with a Clover kext patch too:

http://clover-wiki.zetam.org/Configuration/KernelAndKextPatches

            <dict>
                <key>Comment</key>
                <string>Trim Enabler</string>
                <key>Find</key>
                <data>
                AEFQUExFIFNTRAA=
                </data>
                <key>Name</key>
                <string>IOAHCIBlockStorage</string>
                <key>Replace</key>
                <data>
                AAAAAAAAAAAAAAA=
                </data>
            </dict>

I will add this to my clover config.plist too! Thank you so much for all your help!!

 

[shinzon]

No problem. It's the .aml file, hope that's fine too?
View attachment 138480

Thanks for sharing.

You've got a ton of stock methods compared to mine. Would have expected our DSDTs to at least be somewhat based on the same code, but your's is almost completely different.

That said, I had a look at EmbeddedControl and there are no feilds larger than 8bits (in fact, there is only a single 8bit feild and the rest are smaller), so you don't need to do anything to get working Battery stats, as you said.

Couple of minor nitpicks:

You don't seem to be running a Darwin patch. If you're not adding Darwin to Method _INI with Clover (I generally stay away from Clover DSDT fixes and try to do things manually when possible), you'll need to apply that patch from RehabMan's repo.

https://github.com/RehabMan/Laptop-DSDT-Patch/blob/master/system/system_OSYS.txt as windows xp

You can edit Windows 2001 in that patch to be any of the appropriate options found in Method _INI. As a whole, this code doesn't seem to do much on your machine. Your DSDT has no conditionals for 0x07D9, Windows 2009 (Windows 7), so it does nothing different than the current code paths. There are a few conditionals that check for 0x07D6, Windows 2006 (Vista). The three Windows 2001 (XP) options have a couple of conditionals between them. If it were me, I'd probably edit the patch to apply for Windows 2009, even though it doesn't effect much.

Also, it doesn't look like you have the MCHC patch.
https://github.com/RehabMan/Laptop-DSDT-Patch/blob/master/system/system_MCHC.txt

For giggles you might try foregoing the ACPI_Backlight.kext by building an injector to use the stock AppleBacklight.kext. I followed this simple guide to get that working on my machine (use v2 script):

http://www.tonymacx86.com/hp-proboo...screen-using-patched-applebacklight-kext.html

Otherwise, looks good; great work.