Re: [PATCH v2 2/6] drm/vkms: Support multiple DRM objects (crtcs, etc.) per VKMS device

2023-08-17 Thread Brandon Ross Pollack
Silly addition but I need to apologize for mispelling somone's name in my reply. Maira, please accept my sincere apology. On 8/18/23 12:36, Brandon Pollack wrote: Thanks for taking the time, everyone! Sorry it took so long, we had some internal shuffling etc going on and I was building out wha

Re: [PATCH v2 2/6] drm/vkms: Support multiple DRM objects (crtcs, etc.) per VKMS device

2023-08-17 Thread Brandon Pollack
Thanks for taking the time, everyone! Sorry it took so long, we had some internal shuffling etc going on and I was building out what we needed these chagnes for in the first place, this will be the first of a few replies followed by a new version of the series to be sent out. First up is a respons

Re: [PATCH v2 2/6] drm/vkms: Support multiple DRM objects (crtcs, etc.) per VKMS device

2023-08-15 Thread Marius Vlad
Hi, See below some comments about vkms_wb_atomic_commit(). On Fri, Jun 23, 2023 at 06:23:44PM -0400, Jim Shargo wrote: > This change supports multiple CRTCs, encoders, connectors instead of one > of each per device. > > Since ConfigFS-based devices will support multiple crtcs, it's useful to > m

Re: [PATCH v2 2/6] drm/vkms: Support multiple DRM objects (crtcs, etc.) per VKMS device

2023-06-25 Thread Maira Canal
Hi Jim, On 6/23/23 19:23, Jim Shargo wrote: This change supports multiple CRTCs, encoders, connectors instead of one of each per device. Since ConfigFS-based devices will support multiple crtcs, it's useful to move all of the writeback/composition data from being per-"output" to being per-CRTC.

[PATCH v2 2/6] drm/vkms: Support multiple DRM objects (crtcs, etc.) per VKMS device

2023-06-23 Thread Jim Shargo
This change supports multiple CRTCs, encoders, connectors instead of one of each per device. Since ConfigFS-based devices will support multiple crtcs, it's useful to move all of the writeback/composition data from being per-"output" to being per-CRTC. Since there's still only ever one CRTC, this