diff options
author | Patrick Williams <iawillia@us.ibm.com> | 2012-09-19 14:23:54 -0500 |
---|---|---|
committer | A. Patrick Williams III <iawillia@us.ibm.com> | 2012-10-09 17:06:49 -0500 |
commit | fb1836fd7b1b8839815595db08ae740ec7b86347 (patch) | |
tree | 54ff93536489c27b80af1f503520bd9894cdcfd3 /src/usr/pnor/pnordd.H | |
parent | 84e4274fe412a577f67805cc701f4fb66a3feb2f (diff) | |
download | talos-hostboot-fb1836fd7b1b8839815595db08ae740ec7b86347.tar.gz talos-hostboot-fb1836fd7b1b8839815595db08ae740ec7b86347.zip |
Support code coverage in extended modules.
- Reduce optimization (to -Os) to fit when doing coverage profile.
- Remove errl storage area from base image.
- Add GCC function attributes to sys library functions.
RTC: 36933
Change-Id: Ic83011a2444ef5b735db0446a14a0af34187eebf
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/1908
Tested-by: Jenkins Server
Reviewed-by: Brian H. Horton <brianh@linux.ibm.com>
Reviewed-by: ADAM R. MUHLE <armuhle@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Reviewed-by: Melissa J. Connell <missyc@us.ibm.com>
Reviewed-by: Paul Nguyen <nguyenp@us.ibm.com>
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Diffstat (limited to 'src/usr/pnor/pnordd.H')
-rw-r--r-- | src/usr/pnor/pnordd.H | 19 |
1 files changed, 12 insertions, 7 deletions
diff --git a/src/usr/pnor/pnordd.H b/src/usr/pnor/pnordd.H index c2f1589a1..1f04157ce 100644 --- a/src/usr/pnor/pnordd.H +++ b/src/usr/pnor/pnordd.H @@ -48,7 +48,7 @@ class PnorDD * @brief Performs a PNOR Read Operation * * @parm o_buffer Buffer to read data into - * @parm io_buflen Input: Number of bytes to read, + * @parm io_buflen Input: Number of bytes to read, * Output: Number of bytes actually read * @parm i_address Offset into flash to read * @@ -62,7 +62,7 @@ class PnorDD * @brief Performs a PNOR Write Operation * * @parm i_buffer Buffer to write data from - * @parm io_buflen Input: Number of bytes to write, + * @parm io_buflen Input: Number of bytes to write, * Output: Number of bytes actually written * @parm i_address Offset into flash to write * @@ -72,6 +72,7 @@ class PnorDD size_t& io_buflen, uint64_t i_address); + // Enumeration values must match those in debug framework. enum PnorMode_t { MODEL_UNKNOWN, /**< Invalid */ MODEL_MEMCPY, /**< No LPC logic, just do memcpy into cache area */ @@ -93,7 +94,7 @@ class PnorDD */ ~PnorDD(); - protected: + protected: /** @@ -106,7 +107,7 @@ class PnorDD LPC_REG_BAR2 = 0x08, /**< BAR2 : LPC Memory space */ LPC_REG_BAR3 = 0x0C, /**< BAR3 : LPC Firmware space */ }; - + /** * @brief SPI Config Info * OP Codes and other MISC info for configuring SFC @@ -437,7 +438,7 @@ class PnorDD * @parm i_address Offset into flash * @parm i_byteSize Number of bytes in range * - * @return Number of full or partial erase blocks + * @return Number of full or partial erase blocks */ uint32_t getNumAffectedBlocks(uint32_t i_address, size_t i_byteSize) @@ -502,7 +503,7 @@ class PnorDD uint64_t address : 32; /**< 32:63 = LPC Address */ }; - ControlReg_t() : data64(LPC_CTL_REG_DEFAULT) {}; + ControlReg_t() : data64(LPC_CTL_REG_DEFAULT) {}; }; /** @@ -526,6 +527,10 @@ class PnorDD }; private: // Variables + + // NOTE: The layout of the variables in this class must be maintained + // along with the offsets in the debug framework. + /** * @brief Mutex to prevent concurrent PNOR accesses * This needs to be static so we can mutex across multiple instances of PnorDD @@ -540,7 +545,7 @@ class PnorDD */ uint8_t* iv_erases; - /** + /** * @brief Determine how much of the PNOR logic to use, * this is required due to different model functionality * in the current VPO and Simics models |