ChefKissInc / NootedRed

The AMD Vega iGPU support patch kext. No commercial use.
https://ChefKissInc.github.io/nred
Other
1.55k stars 653 forks source link

Brightness control on Raven is broken #236

Open MizuKumi opened 3 months ago

MizuKumi commented 3 months ago

macOS Version

Ventura

What is your CPU's model?

AMD Ryzen 5 PRO 3500U

Please describe the behaviour in detail.

After upgrading to the latest version of NootedRed: https://github.com/ChefKissInc/NootedRed/actions/runs/8277805359 (commit [274ba77])

The brightness control stopped working properly, the brightness reduction via the native "Brightness Slider" from macOS only goes down one point from the maximum brightness value, the brightness does not go any lower after moving the slider.

In the previous version of Nootedred: https://github.com/ChefKissInc/NootedRed/actions/runs/7765787829 (Commit [eff598f])

The brightness control was working properly. Via the native "Brightness Slider" from macOS.

Videos demonstrating the problem are attached in two files.

Before: https://github.com/ChefKissInc/NootedRed/assets/116596455/5e0ec249-8dbe-4360-8178-868efe9eb4d6

After: https://github.com/ChefKissInc/NootedRed/assets/116596455/285fd8c8-38a0-4218-a8a4-97404bcb6f7d

What should've happened instead?

Brightness control, well, what else is there? :D

If applicable, attach the .gpuRestart, .panic, etc file related to this issue.

Pointless, the problem is inside Nootedred, apparently this problem appeared after commit d6554a3: https://github.com/ChefKissInc/NootedRed/actions/runs/8253163850

Bug report submission terms

VisualEhrmanntraut commented 3 months ago

What is the display type?

MizuKumi commented 3 months ago

Where can I see this in macOS?

VisualEhrmanntraut commented 3 months ago

I meant LCD, OLED, ... if it has HDR, etc.

MizuKumi commented 3 months ago

does not have HDR

Screenshot 2024-03-14 at 17 01 32
VisualEhrmanntraut commented 3 months ago

It seems like this issue is specific to Raven (DCN 1). I have not seen the issue occur on Renoir (DCN 2). It is possible that AMDRadeonX6000Framebuffer has a bug with the DMCU control. I will investigate how to solve this without removing the newly added Catalina support for brightness control and etc.

MizuKumi commented 3 months ago

Okay. I look forward to your fix. And yes, thanks for such a great project, you made it possible for me not to buy a macbook for overprice. :)

MacintoshSSD commented 4 days ago

An idea:maybe we can design a command for users to add boot-args to disable Catalina brightness control code, or delete Catalina brightness control code, because Catalina is outdated.