-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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-PNLF Fix code for CFL + (whiskey lake fix) #284
Conversation
Please explain why deleting the code is legit here. |
that code I deleted on cfl+ is not essential, keeping that code works on cfl, cml and icl but on whiskey lake creates problems since it makes it load uid 10 and not 13 |
but I repeat it is a problem only with whiskey lake generation, the others work well with current pnlf release. |
Reported by my mates that some CML machines can not change brightness in 3 minutes. After 3 minutes, everything works well. No log for now. Occured since updated SSDT-PNLF in cbea3f1 |
I have a i7 10510u cml and it works perfectly. Also with the first version.. Never had this problem so maybe it comes from other stuff in their efi? |
Probably is this: Just add to their efi this Bootarg in nvram "-igfxblr" or in the Dev prop patch "enable-backlight-registers-fix" with 01000000 DATA And please, study and read before apply something... |
@Vorshim92 Thanks for the reply. I am still confused about CFL part in SSDT-PNLF. Why we need to assign COFFEELAKE_PWMMAX to CFL pwm in SSDT-PNLF while in SSDT-PNLFCFL, we only need to change UID to 13? To me, SSDT-PNLFCFL is a much cleaner and native way to inject UID without modifying PWM max value. |
sorry meddling, but you did not understand how it works you just have to fill in aml, and then he alone according to your device id of the igpu, inject in automatic correct uid, in your case sect 13 in automatic since you have cfl. |
@Lorys89 Thanks for the reply. SSDT-PNLF has detailed notations so it's not difficult to understand how it works on assigning PWM value. The auto device-id detection is indeed good for newcomers. What I doubted is the necessity for CFL users transition from SSDT-PNLFCFL to SSDT-PNLF since PNLFCFL seems to be a much decent UID injection targeted for CFL+ users. -igfxblr is not needed on my friend's machine when he uses SSDT-PNLFCFL. I will ask him to try this with PNLF. BTW, still hoping that AppleBacklightSmoother could be updated as it was also a great solution for brightness adjustion. |
What I suggest and I believe the behind reason for Rehabman having a separated PNLFCFL is letting CFL machine's LMAX and PWM untouched. In the new version of PNLF, CFL machine's UID is set to 19 and triggers F19Txxxx backlight data. While UID 13 in SSDT-PNLFCFL will call native AppleBacklight.kext. (For some reason, machine within PNLFCFL's UID is still set to 19?) |
then you know that uid 19 (decimal) converted is uid 0x13 (hex) |
goes back to SSDT-PNLFCFL that is enough for Coffee Lake - acidanthera/OpenCorePkg#284 Changes in config.plist kernel quirks (RebuildAppleMemoryMap enabled with SyncRuntimePermissions; EnableWriteUnprotector Disabled) Disabled Hibernation (I don't use it often)
@1alessandro1 , thank you for the follow-up. @Lorys89 did not delete my comment. I deleted my comment (which originally asked if SSDT-PNLF might cause the kernel panic) because the kernel panic still happens with the the non-CFL version of SSDT-PNLF. I have now reverted to OC 0.7.1 and corresponding kexts released with OC 0.7.1 (WEG, Lilu, etc) in an attempt to isolate the kernel panic. I apologize for the false alarm - I deleted my comment, because I don't think my kernel panic is caused by SSDT-PNLF. |
ssdt pnlf cannot cause kp, or starts or does not start (does not load) investigates ssdt pnlf cannot cause kp, it's next time write about acidanthera bugtracker. |
My hack (HP Envy Kabylake R i5-8250U/UHD620) BIOS does not permit configuration of video RAM, so I configured framebuffer-fbmem and framebuffer-stolenmem as I understood from here. I used the 9MB and 19MB values respectively from the Dortania post-install guide. After increasing these values (to 21MB and 39MB respectively), I am no longer observing the Kabylake kernel panic. I will continue to monitor and report back if anything changes. UPDATE: I have removed framebuffer-fbmem and framebuffer-stolenmem from my OC 0.7.3 config.plist and am not experiencing any kernel panics. I re-read this and now understand that I misinterpretted the Dortania instruction. I incorrectly thought that an inability to configure VRAM in BIOS meant an automatic need to configure framebuffer-fbmem and framebuffer-stolenmem in OC's config.plist DeviceProperties. I now realize that configuring these is only necessary if VRAM is not sufficient to support fbmem and stolenmem required by the chosen framebuffer (AAPL,ig-platform-id). I suspect that if I booted Windows on my rig (I don't have Windows installed and only run Big Sur 11.6), I would see that available video memory is more than adequate for my chosen framebuffer. |
in the previous version it only had problem with some laptops with whiskey lake cpu, but it worked on cfl and cml and icl, with new modification solved the problem.