Jump to content

SOLVED -- EC -Not sure if I get this?


makk
 Share

29 posts in this topic

Recommended Posts

I would like to know if my EC is correctly set and working


As far as I understand this correctly as long as the AppleACPIEC is not attaching to EC0?

 

ECEnabler question.  When this is enable what should be the indicator to show it is working?

 

The config.plist has battery patches.

 

SSDT-EC.aml is loading

Rename EC0 to EC is enabled.

ECEnabler should be enabled but currently off, due to misnomers on how this is supposed to work.

 

Configurations tried.

 

A- With ECEnabler.kext enabled,  rename disabled, SSDT-EC.aml loading, no Battery indicator, no trackpad.

But USB splits to 2.0 and 3.0.

 

B- With ECEnabler.kext disabled, rename enabled, SSDT-EC.aml loading, Battery works, trackpad on

but USB is 3.0 only.

 

The screen shot is with the latter configured state.

 

C- With ECEnabler.kext enabled, rename enabled, SSDT-EC.aml loading, have to check, don't recall. 

 

 

Screen Shot 2021-09-27 at 3.48.12 AM.png

Edited by makk
SOLVED
Link to comment
Share on other sites

I think there is a misunderstanding. As far as I know, ECEnabler.kext doesn't enable EC. It enables macOS to read Data from EC.

So therefore, SSDT-EC.aml needs to be enabled, so that PNP0C09 is present.

 

If Battery Status Indicator doesn't work afterwards then you must use the old methods: binary renames and a SSDT-Battery.aml

Link to comment
Share on other sites

First of all: I know that! I just wanted to use a more understandable language. To quote the dev: "ECEnabler is a Lilu plugin which allows macOS to read any size of EC field, meaning that no more ACPI patches are needed to split EC fields up within ACPI. I got tired of trying to make a complicated automated ACPI battery patcher […] I eventually had the idea to try patching macOS instead, which turns out to be much easier. "

 

Second of all. The point is: ECEnabler.kext and SSDT-EC.aml are required in order to make the combo work. I didn't say that SSDT-EC creates PNP0C09 – don't twist my words around!

Link to comment
Share on other sites

Ok, I'm more confused.

 

So what's the correct way on My HP Probook 450G2?

 

SSDT-EC.aml - without this no boot?  obvious

EC0 to EC Rename?  without this no Battery and no Trackpad.

 

The Vanilla DSDT has Embedded Controller EC0 and it is not showing 

like your screen shot above: EC 08. Completely different sets with (random letters here with 8, some 16, some 1)

 

WIthout this as mentioned no Battery, trackpad, and EH01 

 

What can be done so the Fake EC loads and all the above works?

 

The screen shot attached shows IOREG

BAT0

BATC

BATZ

 

Disabling EC0 to EC Rename, BATC does not engage.

BATC is where the Battery, Trackpad and another thing is using.

But USB2.0 and USB3. splits correctly or is correctly showing

 

The screen shot from System Report shows USB 3 only when using the Rename of EC0 to EC.

Without the Rename both USB2 and USB 3 are in place.

 

In the Vanilla DSDT shows BAT0 and BAT1

 

In config.plist have a rename to BAT0,1 to BATC -- Notification patch

 

I've tried all manner of enable disable, different versions of SSDT-EC and to no avial, cannot get the Fake EC to connect

and bring up Battery and Trackpad.

 

Done this for days and hours and the only option I am left with in the current configs, is

 

SSDT-EC to enable and load to boot the OS.

EC0 to EC Rename enabled.

 

I've tried the ECEnabler to see if the Battery would be read and have Battery and Trackpad.

 

I even disabled the EHC1 to EH01, all the battery patches, all manner of combinations. But something is still missing

to bring is all together.

 

Thank you Herve' 

 

I understand 5T33Z0 what you mean. 

 

I think I read where 1REVENGER1 had said the new version reads from 1 byte?  1.02 

 

From the post below

 

ECEnabler

Allows reading Embedded Controller fields over 1 byte long, vastly reducing the amount of ACPI modification needed (if any) for working battery status.

Supports Lion to Monterey (10.7-12)

 

 

Screen Shot 2021-09-27 at 7.40.17 PM.png

Screen Shot 2021-09-27 at 7.42.51 PM.png

Screen Shot 2021-09-27 at 7.53.01 PM.png

Edited by makk
Link to comment
Share on other sites

You either use SSDT-EC or the rename - not both. Usually go with SSDT-EC as the rename breaks Windows in my experience.

 

