On 03/24, Maíra Canal wrote:
> This patchset focus on the proper handling of the primary plane after the
> modifications introduced in commit bc0d7fdefec6 ("drm: vkms: Supports to the
> case where primary plane doesn't match the CRTC"). Before this commit, the
> composition was executed on top of the primary plane. Therefore, the primary
> plane needed to cover the entire CRTC and it needed to be visible. After
> commit
> bc0d7fdefec6, this is no longer necessary, as the composition is now executed
> on
> top of the CRTC. So, remove those restrictions to the primary plane.
>
> This patchset increased the vkms IGT test coverage. The following tests are
> now
> passing:
>
> - igt@kms_atomic_transition@plane-toggle-modeset-transition
> - igt@kms_atomic_transition@modeset-transition
> - igt@kms_atomic_transition@modeset-transition-fencing
> - igt@kms_atomic_transition@modeset-transition-nonblocking
> - igt@kms_atomic_transition@modeset-transition-nonblocking-fencing
> - igt@kms_universal_plane@disable-primary-vs-flip-pipe-a
> - igt@kms_universal_plane@universal-plane-pageflip-windowed-pipe-a
> - igt@kms_universal_plane@universal-plane-pipe-a-functional
>
> Moreover, this patchset was tested with the vkms IGT testlist and all tests
> passed successfully.Nice catch. Thanks! This series is: Reviewed-by: Melissa Wen <[email protected]> > > Best Regards, > - Maíra Canal > > v1 -> v2: > https://lore.kernel.org/dri-devel/[email protected]/T/ > > * Introduce a second patch to allow the primary plane to be positioned. > > --- > > Maíra Canal (2): > drm/vkms: remove the need for the primary plane to be visible > drm/vkms: allow the primary plane to be positioned > > drivers/gpu/drm/vkms/vkms_plane.c | 10 +--------- > 1 file changed, 1 insertion(+), 9 deletions(-) > > -- > 2.39.2 >
signature.asc
Description: PGP signature
