On 5/29/26 11:56, Dmitry Baryshkov wrote:
On Thu, May 28, 2026 at 12:49:05AM -0500, Mario Limonciello (AMD) wrote:
This will show which connector in sysfs matches which backlight.

Signed-off-by: Mario Limonciello (AMD) <[email protected]>
---
  drivers/gpu/drm/Kconfig             |   1 +
  drivers/gpu/drm/Makefile            |   2 +
  drivers/gpu/drm/drm_atomic_helper.c |   8 +
  drivers/gpu/drm/drm_backlight.c     | 430 ++++++++++++++++++++++++++++
  drivers/gpu/drm/drm_connector.c     |  12 +
  drivers/gpu/drm/drm_drv.c           |   8 +
  drivers/gpu/drm/drm_mode_config.c   |   7 +
  drivers/gpu/drm/drm_mode_object.c   |  66 ++++-
  drivers/gpu/drm/drm_sysfs.c         |  28 +-
  drivers/video/backlight/backlight.c |  17 ++
  include/drm/drm_backlight.h         |  46 +++
  include/drm/drm_connector.h         |   3 +
  include/drm/drm_mode_config.h       |   5 +
  include/linux/backlight.h           |  13 +
  14 files changed, 641 insertions(+), 5 deletions(-)
  create mode 100644 drivers/gpu/drm/drm_backlight.c
  create mode 100644 include/drm/drm_backlight.h

+
+static void __drm_backlight_worker(struct work_struct *w)
+{
+       struct drm_backlight *b = container_of(w, struct drm_backlight, work);
+       static const char * const ep[] = { "BACKLIGHT=1", NULL };

This caused compile errors on linux-next, I had to drop both const
specifiers from this declaration.

Yeah - my fault for trusting checkpatch suggestion right before sending upstream and not compile testing because it looked "trivial".

Will fix in next spin.


+       struct backlight_device *bd;
+       bool send_uevent;
+       unsigned int v;
+

Nevertheless:


Tested-by: Dmitry Baryshkov <[email protected]> # SM8150-HDK




Much appreciated!

Reply via email to