linux icon indicating copy to clipboard operation
linux copied to clipboard

drm/panel: s6e8aa5x01-ams497hy01 Samsung J5 backlight support

Open chr-chr opened this issue 2 years ago • 6 comments
trafficstars

The code was shamelessly stolen from ams520kt01, seems to work the same way. The data was mined from downstream.

Tested, works for me.

chr-chr avatar Oct 17 '23 17:10 chr-chr

@Grimler91 Do you mind taking a look at these changes?

stephan-gh avatar Oct 17 '23 17:10 stephan-gh

f this ... I donate a space ...

chr-chr avatar Oct 18 '23 01:10 chr-chr

Not J5X, but I would like to cc @B83C

Not J5X, but I would like to cc @B83C

? J5x having AMS520KT01 no?

chr-chr avatar Oct 18 '23 15:10 chr-chr

Oo,

found some bug that shall be fixed ;)

chr-chr avatar Oct 24 '23 21:10 chr-chr

code cleanup. @B83C: I guess the ams520kt01 can use this aswell. So stop copy and paste mistakes from each other. I guess the Author was oversleept like me ;)

 static int s6e8aa5x01_ams497hy01_enable(struct drm_panel *panel)
 {
...
        if (ctx->prepared)
                return 0;

Whats left? Actually one brightness cmd takes up to 400ms and when juggling a brightness slider I occasional get:

dsi_err_worker: status=4

but that did not hurt so far.

@B83C: I'd like some enlightenment on the Temp1? / Temp2? cmds ... do we need them?!

chr-chr avatar Oct 27 '23 16:10 chr-chr