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

SSDT DMAC - FWHD #121

Closed
wants to merge 39 commits into from
Closed

SSDT DMAC - FWHD #121

wants to merge 39 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Sep 22, 2020

in new intel generations, dmac and fwhd are missing.

Devil66 added 8 commits September 22, 2020 11:07
in new intel generations, dmac and fwhd are missing, instead b0d4 is used to correct errors that dsdt has and creates benefit. I propose this addition, of b0d4 I can post the photos of before and after putting it, so you can see it too.
thank you
identation fixed
@ghost
Copy link

ghost commented Sep 22, 2020

which intel generations? what is it trying to fix?


Scope (_SB.PCI0.B0D4)
{
If (_OSI ("Darwin"))
Copy link
Contributor

Choose a reason for hiding this comment

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

Some method is missing here.

Copy link
Author

Choose a reason for hiding this comment

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

there is no lack of method, it is just so. see photo above

Copy link
Contributor

Choose a reason for hiding this comment

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

Then how does it work? You cannot return a value from just the scope, it is not a function. Read the ACPI spec.

@@ -0,0 +1,16 @@
/*
* Only necessary when B0D4 device is present in the (INTEL) DSDT.
* For all intel generations, This Configuration Adjust B0D4 device Return (Fix DSDT Warnings and error).
Copy link
Contributor

Choose a reason for hiding this comment

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

I still cannot make sense what this SSDT does from the description.

@@ -0,0 +1,57 @@
/*
* Only necessary when no DMAC device (with any name) is present in the DSDT and a custom device-id
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is that device needed? Also not very descriptive to be honest.

@@ -0,0 +1,35 @@
/*
* Only necessary when no FWHD device (with any name) is present in the DSDT and a custom device-id
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as DMAC.

@vit9696
Copy link
Contributor

vit9696 commented Sep 22, 2020

This needs more background information on what is being done. Both in the PR description and in the dsl tables themselves.

@foskvs
Copy link

foskvs commented Sep 22, 2020

I guess somebody doesn’t even bother reading ACPI speciifications and doesn’t have a clue on coding syntax...

this one doesn't. Ok
@Tandem91
Copy link

Next step? Could you also work on SSDT_SMC? It would help to avoid using a SMC emulator.

@vit9696
Copy link
Contributor

vit9696 commented Sep 22, 2020

It is unjustified why these devices are needed on our machines. Just the fact they are present in Apple ACPI does not make it a requirement for our ACPI. Closing until more information is available.

@vit9696 vit9696 closed this Sep 22, 2020
@ghost ghost changed the title SSDT DMAC - FWHD - B0D4 SSDT DMAC - FWHD Sep 22, 2020
@dreamwhite
Copy link
Contributor

It is unjustified why these devices are needed on our machines. Just the fact they are present in Apple ACPI does not make it a requirement for our ACPI. Closing until more information is available.

This should be said also to Mald0n tbh ._. but hey "APPLE USES THOSE DEVICES SO WE SHOULD INCLUDE THEM WITHOUT ANY KIND OF DOCUMENTATION". Pretty ridiculous

@mhaeuser
Copy link
Member

@foskvs @dreamwhite Please refrain from making any comments that do not detail anything about the PR's content, this is not a gossip forum.

@Tandem91 That is not how ACPI works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
6 participants