From 413b4e7431b20d9b29dbf66d6677a60205ddd357 Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Fri, 12 Oct 2018 04:57:29 -0700 Subject: x86: Use little endian packet accessors. We know data is little endian, so we can use those accessors explicitly. Change-Id: I09aa7f1e525ad1346e932ce4a772b64bf59dc350 Reviewed-on: https://gem5-review.googlesource.com/c/13456 Reviewed-by: Andreas Sandberg Maintainer: Gabe Black --- src/dev/x86/cmos.cc | 8 ++++---- src/dev/x86/i8042.cc | 6 +++--- src/dev/x86/i82094aa.cc | 8 ++++---- src/dev/x86/i8237.cc | 2 +- src/dev/x86/i8254.cc | 8 ++++---- src/dev/x86/i8259.cc | 8 ++++---- src/dev/x86/speaker.cc | 4 ++-- 7 files changed, 22 insertions(+), 22 deletions(-) (limited to 'src/dev') diff --git a/src/dev/x86/cmos.cc b/src/dev/x86/cmos.cc index 16286f07e..41009c6d1 100644 --- a/src/dev/x86/cmos.cc +++ b/src/dev/x86/cmos.cc @@ -50,10 +50,10 @@ X86ISA::Cmos::read(PacketPtr pkt) switch(pkt->getAddr() - pioAddr) { case 0x0: - pkt->set(address); + pkt->setLE(address); break; case 0x1: - pkt->set(readRegister(address)); + pkt->setLE(readRegister(address)); break; default: panic("Read from undefined CMOS port.\n"); @@ -69,10 +69,10 @@ X86ISA::Cmos::write(PacketPtr pkt) switch(pkt->getAddr() - pioAddr) { case 0x0: - address = pkt->get(); + address = pkt->getLE(); break; case 0x1: - writeRegister(address, pkt->get()); + writeRegister(address, pkt->getLE()); break; default: panic("Write to undefined CMOS port.\n"); diff --git a/src/dev/x86/i8042.cc b/src/dev/x86/i8042.cc index 8ab0d4047..692f4afa0 100644 --- a/src/dev/x86/i8042.cc +++ b/src/dev/x86/i8042.cc @@ -118,10 +118,10 @@ X86ISA::I8042::read(PacketPtr pkt) if (addr == dataPort) { uint8_t data = readDataOut(); //DPRINTF(I8042, "Read from data port got %#02x.\n", data); - pkt->set(data); + pkt->setLE(data); } else if (addr == commandPort) { //DPRINTF(I8042, "Read status as %#02x.\n", (uint8_t)statusReg); - pkt->set((uint8_t)statusReg); + pkt->setLE((uint8_t)statusReg); } else { panic("Read from unrecognized port %#x.\n", addr); } @@ -134,7 +134,7 @@ X86ISA::I8042::write(PacketPtr pkt) { assert(pkt->getSize() == 1); Addr addr = pkt->getAddr(); - uint8_t data = pkt->get(); + uint8_t data = pkt->getLE(); if (addr == dataPort) { statusReg.commandLast = 0; switch (lastCommand) { diff --git a/src/dev/x86/i82094aa.cc b/src/dev/x86/i82094aa.cc index 51304f25a..2b09f14a5 100644 --- a/src/dev/x86/i82094aa.cc +++ b/src/dev/x86/i82094aa.cc @@ -103,10 +103,10 @@ X86ISA::I82094AA::read(PacketPtr pkt) Addr offset = pkt->getAddr() - pioAddr; switch(offset) { case 0: - pkt->set(regSel); + pkt->setLE(regSel); break; case 16: - pkt->set(readReg(regSel)); + pkt->setLE(readReg(regSel)); break; default: panic("Illegal read from I/O APIC.\n"); @@ -122,10 +122,10 @@ X86ISA::I82094AA::write(PacketPtr pkt) Addr offset = pkt->getAddr() - pioAddr; switch(offset) { case 0: - regSel = pkt->get(); + regSel = pkt->getLE(); break; case 16: - writeReg(regSel, pkt->get()); + writeReg(regSel, pkt->getLE()); break; default: panic("Illegal write to I/O APIC.\n"); diff --git a/src/dev/x86/i8237.cc b/src/dev/x86/i8237.cc index a9ef53c2c..2f17fae4a 100644 --- a/src/dev/x86/i8237.cc +++ b/src/dev/x86/i8237.cc @@ -100,7 +100,7 @@ X86ISA::I8237::write(PacketPtr pkt) panic("Write to i8237 request register unimplemented.\n"); case 0xa: { - uint8_t command = pkt->get(); + uint8_t command = pkt->getLE(); uint8_t select = bits(command, 1, 0); uint8_t bitVal = bits(command, 2); if (!bitVal) diff --git a/src/dev/x86/i8254.cc b/src/dev/x86/i8254.cc index 457db13f7..1c2780ab6 100644 --- a/src/dev/x86/i8254.cc +++ b/src/dev/x86/i8254.cc @@ -52,9 +52,9 @@ X86ISA::I8254::read(PacketPtr pkt) assert(pkt->getSize() == 1); Addr offset = pkt->getAddr() - pioAddr; if (offset < 3) { - pkt->set(pit.readCounter(offset)); + pkt->setLE(pit.readCounter(offset)); } else if (offset == 3) { - pkt->set(uint8_t(-1)); + pkt->setLE(uint8_t(-1)); } else { panic("Read from undefined i8254 register.\n"); } @@ -68,9 +68,9 @@ X86ISA::I8254::write(PacketPtr pkt) assert(pkt->getSize() == 1); Addr offset = pkt->getAddr() - pioAddr; if (offset < 3) { - pit.writeCounter(offset, pkt->get()); + pit.writeCounter(offset, pkt->getLE()); } else if (offset == 3) { - pit.writeControl(pkt->get()); + pit.writeControl(pkt->getLE()); } else { panic("Write to undefined i8254 register.\n"); } diff --git a/src/dev/x86/i8259.cc b/src/dev/x86/i8259.cc index 03c5cb948..4c3b9b272 100644 --- a/src/dev/x86/i8259.cc +++ b/src/dev/x86/i8259.cc @@ -56,15 +56,15 @@ X86ISA::I8259::read(PacketPtr pkt) case 0x0: if (readIRR) { DPRINTF(I8259, "Reading IRR as %#x.\n", IRR); - pkt->set(IRR); + pkt->setLE(IRR); } else { DPRINTF(I8259, "Reading ISR as %#x.\n", ISR); - pkt->set(ISR); + pkt->setLE(ISR); } break; case 0x1: DPRINTF(I8259, "Reading IMR as %#x.\n", IMR); - pkt->set(IMR); + pkt->setLE(IMR); break; } pkt->makeAtomicResponse(); @@ -75,7 +75,7 @@ Tick X86ISA::I8259::write(PacketPtr pkt) { assert(pkt->getSize() == 1); - uint8_t val = pkt->get(); + uint8_t val = pkt->getLE(); switch (pkt->getAddr() - pioAddr) { case 0x0: if (bits(val, 4)) { diff --git a/src/dev/x86/speaker.cc b/src/dev/x86/speaker.cc index 4d39903e2..d41f83ac5 100644 --- a/src/dev/x86/speaker.cc +++ b/src/dev/x86/speaker.cc @@ -48,7 +48,7 @@ X86ISA::Speaker::read(PacketPtr pkt) controlVal.gate ? "on" : "off", controlVal.speaker ? "on" : "off", controlVal.timer ? "on" : "off"); - pkt->set((uint8_t)controlVal); + pkt->setLE((uint8_t)controlVal); pkt->makeAtomicResponse(); return latency; } @@ -58,7 +58,7 @@ X86ISA::Speaker::write(PacketPtr pkt) { assert(pkt->getAddr() == pioAddr); assert(pkt->getSize() == 1); - SpeakerControl val = pkt->get(); + SpeakerControl val = pkt->getLE(); controlVal.gate = val.gate; //Change the gate value in the timer. if (!val.gate) -- cgit v1.2.3