Skip to content

drm/vc4: Fix atomic_async_check to call the right mode_set function #5860

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 1 commit into from
Jan 17, 2024

Conversation

6by9
Copy link
Contributor

@6by9 6by9 commented Jan 17, 2024

vc4_plane_atomic_async_check was always calling vc4_plane_mode_set to validate and generate the dlist for the check. If async_check decided it had to fall back to a sync commit, then this GEN4/5 dlist could get used on GEN6.

Call either vc4_plane_mode_set or vc6_plane_mode_set as appropriate.

Fixes: 1ab1fbb ("drm/vc4: hvs: Support BCM2712 HVS")

@6by9
Copy link
Contributor Author

6by9 commented Jan 17, 2024

Apologies - missed getting the vc4 pointer. Updating.

vc4_plane_atomic_async_check was always calling vc4_plane_mode_set
to validate and generate the dlist for the check. If async_check
decided it had to fall back to a sync commit, then this GEN4/5
dlist could get used on GEN6.

Call either vc4_plane_mode_set or vc6_plane_mode_set as appropriate.

Fixes: 1ab1fbb ("drm/vc4: hvs: Support BCM2712 HVS")
Signed-off-by: Dave Stevenson <[email protected]>
@6by9
Copy link
Contributor Author

6by9 commented Jan 17, 2024

Should be fixed now.

@pelwell pelwell merged commit 4a2a333 into raspberrypi:rpi-6.6.y Jan 17, 2024
popcornmix added a commit to raspberrypi/firmware that referenced this pull request Jan 18, 2024
See: raspberrypi/linux#5827

kernel: drm/vc4: Fix atomic_async_check to call the right mode_set function
See: raspberrypi/linux#5860
popcornmix added a commit to raspberrypi/rpi-firmware that referenced this pull request Jan 18, 2024
See: raspberrypi/linux#5827

kernel: drm/vc4: Fix atomic_async_check to call the right mode_set function
See: raspberrypi/linux#5860
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.

3 participants