commit 421c2e2b2186551cd9310b30d26a4805dfa0482e
author Gabe Black <gabeblack@google.com> Mon Apr 02 18:20:55 2018 -0700
committer Gabe Black <gabeblack@google.com> Tue Apr 10 11:46:52 2018 +0000
tree e2957c15bf048f7a6911a20499a1e907d4186e45
parent 3bd9a8968a2ccae1065526572e22cbdcd66b9cbd

dev: arm: SetScaling commands don't send parameter bytes.

These are single byte commands which change the mode of the mouse. They don't
take any additional parameters like the SetRate or SetResolution.

Change-Id: I29194916cfed5d3f4893947ef6d6cc636aee2419
Reviewed-on: https://gem5-review.googlesource.com/9701
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>

src/dev/arm/kmi.cc

diff --git a/src/dev/arm/kmi.cc b/src/dev/arm/kmi.cc
index 8db9254..8b373b8 100644
--- a/src/dev/arm/kmi.cc
+++ b/src/dev/arm/kmi.cc
@@ -188,8 +188,6 @@
       case SetResolution:
       case SetRate:
       case SetStatusLed:
-      case SetScaling1_1:
-      case SetScaling1_2:
         rxQueue.push_back(Ack);
         ackNext = 1;
         break;
@@ -211,6 +209,8 @@
       case Disable:
       case Enable:
       case SetDefaults:
+      case SetScaling1_1:
+      case SetScaling1_2:
         rxQueue.push_back(Ack);
         break;
       case StatusRequest: