tree 2b2c8d0f555135ec78eeee6c2eecc0d73a8e4c87
parent 85eebdf456b67df1386209b78d2d7f833f7b43a0
author Daniel Stone <daniels@collabora.com> 1526400593 +0100
committer Daniel Stone <daniels@collabora.com> 1530875739 +0100

compositor-drm: Remove unnecessary picked_scanout variable

e2e80136334f fixed the same issue as df573031d0ba in a different way.
The latter commit (applied earlier in the upstream tree) adds a variable
to assign_planes to keep track of when we successfully assign a view to
the scanout plane, and doesn't call prepare_scanout_view if we have.

The former commit adds this checking inside prepare_scanout_view: if the
pending output state already has a framebuffer assigned to the scanout
plane, we drop out of prepare_scanout_view early. The picked_scanout
variable inside assign_planes can thus be removed.

Signed-off-by: Daniel Stone <daniels@collabora.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Tested-by: Emre Ucan <eucan@de.adit-jv.com>