[PATCH v10 00/61] Atomic modesetting rides again

[Pekka Paalanen]

On Thu, 13 Jul 2017 14:47:03 +0000
Fabien DESSENNE <fabien.dessenne at st.com> wrote:

> Hi all,
> 
> It looks like the great work proposed by Daniel to have Atomic
> Modesetting in Weston did not find the success it deserves. We tested
> it successfully on STMicroelectronics boards and it works great.
> 
> The kernel DRM framework has been reworked in the past months (years)
> in order to support Atomic, but one of the reference compositor,
> Weston, does not support this feature. And as long as this is not
> supported, there is no way to make use of HW planes ('sprites are
> broken'): all of the composition has to be done by the GPU.
> 
> So, for the time being, we have a nice "HW plane enable" kernel, but
> there is no userland compositor in front of it! (except Android
> HWC...)
> 
> Can some of you review those patches and help to get it merged?

Hi,

I have been reviewing it when I can, but this year I have been swamped
in other projects. Reviewing the atomic series is still my #1 priority
when I can.

> Without review from the community, shall we conclude that only us are
> interested in using HW planes in Weston?

I would hope it's not lack of interest, it's circumstances forcing
people to do other things. At least that's it for me.

Maybe another factor is that people are not comfortable reviewing
Weston patches.

I have already given an Ack for the overall design/architecture in the
atomic series, we just need the detailed review to check the code is
correct.

I believe the latest code is still
https://git.collabora.com/cgit/user/daniels/weston.git/log/?h=wip/2017-04/atomic-v11-WIP
while the latest series with review comments is the v10 you can find
here:
https://patchwork.freedesktop.org/project/wayland/patches/?submitter=18&state=*&q=&archive=&delegate=

The last patch I reviewed so far was "[PATCH weston v10 21/61]
compositor-drm: Track all plane types", so review from that one onwards
would be nice. Mind the review comments given to the patches that are
marked as Changes Requested in Patchwork, I recall they still apply in
the v11-WIP branch.


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20170714/79d7bcd1/attachment.sig>