From 70e99e0b915fa7ed9ac682af6f68f077799ddea7 Mon Sep 17 00:00:00 2001 From: Andreas Hansson Date: Tue, 21 Aug 2012 05:50:03 -0400 Subject: Device: Remove overloaded pio_latency parameter This patch removes the overloading of the parameter, which seems both redundant, and possibly incorrect. The PciConfigAll now also uses a Param.Latency rather than a Param.Tick. For backwards compatibility it still sets the pio_latency to 1 tick. All the comments have also been updated to not state that it is in simticks when it is not necessarily the case. --- src/dev/Pci.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'src/dev/Pci.py') diff --git a/src/dev/Pci.py b/src/dev/Pci.py index c866f9386..db2791ed5 100644 --- a/src/dev/Pci.py +++ b/src/dev/Pci.py @@ -34,7 +34,9 @@ from Device import BasicPioDevice, DmaDevice, PioDevice class PciConfigAll(PioDevice): type = 'PciConfigAll' platform = Param.Platform(Parent.any, "Platform this device is part of.") - pio_latency = Param.Tick(1, "Programmed IO latency in simticks") + # @todo: This latency is unrealistically low and only kept at 1 tick + # to not change any regressions + pio_latency = Param.Latency('1t', "Programmed IO latency") bus = Param.UInt8(0x00, "PCI bus to act as config space for") size = Param.MemorySize32('16MB', "Size of config space") @@ -47,7 +49,7 @@ class PciDevice(DmaDevice): pci_bus = Param.Int("PCI bus") pci_dev = Param.Int("PCI device number") pci_func = Param.Int("PCI function code") - pio_latency = Param.Latency('1ns', "Programmed IO latency in simticks") + pio_latency = Param.Latency('1ns', "Programmed IO latency") config_latency = Param.Latency('20ns', "Config read or write latency") VendorID = Param.UInt16("Vendor ID") -- cgit v1.2.3