diff options
author | Sudhanshu Jha <sudhanshu.jha@arm.com> | 2017-02-24 13:34:22 +0000 |
---|---|---|
committer | Andreas Sandberg <andreas.sandberg@arm.com> | 2017-03-07 11:14:28 +0000 |
commit | 9dd54d10ab63c395b84bff31b5ff94b35c44e18f (patch) | |
tree | 80c7755669f3db6651b424ba5e2fae7f687b5b56 /src/dev/arm | |
parent | 82a8230aa761e193a91ade0fa3c109a5c0f08aed (diff) | |
download | gem5-9dd54d10ab63c395b84bff31b5ff94b35c44e18f.tar.xz |
dev: Add support for single-pass scan out in the PixelPump
Add a helper function to scan out an entire frame in one time
step. This requires the public PixelPump to be changed somewhat to
separate timing updates from general PixelPump control. Instead of
calling PixelPump::start(timings), timings now need to be updated
using a separate call to PixelPump::updateTimings(timings) before
calling PixelPump::start().
Display controllers that don't need accurate timing (e.g., in KVM
mode), can use the new PixelPump::renderFrame() API to render an
entire frame in one step. This call results in the same callbacks
(e.g., calls to nextPixel()) as the timing calls, but they all happen
in immediately. Unlike the timing counterpart, renderFrame() doesn't
support buffer underruns and will panic if nextPixle() indicates an
underrun.
Change-Id: I76c84db04249b02d4207c5281d82aa693d0881be
Signed-off-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/2241
Reviewed-by: Rahul Thakur <rjthakur@google.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Diffstat (limited to 'src/dev/arm')
-rw-r--r-- | src/dev/arm/hdlcd.cc | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/src/dev/arm/hdlcd.cc b/src/dev/arm/hdlcd.cc index 363aeba94..a92ae4627 100644 --- a/src/dev/arm/hdlcd.cc +++ b/src/dev/arm/hdlcd.cc @@ -1,5 +1,5 @@ /* - * Copyright (c) 2010-2013, 2015 ARM Limited + * Copyright (c) 2010-2013, 2015, 2017 ARM Limited * All rights reserved * * The license below extends only to copyright in the software and shall @@ -204,8 +204,11 @@ HDLcd::drainResume() // We restored from an old checkpoint without a pixel pump, start // an new refresh. This typically happens when restoring from old // checkpoints. - if (enabled() && !pixelPump.active()) - pixelPump.start(displayTimings()); + if (enabled() && !pixelPump.active()) { + // Update timing parameter before rendering frames + pixelPump.updateTimings(displayTimings()); + pixelPump.start(); + } // We restored from a checkpoint and need to update the VNC server if (pixelPump.active() && vnc) @@ -476,7 +479,10 @@ HDLcd::cmdEnable() { createDmaEngine(); conv = pixelConverter(); - pixelPump.start(displayTimings()); + + // Update timing parameter before rendering frames + pixelPump.updateTimings(displayTimings()); + pixelPump.start(); } void |