Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48924 )

Change subject: dev: Fix style in i8254xGBe_defs.hh.
......................................................................

dev: Fix style in i8254xGBe_defs.hh.

Functions with more than one line in them put on a single line, lines
over 79 characters, missing spaces after ","s.

Change-Id: If3a1bf7d9b9068608ca70edf9577c0b2e1b44306
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/48924
Reviewed-by: Daniel Carvalho <[email protected]>
Maintainer: Gabe Black <[email protected]>
Tested-by: kokoro <[email protected]>
---
M src/dev/net/i8254xGBe_defs.hh
1 file changed, 387 insertions(+), 272 deletions(-)

Approvals:
  Daniel Carvalho: Looks good to me, approved
  Gabe Black: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/dev/net/i8254xGBe_defs.hh b/src/dev/net/i8254xGBe_defs.hh
index e2508fa..015ca7d 100644
--- a/src/dev/net/i8254xGBe_defs.hh
+++ b/src/dev/net/i8254xGBe_defs.hh
@@ -248,57 +248,179 @@
 const uint8_t TXD_ADVCNXT = 0x2;
 const uint8_t TXD_ADVDATA = 0x3;

-inline bool isLegacy(TxDesc *d) { return !bits(d->d2,29,29); }
-inline uint8_t getType(TxDesc *d) { return bits(d->d2, 23,20); }
+inline bool isLegacy(TxDesc *d) { return !bits(d->d2, 29); }
+inline uint8_t getType(TxDesc *d) { return bits(d->d2, 23, 20); }
 inline bool isType(TxDesc *d, uint8_t type) { return getType(d) == type; }
-inline bool isTypes(TxDesc *d, uint8_t t1, uint8_t t2) { return isType(d, t1) || isType(d, t2); } -inline bool isAdvDesc(TxDesc *d) { return !isLegacy(d) && isTypes(d, TXD_ADVDATA,TXD_ADVCNXT); } -inline bool isContext(TxDesc *d) { return !isLegacy(d) && isTypes(d,TXD_CNXT, TXD_ADVCNXT); } -inline bool isData(TxDesc *d) { return !isLegacy(d) && isTypes(d, TXD_DATA, TXD_ADVDATA); }
+inline bool
+isTypes(TxDesc *d, uint8_t t1, uint8_t t2)
+{
+    return isType(d, t1) || isType(d, t2);
+}
+inline bool
+isAdvDesc(TxDesc *d)
+{
+    return !isLegacy(d) && isTypes(d, TXD_ADVDATA,TXD_ADVCNXT);
+}
+inline bool
+isContext(TxDesc *d)
+{
+    return !isLegacy(d) && isTypes(d,TXD_CNXT, TXD_ADVCNXT);
+}
+inline bool
+isData(TxDesc *d)
+{
+    return !isLegacy(d) && isTypes(d, TXD_DATA, TXD_ADVDATA);
+}

-inline Addr getBuf(TxDesc *d) { assert(isLegacy(d) || isData(d)); return d->d1; } -inline Addr getLen(TxDesc *d) { if (isLegacy(d)) return bits(d->d2,15,0); else return bits(d->d2, 19,0); }
+inline Addr
+getBuf(TxDesc *d)
+{
+    assert(isLegacy(d) || isData(d));
+    return d->d1;
+}
+inline Addr
+getLen(TxDesc *d)
+{
+    if (isLegacy(d))
+        return bits(d->d2, 15, 0);
+    else
+        return bits(d->d2, 19, 0);
+}
 inline void setDd(TxDesc *d) { replaceBits(d->d2, 35, 32, 1ULL); }

-inline bool ide(TxDesc *d) { return bits(d->d2, 31,31) && (getType(d) == TXD_DATA || isLegacy(d)); } -inline bool vle(TxDesc *d) { assert(isLegacy(d) || isData(d)); return bits(d->d2, 30,30); }
-inline bool rs(TxDesc *d)   { return bits(d->d2, 27,27); }
-inline bool ic(TxDesc *d) { assert(isLegacy(d) || isData(d)); return isLegacy(d) && bits(d->d2, 26,26); }
-inline bool tse(TxDesc *d)  {
+inline bool
+ide(TxDesc *d)
+{
+    return bits(d->d2, 31, 31) && (getType(d) == TXD_DATA || isLegacy(d));
+}
+inline bool
+vle(TxDesc *d)
+{
+    assert(isLegacy(d) || isData(d));
+    return bits(d->d2, 30);
+}
+inline bool rs(TxDesc *d) { return bits(d->d2, 27); }
+inline bool
+ic(TxDesc *d)
+{
+    assert(isLegacy(d) || isData(d));
+    return isLegacy(d) && bits(d->d2, 26);
+}
+inline bool
+tse(TxDesc *d)
+{
     if (isTypes(d, TXD_CNXT, TXD_DATA))
-        return bits(d->d2, 26,26);
+        return bits(d->d2, 26);
     if (isType(d, TXD_ADVDATA))
-        return bits(d->d2, 31, 31);
+        return bits(d->d2, 31);
     return false;
 }

-inline bool ifcs(TxDesc *d) { assert(isLegacy(d) || isData(d)); return bits(d->d2, 25,25); } -inline bool eop(TxDesc *d) { assert(isLegacy(d) || isData(d)); return bits(d->d2, 24,24); } -inline bool ip(TxDesc *d) { assert(isContext(d)); return bits(d->d2, 25,25); } -inline bool tcp(TxDesc *d) { assert(isContext(d)); return bits(d->d2, 24,24); }
-
-inline uint8_t getCso(TxDesc *d) { assert(isLegacy(d)); return bits(d->d2, 23,16); } -inline uint8_t getCss(TxDesc *d) { assert(isLegacy(d)); return bits(d->d2, 47,40); }
-
-inline bool ixsm(TxDesc *d)  { return isData(d) && bits(d->d2, 40,40); }
-inline bool txsm(TxDesc *d)  { return isData(d) && bits(d->d2, 41,41); }
-
-inline int tucse(TxDesc *d) { assert(isContext(d)); return bits(d->d1,63,48); } -inline int tucso(TxDesc *d) { assert(isContext(d)); return bits(d->d1,47,40); } -inline int tucss(TxDesc *d) { assert(isContext(d)); return bits(d->d1,39,32); } -inline int ipcse(TxDesc *d) { assert(isContext(d)); return bits(d->d1,31,16); } -inline int ipcso(TxDesc *d) { assert(isContext(d)); return bits(d->d1,15,8); } -inline int ipcss(TxDesc *d) { assert(isContext(d)); return bits(d->d1,7,0); } -inline int mss(TxDesc *d) { assert(isContext(d)); return bits(d->d2,63,48); }
-inline int hdrlen(TxDesc *d) {
+inline bool
+ifcs(TxDesc *d)
+{
+    assert(isLegacy(d) || isData(d));
+    return bits(d->d2, 25);
+}
+inline bool
+eop(TxDesc *d)
+{
+    assert(isLegacy(d) || isData(d));
+    return bits(d->d2, 24);
+}
+inline bool
+ip(TxDesc *d)
+{
     assert(isContext(d));
-    if (!isAdvDesc(d))
-        return bits(d->d2,47,40);
-    return bits(d->d2, 47,40) + bits(d->d1, 8,0) + bits(d->d1, 15, 9);
+    return bits(d->d2, 25);
+}
+inline bool
+tcp(TxDesc *d)
+{
+    assert(isContext(d));
+    return bits(d->d2, 24);
 }

-inline int getTsoLen(TxDesc *d) { assert(isType(d, TXD_ADVDATA)); return bits(d->d2, 63,46); } -inline int utcmd(TxDesc *d) { assert(isContext(d)); return bits(d->d2,24,31); }
+inline uint8_t
+getCso(TxDesc *d)
+{
+    assert(isLegacy(d));
+    return bits(d->d2, 23, 16);
+}
+inline uint8_t
+getCss(TxDesc *d)
+{
+    assert(isLegacy(d));
+    return bits(d->d2, 47, 40);
+}
+
+inline bool ixsm(TxDesc *d)  { return isData(d) && bits(d->d2, 40, 40); }
+inline bool txsm(TxDesc *d)  { return isData(d) && bits(d->d2, 41, 41); }
+
+inline int
+tucse(TxDesc *d)
+{
+    assert(isContext(d));
+    return bits(d->d1, 63, 48);
+}
+inline int
+tucso(TxDesc *d)
+{
+    assert(isContext(d));
+    return bits(d->d1, 47, 40);
+}
+inline int
+tucss(TxDesc *d)
+{
+    assert(isContext(d));
+    return bits(d->d1, 39, 32);
+}
+inline int
+ipcse(TxDesc *d)
+{
+    assert(isContext(d));
+    return bits(d->d1, 31, 16);
+}
+inline int
+ipcso(TxDesc *d)
+{
+    assert(isContext(d));
+    return bits(d->d1, 15, 8);
+}
+inline int
+ipcss(TxDesc *d)
+{
+    assert(isContext(d));
+    return bits(d->d1, 7, 0);
+}
+inline int
+mss(TxDesc *d)
+{
+    assert(isContext(d));
+    return bits(d->d2, 63, 48);
+}
+inline int
+hdrlen(TxDesc *d)
+{
+    assert(isContext(d));
+    if (!isAdvDesc(d))
+        return bits(d->d2, 47, 40);
+    return bits(d->d2, 47, 40) + bits(d->d1, 8, 0) + bits(d->d1, 15, 9);
+}
+
+inline int
+getTsoLen(TxDesc *d)
+{
+    assert(isType(d, TXD_ADVDATA));
+    return bits(d->d2, 63, 46);
+}
+inline int
+utcmd(TxDesc *d)
+{
+    assert(isContext(d));
+    return bits(d->d2, 24, 31);
+}
+
 } // namespace txd_op


@@ -335,37 +457,37 @@
     {
          // 0x0000 CTRL Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(fd,0,1);       // full duplex
-        ADD_FIELD32(bem,1,1);      // big endian mode
-        ADD_FIELD32(pcipr,2,1);    // PCI priority
-        ADD_FIELD32(lrst,3,1);     // link reset
-        ADD_FIELD32(tme,4,1);      // test mode enable
-        ADD_FIELD32(asde,5,1);     // Auto-speed detection
-        ADD_FIELD32(slu,6,1);      // Set link up
-        ADD_FIELD32(ilos,7,1);     // invert los-of-signal
-        ADD_FIELD32(speed,8,2);    // speed selection bits
-        ADD_FIELD32(be32,10,1);    // big endian mode 32
-        ADD_FIELD32(frcspd,11,1);  // force speed
-        ADD_FIELD32(frcdpx,12,1);  // force duplex
-        ADD_FIELD32(duden,13,1);   // dock/undock enable
-        ADD_FIELD32(dudpol,14,1);  // dock/undock polarity
-        ADD_FIELD32(fphyrst,15,1); // force phy reset
-        ADD_FIELD32(extlen,16,1);  // external link status enable
-        ADD_FIELD32(rsvd,17,1);    // reserved
-        ADD_FIELD32(sdp0d,18,1);   // software controlled pin data
-        ADD_FIELD32(sdp1d,19,1);   // software controlled pin data
-        ADD_FIELD32(sdp2d,20,1);   // software controlled pin data
-        ADD_FIELD32(sdp3d,21,1);   // software controlled pin data
-        ADD_FIELD32(sdp0i,22,1);   // software controlled pin dir
-        ADD_FIELD32(sdp1i,23,1);   // software controlled pin dir
-        ADD_FIELD32(sdp2i,24,1);   // software controlled pin dir
-        ADD_FIELD32(sdp3i,25,1);   // software controlled pin dir
-        ADD_FIELD32(rst,26,1);     // reset
-        ADD_FIELD32(rfce,27,1);    // receive flow control enable
-        ADD_FIELD32(tfce,28,1);    // transmit flow control enable
-        ADD_FIELD32(rte,29,1);     // routing tag enable
-        ADD_FIELD32(vme,30,1);     // vlan enable
-        ADD_FIELD32(phyrst,31,1);  // phy reset
+        ADD_FIELD32(fd, 0, 1);       // full duplex
+        ADD_FIELD32(bem, 1, 1);      // big endian mode
+        ADD_FIELD32(pcipr, 2, 1);    // PCI priority
+        ADD_FIELD32(lrst, 3, 1);     // link reset
+        ADD_FIELD32(tme, 4, 1);      // test mode enable
+        ADD_FIELD32(asde, 5, 1);     // Auto-speed detection
+        ADD_FIELD32(slu, 6, 1);      // Set link up
+        ADD_FIELD32(ilos, 7, 1);     // invert los-of-signal
+        ADD_FIELD32(speed, 8, 2);    // speed selection bits
+        ADD_FIELD32(be32, 10, 1);    // big endian mode 32
+        ADD_FIELD32(frcspd, 11, 1);  // force speed
+        ADD_FIELD32(frcdpx, 12, 1);  // force duplex
+        ADD_FIELD32(duden, 13, 1);   // dock/undock enable
+        ADD_FIELD32(dudpol, 14, 1);  // dock/undock polarity
+        ADD_FIELD32(fphyrst, 15, 1); // force phy reset
+        ADD_FIELD32(extlen, 16, 1);  // external link status enable
+        ADD_FIELD32(rsvd, 17, 1);    // reserved
+        ADD_FIELD32(sdp0d, 18, 1);   // software controlled pin data
+        ADD_FIELD32(sdp1d, 19, 1);   // software controlled pin data
+        ADD_FIELD32(sdp2d, 20, 1);   // software controlled pin data
+        ADD_FIELD32(sdp3d, 21, 1);   // software controlled pin data
+        ADD_FIELD32(sdp0i, 22, 1);   // software controlled pin dir
+        ADD_FIELD32(sdp1i, 23, 1);   // software controlled pin dir
+        ADD_FIELD32(sdp2i, 24, 1);   // software controlled pin dir
+        ADD_FIELD32(sdp3i, 25, 1);   // software controlled pin dir
+        ADD_FIELD32(rst, 26, 1);     // reset
+        ADD_FIELD32(rfce, 27, 1);    // receive flow control enable
+        ADD_FIELD32(tfce, 28, 1);    // transmit flow control enable
+        ADD_FIELD32(rte, 29, 1);     // routing tag enable
+        ADD_FIELD32(vme, 30, 1);     // vlan enable
+        ADD_FIELD32(phyrst, 31, 1);  // phy reset
     };
     CTRL ctrl;

@@ -373,18 +495,18 @@
     {
         // 0x0008 STATUS Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(fd,0,1);       // full duplex
-        ADD_FIELD32(lu,1,1);       // link up
-        ADD_FIELD32(func,2,2);     // function id
-        ADD_FIELD32(txoff,4,1);    // transmission paused
-        ADD_FIELD32(tbimode,5,1);  // tbi mode
-        ADD_FIELD32(speed,6,2);    // link speed
-        ADD_FIELD32(asdv,8,2);     // auto speed detection value
-        ADD_FIELD32(mtxckok,10,1); // mtx clock running ok
-        ADD_FIELD32(pci66,11,1);   // In 66Mhz pci slot
-        ADD_FIELD32(bus64,12,1);   // in 64 bit slot
-        ADD_FIELD32(pcix,13,1);    // Pci mode
-        ADD_FIELD32(pcixspd,14,2); // pci x speed
+        ADD_FIELD32(fd, 0, 1);       // full duplex
+        ADD_FIELD32(lu, 1, 1);       // link up
+        ADD_FIELD32(func, 2, 2);     // function id
+        ADD_FIELD32(txoff, 4, 1);    // transmission paused
+        ADD_FIELD32(tbimode, 5, 1);  // tbi mode
+        ADD_FIELD32(speed, 6, 2);    // link speed
+        ADD_FIELD32(asdv, 8, 2);     // auto speed detection value
+        ADD_FIELD32(mtxckok, 10, 1); // mtx clock running ok
+        ADD_FIELD32(pci66, 11, 1);   // In 66Mhz pci slot
+        ADD_FIELD32(bus64, 12, 1);   // in 64 bit slot
+        ADD_FIELD32(pcix, 13, 1);    // Pci mode
+        ADD_FIELD32(pcixspd, 14, 2); // pci x speed
     };
     STATUS sts;

@@ -392,18 +514,18 @@
     {
         // 0x0010 EECD Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(sk,0,1);       // clack input to the eeprom
-        ADD_FIELD32(cs,1,1);       // chip select to eeprom
-        ADD_FIELD32(din,2,1);      // data input to eeprom
-        ADD_FIELD32(dout,3,1);     // data output bit
-        ADD_FIELD32(fwe,4,2);      // flash write enable
-        ADD_FIELD32(ee_req,6,1);   // request eeprom access
-        ADD_FIELD32(ee_gnt,7,1);   // grant eeprom access
-        ADD_FIELD32(ee_pres,8,1);  // eeprom present
-        ADD_FIELD32(ee_size,9,1);  // eeprom size
-        ADD_FIELD32(ee_sz1,10,1);  // eeprom size
-        ADD_FIELD32(rsvd,11,2);    // reserved
-        ADD_FIELD32(ee_type,13,1); // type of eeprom
+        ADD_FIELD32(sk, 0, 1);       // clack input to the eeprom
+        ADD_FIELD32(cs, 1, 1);       // chip select to eeprom
+        ADD_FIELD32(din, 2, 1);      // data input to eeprom
+        ADD_FIELD32(dout, 3, 1);     // data output bit
+        ADD_FIELD32(fwe, 4, 2);      // flash write enable
+        ADD_FIELD32(ee_req, 6, 1);   // request eeprom access
+        ADD_FIELD32(ee_gnt, 7, 1);   // grant eeprom access
+        ADD_FIELD32(ee_pres, 8, 1);  // eeprom present
+        ADD_FIELD32(ee_size, 9, 1);  // eeprom size
+        ADD_FIELD32(ee_sz1, 10, 1);  // eeprom size
+        ADD_FIELD32(rsvd, 11, 2);    // reserved
+        ADD_FIELD32(ee_type, 13, 1); // type of eeprom
     } ;
     EECD eecd;

@@ -411,10 +533,10 @@
     {
         // 0x0014 EERD Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(start,0,1);  // start read
-        ADD_FIELD32(done,1,1);   // done read
-        ADD_FIELD32(addr,2,14);   // address
-        ADD_FIELD32(data,16,16); // data
+        ADD_FIELD32(start, 0, 1);  // start read
+        ADD_FIELD32(done, 1, 1);   // done read
+        ADD_FIELD32(addr, 2, 14);   // address
+        ADD_FIELD32(data, 16, 16); // data
     };
     EERD eerd;

@@ -422,21 +544,23 @@
     {
         // 0x0018 CTRL_EXT Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(gpi_en,0,4);      // enable interrupts from gpio
-        ADD_FIELD32(phyint,5,1);      // reads the phy internal int status
-        ADD_FIELD32(sdp2_data,6,1);   // data from gpio sdp
-        ADD_FIELD32(spd3_data,7,1);   // data frmo gpio sdp
-        ADD_FIELD32(spd2_iodir,10,1); // direction of sdp2
-        ADD_FIELD32(spd3_iodir,11,1); // direction of sdp2
-        ADD_FIELD32(asdchk,12,1);     // initiate auto-speed-detection
-        ADD_FIELD32(eerst,13,1);      // reset the eeprom
-        ADD_FIELD32(spd_byps,15,1);   // bypass speed select
-        ADD_FIELD32(ro_dis,17,1);     // disable relaxed memory ordering
-        ADD_FIELD32(vreg,21,1);       // power down the voltage regulator
-        ADD_FIELD32(link_mode,22,2);  // interface to talk to the link
-        ADD_FIELD32(iame, 27,1);      // interrupt acknowledge auto-mask ??
- ADD_FIELD32(drv_loaded, 28,1);// driver is loaded and incharge of device - ADD_FIELD32(timer_clr, 29,1); // clear interrupt timers after IMS clear ??
+        ADD_FIELD32(gpi_en, 0, 4);      // enable interrupts from gpio
+ ADD_FIELD32(phyint, 5, 1); // reads the phy internal int status
+        ADD_FIELD32(sdp2_data, 6, 1);   // data from gpio sdp
+        ADD_FIELD32(spd3_data, 7, 1);   // data frmo gpio sdp
+        ADD_FIELD32(spd2_iodir, 10, 1); // direction of sdp2
+        ADD_FIELD32(spd3_iodir, 11, 1); // direction of sdp2
+        ADD_FIELD32(asdchk, 12, 1);     // initiate auto-speed-detection
+        ADD_FIELD32(eerst, 13, 1);      // reset the eeprom
+        ADD_FIELD32(spd_byps, 15, 1);   // bypass speed select
+        ADD_FIELD32(ro_dis, 17, 1);     // disable relaxed memory ordering
+        ADD_FIELD32(vreg, 21, 1);       // power down the voltage regulator
+        ADD_FIELD32(link_mode, 22, 2);  // interface to talk to the link
+ ADD_FIELD32(iame, 27, 1); // interrupt acknowledge auto-mask ??
+        ADD_FIELD32(drv_loaded, 28, 1); // driver is loaded and incharge of
+                                        //   device
+        ADD_FIELD32(timer_clr, 29, 1);  // clear interrupt timers after IMS
+                                        //   clear ??
     };
     CTRL_EXT ctrl_ext;

@@ -444,13 +568,13 @@
     {
         // 0x0020 MDIC Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(data,0,16);   // data
-        ADD_FIELD32(regadd,16,5); // register address
-        ADD_FIELD32(phyadd,21,5); // phy addresses
-        ADD_FIELD32(op,26,2);     // opcode
-        ADD_FIELD32(r,28,1);      // ready
-        ADD_FIELD32(i,29,1);      // interrupt
-        ADD_FIELD32(e,30,1);      // error
+        ADD_FIELD32(data, 0, 16);   // data
+        ADD_FIELD32(regadd, 16, 5); // register address
+        ADD_FIELD32(phyadd, 21, 5); // phy addresses
+        ADD_FIELD32(op, 26, 2);     // opcode
+        ADD_FIELD32(r, 28, 1);      // ready
+        ADD_FIELD32(i, 29, 1);      // interrupt
+        ADD_FIELD32(e, 30, 1);      // error
     };
     MDIC mdic;

@@ -458,23 +582,23 @@
     {
         // 0x00C0 ICR Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(txdw,0,1)   // tx descr witten back
-        ADD_FIELD32(txqe,1,1)   // tx queue empty
-        ADD_FIELD32(lsc,2,1)    // link status change
-        ADD_FIELD32(rxseq,3,1)  // rcv sequence error
-        ADD_FIELD32(rxdmt0,4,1) // rcv descriptor min thresh
-        ADD_FIELD32(rsvd1,5,1)  // reserved
-        ADD_FIELD32(rxo,6,1)    // receive overrunn
-        ADD_FIELD32(rxt0,7,1)   // receiver timer interrupt
-        ADD_FIELD32(mdac,9,1)   // mdi/o access complete
-        ADD_FIELD32(rxcfg,10,1)  // recv /c/ ordered sets
-        ADD_FIELD32(phyint,12,1) // phy interrupt
-        ADD_FIELD32(gpi1,13,1)   // gpi int 1
-        ADD_FIELD32(gpi2,14,1)   // gpi int 2
-        ADD_FIELD32(txdlow,15,1) // transmit desc low thresh
-        ADD_FIELD32(srpd,16,1)   // small receive packet detected
-        ADD_FIELD32(ack,17,1);    // receive ack frame
- ADD_FIELD32(int_assert, 31,1); // interrupt caused a system interrupt
+        ADD_FIELD32(txdw, 0, 1)   // tx descr witten back
+        ADD_FIELD32(txqe, 1, 1)   // tx queue empty
+        ADD_FIELD32(lsc, 2, 1)    // link status change
+        ADD_FIELD32(rxseq, 3, 1)  // rcv sequence error
+        ADD_FIELD32(rxdmt0, 4, 1) // rcv descriptor min thresh
+        ADD_FIELD32(rsvd1, 5, 1)  // reserved
+        ADD_FIELD32(rxo, 6, 1)    // receive overrunn
+        ADD_FIELD32(rxt0, 7, 1)   // receiver timer interrupt
+        ADD_FIELD32(mdac, 9, 1)   // mdi/o access complete
+        ADD_FIELD32(rxcfg, 10, 1)  // recv /c/ ordered sets
+        ADD_FIELD32(phyint, 12, 1) // phy interrupt
+        ADD_FIELD32(gpi1, 13, 1)   // gpi int 1
+        ADD_FIELD32(gpi2, 14, 1)   // gpi int 2
+        ADD_FIELD32(txdlow, 15, 1) // transmit desc low thresh
+        ADD_FIELD32(srpd, 16, 1)   // small receive packet detected
+        ADD_FIELD32(ack, 17, 1);    // receive ack frame
+ ADD_FIELD32(int_assert, 31, 1); // interrupt caused a system interrupt
     };
     ICR icr;

@@ -484,8 +608,8 @@
     {
         // 0x00C4 ITR Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(interval, 0,16); // minimum inter-interrutp inteval
-                                     // specified in 256ns interrupts
+        ADD_FIELD32(interval, 0, 16); // minimum inter-interrutp inteval
+                                      // specified in 256ns interrupts
     };
     ITR itr;

@@ -498,25 +622,25 @@
     {
         // 0x0100 RCTL Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(rst,0,1);   // Reset
-        ADD_FIELD32(en,1,1);    // Enable
-        ADD_FIELD32(sbp,2,1);   // Store bad packets
-        ADD_FIELD32(upe,3,1);   // Unicast Promiscuous enabled
-        ADD_FIELD32(mpe,4,1);   // Multicast promiscuous enabled
-        ADD_FIELD32(lpe,5,1);   // long packet reception enabled
-        ADD_FIELD32(lbm,6,2);   //
-        ADD_FIELD32(rdmts,8,2); //
-        ADD_FIELD32(mo,12,2);    //
-        ADD_FIELD32(mdr,14,1);   //
-        ADD_FIELD32(bam,15,1);   //
-        ADD_FIELD32(bsize,16,2); //
-        ADD_FIELD32(vfe,18,1);   //
-        ADD_FIELD32(cfien,19,1); //
-        ADD_FIELD32(cfi,20,1);   //
-        ADD_FIELD32(dpf,22,1);   // discard pause frames
-        ADD_FIELD32(pmcf,23,1);  // pass mac control  frames
-        ADD_FIELD32(bsex,25,1);  // buffer size extension
-        ADD_FIELD32(secrc,26,1); // strip ethernet crc from incoming packet
+        ADD_FIELD32(rst, 0, 1);   // Reset
+        ADD_FIELD32(en, 1, 1);    // Enable
+        ADD_FIELD32(sbp, 2, 1);   // Store bad packets
+        ADD_FIELD32(upe, 3, 1);   // Unicast Promiscuous enabled
+        ADD_FIELD32(mpe, 4, 1);   // Multicast promiscuous enabled
+        ADD_FIELD32(lpe, 5, 1);   // long packet reception enabled
+        ADD_FIELD32(lbm, 6, 2);   //
+        ADD_FIELD32(rdmts, 8, 2); //
+        ADD_FIELD32(mo, 12, 2);    //
+        ADD_FIELD32(mdr, 14, 1);   //
+        ADD_FIELD32(bam, 15, 1);   //
+        ADD_FIELD32(bsize, 16, 2); //
+        ADD_FIELD32(vfe, 18, 1);   //
+        ADD_FIELD32(cfien, 19, 1); //
+        ADD_FIELD32(cfi, 20, 1);   //
+        ADD_FIELD32(dpf, 22, 1);   // discard pause frames
+        ADD_FIELD32(pmcf, 23, 1);  // pass mac control  frames
+        ADD_FIELD32(bsex, 25, 1);  // buffer size extension
+ ADD_FIELD32(secrc, 26, 1); // strip ethernet crc from incoming packet
         unsigned descSize()
         {
             switch(bsize()) {
@@ -535,7 +659,7 @@
     {
         // 0x0170 FCTTV
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(ttv,0,16);    // Transmit Timer Value
+        ADD_FIELD32(ttv, 0, 16);    // Transmit Timer Value
     };
     FCTTV fcttv;

@@ -543,17 +667,17 @@
     {
         // 0x0400 TCTL Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(rst,0,1);    // Reset
-        ADD_FIELD32(en,1,1);     // Enable
-        ADD_FIELD32(bce,2,1);    // busy check enable
-        ADD_FIELD32(psp,3,1);    // pad short packets
-        ADD_FIELD32(ct,4,8);     // collision threshold
-        ADD_FIELD32(cold,12,10); // collision distance
-        ADD_FIELD32(swxoff,22,1); // software xoff transmission
-        ADD_FIELD32(pbe,23,1);    // packet burst enable
-        ADD_FIELD32(rtlc,24,1);   // retransmit late collisions
-        ADD_FIELD32(nrtu,25,1);   // on underrun no TX
-        ADD_FIELD32(mulr,26,1);   // multiple request
+        ADD_FIELD32(rst, 0, 1);    // Reset
+        ADD_FIELD32(en, 1, 1);     // Enable
+        ADD_FIELD32(bce, 2, 1);    // busy check enable
+        ADD_FIELD32(psp, 3, 1);    // pad short packets
+        ADD_FIELD32(ct, 4, 8);     // collision threshold
+        ADD_FIELD32(cold, 12, 10); // collision distance
+        ADD_FIELD32(swxoff, 22, 1); // software xoff transmission
+        ADD_FIELD32(pbe, 23, 1);    // packet burst enable
+        ADD_FIELD32(rtlc, 24, 1);   // retransmit late collisions
+        ADD_FIELD32(nrtu, 25, 1);   // on underrun no TX
+        ADD_FIELD32(mulr, 26, 1);   // multiple request
     };
     TCTL tctl;

@@ -561,8 +685,8 @@
     {
         // 0x1000 PBA Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(rxa,0,16);
-        ADD_FIELD32(txa,16,16);
+        ADD_FIELD32(rxa, 0, 16);
+        ADD_FIELD32(txa, 16, 16);
     };
     PBA pba;

@@ -570,9 +694,9 @@
     {
         // 0x2160 FCRTL Register
         using Reg<uint32_t>::operator=;
- ADD_FIELD32(rtl,3,28); // make this bigger than the spec so we can have
-                               // a larger buffer
-        ADD_FIELD32(xone, 31,1);
+ ADD_FIELD32(rtl, 3, 28); // make this bigger than the spec so we can
+                                 // have a larger buffer
+        ADD_FIELD32(xone, 31, 1);
     };
     FCRTL fcrtl;

@@ -580,9 +704,9 @@
     {
         // 0x2168 FCRTL Register
         using Reg<uint32_t>::operator=;
- ADD_FIELD32(rth,3,13); // make this bigger than the spec so we can have
-                               //a larger buffer
-        ADD_FIELD32(xfce, 31,1);
+ ADD_FIELD32(rth, 3, 13); // make this bigger than the spec so we can
+                                 // have a larger buffer
+        ADD_FIELD32(xfce, 31, 1);
     };
     FCRTH fcrth;

@@ -590,8 +714,8 @@
     {
         // 0x2800 RDBA Register
         using Reg<uint64_t>::operator=;
-        ADD_FIELD64(rdbal,0,32); // base address of rx descriptor ring
-        ADD_FIELD64(rdbah,32,32); // base address of rx descriptor ring
+        ADD_FIELD64(rdbal, 0, 32); // base address of rx descriptor ring
+        ADD_FIELD64(rdbah, 32, 32); // base address of rx descriptor ring
     };
     RDBA rdba;

@@ -599,7 +723,7 @@
     {
         // 0x2808 RDLEN Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(len,7,13); // number of bytes in the descriptor buffer
+ ADD_FIELD32(len, 7, 13); // number of bytes in the descriptor buffer
     };
     RDLEN rdlen;

@@ -609,8 +733,8 @@
         using Reg<uint32_t>::operator=;
         ADD_FIELD32(pktlen, 0, 8);
         ADD_FIELD32(hdrlen, 8, 8); // guess based on header, not documented
- ADD_FIELD32(desctype, 25,3); // type of descriptor 000 legacy, 001 adv,
-                                     //101 hdr split
+        ADD_FIELD32(desctype, 25, 3); // type of descriptor 000 legacy,
+                                      // 001 adv, 101 hdr split
         unsigned bufLen() { return pktlen() << 10; }
         unsigned hdrLen() { return hdrlen() << 6; }
     };
@@ -620,7 +744,7 @@
     {
         // 0x2810 RDH Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(rdh,0,16); // head of the descriptor ring
+        ADD_FIELD32(rdh, 0, 16); // head of the descriptor ring
     };
     RDH rdh;

@@ -628,7 +752,7 @@
     {
         // 0x2818 RDT Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(rdt,0,16); // tail of the descriptor ring
+        ADD_FIELD32(rdt, 0, 16); // tail of the descriptor ring
     };
     RDT rdt;

@@ -636,8 +760,8 @@
     {
         // 0x2820 RDTR Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(delay,0,16); // receive delay timer
-        ADD_FIELD32(fpd, 31,1);   // flush partial descriptor block ??
+        ADD_FIELD32(delay, 0, 16); // receive delay timer
+        ADD_FIELD32(fpd, 31, 1);   // flush partial descriptor block ??
     };
     RDTR rdtr;

@@ -645,12 +769,12 @@
     {
         // 0x2828 RXDCTL Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(pthresh,0,6);   // prefetch threshold, less that this
-                                    // consider prefetch
-        ADD_FIELD32(hthresh,8,6);   // number of descriptors in host mem to
-                                    // consider prefetch
-        ADD_FIELD32(wthresh,16,6);  // writeback threshold
-        ADD_FIELD32(gran,24,1);     // granularity 0 = desc, 1 = cacheline
+        ADD_FIELD32(pthresh, 0, 6);   // prefetch threshold, less that this
+                                      // consider prefetch
+ ADD_FIELD32(hthresh, 8, 6); // number of descriptors in host mem to
+                                      // consider prefetch
+        ADD_FIELD32(wthresh, 16, 6);  // writeback threshold
+ ADD_FIELD32(gran, 24, 1); // granularity 0 = desc, 1 = cacheline
     };
     RXDCTL rxdctl;

@@ -658,7 +782,7 @@
     {
         // 0x282C RADV Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(idv,0,16); // absolute interrupt delay
+        ADD_FIELD32(idv, 0, 16); // absolute interrupt delay
     };
     RADV radv;

@@ -666,7 +790,7 @@
     {
         // 0x2C00 RSRPD Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(idv,0,12); // size to interrutp on small packets
+        ADD_FIELD32(idv, 0, 12); // size to interrutp on small packets
     };
     RSRPD rsrpd;

@@ -674,8 +798,8 @@
     {
         // 0x3800 TDBAL Register
         using Reg<uint64_t>::operator=;
- ADD_FIELD64(tdbal,0,32); // base address of transmit descriptor ring - ADD_FIELD64(tdbah,32,32); // base address of transmit descriptor ring + ADD_FIELD64(tdbal, 0, 32); // base address of transmit descriptor ring + ADD_FIELD64(tdbah, 32, 32); // base address of transmit descriptor ring
     };
     TDBA tdba;

@@ -683,7 +807,7 @@
     {
         // 0x3808 TDLEN Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(len,7,13); // number of bytes in the descriptor buffer
+ ADD_FIELD32(len, 7, 13); // number of bytes in the descriptor buffer
     };
     TDLEN tdlen;

@@ -691,7 +815,7 @@
     {
         // 0x3810 TDH Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(tdh,0,16); // head of the descriptor ring
+        ADD_FIELD32(tdh, 0, 16); // head of the descriptor ring
     };
     TDH tdh;

@@ -700,7 +824,7 @@
         // 0x3814 TXDCA_CTL Register
         using Reg<uint32_t>::operator=;
         ADD_FIELD32(cpu_mask, 0, 5);
-        ADD_FIELD32(enabled, 5,1);
+        ADD_FIELD32(enabled, 5, 1);
         ADD_FIELD32(relax_ordering, 6, 1);
     };
     TXDCA_CTL txdca_ctl;
@@ -709,7 +833,7 @@
     {
         // 0x3818 TDT Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(tdt,0,16); // tail of the descriptor ring
+        ADD_FIELD32(tdt, 0, 16); // tail of the descriptor ring
     };
     TDT tdt;

@@ -717,7 +841,7 @@
     {
         // 0x3820 TIDV Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(idv,0,16); // interrupt delay
+        ADD_FIELD32(idv, 0, 16); // interrupt delay
     };
     TIDV tidv;

@@ -725,16 +849,17 @@
     {
         // 0x3828 TXDCTL Register
         using Reg<uint32_t>::operator=;
- ADD_FIELD32(pthresh, 0,6); // if number of descriptors control has is - // below this number, a prefetch is considered - ADD_FIELD32(hthresh,8,8); // number of valid descriptors is host memory
-                                    // before a prefetch is considered
-        ADD_FIELD32(wthresh,16,6);  // number of descriptors to keep until
+ ADD_FIELD32(pthresh, 0, 6); // if number of descriptors control has is
+                                    // below this number, a prefetch is
+                                    //considered
+        ADD_FIELD32(hthresh, 8, 8); // number of valid descriptors is host
+ // memory before a prefetch is considered
+        ADD_FIELD32(wthresh, 16, 6);// number of descriptors to keep until
                                     // writeback is considered
- ADD_FIELD32(gran, 24,1); // granulatiry of above values (0 = cacheline,
-                                    // 1 == desscriptor)
- ADD_FIELD32(lwthresh,25,7); // xmit descriptor low thresh, interrupt
-                                    // below this level
+        ADD_FIELD32(gran, 24, 1);   // granulatiry of above values
+                                    // (0 = cacheline, 1 == desscriptor)
+ ADD_FIELD32(lwthresh, 25, 7); // xmit descriptor low thresh, interrupt
+                                      // below this level
     };
     TXDCTL txdctl;

@@ -742,29 +867,19 @@
     {
         // 0x382C TADV Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(idv,0,16); // absolute interrupt delay
+        ADD_FIELD32(idv, 0, 16); // absolute interrupt delay
     };
     TADV tadv;
-/*
-    struct TDWBA : public Reg<uint64_t>
-    {
-        // 0x3838 TDWBA Register
-        using Reg<uint64_t>::operator=;
- ADD_FIELD64(en,0,1); // enable transmit description ring address writeback - ADD_FIELD64(tdwbal,2,32); // base address of transmit descriptor ring address writeback - ADD_FIELD64(tdwbah,32,32); // base address of transmit descriptor ring
-    };
-    TDWBA tdwba;*/
     uint64_t tdwba;

     struct RXCSUM : public Reg<uint32_t>
     {
         // 0x5000 RXCSUM Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(pcss,0,8);
-        ADD_FIELD32(ipofld,8,1);
-        ADD_FIELD32(tuofld,9,1);
-        ADD_FIELD32(pcsd, 13,1);
+        ADD_FIELD32(pcss, 0, 8);
+        ADD_FIELD32(ipofld, 8, 1);
+        ADD_FIELD32(tuofld, 9, 1);
+        ADD_FIELD32(pcsd, 13, 1);
     };
     RXCSUM rxcsum;

@@ -774,16 +889,16 @@
     {
         // 0x5008 RFCTL Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(iscsi_dis,0,1);
-        ADD_FIELD32(iscsi_dwc,1,5);
-        ADD_FIELD32(nfsw_dis,6,1);
-        ADD_FIELD32(nfsr_dis,7,1);
-        ADD_FIELD32(nfs_ver,8,2);
-        ADD_FIELD32(ipv6_dis,10,1);
-        ADD_FIELD32(ipv6xsum_dis,11,1);
-        ADD_FIELD32(ackdis,13,1);
-        ADD_FIELD32(ipfrsp_dis,14,1);
-        ADD_FIELD32(exsten,15,1);
+        ADD_FIELD32(iscsi_dis, 0, 1);
+        ADD_FIELD32(iscsi_dwc, 1, 5);
+        ADD_FIELD32(nfsw_dis, 6, 1);
+        ADD_FIELD32(nfsr_dis, 7, 1);
+        ADD_FIELD32(nfs_ver, 8, 2);
+        ADD_FIELD32(ipv6_dis, 10, 1);
+        ADD_FIELD32(ipv6xsum_dis, 11, 1);
+        ADD_FIELD32(ackdis, 13, 1);
+        ADD_FIELD32(ipfrsp_dis, 14, 1);
+        ADD_FIELD32(exsten, 15, 1);
     };
     RFCTL rfctl;

@@ -791,33 +906,33 @@
     {
         // 0x5820 MANC Register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(smbus,0,1);    // SMBus enabled #####
-        ADD_FIELD32(asf,1,1);      // ASF enabled #####
-        ADD_FIELD32(ronforce,2,1); // reset of force
-        ADD_FIELD32(rsvd,3,5);     // reserved
-        ADD_FIELD32(rmcp1,8,1);    // rcmp1 filtering
-        ADD_FIELD32(rmcp2,9,1);    // rcmp2 filtering
-        ADD_FIELD32(ipv4,10,1);     // enable ipv4
-        ADD_FIELD32(ipv6,11,1);     // enable ipv6
-        ADD_FIELD32(snap,12,1);     // accept snap
-        ADD_FIELD32(arp,13,1);      // filter arp #####
-        ADD_FIELD32(neighbor,14,1); // neighbor discovery
-        ADD_FIELD32(arp_resp,15,1); // arp response
-        ADD_FIELD32(tcorst,16,1);   // tco reset happened
-        ADD_FIELD32(rcvtco,17,1);   // receive tco enabled ######
-        ADD_FIELD32(blkphyrst,18,1);// block phy resets ########
-        ADD_FIELD32(rcvall,19,1);   // receive all
-        ADD_FIELD32(macaddrfltr,20,1); // mac address filtering ######
-        ADD_FIELD32(mng2host,21,1); // mng2 host packets #######
-        ADD_FIELD32(ipaddrfltr,22,1); // ip address filtering
-        ADD_FIELD32(xsumfilter,23,1); // checksum filtering
-        ADD_FIELD32(brfilter,24,1); // broadcast filtering
-        ADD_FIELD32(smbreq,25,1);   // smb request
-        ADD_FIELD32(smbgnt,26,1);   // smb grant
-        ADD_FIELD32(smbclkin,27,1); // smbclkin
-        ADD_FIELD32(smbdatain,28,1); // smbdatain
-        ADD_FIELD32(smbdataout,29,1); // smb data out
-        ADD_FIELD32(smbclkout,30,1); // smb clock out
+        ADD_FIELD32(smbus, 0, 1);    // SMBus enabled #####
+        ADD_FIELD32(asf, 1, 1);      // ASF enabled #####
+        ADD_FIELD32(ronforce, 2, 1); // reset of force
+        ADD_FIELD32(rsvd, 3, 5);     // reserved
+        ADD_FIELD32(rmcp1, 8, 1);    // rcmp1 filtering
+        ADD_FIELD32(rmcp2, 9, 1);    // rcmp2 filtering
+        ADD_FIELD32(ipv4, 10, 1);     // enable ipv4
+        ADD_FIELD32(ipv6, 11, 1);     // enable ipv6
+        ADD_FIELD32(snap, 12, 1);     // accept snap
+        ADD_FIELD32(arp, 13, 1);      // filter arp #####
+        ADD_FIELD32(neighbor, 14, 1); // neighbor discovery
+        ADD_FIELD32(arp_resp, 15, 1); // arp response
+        ADD_FIELD32(tcorst, 16, 1);   // tco reset happened
+        ADD_FIELD32(rcvtco, 17, 1);   // receive tco enabled ######
+        ADD_FIELD32(blkphyrst, 18, 1);// block phy resets ########
+        ADD_FIELD32(rcvall, 19, 1);   // receive all
+        ADD_FIELD32(macaddrfltr, 20, 1); // mac address filtering ######
+        ADD_FIELD32(mng2host, 21, 1); // mng2 host packets #######
+        ADD_FIELD32(ipaddrfltr, 22, 1); // ip address filtering
+        ADD_FIELD32(xsumfilter, 23, 1); // checksum filtering
+        ADD_FIELD32(brfilter, 24, 1); // broadcast filtering
+        ADD_FIELD32(smbreq, 25, 1);   // smb request
+        ADD_FIELD32(smbgnt, 26, 1);   // smb grant
+        ADD_FIELD32(smbclkin, 27, 1); // smbclkin
+        ADD_FIELD32(smbdatain, 28, 1); // smbdatain
+        ADD_FIELD32(smbdataout, 29, 1); // smb data out
+        ADD_FIELD32(smbclkout, 30, 1); // smb clock out
     };
     MANC manc;

@@ -825,9 +940,9 @@
     {
         // 0x5B50 SWSM register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(smbi,0,1); // Semaphone bit
-        ADD_FIELD32(swesmbi, 1,1); // Software eeporm semaphore
-        ADD_FIELD32(wmng, 2,1); // Wake MNG clock
+        ADD_FIELD32(smbi, 0, 1); // Semaphone bit
+        ADD_FIELD32(swesmbi, 1, 1); // Software eeporm semaphore
+        ADD_FIELD32(wmng, 2, 1); // Wake MNG clock
         ADD_FIELD32(reserved, 3, 29);
     };
     SWSM swsm;
@@ -836,12 +951,12 @@
     {
         // 0x5B54 FWSM register
         using Reg<uint32_t>::operator=;
-        ADD_FIELD32(eep_fw_semaphore,0,1);
-        ADD_FIELD32(fw_mode, 1,3);
-        ADD_FIELD32(ide, 4,1);
-        ADD_FIELD32(sol, 5,1);
-        ADD_FIELD32(eep_roload, 6,1);
-        ADD_FIELD32(reserved, 7,8);
+        ADD_FIELD32(eep_fw_semaphore, 0, 1);
+        ADD_FIELD32(fw_mode, 1, 3);
+        ADD_FIELD32(ide, 4, 1);
+        ADD_FIELD32(sol, 5, 1);
+        ADD_FIELD32(eep_roload, 6, 1);
+        ADD_FIELD32(reserved, 7, 8);
         ADD_FIELD32(fw_val_bit, 15, 1);
         ADD_FIELD32(reset_cnt, 16, 3);
         ADD_FIELD32(ext_err_ind, 19, 6);



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/48924
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: If3a1bf7d9b9068608ca70edf9577c0b2e1b44306
Gerrit-Change-Number: 48924
Gerrit-PatchSet: 3
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Nikos Nikoleris <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to