I'd start from scratch ACPI wise, at least for the battery. Then use "log show --last boot | grep -i ACPI" to see if you have any ACPI errors. This will show tables that won't load as well as any runtime errors while trying to read the battery info.

 

SSDT-EC should not disable EC0 - all it needs to do is create a fake EC device to make AppleACPIPlatformExpert not deadlock.

Link to comment
Share on other sites

4 hours ago, 1Revenger1 said:

You either use SSDT-EC or the rename - not both. Usually go with SSDT-EC as the rename breaks Windows in my experience.

 

I'd start from scratch ACPI wise, at least for the battery. Then use "log show --last boot | grep -i ACPI" to see if you have any ACPI errors. This will show tables that won't load as well as any runtime errors while trying to read the battery info.

 

SSDT-EC should not disable EC0 - all it needs to do is create a fake EC device to make AppleACPIPlatformExpert not deadlock.

 

Thanks!

 

I'll leave it with Rename EC0 otherwise just with SSDT-EC no battery and trackpad.

 

Log file to show Cannot find EHC1

 

log.txt

Link to comment
Share on other sites

Why did you rename it? This only changes the name of the USB Controller for older CPU families. If you renamed it from EHC1 to EH01 it should be displayed in Hackintool in the "USB" Section as EH01. I don't see the relation to the battery status indicator here, though. I can disable these renames and still Battery status works if I use the old patching method.

 

 

Link to comment
Share on other sites

15 minutes ago, 5T33Z0 said:

Why did you rename it? This only changes the name of the USB Controller for older CPU families. If you renamed it from EHC1 to EH01 it should be displayed in Hackintool in the "USB" Section as EH01. I don't see the relation to the battery status indicator here, though. I can disable these renames and still Battery status works if I use the old patching method.

 

 

 

That's how it works here.

HP 

 

I am using old patch methods.

 

Don't know anyone with the same laptop to exchange with so there's another issue.

 

 

Link to comment
Share on other sites

On 9/28/2021 at 1:25 PM, Hervé said:

Why don't you post your extracted BIOS tables and a zipped copy of your bootloader's EFI folder? You may also want to consider switching to MBP12,1 SMBIOS as is usually the norm for Broadwell/HD5500 laptops rather than some MacBookAir SMBIOS.

https://dortania.github.io/OpenCore-Install-Guide/config-laptop.plist/broadwell.html#starting-point

 

Thanks  Herve', give me sometime to post these.

 

I am working on another issue with waking from RTC (Alarm)

 

 

Link to comment
Share on other sites

Herve'

EFI

Deleted see next post below

 

MacBook Pro12,1 model

 

Thank you!

 

Some side notes: Depending on the version of SSDT-EC.aml they load or don't load due to Name already in tables.

There is two SSDT-EC.aml in the ACPI section.

 

The current one SSDT loads and no ACPI errors in the log. The only issue being the Battery Icon in Menu Bar is no longer present.

When the other verisons of SSDT-EC do not load, the Battery Icon is present, due to no loading happens -- name exists in table.


So the issue is how to write a proper EC that loads and keeps icon present.

 

 

 

Edited by makk
Link to comment
Share on other sites

5 minutes ago, Hervé said:

Please post your extracted ACPI tables here rather than on a locked Google drive. There should be no issue doing this, zip them all in a simple common archive. It cannot be tens of MB large.

 

Herve' -- Apology if the EFI folder is locked, it's pretty big, what do you need from EFI as in specifics?

 

the ACIP Tables are in the SystemReport ZIP filed above.


Were you able to get them?

 

The EFI folder also was it locked? 

 

Thanks Herve'

ACPI.zip EFI OC 73.zip

Edited by makk
Link to comment
Share on other sites

3 minutes ago, Hervé said:

Sorry, it was the EFI (not your ACPI tables) that was on a protected Google drive.

 

