ozone: Select a crtc even when KMS doesn't report any plane for it.

ozone: Select a crtc even when KMS doesn't report any plane for it.

crrev.com/2420083003 changed the logic we use to select a crtc.

The new code assumes that KMS advertises at least one plane
per crtc. This is not true, and it caused not to select any crtc for
certain devices.

This change removes that assumption.

BUG=680798
Review-Url: https://codereview.chromium.org/2629773004
Cr-Commit-Position: refs/heads/master@{#443766}
Committed: https://chromium.googlesource.com/chromium/src/+/aa96a2c8835f1681fa482a9b31316d2f3cd2ccf6

[Daniele Castagna, 2017-01-14 01:05:54]

The CQ bit was checked by dcastagna@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-14 01:06:09]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Daniele Castagna, 2017-01-14 01:07:30]

Description was changed from

==========
ozone: Select a crtc even when KMS doesn't report any plane for it.

crrev.com/2420083003 changed the logic we use to select a CRTc.

The new code assumed that KMS would advertise at least one plane
per CRTc. This is not true, and it caused not to select any CRTc for
certain devices.

This change removes that assumption.

BUG=680798
==========

to

==========
ozone: Select a crtc even when KMS doesn't report any plane for it.

crrev.com/2420083003 changed the logic we use to select a crtc.

The new code assumes that KMS advertises at least one plane
per crtc. This is not true, and it caused not to select any crtc for
certain devices.

This change removes that assumption.

BUG=680798
==========

[Daniele Castagna, 2017-01-14 01:07:35]

dcastagna@chromium.org changed reviewers:
+ hoegsberg@chromium.org, marcheu@chromium.org

[marcheu, 2017-01-14 01:12:04]

lgtm

[Daniele Castagna, 2017-01-14 01:21:31]

dcastagna@chromium.org changed reviewers:
+ hshi@chromium.org

[hshi1, 2017-01-14 01:22:15]

lgtm

[malaykeshav, 2017-01-14 01:24:07]

malaykeshav@chromium.org changed reviewers:
+ malaykeshav@chromium.org

[malaykeshav, 2017-01-14 01:24:08]

lgtm

[commit-bot: I haz the power, 2017-01-14 01:44:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-14 01:44:01]

Dry run: This issue passed the CQ dry run.

[Daniele Castagna, 2017-01-14 01:44:15]

The CQ bit was checked by dcastagna@chromium.org

[commit-bot: I haz the power, 2017-01-14 01:44:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-14 01:48:53]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484358255676810, "parent_rev":
"848c2e8399cd8aca05c31f555d29ba6b9a68d3ce", "commit_rev":
"aa96a2c8835f1681fa482a9b31316d2f3cd2ccf6"}

[commit-bot: I haz the power, 2017-01-14 01:49:26]

Description was changed from

==========
ozone: Select a crtc even when KMS doesn't report any plane for it.

crrev.com/2420083003 changed the logic we use to select a crtc.

The new code assumes that KMS advertises at least one plane
per crtc. This is not true, and it caused not to select any crtc for
certain devices.

This change removes that assumption.

BUG=680798
==========

to

==========
ozone: Select a crtc even when KMS doesn't report any plane for it.

crrev.com/2420083003 changed the logic we use to select a crtc.

The new code assumes that KMS advertises at least one plane
per crtc. This is not true, and it caused not to select any crtc for
certain devices.

This change removes that assumption.

BUG=680798

Review-Url: https://codereview.chromium.org/2629773004
Cr-Commit-Position: refs/heads/master@{#443766}
Committed:
https://chromium.googlesource.com/chromium/src/+/aa96a2c8835f1681fa482a9b3131...
==========

[commit-bot: I haz the power, 2017-01-14 01:49:27]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/aa96a2c8835f1681fa482a9b3131...