[gem5-dev] changeset in gem5: arm: Workaround incorrect HDLCD register orde...

[Andreas Sandberg]

changeset 48039363f67a in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=48039363f67a
description:
        arm: Workaround incorrect HDLCD register order in kernel

        Some versions of the kernel incorrectly swap the red and blue color
        select registers. This changeset adds a workaround for that by
        swapping them when instantiating a PixelConverter.

diffstat:

 src/dev/arm/RealView.py |   2 ++
 src/dev/arm/hdlcd.cc    |  24 +++++++++++++++++++-----
 src/dev/arm/hdlcd.hh    |   2 ++
 3 files changed, 23 insertions(+), 5 deletions(-)

diffs (66 lines):

diff -r 10cac0f0f419 -r 48039363f67a src/dev/arm/RealView.py
--- a/src/dev/arm/RealView.py   Sat May 23 13:37:03 2015 +0100
+++ b/src/dev/arm/RealView.py   Sat May 23 13:37:04 2015 +0100
@@ -175,6 +175,8 @@
     vnc = Param.VncInput(Parent.any, "Vnc server for remote frame buffer "
                                      "display")
     amba_id = 0x00141000
+    workaround_swap_rb = Param.Bool(True, "Workaround incorrect color "
+                                    "selector order in some kernels")
     enable_capture = Param.Bool(True, "capture frame to 
system.framebuffer.bmp")
 
 class RealView(Platform):
diff -r 10cac0f0f419 -r 48039363f67a src/dev/arm/hdlcd.cc
--- a/src/dev/arm/hdlcd.cc      Sat May 23 13:37:03 2015 +0100
+++ b/src/dev/arm/hdlcd.cc      Sat May 23 13:37:04 2015 +0100
@@ -74,7 +74,8 @@
       fillPixelBufferEvent(this), intEvent(this),
       dmaDoneEventAll(MAX_OUTSTANDING_DMA_REQ_CAPACITY, this),
       dmaDoneEventFree(MAX_OUTSTANDING_DMA_REQ_CAPACITY),
-      enableCapture(p->enable_capture)
+      enableCapture(p->enable_capture),
+      workaround_swap_rb(p->workaround_swap_rb)
 {
     pioSize = 0xFFFF;
 
@@ -504,11 +505,24 @@
 PixelConverter
 HDLcd::pixelConverter() const
 {
-    return PixelConverter(
-        bytesPerPixel(),
-        red_select.offset, green_select.offset, blue_select.offset,
-        red_select.size, green_select.size, blue_select.size,
+    ByteOrder byte_order(
         pixel_format.big_endian ? BigEndianByteOrder : LittleEndianByteOrder);
+
+    /* Some Linux kernels have a broken driver that swaps the red and
+     * blue color select registers. */
+    if (!workaround_swap_rb) {
+        return PixelConverter(
+            bytesPerPixel(),
+            red_select.offset, green_select.offset, blue_select.offset,
+            red_select.size, green_select.size, blue_select.size,
+            byte_order);
+    } else {
+        return PixelConverter(
+            bytesPerPixel(),
+            blue_select.offset, green_select.offset, red_select.offset,
+            blue_select.size, green_select.size, red_select.size,
+            byte_order);
+    }
 }
 
 void
diff -r 10cac0f0f419 -r 48039363f67a src/dev/arm/hdlcd.hh
--- a/src/dev/arm/hdlcd.hh      Sat May 23 13:37:03 2015 +0100
+++ b/src/dev/arm/hdlcd.hh      Sat May 23 13:37:04 2015 +0100
@@ -486,6 +486,8 @@
 
     bool enableCapture;
 
+    const bool workaround_swap_rb;
+
   public:
     typedef HDLcdParams Params;
 
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev