Skip to content

Add association between DRM panels and backlight where defined #6177

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 22, 2024

Conversation

6by9
Copy link
Contributor

@6by9 6by9 commented May 17, 2024

For comment from SimonL for UI backlight control.

For ease of testing this also includes the patches from 6176 as it's useful to have the panel fully working.

6by9 added 4 commits May 17, 2024 15:41
Add shutdown and remove hooks so that the panel gets powered
off with the system.

Signed-off-by: Dave Stevenson <[email protected]>
As with many microcontrollers, the panel wants to use clock
stretching during a read so that the appropriate return value
can be generated and programmed into the hardware.
With Pi0-3 really not supporting clock stretching, this resulted
in the panel firmware dying.

Insert a delay between the write and the read to give the firmware
a chance to generate the relevant return value.

Signed-off-by: Dave Stevenson <[email protected]>
The naming of backlight devices is not terribly useful for
associating a backlight controller with a display (assuming
it is attached to one).

Add a sysfs node that will return a display name that can be set
by other subsystems.

Signed-off-by: Dave Stevenson <[email protected]>
Pass the DRM connector name to any configured backlight
device so that userspace can associate the two items.

Ideally this should be in drm_panel, but it is bridge/panel
that creates the drm_connector and therefore knows the name.

Signed-off-by: Dave Stevenson <[email protected]>
@6by9 6by9 force-pushed the rpi-6.6.y-backlight branch from 97d00cb to 132d359 Compare May 17, 2024 17:04
@6by9
Copy link
Contributor Author

6by9 commented May 22, 2024

@pelwell @popcornmix
Simon has written a widget to control the backlight based on this. Are we happy with the core mods as downstream patches? I don't think we'll get much traction in upstreaming this sort of thing.

@pelwell pelwell merged commit 573f8fd into raspberrypi:rpi-6.6.y May 22, 2024
12 checks passed
popcornmix added a commit to raspberrypi/firmware that referenced this pull request May 24, 2024
See: raspberrypi/linux#6180

kernel: drivers: staging: bcm2835-isp: Respect caller's stride value
See: raspberrypi/linux#5960

kernel: drivers: pcie-brcmstb: add best-effort workaround for QoS bug on bcm2712
See: raspberrypi/linux#6183

kernel: ARM: dts: Fix camera sync parameters
See: raspberrypi/linux#6184

kernel: Add association between DRM panels and backlight where defined
See: raspberrypi/linux#6177
popcornmix added a commit to raspberrypi/rpi-firmware that referenced this pull request May 24, 2024
See: raspberrypi/linux#6180

kernel: drivers: staging: bcm2835-isp: Respect caller's stride value
See: raspberrypi/linux#5960

kernel: drivers: pcie-brcmstb: add best-effort workaround for QoS bug on bcm2712
See: raspberrypi/linux#6183

kernel: ARM: dts: Fix camera sync parameters
See: raspberrypi/linux#6184

kernel: Add association between DRM panels and backlight where defined
See: raspberrypi/linux#6177
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants