summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGabe Black <gabeblack@google.com>2018-10-12 04:58:50 -0700
committerGabe Black <gabeblack@google.com>2018-10-12 23:45:51 +0000
commit9125a43f624653e6238dbb8713658cae2c5d43cd (patch)
tree268bab703026e5d6497492c97b736f125e875c59
parent413b4e7431b20d9b29dbf66d6677a60205ddd357 (diff)
downloadgem5-9125a43f624653e6238dbb8713658cae2c5d43cd.tar.xz
arm: Use little endian packet accessors.
We know data is little endian, so we can use those accessors explicitly. Change-Id: Iee337109fcda134e1ac5a700e5141fd7060f9c45 Reviewed-on: https://gem5-review.googlesource.com/c/13457 Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com> Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>
-rw-r--r--src/dev/arm/a9scu.cc4
-rw-r--r--src/dev/arm/amba_device.cc2
-rw-r--r--src/dev/arm/amba_fake.cc2
-rw-r--r--src/dev/arm/energy_ctrl.cc6
-rw-r--r--src/dev/arm/generic_timer.cc6
-rw-r--r--src/dev/arm/gic_v2.cc16
-rw-r--r--src/dev/arm/gic_v2m.cc8
-rw-r--r--src/dev/arm/gpu_nomali.cc4
-rw-r--r--src/dev/arm/hdlcd.cc4
-rw-r--r--src/dev/arm/kmi.cc2
-rw-r--r--src/dev/arm/pl011.cc16
-rw-r--r--src/dev/arm/pl111.cc16
-rw-r--r--src/dev/arm/rtc_pl031.cc16
-rw-r--r--src/dev/arm/rv_ctrl.cc46
-rw-r--r--src/dev/arm/timer_a9global.cc26
-rw-r--r--src/dev/arm/timer_cpulocal.cc32
-rw-r--r--src/dev/arm/timer_sp804.cc24
-rw-r--r--src/dev/arm/ufs_device.cc8
-rw-r--r--src/dev/arm/vgic.cc47
-rw-r--r--src/dev/arm/vio_mmio.cc6
20 files changed, 148 insertions, 143 deletions
diff --git a/src/dev/arm/a9scu.cc b/src/dev/arm/a9scu.cc
index 86a50ae8e..677101b36 100644
--- a/src/dev/arm/a9scu.cc
+++ b/src/dev/arm/a9scu.cc
@@ -59,7 +59,7 @@ A9SCU::read(PacketPtr pkt)
switch(daddr) {
case Control:
- pkt->set(1); // SCU already enabled
+ pkt->setLE(1); // SCU already enabled
break;
case Config:
/* Without making a completely new SCU, we can use the core count field
@@ -75,7 +75,7 @@ A9SCU::read(PacketPtr pkt)
int smp_bits, core_cnt;
smp_bits = power(2,sys->numContexts()) - 1;
core_cnt = sys->numContexts() - 1;
- pkt->set(smp_bits << 4 | core_cnt);
+ pkt->setLE(smp_bits << 4 | core_cnt);
break;
default:
// Only configuration register is implemented
diff --git a/src/dev/arm/amba_device.cc b/src/dev/arm/amba_device.cc
index d9acc9a00..a8404a42c 100644
--- a/src/dev/arm/amba_device.cc
+++ b/src/dev/arm/amba_device.cc
@@ -83,6 +83,6 @@ AmbaDevice::readId(PacketPtr pkt, uint64_t amba_id, Addr pio_addr)
(amba_id >> byte) & 0xFF,
pkt->getAddr() - pio_addr, byte);
assert(pkt->getSize() == 4);
- pkt->set<uint32_t>((amba_id >> byte) & 0xFF);
+ pkt->setLE<uint32_t>((amba_id >> byte) & 0xFF);
return true;
}
diff --git a/src/dev/arm/amba_fake.cc b/src/dev/arm/amba_fake.cc
index 475ff768a..9c14140fb 100644
--- a/src/dev/arm/amba_fake.cc
+++ b/src/dev/arm/amba_fake.cc
@@ -61,7 +61,7 @@ AmbaFake::read(PacketPtr pkt)
DPRINTF(AMBA, " read register %#x\n", daddr);
- pkt->set<uint32_t>(0);
+ pkt->setLE<uint32_t>(0);
if (!readId(pkt, ambaId, pioAddr) && !params()->ignore_access)
panic("Tried to read AmbaFake at offset %#x that doesn't exist\n", daddr);
diff --git a/src/dev/arm/energy_ctrl.cc b/src/dev/arm/energy_ctrl.cc
index 9cfce3a48..ba2d3e942 100644
--- a/src/dev/arm/energy_ctrl.cc
+++ b/src/dev/arm/energy_ctrl.cc
@@ -72,7 +72,7 @@ EnergyCtrl::read(PacketPtr pkt)
if (!dvfsHandler->isEnabled()) {
// NB: Zero is a good response if the handler is disabled
- pkt->set<uint32_t>(0);
+ pkt->setLE<uint32_t>(0);
warn_once("EnergyCtrl: Disabled handler, ignoring read from reg %i\n",
reg);
DPRINTF(EnergyCtrl, "dvfs handler disabled, return 0 for read from "\
@@ -141,7 +141,7 @@ EnergyCtrl::read(PacketPtr pkt)
panic("Tried to read EnergyCtrl at offset %#x / reg %i\n", daddr,
reg);
}
- pkt->set<uint32_t>(result);
+ pkt->setLE<uint32_t>(result);
pkt->makeAtomicResponse();
return pioDelay;
}
@@ -153,7 +153,7 @@ EnergyCtrl::write(PacketPtr pkt)
assert(pkt->getSize() == 4);
uint32_t data;
- data = pkt->get<uint32_t>();
+ data = pkt->getLE<uint32_t>();
Addr daddr = pkt->getAddr() - pioAddr;
assert((daddr & 3) == 0);
diff --git a/src/dev/arm/generic_timer.cc b/src/dev/arm/generic_timer.cc
index 92c39c2ae..0994f6dd4 100644
--- a/src/dev/arm/generic_timer.cc
+++ b/src/dev/arm/generic_timer.cc
@@ -583,9 +583,9 @@ GenericTimerMem::read(PacketPtr pkt)
DPRINTF(Timer, "Read 0x%x <- 0x%x(%i)\n", value, addr, size);
if (size == 8) {
- pkt->set<uint64_t>(value);
+ pkt->setLE<uint64_t>(value);
} else if (size == 4) {
- pkt->set<uint32_t>(value);
+ pkt->setLE<uint32_t>(value);
} else {
panic("Unexpected access size: %i\n", size);
}
@@ -602,7 +602,7 @@ GenericTimerMem::write(PacketPtr pkt)
const Addr addr(pkt->getAddr());
const uint64_t value(size == 8 ?
- pkt->get<uint64_t>() : pkt->get<uint32_t>());
+ pkt->getLE<uint64_t>() : pkt->getLE<uint32_t>());
DPRINTF(Timer, "Write 0x%x -> 0x%x(%i)\n", value, addr, size);
if (ctrlRange.contains(addr)) {
diff --git a/src/dev/arm/gic_v2.cc b/src/dev/arm/gic_v2.cc
index 99124b08c..a24e56391 100644
--- a/src/dev/arm/gic_v2.cc
+++ b/src/dev/arm/gic_v2.cc
@@ -145,13 +145,13 @@ GicV2::readDistributor(PacketPtr pkt)
switch (pkt->getSize()) {
case 1:
- pkt->set<uint8_t>(resp);
+ pkt->setLE<uint8_t>(resp);
break;
case 2:
- pkt->set<uint16_t>(resp);
+ pkt->setLE<uint16_t>(resp);
break;
case 4:
- pkt->set<uint32_t>(resp);
+ pkt->setLE<uint32_t>(resp);
break;
default:
panic("Invalid size while reading Distributor regs in GIC: %d\n",
@@ -296,7 +296,7 @@ GicV2::readCpu(PacketPtr pkt)
DPRINTF(GIC, "gic cpu read register %#x cpu context: %d\n", daddr,
ctx);
- pkt->set<uint32_t>(readCpu(ctx, daddr));
+ pkt->setLE<uint32_t>(readCpu(ctx, daddr));
pkt->makeAtomicResponse();
return cpuPioDelay;
@@ -395,13 +395,13 @@ GicV2::writeDistributor(PacketPtr pkt)
switch (data_sz)
{
case 1:
- pkt_data = pkt->get<uint8_t>();
+ pkt_data = pkt->getLE<uint8_t>();
break;
case 2:
- pkt_data = pkt->get<uint16_t>();
+ pkt_data = pkt->getLE<uint16_t>();
break;
case 4:
- pkt_data = pkt->get<uint32_t>();
+ pkt_data = pkt->getLE<uint32_t>();
break;
default:
panic("Invalid size when writing to priority regs in Gic: %d\n",
@@ -558,7 +558,7 @@ GicV2::writeCpu(PacketPtr pkt)
assert(pkt->req->hasContextId());
const ContextID ctx = pkt->req->contextId();
- const uint32_t data = pkt->get<uint32_t>();
+ const uint32_t data = pkt->getLE<uint32_t>();
DPRINTF(GIC, "gic cpu write register cpu:%d %#x val: %#x\n",
ctx, daddr, data);
diff --git a/src/dev/arm/gic_v2m.cc b/src/dev/arm/gic_v2m.cc
index d90cb986e..9ae0f68ee 100644
--- a/src/dev/arm/gic_v2m.cc
+++ b/src/dev/arm/gic_v2m.cc
@@ -114,18 +114,18 @@ Gicv2m::read(PacketPtr pkt)
switch (offset) {
case MSI_TYPER:
- pkt->set<uint32_t>((frames[frame]->spi_base << 16) |
+ pkt->setLE<uint32_t>((frames[frame]->spi_base << 16) |
frames[frame]->spi_len);
break;
case PER_ID4:
- pkt->set<uint32_t>(0x4 | ((4+log2framenum) << 4));
+ pkt->setLE<uint32_t>(0x4 | ((4+log2framenum) << 4));
// Nr of 4KB blocks used by component. This is messy as frames are 64K
// (16, ie 2^4) and we should assert we're given a Po2 number of frames.
break;
default:
DPRINTF(GICV2M, "GICv2m: Read of unk reg %#x\n", offset);
- pkt->set<uint32_t>(0);
+ pkt->setLE<uint32_t>(0);
};
pkt->makeAtomicResponse();
@@ -144,7 +144,7 @@ Gicv2m::write(PacketPtr pkt)
if (offset == MSI_SETSPI_NSR) {
/* Is payload SPI number within range? */
- uint32_t m = pkt->get<uint32_t>();
+ uint32_t m = pkt->getLE<uint32_t>();
if (m >= frames[frame]->spi_base &&
m < (frames[frame]->spi_base + frames[frame]->spi_len)) {
DPRINTF(GICV2M, "GICv2m: Frame %d raising MSI %d\n", frame, m);
diff --git a/src/dev/arm/gpu_nomali.cc b/src/dev/arm/gpu_nomali.cc
index a6c3e29e5..c8630c707 100644
--- a/src/dev/arm/gpu_nomali.cc
+++ b/src/dev/arm/gpu_nomali.cc
@@ -160,7 +160,7 @@ NoMaliGpu::read(PacketPtr pkt)
else if (addr & 0x3)
panic("Unaligned GPU read: %i\n", size);
- pkt->set<uint32_t>(readReg(addr));
+ pkt->setLE<uint32_t>(readReg(addr));
pkt->makeResponse();
return 0;
@@ -181,7 +181,7 @@ NoMaliGpu::write(PacketPtr pkt)
else if (addr & 0x3)
panic("Unaligned GPU write: %i\n", size);
- writeReg(addr, pkt->get<uint32_t>());
+ writeReg(addr, pkt->getLE<uint32_t>());
pkt->makeAtomicResponse();
return 0;
diff --git a/src/dev/arm/hdlcd.cc b/src/dev/arm/hdlcd.cc
index 20dc7d44e..113c1c358 100644
--- a/src/dev/arm/hdlcd.cc
+++ b/src/dev/arm/hdlcd.cc
@@ -248,7 +248,7 @@ HDLcd::read(PacketPtr pkt)
const uint32_t data(readReg(daddr));
DPRINTF(HDLcd, "read register 0x%04x: 0x%x\n", daddr, data);
- pkt->set<uint32_t>(data);
+ pkt->setLE<uint32_t>(data);
pkt->makeAtomicResponse();
return pioDelay;
}
@@ -264,7 +264,7 @@ HDLcd::write(PacketPtr pkt)
panic_if(pkt->getSize() != 4,
"Unhandled read size (address: 0x.4x, size: %u)",
daddr, pkt->getSize());
- const uint32_t data(pkt->get<uint32_t>());
+ const uint32_t data(pkt->getLE<uint32_t>());
DPRINTF(HDLcd, "write register 0x%04x: 0x%x\n", daddr, data);
writeReg(daddr, data);
diff --git a/src/dev/arm/kmi.cc b/src/dev/arm/kmi.cc
index 8055afe2e..3cb598d68 100644
--- a/src/dev/arm/kmi.cc
+++ b/src/dev/arm/kmi.cc
@@ -98,7 +98,7 @@ Pl050::read(PacketPtr pkt)
default:
if (readId(pkt, ambaId, pioAddr)) {
// Hack for variable size accesses
- data = pkt->get<uint32_t>();
+ data = pkt->getLE<uint32_t>();
break;
}
diff --git a/src/dev/arm/pl011.cc b/src/dev/arm/pl011.cc
index 5ca9f6087..fe73d9674 100644
--- a/src/dev/arm/pl011.cc
+++ b/src/dev/arm/pl011.cc
@@ -136,7 +136,7 @@ Pl011::read(PacketPtr pkt)
default:
if (readId(pkt, AMBA_ID, pioAddr)) {
// Hack for variable size accesses
- data = pkt->get<uint32_t>();
+ data = pkt->getLE<uint32_t>();
break;
}
@@ -146,13 +146,13 @@ Pl011::read(PacketPtr pkt)
switch(pkt->getSize()) {
case 1:
- pkt->set<uint8_t>(data);
+ pkt->setLE<uint8_t>(data);
break;
case 2:
- pkt->set<uint16_t>(data);
+ pkt->setLE<uint16_t>(data);
break;
case 4:
- pkt->set<uint32_t>(data);
+ pkt->setLE<uint32_t>(data);
break;
default:
panic("Uart read size too big?\n");
@@ -173,7 +173,7 @@ Pl011::write(PacketPtr pkt)
Addr daddr = pkt->getAddr() - pioAddr;
DPRINTF(Uart, " write register %#x value %#x size=%d\n", daddr,
- pkt->get<uint8_t>(), pkt->getSize());
+ pkt->getLE<uint8_t>(), pkt->getSize());
// use a temporary data since the uart registers are read/written with
// different size operations
@@ -182,13 +182,13 @@ Pl011::write(PacketPtr pkt)
switch(pkt->getSize()) {
case 1:
- data = pkt->get<uint8_t>();
+ data = pkt->getLE<uint8_t>();
break;
case 2:
- data = pkt->get<uint16_t>();
+ data = pkt->getLE<uint16_t>();
break;
case 4:
- data = pkt->get<uint32_t>();
+ data = pkt->getLE<uint32_t>();
break;
default:
panic("Uart write size too big?\n");
diff --git a/src/dev/arm/pl111.cc b/src/dev/arm/pl111.cc
index 344eb0864..7560ec4b9 100644
--- a/src/dev/arm/pl111.cc
+++ b/src/dev/arm/pl111.cc
@@ -185,7 +185,7 @@ Pl111::read(PacketPtr pkt)
default:
if (readId(pkt, AMBA_ID, pioAddr)) {
// Hack for variable size accesses
- data = pkt->get<uint32_t>();
+ data = pkt->getLE<uint32_t>();
break;
} else if (daddr >= CrsrImage && daddr <= 0xBFC) {
// CURSOR IMAGE
@@ -208,13 +208,13 @@ Pl111::read(PacketPtr pkt)
switch(pkt->getSize()) {
case 1:
- pkt->set<uint8_t>(data);
+ pkt->setLE<uint8_t>(data);
break;
case 2:
- pkt->set<uint16_t>(data);
+ pkt->setLE<uint16_t>(data);
break;
case 4:
- pkt->set<uint32_t>(data);
+ pkt->setLE<uint32_t>(data);
break;
default:
panic("CLCD controller read size too big?\n");
@@ -236,13 +236,13 @@ Pl111::write(PacketPtr pkt)
switch(pkt->getSize()) {
case 1:
- data = pkt->get<uint8_t>();
+ data = pkt->getLE<uint8_t>();
break;
case 2:
- data = pkt->get<uint16_t>();
+ data = pkt->getLE<uint16_t>();
break;
case 4:
- data = pkt->get<uint32_t>();
+ data = pkt->getLE<uint32_t>();
break;
default:
panic("PL111 CLCD controller write size too big?\n");
@@ -255,7 +255,7 @@ Pl111::write(PacketPtr pkt)
Addr daddr = pkt->getAddr() - pioAddr;
DPRINTF(PL111, " write register %#x value %#x size=%d\n", daddr,
- pkt->get<uint8_t>(), pkt->getSize());
+ pkt->getLE<uint8_t>(), pkt->getSize());
switch (daddr) {
case LcdTiming0:
diff --git a/src/dev/arm/rtc_pl031.cc b/src/dev/arm/rtc_pl031.cc
index 5b6ce5288..9abb77629 100644
--- a/src/dev/arm/rtc_pl031.cc
+++ b/src/dev/arm/rtc_pl031.cc
@@ -92,7 +92,7 @@ PL031::read(PacketPtr pkt)
default:
if (readId(pkt, ambaId, pioAddr)) {
// Hack for variable sized access
- data = pkt->get<uint32_t>();
+ data = pkt->getLE<uint32_t>();
break;
}
panic("Tried to read PL031 at offset %#x that doesn't exist\n", daddr);
@@ -101,13 +101,13 @@ PL031::read(PacketPtr pkt)
switch(pkt->getSize()) {
case 1:
- pkt->set<uint8_t>(data);
+ pkt->setLE<uint8_t>(data);
break;
case 2:
- pkt->set<uint16_t>(data);
+ pkt->setLE<uint16_t>(data);
break;
case 4:
- pkt->set<uint32_t>(data);
+ pkt->setLE<uint32_t>(data);
break;
default:
panic("Uart read size too big?\n");
@@ -131,22 +131,22 @@ PL031::write(PacketPtr pkt)
case DataReg:
break;
case MatchReg:
- matchVal = pkt->get<uint32_t>();
+ matchVal = pkt->getLE<uint32_t>();
resyncMatch();
break;
case LoadReg:
lastWrittenTick = curTick();
- timeVal = pkt->get<uint32_t>();
+ timeVal = pkt->getLE<uint32_t>();
loadVal = timeVal;
resyncMatch();
break;
case ControlReg:
break; // Can't stop when started
case IntMask:
- maskInt = pkt->get<uint32_t>();
+ maskInt = pkt->getLE<uint32_t>();
break;
case IntClear:
- if (pkt->get<uint32_t>()) {
+ if (pkt->getLE<uint32_t>()) {
rawInt = false;
pendingInt = false;
}
diff --git a/src/dev/arm/rv_ctrl.cc b/src/dev/arm/rv_ctrl.cc
index 954be6597..9f9e51f09 100644
--- a/src/dev/arm/rv_ctrl.cc
+++ b/src/dev/arm/rv_ctrl.cc
@@ -61,66 +61,66 @@ RealViewCtrl::read(PacketPtr pkt)
switch(daddr) {
case ProcId0:
- pkt->set(params()->proc_id0);
+ pkt->setLE(params()->proc_id0);
break;
case ProcId1:
- pkt->set(params()->proc_id1);
+ pkt->setLE(params()->proc_id1);
break;
case Clock24:
Tick clk;
clk = SimClock::Float::MHz * curTick() * 24;
- pkt->set((uint32_t)(clk));
+ pkt->setLE((uint32_t)(clk));
break;
case Clock100:
Tick clk100;
clk100 = SimClock::Float::MHz * curTick() * 100;
- pkt->set((uint32_t)(clk100));
+ pkt->setLE((uint32_t)(clk100));
break;
case Flash:
- pkt->set<uint32_t>(0);
+ pkt->setLE<uint32_t>(0);
break;
case Clcd:
- pkt->set<uint32_t>(0x00001F00);
+ pkt->setLE<uint32_t>(0x00001F00);
break;
case Osc0:
- pkt->set<uint32_t>(0x00012C5C);
+ pkt->setLE<uint32_t>(0x00012C5C);
break;
case Osc1:
- pkt->set<uint32_t>(0x00002CC0);
+ pkt->setLE<uint32_t>(0x00002CC0);
break;
case Osc2:
- pkt->set<uint32_t>(0x00002C75);
+ pkt->setLE<uint32_t>(0x00002C75);
break;
case Osc3:
- pkt->set<uint32_t>(0x00020211);
+ pkt->setLE<uint32_t>(0x00020211);
break;
case Osc4:
- pkt->set<uint32_t>(0x00002C75);
+ pkt->setLE<uint32_t>(0x00002C75);
break;
case Lock:
- pkt->set<uint32_t>(sysLock);
+ pkt->setLE<uint32_t>(sysLock);
break;
case Flags:
- pkt->set<uint32_t>(flags);
+ pkt->setLE<uint32_t>(flags);
break;
case IdReg:
- pkt->set<uint32_t>(params()->idreg);
+ pkt->setLE<uint32_t>(params()->idreg);
break;
case CfgStat:
- pkt->set<uint32_t>(1);
+ pkt->setLE<uint32_t>(1);
break;
case CfgData:
- pkt->set<uint32_t>(scData);
+ pkt->setLE<uint32_t>(scData);
DPRINTF(RVCTRL, "Read %#x from SCReg\n", scData);
break;
case CfgCtrl:
- pkt->set<uint32_t>(0); // not busy
+ pkt->setLE<uint32_t>(0); // not busy
DPRINTF(RVCTRL, "Read 0 from CfgCtrl\n");
break;
default:
warn("Tried to read RealView I/O at offset %#x that doesn't exist\n",
daddr);
- pkt->set<uint32_t>(0);
+ pkt->setLE<uint32_t>(0);
break;
}
pkt->makeAtomicResponse();
@@ -144,26 +144,26 @@ RealViewCtrl::write(PacketPtr pkt)
case Osc4:
break;
case Lock:
- sysLock.lockVal = pkt->get<uint16_t>();
+ sysLock.lockVal = pkt->getLE<uint16_t>();
break;
case ResetCtl:
// Ignore writes to reset control
warn_once("Ignoring write to reset control\n");
break;
case Flags:
- flags = pkt->get<uint32_t>();
+ flags = pkt->getLE<uint32_t>();
break;
case FlagsClr:
flags = 0;
break;
case CfgData:
- scData = pkt->get<uint32_t>();
+ scData = pkt->getLE<uint32_t>();
break;
case CfgCtrl: {
// A request is being submitted to read/write the system control
// registers. See
// http://infocenter.arm.com/help/topic/com.arm.doc.dui0447h/CACDEFGH.html
- CfgCtrlReg req = pkt->get<uint32_t>();
+ CfgCtrlReg req = pkt->getLE<uint32_t>();
if (!req.start) {
DPRINTF(RVCTRL, "SCReg: write %#x to ctrl but not starting\n",
req);
@@ -195,7 +195,7 @@ RealViewCtrl::write(PacketPtr pkt)
case CfgStat: // Weird to write this
default:
warn("Tried to write RVIO at offset %#x (data %#x) that doesn't exist\n",
- daddr, pkt->get<uint32_t>());
+ daddr, pkt->getLE<uint32_t>());
break;
}
pkt->makeAtomicResponse();
diff --git a/src/dev/arm/timer_a9global.cc b/src/dev/arm/timer_a9global.cc
index 742ed76cd..1d4089f7f 100644
--- a/src/dev/arm/timer_a9global.cc
+++ b/src/dev/arm/timer_a9global.cc
@@ -94,19 +94,19 @@ A9GlobalTimer::Timer::read(PacketPtr pkt, Addr daddr)
case CounterRegLow32:
time = getTimeCounterFromTicks(curTick());
DPRINTF(Timer, "-- returning lower 32-bits of counter: %u\n", time);
- pkt->set<uint32_t>(time);
+ pkt->setLE<uint32_t>(time);
break;
case CounterRegHigh32:
time = getTimeCounterFromTicks(curTick());
time >>= 32;
DPRINTF(Timer, "-- returning upper 32-bits of counter: %u\n", time);
- pkt->set<uint32_t>(time);
+ pkt->setLE<uint32_t>(time);
break;
case ControlReg:
- pkt->set<uint32_t>(control);
+ pkt->setLE<uint32_t>(control);
break;
case IntStatusReg:
- pkt->set<uint32_t>(rawInt);
+ pkt->setLE<uint32_t>(rawInt);
break;
case CmpValRegLow32:
DPRINTF(Timer, "Event schedule for %d, clock=%d, prescale=%d\n",
@@ -117,7 +117,7 @@ A9GlobalTimer::Timer::read(PacketPtr pkt, Addr daddr)
time = 0;
}
DPRINTF(Timer, "-- returning lower 32-bits of comparator: %u\n", time);
- pkt->set<uint32_t>(time);
+ pkt->setLE<uint32_t>(time);
break;
case CmpValRegHigh32:
DPRINTF(Timer, "Event schedule for %d, clock=%d, prescale=%d\n",
@@ -129,17 +129,17 @@ A9GlobalTimer::Timer::read(PacketPtr pkt, Addr daddr)
time = 0;
}
DPRINTF(Timer, "-- returning upper 32-bits of comparator: %u\n", time);
- pkt->set<uint32_t>(time);
+ pkt->setLE<uint32_t>(time);
break;
case AutoIncrementReg:
- pkt->set<uint32_t>(autoIncValue);
+ pkt->setLE<uint32_t>(autoIncValue);
break;
default:
panic("Tried to read A9GlobalTimer at offset %#x\n", daddr);
break;
}
DPRINTF(Timer, "Reading %#x from A9GlobalTimer at offset: %#x\n",
- pkt->get<uint32_t>(), daddr);
+ pkt->getLE<uint32_t>(), daddr);
}
Tick
@@ -165,7 +165,7 @@ void
A9GlobalTimer::Timer::write(PacketPtr pkt, Addr daddr)
{
DPRINTF(Timer, "Writing %#x to A9GlobalTimer at offset: %#x\n",
- pkt->get<uint32_t>(), daddr);
+ pkt->getLE<uint32_t>(), daddr);
switch (daddr) {
case CounterRegLow32:
case CounterRegHigh32:
@@ -176,7 +176,7 @@ A9GlobalTimer::Timer::write(PacketPtr pkt, Addr daddr)
bool old_cmpEnable;
old_enable = control.enable;
old_cmpEnable = control.cmpEnable;
- control = pkt->get<uint32_t>();
+ control = pkt->getLE<uint32_t>();
if ((old_enable == 0) && control.enable)
restartCounter();
if ((old_cmpEnable == 0) && control.cmpEnable)
@@ -193,14 +193,14 @@ A9GlobalTimer::Timer::write(PacketPtr pkt, Addr daddr)
break;
case CmpValRegLow32:
cmpVal &= 0xFFFFFFFF00000000ULL;
- cmpVal |= (uint64_t)pkt->get<uint32_t>();
+ cmpVal |= (uint64_t)pkt->getLE<uint32_t>();
break;
case CmpValRegHigh32:
cmpVal &= 0x00000000FFFFFFFFULL;
- cmpVal |= ((uint64_t)pkt->get<uint32_t>() << 32);
+ cmpVal |= ((uint64_t)pkt->getLE<uint32_t>() << 32);
break;
case AutoIncrementReg:
- autoIncValue = pkt->get<uint32_t>();
+ autoIncValue = pkt->getLE<uint32_t>();
break;
default:
panic("Tried to write A9GlobalTimer at offset %#x\n", daddr);
diff --git a/src/dev/arm/timer_cpulocal.cc b/src/dev/arm/timer_cpulocal.cc
index 9e9946b1f..a0825232f 100644
--- a/src/dev/arm/timer_cpulocal.cc
+++ b/src/dev/arm/timer_cpulocal.cc
@@ -116,7 +116,7 @@ CpuLocalTimer::Timer::read(PacketPtr pkt, Addr daddr)
switch(daddr) {
case TimerLoadReg:
- pkt->set<uint32_t>(timerLoadValue);
+ pkt->setLE<uint32_t>(timerLoadValue);
break;
case TimerCounterReg:
DPRINTF(Timer, "Event schedule for timer %d, clock=%d, prescale=%d\n",
@@ -126,16 +126,16 @@ CpuLocalTimer::Timer::read(PacketPtr pkt, Addr daddr)
time = time / parent->clockPeriod() /
power(16, timerControl.prescalar);
DPRINTF(Timer, "-- returning counter at %d\n", time);
- pkt->set<uint32_t>(time);
+ pkt->setLE<uint32_t>(time);
break;
case TimerControlReg:
- pkt->set<uint32_t>(timerControl);
+ pkt->setLE<uint32_t>(timerControl);
break;
case TimerIntStatusReg:
- pkt->set<uint32_t>(rawIntTimer);
+ pkt->setLE<uint32_t>(rawIntTimer);
break;
case WatchdogLoadReg:
- pkt->set<uint32_t>(watchdogLoadValue);
+ pkt->setLE<uint32_t>(watchdogLoadValue);
break;
case WatchdogCounterReg:
DPRINTF(Timer,
@@ -146,16 +146,16 @@ CpuLocalTimer::Timer::read(PacketPtr pkt, Addr daddr)
time = time / parent->clockPeriod() /
power(16, watchdogControl.prescalar);
DPRINTF(Timer, "-- returning counter at %d\n", time);
- pkt->set<uint32_t>(time);
+ pkt->setLE<uint32_t>(time);
break;
case WatchdogControlReg:
- pkt->set<uint32_t>(watchdogControl);
+ pkt->setLE<uint32_t>(watchdogControl);
break;
case WatchdogIntStatusReg:
- pkt->set<uint32_t>(rawIntWatchdog);
+ pkt->setLE<uint32_t>(rawIntWatchdog);
break;
case WatchdogResetStatusReg:
- pkt->set<uint32_t>(rawResetWatchdog);
+ pkt->setLE<uint32_t>(rawResetWatchdog);
break;
case WatchdogDisableReg:
panic("Tried to read from WatchdogDisableRegister\n");
@@ -197,16 +197,16 @@ CpuLocalTimer::Timer::write(PacketPtr pkt, Addr daddr)
case TimerLoadReg:
// Writing to this register also resets the counter register and
// starts decrementing if the counter is enabled.
- timerLoadValue = pkt->get<uint32_t>();
+ timerLoadValue = pkt->getLE<uint32_t>();
restartTimerCounter(timerLoadValue);
break;
case TimerCounterReg:
// Can be written, doesn't start counting unless the timer is enabled
- restartTimerCounter(pkt->get<uint32_t>());
+ restartTimerCounter(pkt->getLE<uint32_t>());
break;
case TimerControlReg:
old_enable = timerControl.enable;
- timerControl = pkt->get<uint32_t>();
+ timerControl = pkt->getLE<uint32_t>();
if ((old_enable == 0) && timerControl.enable)
restartTimerCounter(timerLoadValue);
break;
@@ -218,19 +218,19 @@ CpuLocalTimer::Timer::write(PacketPtr pkt, Addr daddr)
}
break;
case WatchdogLoadReg:
- watchdogLoadValue = pkt->get<uint32_t>();
+ watchdogLoadValue = pkt->getLE<uint32_t>();
restartWatchdogCounter(watchdogLoadValue);
break;
case WatchdogCounterReg:
// Can't be written when in watchdog mode, but can in timer mode
if (!watchdogControl.watchdogMode) {
- restartWatchdogCounter(pkt->get<uint32_t>());
+ restartWatchdogCounter(pkt->getLE<uint32_t>());
}
break;
case WatchdogControlReg:
old_enable = watchdogControl.enable;
old_wd_mode = watchdogControl.watchdogMode;
- watchdogControl = pkt->get<uint32_t>();
+ watchdogControl = pkt->getLE<uint32_t>();
if ((old_enable == 0) && watchdogControl.enable)
restartWatchdogCounter(watchdogLoadValue);
// cannot disable watchdog using control register
@@ -250,7 +250,7 @@ CpuLocalTimer::Timer::write(PacketPtr pkt, Addr daddr)
break;
case WatchdogDisableReg:
old_val = watchdogDisableReg;
- watchdogDisableReg = pkt->get<uint32_t>();
+ watchdogDisableReg = pkt->getLE<uint32_t>();
// if this sequence is observed, turn off watchdog mode
if (old_val == 0x12345678 && watchdogDisableReg == 0x87654321)
watchdogControl.watchdogMode = 0;
diff --git a/src/dev/arm/timer_sp804.cc b/src/dev/arm/timer_sp804.cc
index f28ba7bf0..0549c6318 100644
--- a/src/dev/arm/timer_sp804.cc
+++ b/src/dev/arm/timer_sp804.cc
@@ -86,7 +86,7 @@ Sp804::Timer::read(PacketPtr pkt, Addr daddr)
{
switch(daddr) {
case LoadReg:
- pkt->set<uint32_t>(loadValue);
+ pkt->setLE<uint32_t>(loadValue);
break;
case CurrentReg:
DPRINTF(Timer, "Event schedule for %d, clock=%d, prescale=%d\n",
@@ -95,25 +95,26 @@ Sp804::Timer::read(PacketPtr pkt, Addr daddr)
time = zeroEvent.when() - curTick();
time = time / clock / power(16, control.timerPrescale);
DPRINTF(Timer, "-- returning counter at %d\n", time);
- pkt->set<uint32_t>(time);
+ pkt->setLE<uint32_t>(time);
break;
case ControlReg:
- pkt->set<uint32_t>(control);
+ pkt->setLE<uint32_t>(control);
break;
case RawISR:
- pkt->set<uint32_t>(rawInt);
+ pkt->setLE<uint32_t>(rawInt);
break;
case MaskedISR:
- pkt->set<uint32_t>(pendingInt);
+ pkt->setLE<uint32_t>(pendingInt);
break;
case BGLoad:
- pkt->set<uint32_t>(loadValue);
+ pkt->setLE<uint32_t>(loadValue);
break;
default:
panic("Tried to read SP804 timer at offset %#x\n", daddr);
break;
}
- DPRINTF(Timer, "Reading %#x from Timer at offset: %#x\n", pkt->get<uint32_t>(), daddr);
+ DPRINTF(Timer, "Reading %#x from Timer at offset: %#x\n",
+ pkt->getLE<uint32_t>(), daddr);
}
Tick
@@ -137,10 +138,11 @@ Sp804::write(PacketPtr pkt)
void
Sp804::Timer::write(PacketPtr pkt, Addr daddr)
{
- DPRINTF(Timer, "Writing %#x to Timer at offset: %#x\n", pkt->get<uint32_t>(), daddr);
+ DPRINTF(Timer, "Writing %#x to Timer at offset: %#x\n",
+ pkt->getLE<uint32_t>(), daddr);
switch (daddr) {
case LoadReg:
- loadValue = pkt->get<uint32_t>();
+ loadValue = pkt->getLE<uint32_t>();
restartCounter(loadValue);
break;
case CurrentReg:
@@ -149,7 +151,7 @@ Sp804::Timer::write(PacketPtr pkt, Addr daddr)
case ControlReg:
bool old_enable;
old_enable = control.timerEnable;
- control = pkt->get<uint32_t>();
+ control = pkt->getLE<uint32_t>();
if ((old_enable == 0) && control.timerEnable)
restartCounter(loadValue);
break;
@@ -162,7 +164,7 @@ Sp804::Timer::write(PacketPtr pkt, Addr daddr)
}
break;
case BGLoad:
- loadValue = pkt->get<uint32_t>();
+ loadValue = pkt->getLE<uint32_t>();
break;
default:
panic("Tried to write SP804 timer at offset %#x\n", daddr);
diff --git a/src/dev/arm/ufs_device.cc b/src/dev/arm/ufs_device.cc
index 137ad8c27..cc2800072 100644
--- a/src/dev/arm/ufs_device.cc
+++ b/src/dev/arm/ufs_device.cc
@@ -1053,7 +1053,7 @@ UFSHostDevice::read(PacketPtr pkt)
break;
}
- pkt->set<uint32_t>(data);
+ pkt->setLE<uint32_t>(data);
pkt->makeResponse();
return pioDelay;
}
@@ -1071,15 +1071,15 @@ UFSHostDevice::write(PacketPtr pkt)
switch (pkt->getSize()) {
case 1:
- data = pkt->get<uint8_t>();
+ data = pkt->getLE<uint8_t>();
break;
case 2:
- data = pkt->get<uint16_t>();
+ data = pkt->getLE<uint16_t>();
break;
case 4:
- data = pkt->get<uint32_t>();
+ data = pkt->getLE<uint32_t>();
break;
default:
diff --git a/src/dev/arm/vgic.cc b/src/dev/arm/vgic.cc
index 529520403..3e295828c 100644
--- a/src/dev/arm/vgic.cc
+++ b/src/dev/arm/vgic.cc
@@ -106,16 +106,16 @@ VGic::readVCpu(PacketPtr pkt)
switch (daddr) {
case GICV_CTLR:
- pkt->set<uint32_t>(vid->vctrl);
+ pkt->setLE<uint32_t>(vid->vctrl);
break;
case GICV_IAR: {
int i = findHighestPendingLR(vid);
if (i < 0 || !vid->vctrl.En) {
- pkt->set<uint32_t>(1023); // "No int" marker
+ pkt->setLE<uint32_t>(1023); // "No int" marker
} else {
ListReg *lr = &vid->LR[i];
- pkt->set<uint32_t>(lr->VirtualID |
+ pkt->setLE<uint32_t>(lr->VirtualID |
(((int)lr->CpuID) << 10));
// We don't support auto-EOI of HW interrupts via real GIC!
// Fortunately, KVM doesn't use this. How about Xen...? Ulp!
@@ -160,15 +160,15 @@ VGic::readCtrl(PacketPtr pkt)
switch (daddr) {
case GICH_HCR:
- pkt->set<uint32_t>(vid->hcr);
+ pkt->setLE<uint32_t>(vid->hcr);
break;
case GICH_VTR:
- pkt->set<uint32_t>(0x44000000 | (NUM_LR - 1));
+ pkt->setLE<uint32_t>(0x44000000 | (NUM_LR - 1));
break;
case GICH_VMCR:
- pkt->set<uint32_t>(
+ pkt->setLE<uint32_t>(
((uint32_t)vid->VMPriMask << 27) |
((uint32_t)vid->VMBP << 21) |
((uint32_t)vid->VMABP << 18) |
@@ -182,15 +182,15 @@ VGic::readCtrl(PacketPtr pkt)
break;
case GICH_MISR:
- pkt->set<uint32_t>(getMISR(vid));
+ pkt->setLE<uint32_t>(getMISR(vid));
break;
case GICH_EISR0:
- pkt->set<uint32_t>(vid->eisr & 0xffffffff);
+ pkt->setLE<uint32_t>(vid->eisr & 0xffffffff);
break;
case GICH_EISR1:
- pkt->set<uint32_t>(vid->eisr >> 32);
+ pkt->setLE<uint32_t>(vid->eisr >> 32);
break;
case GICH_ELSR0: {
@@ -199,7 +199,7 @@ VGic::readCtrl(PacketPtr pkt)
if (!vid->LR[i].State)
bm |= 1 << i;
}
- pkt->set<uint32_t>(bm);
+ pkt->setLE<uint32_t>(bm);
} break;
case GICH_ELSR1: {
@@ -208,19 +208,19 @@ VGic::readCtrl(PacketPtr pkt)
if (!vid->LR[i].State)
bm |= 1 << (i-32);
}
- pkt->set<uint32_t>(bm);
+ pkt->setLE<uint32_t>(bm);
} break;
case GICH_APR0:
warn_once("VGIC GICH_APR read!\n");
- pkt->set<uint32_t>(0);
+ pkt->setLE<uint32_t>(0);
break;
case GICH_LR0:
case GICH_LR1:
case GICH_LR2:
case GICH_LR3:
- pkt->set<uint32_t>(vid->LR[(daddr - GICH_LR0) >> 2]);
+ pkt->setLE<uint32_t>(vid->LR[(daddr - GICH_LR0) >> 2]);
break;
default:
@@ -240,20 +240,21 @@ VGic::writeVCpu(PacketPtr pkt)
assert(ctx_id < VGIC_CPU_MAX);
struct vcpuIntData *vid = &vcpuData[ctx_id];
- DPRINTF(VGIC, "VGIC VCPU write register %#x <= %#x\n", daddr, pkt->get<uint32_t>());
+ DPRINTF(VGIC, "VGIC VCPU write register %#x <= %#x\n",
+ daddr, pkt->getLE<uint32_t>());
switch (daddr) {
case GICV_CTLR:
- vid->vctrl = pkt->get<uint32_t>();
+ vid->vctrl = pkt->getLE<uint32_t>();
break;
case GICV_PMR:
- vid->VMPriMask = pkt->get<uint32_t>();
+ vid->VMPriMask = pkt->getLE<uint32_t>();
break;
case GICV_EOIR: {
// We don't handle the split EOI-then-DIR mode. Linux (guest)
// doesn't need it though.
assert(!vid->vctrl.EOImode);
- uint32_t w = pkt->get<uint32_t>();
+ uint32_t w = pkt->getLE<uint32_t>();
unsigned int virq = w & 0x3ff;
unsigned int vcpu = (w >> 10) & 7;
int i = findLRForVIRQ(vid, virq, vcpu);
@@ -268,7 +269,8 @@ VGic::writeVCpu(PacketPtr pkt)
}
} break;
default:
- panic("VGIC VCPU write %#x to unk address %#x\n", pkt->get<uint32_t>(), daddr);
+ panic("VGIC VCPU write %#x to unk address %#x\n",
+ pkt->getLE<uint32_t>(), daddr);
}
// This updates the EISRs and flags IRQs:
@@ -285,7 +287,8 @@ VGic::writeCtrl(PacketPtr pkt)
ContextID ctx_id = pkt->req->contextId();
- DPRINTF(VGIC, "VGIC HVCtrl write register %#x <= %#x\n", daddr, pkt->get<uint32_t>());
+ DPRINTF(VGIC, "VGIC HVCtrl write register %#x <= %#x\n",
+ daddr, pkt->getLE<uint32_t>());
/* Munge the address: 0-0xfff is the usual space banked by requester CPU.
* Anything > that is 0x200-sized slices of 'per CPU' regs.
@@ -301,12 +304,12 @@ VGic::writeCtrl(PacketPtr pkt)
switch (daddr) {
case GICH_HCR:
- vid->hcr = pkt->get<uint32_t>();
+ vid->hcr = pkt->getLE<uint32_t>();
// update int state
break;
case GICH_VMCR: {
- uint32_t d = pkt->get<uint32_t>();
+ uint32_t d = pkt->getLE<uint32_t>();
vid->VMPriMask = d >> 27;
vid->VMBP = (d >> 21) & 7;
vid->VMABP = (d >> 18) & 7;
@@ -326,7 +329,7 @@ VGic::writeCtrl(PacketPtr pkt)
case GICH_LR1:
case GICH_LR2:
case GICH_LR3:
- vid->LR[(daddr - GICH_LR0) >> 2] = pkt->get<uint32_t>();
+ vid->LR[(daddr - GICH_LR0) >> 2] = pkt->getLE<uint32_t>();
// update int state
break;
diff --git a/src/dev/arm/vio_mmio.cc b/src/dev/arm/vio_mmio.cc
index 9f7bf1333..b9ecce918 100644
--- a/src/dev/arm/vio_mmio.cc
+++ b/src/dev/arm/vio_mmio.cc
@@ -79,7 +79,7 @@ MmioVirtIO::read(PacketPtr pkt)
const uint32_t value = read(offset);
DPRINTF(VIOIface, " value: 0x%x\n", value);
pkt->makeResponse();
- pkt->set<uint32_t>(value);
+ pkt->setLE<uint32_t>(value);
return 0;
}
@@ -173,9 +173,9 @@ MmioVirtIO::write(PacketPtr pkt)
}
panic_if(size != 4, "Unexpected write size @ 0x%x: %u\n", offset, size);
- DPRINTF(VIOIface, " value: 0x%x\n", pkt->get<uint32_t>());
+ DPRINTF(VIOIface, " value: 0x%x\n", pkt->getLE<uint32_t>());
pkt->makeResponse();
- write(offset, pkt->get<uint32_t>());
+ write(offset, pkt->getLE<uint32_t>());
return 0;
}