[coreboot-gerrit] New patch to review for coreboot: c1f6faf src/device: Doxygen fixes
Martin Roth (gaumless@gmail.com)
gerrit at coreboot.org
Thu Jan 8 22:33:01 CET 2015
Martin Roth (gaumless at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8173
-gerrit
commit c1f6fafbe5641fa3d65b5ab0f7ec50219faeed6e
Author: Martin Roth <gaumless at gmail.com>
Date: Tue Jan 6 21:02:52 2015 -0700
src/device: Doxygen fixes
- Add missing parameters
- add missing @param commands
Change-Id: I029b5dafde94bd250800b06c0e9bd2118f10ef48
Signed-off-by: Martin Roth <gaumless at gmail.com>
---
src/device/dram/ddr3.c | 11 ++++++++++-
src/device/pci_device.c | 4 ++--
2 files changed, 12 insertions(+), 3 deletions(-)
diff --git a/src/device/dram/ddr3.c b/src/device/dram/ddr3.c
index 86abea8..8170ae1 100644
--- a/src/device/dram/ddr3.c
+++ b/src/device/dram/ddr3.c
@@ -18,7 +18,7 @@
*/
/**
- * @file ddr3_util.h
+ * @file ddr3.c
*
* \brief Utilities for decoding DDR3 SPDs
*/
@@ -452,8 +452,13 @@ static u16 ddr3_cas_to_mr0_map(u8 cas)
* write_recovery and cas are given in clock cycles. For example, a CAS of 7T
* should be given as 7.
*
+ * @param precharge_pd
* @param write_recovery Write recovery latency, tWR in clock cycles.
+ * @param dll_reset
+ * @param mode
* @param cas CAS latency in clock cycles.
+ * @param burst_type
+ * @param burst_length
*/
mrs_cmd_t ddr3_get_mr0(enum ddr3_mr0_precharge precharge_pd,
u8 write_recovery,
@@ -555,8 +560,12 @@ mrs_cmd_t ddr3_get_mr1(enum ddr3_mr1_qoff qoff,
* cas_cwl is given in clock cycles. For example, a cas_cwl of 7T should be
* given as 7.
*
+ * @param rtt_wr
+ * @param extended_temp
+ * @param self_refresh
* @param cas_cwl CAS write latency in clock cycles.
*/
+
mrs_cmd_t ddr3_get_mr2(enum ddr3_mr2_rttwr rtt_wr,
enum ddr3_mr2_srt_range extended_temp,
enum ddr3_mr2_asr self_refresh, u8 cas_cwl)
diff --git a/src/device/pci_device.c b/src/device/pci_device.c
index 22454d6..862ed0e 100644
--- a/src/device/pci_device.c
+++ b/src/device/pci_device.c
@@ -818,8 +818,8 @@ static struct device_operations *get_pci_bridge_ops(device_t dev)
* The driver entry can either point at a zero terminated array of acceptable
* device IDs, or include a single device ID.
*
- * @driver pointer to the PCI driver entry being checked
- * @device_id PCI device ID of the device being matched
+ * @param driver pointer to the PCI driver entry being checked
+ * @param device_id PCI device ID of the device being matched
*/
static int device_id_match(struct pci_driver *driver, unsigned short device_id)
{
More information about the coreboot-gerrit
mailing list