On Sun, 2025-08-24 at 21:33 +0200, Antheas Kapenekakis wrote: > Well Phil managed to fall into the value 332800, which has a 0 minor > bit. Unfortunate. In hindsight, every 256 hundreds there would be a > zero anyway. > > Before I made this patch I made a partial refactor of panel-quirks > where a quirk like this could go to. But I would really prefer not to > do quirks. Ill send that too. > > Antheas
I was already looking into that OLED issue for several weeks. Changing granularity might hid the root cause and you might hit the issue less frequently. Currently checking [1] which changes the first byte to 3 since when DP_SOURCE_BACKLIGHT_LEVEL is written to with the first byte being 0x00 and sometimes 0x01, the panel forcibly turns off until the device sleeps again. In the end the panel vendor has to fix it in firmware. If not a quirk might be better specific for each panel vendor. I'm still not sure if that refactoring is needed, or a separate quirk function is more logical to be accepted upstream. [1] https://lore.kernel.org/lkml/20250824200202.1744335-5-l...@antheas.dev/T/#u
