Skip to content

drm:vc4 Added calls for firmware display blank/unblank#3289

Merged
popcornmix merged 1 commit intorpi-4.19.yfrom
pi4_screen_blanking
Oct 28, 2019
Merged

drm:vc4 Added calls for firmware display blank/unblank#3289
popcornmix merged 1 commit intorpi-4.19.yfrom
pi4_screen_blanking

Conversation

@JamesH65
Copy link
Contributor

Requires new display power mailbox call to be present.

Signed-off-by: James Hughes james.hughes@raspberrypi.org

@JamesH65
Copy link
Contributor Author

Fixes #3050

@JamesH65 JamesH65 force-pushed the pi4_screen_blanking branch from b929714 to e9c52f7 Compare October 16, 2019 14:26
struct drm_encoder base;
bool hdmi_monitor;
bool rgb_range_selectable;
int display_num;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel that this should be accessible from either the crtc or connector (the bits either side of the encoder).
In reality I get lost with the potential dereferencing between the structures, and this driver is fairly monolithic anyway.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did spend about 45 minutes trying to find a way of getting this information from the existing structures, and it ended up being so convoluted that adding this parameter was a hugely simpler option - both from an implementation and understanding point of view.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough. I know that finding the right linkings can be less than obvious.

Requires new display power mailbox call to be present.

Signed-off-by: James Hughes <james.hughes@raspberrypi.org>
@JamesH65 JamesH65 force-pushed the pi4_screen_blanking branch from e9c52f7 to 11e0604 Compare October 16, 2019 15:07
@JamesH65
Copy link
Contributor Author

@popcornmix This can be merged now that that required mailbox command in the firmware has been merged, unless you want a short delay to wait for it to go out in rpi-update.

@popcornmix popcornmix merged commit f2aa9c0 into rpi-4.19.y Oct 28, 2019
@popcornmix popcornmix deleted the pi4_screen_blanking branch October 28, 2019 18:06
popcornmix added a commit to raspberrypi/firmware that referenced this pull request Oct 28, 2019
See: raspberrypi/linux#3296

kernel: dts: bcm2838: Disable DWC OTG block by default

kernel: ALSA: usb-audio: Add Gustard U16/X26 to quirks for native DSD support
See: raspberrypi/linux#3295

kernel: bcm2835-v4l2-codec fixes
See: raspberrypi/linux#3293

kernel: drm:vc4 Added calls for firmware display blank/unblank
See: raspberrypi/linux#3289

firmware: arm_loader: Fix explicit kernel name handling
See: #1277

firmware: Added a new display power mailbox call
firmware: Update display_power gencmd with optional display id
See: raspberrypi/linux#3050

firmware: Remove legacy pkgconfig to avoid Mesa conflicts
See: raspberrypi/userland#585
popcornmix added a commit to Hexxeh/rpi-firmware that referenced this pull request Oct 28, 2019
See: raspberrypi/linux#3296

kernel: dts: bcm2838: Disable DWC OTG block by default

kernel: ALSA: usb-audio: Add Gustard U16/X26 to quirks for native DSD support
See: raspberrypi/linux#3295

kernel: bcm2835-v4l2-codec fixes
See: raspberrypi/linux#3293

kernel: drm:vc4 Added calls for firmware display blank/unblank
See: raspberrypi/linux#3289

firmware: arm_loader: Fix explicit kernel name handling
See: raspberrypi/firmware#1277

firmware: Added a new display power mailbox call
firmware: Update display_power gencmd with optional display id
See: raspberrypi/linux#3050

firmware: Remove legacy pkgconfig to avoid Mesa conflicts
See: raspberrypi/userland#585
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.

4 participants