Had a look at your Embedded Controller EC0 and, as expected, it does contain variables > 8bit:

                    OperationRegion (ECRM, EmbeddedControl, 0x00, 0xFF)
                    Field (ECRM, ByteAcc, NoLock, Preserve)
                    {
                        PMCD,   32, 
                        S0FL,   8, 
                        SXF0,   8, 
                        SXF1,   8, 
                        CPWR,   16, 
                        CVLT,   16, 
                        CCUR,   16, 
                        DIDX,   8, 
                        CIDX,   8, 
                        PMCC,   8, 
                        PMEP,   8, 
                        Offset (0x22), 
                        CRZN,   8, 
                        THTA,   8, 
                        HYST,   8, 
                        CRIT,   8, 
                        TEMP,   8, 
                        TENA,   8, 
                        Offset (0x29), 
                        TOAD,   8, 
                        PHTP,   8, 
                        THEM,   8, 
                        TMPO,   8, 
                        Offset (0x2E), 
                        FRDC,   8, 
                        FTGC,   8, 
                        PLTP,   8, 
                        Offset (0x32), 
                        DTMP,   8, 
                        Offset (0x35), 
                        PL1,    8, 
                        PL2,    8, 
                        BCVD,   8, 
                        Offset (0x3A), 
                        LUXH,   8, 
                        LUXL,   8, 
                        BLXH,   8, 
                        BLXL,   8, 
                        ALST,   8, 
                        Offset (0x40), 
                        ABDI,   8, 
                        ABAD,   8, 
                        ABIX,   8, 
                        ABDA,   8, 
                        ABST,   8, 
                        PORI,   8, 
                        Offset (0x47), 
                        PORL,   8, 
                        Offset (0x4C), 
                        PSSB,   8, 
                        Offset (0x50), 
                        FBCM,   8, 
                        FBGI,   8, 
                        FBAE,   8, 
                        FBCB,   8, 
                        FBW1,   8, 
                        FBW2,   8, 
                        Offset (0x57), 
                        FBID,   8, 
                        FUAE,   8, 
                        FRPS,   8, 
                        Offset (0x60), 
                        SCCB,   8, 
                        SCCS,   8, 
                        SCWT,   16, 
                        SCWD,   16, 
                        Offset (0x80), 
                        Offset (0x81), 
                            ,   4, 
                        SLPT,   4, 
                        FNSW,   1, 
                        SFNC,   1, 
                        ACPI,   1, 
                            ,   1, 
                            ,   1, 
                            ,   1, 
                            ,   1, 
                        DETF,   1, 
                        LIDS,   1, 
                        TBLT,   1, 
                            ,   1, 
                            ,   1, 
                            ,   1, 
                        COMM,   1, 
                        PME,    1, 
                        SBVD,   1, 
                        ADP,    1, 
                        ADID,   2, 
                        LCTV,   1, 
                        BATP,   4, 
                        BPU,    1, 
                        Offset (0x86), 
                        BSEL,   4, 
                        Offset (0x87), 
                        LB1,    8, 
                        LB2,    8, 
                        BDC,    16, 
                        Offset (0x8D), 
                        BFC,    16, 
                        BRTE,   16, 
                        BTC,    1, 
                        Offset (0x92), 
                        BME,    16, 
                        BDN,    8, 
                        BDV,    16, 
                        BCV1,   16, 
                        BST,    4, 
                        Offset (0x9B), 
                        BATE,   16, 
                        BPR,    16, 
                        BCR,    16, 
                        BRC,    16, 
                        BCC,    16, 
                        BPV,    16, 
                        BCV2,   16, 
                        BCV3,   16, 
                        BCV4,   16, 
                        BCW,    16, 
                        BATF,   16, 
                        BCL,    16, 
                        MAXC,   16, 
                        BCG1,   8, 
                        BT1I,   1, 
                        BT2I,   1, 
                            ,   2, 
                        BATN,   4, 
                        BSTS,   16, 
                        BCG2,   8, 
                        Offset (0xBD), 
                        BMO,    8, 
                        Offset (0xBF), 
                        BRCV,   8, 
                        Offset (0xC1), 
                        BIF,    8, 
                        BRCC,   8, 
                        Offset (0xC9), 
                        BSN,    16, 
                        BDAT,   16, 
                        BMF,    8, 
                        Offset (0xCF), 
                        CTLB,   8, 
                        Offset (0xD1), 
                        BTY,    8, 
                        Offset (0xD5), 
                        MFAC,   8, 
                        CFAN,   8, 
                        PFAN,   8, 
                        OCPS,   8, 
                        OCPR,   8, 
                        OCPE,   8, 
                        TMP1,   8, 
                        TMP2,   8, 
                        NABT,   4, 
                        BCM,    4, 
                        CCBQ,   16, 
                        CBT,    16, 
                        Offset (0xE3), 
                        OST,    4, 
                        Offset (0xE4), 
                        Offset (0xE5), 
                        TPTE,   1, 
                        TBBN,   1, 
                            ,   3, 
                        TP,     1, 
                        Offset (0xE6), 
                        SHK,    8, 
                        AUDS,   1, 
                        SPKR,   1, 
                        Offset (0xE8), 
                        HSEN,   4, 
                        HSST,   4, 
                        Offset (0xEA), 
                            ,   2, 
                        WWP,    1, 
                        WLP,    1, 
                            ,   1, 
                        WWS3,   1, 
                        WLS3,   1, 
                        Offset (0xEB), 
                        DKPS,   1, 
                        DKTP,   1, 
                        Offset (0xEC), 
                            ,   4, 
                        PB10,   1, 
                        ERWB,   1, 
                            ,   1, 
                        TBOS,   1, 
                            ,   3, 
                        FPPE,   1, 
                        Offset (0xEF), 
                        INCH,   2, 
                        IDIS,   2, 
                        INAC,   1
                    }

The ECEnabler kext may therefore be of some help.

 

 

 

 

 

I'm not 100% sure if this is considered a dual battery. where BTIX is needed in the hot patch. Extended Battery Info.

There's BAT0 and BAT1 in the DSDT.

 

When using SSDT-EC and disabling the EC0 to EC rename, must have the rename EHCI to EHO1 enabled or no boot. ( the onscreen states Mac Frame work ) then stops loading. 

 

Right now it is using SSDT-EC-LAPTOP and the renames EC0, EHCI, XHCI, without these USB has undesirable result.

 

The way the SSDT-4xG2b.aml is written, the rename is a must.  was written for High Sierra then modified for Mojave by Rehabman, it's all his

 work.  Which I used to get this HP up and running.  I used the normal upgrade path afterwards having to make changes in config.plist to find my way through.

 

 

Link to comment
Share on other sites

Guest 5T33Z0

I had a look at the ACPI Patches. These could be problematic:

I think that these are part of the problems why the system doesn't work correctly.

 

 

Edited by 5T33Z0
Link to comment
Share on other sites

20 hours ago, 5T33Z0 said:

I had a look at the ACPI Patches. These could be problematic:

I think that these are part of the problems why the system doesn't work correctly.

 

 

 

Thank you for taking the time 5T!

 

I've tried those methods before.  

 

It is pretty extensive. 

 

With this SSDT-4x0Gb2.aml <> hotpatch all those renames you mentioned are taking effect.

Such as UIAC, XOSI, XDSM, ZWAK, GPRW, BTIF, SBTI, BTI0, LPCB, others.

 

Attached is System DSDT.dsl.  >< .aml errors out.  In this System DSDT you can see the changes made

from renames and Hotpatch.

 

Thank you so very much!

 

 

 

SSDT-4x0G2b.aml System DSDT.dsl

MacBook Pro.acpi

Edited by makk
Link to comment
Share on other sites

On 10/3/2021 at 2:39 AM, 5T33Z0 said:

Unfortunately, I can't help you further with that since my ASL skills are basic at best.

 

 

Not a problem I am almost done.  

It's the way the the hotpathes were implemented.  

 

Plus this an HP, and old one.

 

I moved back to Clover, HDMI Audio does not want to work no matter what we try.

 

On Clover working perfectly.  

 

 

Link to comment
Share on other sites

  • 2 months later...

Has it been resolved. I can not see this kext being loaded from Clover kext folder at all. Like I never placed it.

 

P.S. Working excellent on my Asus Haswell laptop on Mojave with Clover. Had to update Lilu hence it did not work 1st time.

Edited by eugene28
update.
Link to comment
Share on other sites

On 10/5/2021 at 7:03 AM, makk said:

 

Not a problem I am almost done.  

It's the way the the hotpathes were implemented.  

 

Plus this an HP, and old one.

 

I moved back to Clover, HDMI Audio does not want to work no matter what we try.

 

On Clover working perfectly.  

 

 

Apart from many unneeded patches you have tooo many other stuff you do not need. If you have USB correctly mapped you do not need USBInjectAll.kext. The reason your Audio not working cause you did not put LayoutID in the PciRoot(0x0)/Pci(0x1B,0x0) Audio properties. Too many device properties in device tree. You only need device propertie for Video, Audio and Wi-fi. No need for IntelMausiEthernet.kext since you using Realtek also.

Edited by eugene28
Link to comment
Share on other sites

  • 2 weeks later...
On 12/31/2021 at 8:16 AM, Baio77 said:

I have prepared this EFI HP 450 G2, if anyone want to try it, I need ioreg of this EFI in case you mention me. EFI.zip

 

Hi Baio77

 

Thank you! I will give it a try just as it is.

Then make a few little bit adjustments as needed.

 

Right now I'm working on my Mojave. working out the bugs.

 

I have a problem with VoodooTrackpad not showing up when I turn of the ECO to EC rename and use the 

SSDT-EC-Laptop.aml.

Also does not map USB3 in my current EFI.   So which is strange.

It is rather unique.

 

 

 

 

Edited by makk
Added more words
Link to comment
Share on other sites

On 12/29/2021 at 9:21 PM, eugene28 said:

Apart from many unneeded patches you have tooo many other stuff you do not need. If you have USB correctly mapped you do not need USBInjectAll.kext. The reason your Audio not working cause you did not put LayoutID in the PciRoot(0x0)/Pci(0x1B,0x0) Audio properties. Too many device properties in device tree. You only need device propertie for Video, Audio and Wi-fi. No need for IntelMausiEthernet.kext since you using Realtek also.

 

Your right I just left the Mausi because Rehabman made this originally and for some reason he leaves in the IntelMausiEthernet.kext, which I have no idea why. Unless there is the hidden thing. Kind of defeats the purpose.  It doesn't load anyway it's just there for looks.

As for the devices in properties, helps in System Information to see if the driver is loading besides IOREG. Two sources to compare and check, sort of redundant but helps the Mac I think.  I had a problem with the USBMap.kext after awhile where the ports did not map after mapping well which was strange.  But thanks man! I'll let you know how the Baio77 EFI works out.


Thank you!

  • Thanks 1
Link to comment
Share on other sites

On 12/31/2021 at 8:16 AM, Baio77 said:

I have prepared this EFI HP 450 G2, if anyone want to try it, I need ioreg of this EFI in case you mention me. EFI.zip

 

Baio77  

 

Thanks again for the EFI, i had to make some changes.  The SSDT-4x02B2.aml which you modded didn't fare well. went back to original

A few changes in config.plist as would not boot.  

 

Right now running Mojave on a different drive.  I like Mojave for the looks overall.  I like it better that the newer OS. 

 

I'm working on the OpenIntel now, having a problem with bluetooth not staying connected to other devices.  Some kinks have to workout.

 

I'll get around to sending EFI and IOREG a little later after I get the bumps and kinks out.

 

My big issue has the EC.  The Battery Icon does not stick and is not registering at all.  Also trackpad does not register.

 

When using USBPorts.kext or USBMap.kext the USB3 speeds remain at 480mbps rather than the 5Ghz.

So that is why I used USBInjectall.kext instead.

 

I downloaded itlwm.kext and using Helliport and it works good, good speeds over the Atheros.

 

Haven't tried the Airportitlwm yet had to get the one for Mojave.

 

UPDATE:  NEWER IOREG + System Information screenshot 

the USB 2 and 3 are now separated and getting the upto 5Ghz speed for USB 3. where previous was 480Mbps slow for USB Drives

 

 

 

 

Screen Shot 2022-01-15 at 8.40.36 AM.png

MacBook Pro.zip

config.plist.zip

Edited by makk
UPDATED IOREG + SCREENSHOT SYS INFO
Link to comment
Share on other sites

Baio77

 

Update on the previous post

 

Here is EFI file minus the 16.3Mb itlwm.kext

 

Try this 

Replaced the SSDT-4x0G2b.aml 

did not need the FakePCIID_Intel_HDMI.kext

audio boot arg as alcid=3; remove the layoutid of 3 from Device Properties

SIP disabled. have to test the airportiltwm.kext with the method described

Several changes

 

The issue now is no battery in Energy and no icon and no trackpad in System Preference.

 

 

EFI OC.zip

Edited by makk
Link to comment
Share on other sites

7 hours ago, makk said:

Baio77

 

Update on the previous post

 

Here is EFI file minus the 16.3Mb itlwm.kext

 

Try this 

Replaced the SSDT-4x0G2b.aml 

did not need the FakePCIID_Intel_HDMI.kext

audio boot arg as alcid=3; remove the layoutid of 3 from Device Properties

SIP disabled. have to test the airportiltwm.kext with the method described

Several changes

 

The issue now is no battery in Energy and no icon and no trackpad in System Preference.

 

 

EFI OC.zip 8.98 MB · 0 downloads

You need latest lilu. Why use using 1.5.7 still?

Link to comment
Share on other sites

2 hours ago, eugene28 said:

You need latest lilu. Why use using 1.5.7 still?

 

2 hours ago, eugene28 said:

You need latest lilu. Why use using 1.5.7 still?

 

Latest Lilu has bug no boot  🤧

 

Causing panic kernel on boot 

 

cause being Lilu; three lines of error outputs

 

Edited by makk
Link to comment
Share on other sites

 Share

×
×
  • Create New...