diff options
author | Aseda Aboagye <aaboagye@chromium.org> | 2021-02-24 21:10:55 +0000 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-02-24 22:51:14 +0000 |
commit | 6acf29d48a6e651c54da0f8057013dea119eadc2 (patch) | |
tree | e4a6b1e04539eddc6307efa7ff54f2cc623f3ba9 /include/timer.h | |
parent | 7ac66e54e8e73018189b0b3dc4e1bae9bfa5bec3 (diff) | |
download | chrome-ec-6acf29d48a6e651c54da0f8057013dea119eadc2.tar.gz |
Revert "pd_alt_mode_dfp: Default to 4 lanes of DP"
This reverts commit 7991e4d1f705c84966b382219f194804380b79ab.
Reason for revert: Maintaining status quo until Chrome OS UI has
the ability for a user to report if this change works well for them.
Original change's description:
> pd_alt_mode_dfp: Default to 4 lanes of DP
>
> Currently our default PD policy states that if dock that supports DP
> alt mode also advertises Multi-function preferred, we configure the
> superspeed muxes to use 2 lanes for DisplayPort and 2 lanes for
> SuperSpeed USB. However, this is done without regard as to what the
> actual usage of the SuperSpeed ports may be.
>
> We've historically made this trade off, but it results in degraded
> display performance. Users may be better served if we prioritize
> display performance over SuperSpeed USB.
>
> The ideal solution involves adding additional plumbing such that we
> can configure the chosen pin configuration from the AP based upon
> other heuristics.
>
> BUG=b:178635286
> BRANCH=as many as feasible?
> TEST=Build and flash DUT, plug in a dock with SuperSpeed USB ports,
> verify that 4 lanes of DP are configured.
>
> Signed-off-by: Aseda Aboagye <aaboagye@google.com>
> Change-Id: Ie2d47588012aceb7f13312b6947b885f8f7034a8
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2653315
> Tested-by: Aseda Aboagye <aaboagye@chromium.org>
> Auto-Submit: Aseda Aboagye <aaboagye@chromium.org>
> Reviewed-by: Rob Barnes <robbarnes@google.com>
> Reviewed-by: Diana Z <dzigterman@chromium.org>
> Commit-Queue: Diana Z <dzigterman@chromium.org>
Bug: b:178635286
Change-Id: I94f107a43a58d512070a0507ebe2529eaedf473e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2718274
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Commit-Queue: Aseda Aboagye <aaboagye@chromium.org>
Tested-by: Aseda Aboagye <aaboagye@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Diffstat (limited to 'include/timer.h')
0 files changed, 0 insertions, 0 deletions