On 25-07-06 17:42:24, Val Packett wrote: > At least some panels using the LSB register are not happy with the > unconditional increase of the command buffer to 3 bytes. > > With the BOE NE14QDM in my Dell Latitude 7455, the recent patches for > luminance based brightness have introduced a regression: the brightness > range stopped being contiguous and became nonsensical (it probably was > interpreting the last 2 bytes of the buffer and not the first 2). > > Change from using a fixed sizeof() to a length variable that's only > set to 3 when luminance is used. Let's leave the default as 2 even for > the single-byte version, since that's how it worked before. > > Fixes: f2db78e37fe7 ("drm/dp: Modify drm_edp_backlight_set_level") > Signed-off-by: Val Packett <v...@packett.cool>
Tested on Dell XPS 13 9345 which has LGD 134WT1 panel. Without this, the panel doesn't come up at all on boot. Tested-by: Abel Vesa <abel.v...@linaro.org>