Skip to content
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

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

Merged
merged 1 commit into from
Oct 28, 2019

Conversation

JamesH65
Copy link
Contributor

Requires new display power mailbox call to be present.

Signed-off-by: James Hughes [email protected]

@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
@@ -272,6 +278,7 @@ struct vc4_fkms_encoder {
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 <[email protected]>
@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