Minor include file & formatting cleanup.
authorSteve Reinhardt <stever@eecs.umich.edu>
Wed, 30 Aug 2006 16:57:46 +0000 (09:57 -0700)
committerSteve Reinhardt <stever@eecs.umich.edu>
Wed, 30 Aug 2006 16:57:46 +0000 (09:57 -0700)
--HG--
extra : convert_revision : fa23563b2897687752379d63ddab5cccb92484ba

src/dev/ide_disk.hh
src/dev/io_device.cc
src/dev/io_device.hh
src/dev/pcidev.cc
src/mem/tport.hh

index fb0614d4d343cc182def273f42278f4aed721e0e..0bc0b73aba0590d8fdd12a144e28266b0a20b8d0 100644 (file)
@@ -43,6 +43,8 @@
 #include "dev/io_device.hh"
 #include "sim/eventq.hh"
 
+class ChunkGenerator;
+
 #define DMA_BACKOFF_PERIOD 200
 
 #define MAX_DMA_SIZE    (131072)  // 128K
index b51a931902f4c8808e6f9866bd43161d5980f541..b06e8b85d4580ffc8e0e5f15e9c8e52198fc166a 100644 (file)
@@ -29,6 +29,7 @@
  *          Nathan Binkert
  */
 
+#include "base/chunk_generator.hh"
 #include "base/trace.hh"
 #include "dev/io_device.hh"
 #include "sim/builder.hh"
index 710b22b2c4ed031979d02ed58998dd779860c6e7..d91304301138c2322a93c7b369fcc842384a4927 100644 (file)
 #ifndef __DEV_IO_DEVICE_HH__
 #define __DEV_IO_DEVICE_HH__
 
-#include "base/chunk_generator.hh"
 #include "mem/mem_object.hh"
 #include "mem/packet_impl.hh"
-#include "sim/eventq.hh"
 #include "sim/sim_object.hh"
 #include "mem/tport.hh"
 
+class Event;
 class Platform;
 class PioDevice;
 class DmaDevice;
@@ -49,8 +48,8 @@ class System;
  * sensitive to an address range use. The port takes all the memory
  * access types and roles them into one read() and write() call that the device
  * must respond to. The device must also provide the addressRanges() function
- * with which it returns the address ranges it is interested in. */
-
+ * with which it returns the address ranges it is interested in.
+ */
 class PioPort : public SimpleTimingPort
 {
   protected:
@@ -73,7 +72,8 @@ class PioPort : public SimpleTimingPort
     virtual void recvStatusChange(Status status)
     { peerStatus = status; }
 
-    virtual void getDeviceAddressRanges(AddrRangeList &resp, AddrRangeList &snoop);
+    virtual void getDeviceAddressRanges(AddrRangeList &resp,
+                                        AddrRangeList &snoop);
 
   public:
     PioPort(PioDevice *dev, System *s, std::string pname = "-pioport");
@@ -132,7 +132,8 @@ class DmaPort : public Port
 
     virtual void recvRetry() ;
 
-    virtual void getDeviceAddressRanges(AddrRangeList &resp, AddrRangeList &snoop)
+    virtual void getDeviceAddressRanges(AddrRangeList &resp,
+                                        AddrRangeList &snoop)
     { resp.clear(); snoop.clear(); }
 
     void sendDma(Packet *pkt, bool front = false);
@@ -155,7 +156,6 @@ class DmaPort : public Port
  * mode we are in, etc is handled by the PioPort so the device doesn't have to
  * bother.
  */
-
 class PioDevice : public MemObject
 {
   protected:
@@ -172,13 +172,14 @@ class PioDevice : public MemObject
 
     virtual void addressRanges(AddrRangeList &range_list) = 0;
 
-    /** As far as the devices are concerned they only accept atomic transactions
-     * which are converted to either a write or a read. */
+    /** As far as the devices are concerned they only accept atomic
+     * transactions which are converted to either a write or a
+     * read. */
     Tick recvAtomic(Packet *pkt)
     { return pkt->isRead() ? this->read(pkt) : this->write(pkt); }
 
-    /** Pure virtual function that the device must implement. Called when a read
-     * command is recieved by the port.
+    /** Pure virtual function that the device must implement. Called
+     * when a read command is recieved by the port.
      * @param pkt Packet describing this request
      * @return number of ticks it took to complete
      */
@@ -192,10 +193,9 @@ class PioDevice : public MemObject
     virtual Tick write(Packet *pkt) = 0;
 
   public:
-    /** Params struct which is extended through each device based on the
-     * parameters it needs. Since we are re-writing everything, we might as well
-     * start from the bottom this time. */
-
+    /** Params struct which is extended through each device based on
+     * the parameters it needs. Since we are re-writing everything, we
+     * might as well start from the bottom this time. */
     struct Params
     {
         std::string name;
@@ -255,7 +255,8 @@ class BasicPioDevice : public PioDevice
 
   public:
     BasicPioDevice(Params *p)
-        : PioDevice(p), pioAddr(p->pio_addr), pioSize(0), pioDelay(p->pio_delay)
+        : PioDevice(p), pioAddr(p->pio_addr), pioSize(0),
+          pioDelay(p->pio_delay)
     {}
 
     /** return the address ranges that this device responds to.
index d0e9d758a60d7ad430ab63735a48123a3f643ae7..a8fb44c39d5f8c4aa113829a39e87111f0681f69 100644 (file)
@@ -39,6 +39,7 @@
 #include <vector>
 
 #include "base/inifile.hh"
+#include "base/intmath.hh" // for isPowerOf2(
 #include "base/misc.hh"
 #include "base/str.hh" // for to_number
 #include "base/trace.hh"
@@ -67,8 +68,8 @@ Tick
 PciDev::PciConfigPort::recvAtomic(Packet *pkt)
 {
     assert(pkt->result == Packet::Unknown);
-    assert(pkt->getAddr() >= configAddr && pkt->getAddr() < configAddr +
-            PCI_CONFIG_SIZE);
+    assert(pkt->getAddr() >= configAddr &&
+           pkt->getAddr() < configAddr + PCI_CONFIG_SIZE);
     return device->recvConfig(pkt);
 }
 
@@ -76,8 +77,8 @@ void
 PciDev::PciConfigPort::recvFunctional(Packet *pkt)
 {
     assert(pkt->result == Packet::Unknown);
-    assert(pkt->getAddr() >= configAddr && pkt->getAddr() < configAddr +
-            PCI_CONFIG_SIZE);
+    assert(pkt->getAddr() >= configAddr &&
+           pkt->getAddr() < configAddr + PCI_CONFIG_SIZE);
     device->recvConfig(pkt);
 }
 
@@ -97,8 +98,8 @@ PciDev::PciConfigPort::recvTiming(Packet *pkt)
         resendNacked(pkt);
     } else {
         assert(pkt->result == Packet::Unknown);
-        assert(pkt->getAddr() >= configAddr && pkt->getAddr() < configAddr +
-                PCI_CONFIG_SIZE);
+        assert(pkt->getAddr() >= configAddr &&
+               pkt->getAddr() < configAddr + PCI_CONFIG_SIZE);
         Tick latency = device->recvConfig(pkt);
         // turn packet around to go back to requester
         pkt->makeTimingResponse();
index 5473e945e9c656cde96ea02dde63fe13038503ae..4d5c4599dce01c6e1634e4687ec244f887b92ba2 100644 (file)
@@ -76,19 +76,21 @@ PioDevice::drain(Event *de)
 class SimpleTimingPort : public Port
 {
   protected:
-    /** A list of outgoing timing response packets that haven't been serviced
-     * yet. */
+    /** A list of outgoing timing response packets that haven't been
+     * serviced yet. */
     std::list<Packet*> transmitList;
     /**
-     * This class is used to implemented sendTiming() with a delay. When a delay
-     * is requested a new event is created. When the event time expires it
-     * attempts to send the packet. If it cannot, the packet is pushed onto the
-     * transmit list to be sent when recvRetry() is called. */
+     * This class is used to implemented sendTiming() with a delay. When
+     * a delay is requested a new event is created. When the event time
+     * expires it attempts to send the packet. If it cannot, the packet
+     * is pushed onto the transmit list to be sent when recvRetry() is
+     * called. */
     class SendEvent : public Event
     {
         SimpleTimingPort *port;
         Packet *packet;
 
+      public:
         SendEvent(SimpleTimingPort *p, Packet *pkt, Tick t)
             : Event(&mainEventQueue), port(p), packet(pkt)
         { setFlags(AutoDelete); schedule(curTick + t); }
@@ -97,8 +99,6 @@ class SimpleTimingPort : public Port
 
         virtual const char *description()
         { return "Future scheduled sendTiming event"; }
-
-        friend class SimpleTimingPort;
     };
 
 
@@ -127,8 +127,6 @@ class SimpleTimingPort : public Port
     {}
 
     unsigned int drain(Event *de);
-
-    friend class SimpleTimingPort::SendEvent;
 };
 
 #endif // __MEM_TPORT_HH__