Re: [Spice-devel] [virtio-dev] [PATCH RFC v4 0/1] Virtio Video Device Specification

2021-01-15 Thread Matti Moell
On 15.01.21 15:25, Keiichi Watanabe wrote: > I think the driver implementation is necessary for the spec to be > merged, but it's not yet clear when we can spend time implementing > drivers. It's likely to be after April or so. > > IIRC, OpenSynergy folks, who implemented the v3 driver, also had

Re: [Spice-devel] [virtio-dev] [PATCH RFC v4 0/1] Virtio Video Device Specification

2021-01-15 Thread Keiichi Watanabe
On Fri, Jan 15, 2021 at 3:18 AM Alex Bennée wrote: > > Keiichi Watanabe writes: > > > This is the v4 RFC of virtio video device specification. > > PDF versions are available at [1, 2]. > > > > Note that this patch depends on a recent patchset "Cross-device resource > > sharing" [3]. > > > > Here

[Spice-devel] [PATCH 09/10] drm/atomic: Pass the full state to planes atomic disable and update

2021-01-15 Thread Maxime Ripard
The current atomic helpers have either their object state being passed as an argument or the full atomic state. The former is the pattern that was done at first, before switching to the latter for new hooks or when it was needed. Let's start convert all the remaining helpers to provide a consiste

[Spice-devel] [PATCH 10/10] drm: Use state helper instead of the plane state pointer

2021-01-15 Thread Maxime Ripard
Many drivers reference the plane->state pointer in order to get the current plane state in their atomic_update or atomic_disable hooks, which would be the new plane state in the global atomic state since _swap_state happened when those hooks are run. Use the drm_atomic_get_new_plane_state helper t

[Spice-devel] [PATCH 02/10] drm: Rename plane atomic_check state names

2021-01-15 Thread Maxime Ripard
Most drivers call the argument to the plane atomic_check hook simply state, which is going to conflict with the global atomic state in a later rework. Let's rename it to new_plane_state (or new_state depending on the convention used in the driver). This was done using the coccinelle script below,

[Spice-devel] [PATCH 04/10] drm/atomic: Pass the full state to planes atomic_check

2021-01-15 Thread Maxime Ripard
The current atomic helpers have either their object state being passed as an argument or the full atomic state. The former is the pattern that was done at first, before switching to the latter for new hooks or when it was needed. Let's start convert all the remaining helpers to provide a consiste

[Spice-devel] [PATCH 07/10] drm: Store new plane state in a variable for atomic_update and disable

2021-01-15 Thread Maxime Ripard
In order to store the new plane state in a subsequent helper, let's move the plane->state dereferences into a variable. This was done using the following coccinelle script, plus some hand changes for vmwgfx: @ plane_atomic_func @ identifier helpers; identifier func; @@ ( static const struct drm

Re: [Spice-devel] [virtio-dev] [PATCH RFC v4 0/1] Virtio Video Device Specification

2021-01-15 Thread Alex Bennée
Keiichi Watanabe writes: > This is the v4 RFC of virtio video device specification. > PDF versions are available at [1, 2]. > > Note that this patch depends on a recent patchset "Cross-device resource > sharing" [3]. > > Here is a list of major changes from v3: > * Redesingned struct definitions