[coreboot-gerrit] New patch to review for coreboot: d723035 northbridg/via/vx900: Doxygen fixes

Martin Roth (gaumless@gmail.com) gerrit at coreboot.org
Wed Jan 7 04:49:23 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/8154

-gerrit

commit d7230356eb8f5f33e11ada8963a4e2f52e106bd5
Author: Martin Roth <gaumless at gmail.com>
Date:   Tue Jan 6 10:20:42 2015 -0700

    northbridg/via/vx900: Doxygen fixes
    
    - @todo has to be lowercase for doxygen
    - Fix some parameters that had changed in the code.
    - The @file entries needed to be more specific.
    
    Change-Id: Icdce08735f581609cd25cce41e986c71435368a4
    Signed-off-by: Martin Roth <gaumless at gmail.com>
---
 src/northbridge/via/vx900/chrome9hd.c   | 2 +-
 src/northbridge/via/vx900/early_smbus.c | 3 ++-
 src/northbridge/via/vx900/lpc.c         | 2 +-
 src/northbridge/via/vx900/northbridge.c | 2 +-
 src/northbridge/via/vx900/pcie.c        | 2 +-
 src/northbridge/via/vx900/sata.c        | 2 +-
 src/northbridge/via/vx900/traf_ctrl.c   | 2 +-
 7 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/northbridge/via/vx900/chrome9hd.c b/src/northbridge/via/vx900/chrome9hd.c
index b9fed6f..326fda1 100644
--- a/src/northbridge/via/vx900/chrome9hd.c
+++ b/src/northbridge/via/vx900/chrome9hd.c
@@ -53,7 +53,7 @@
  * the IGP is able to use it. GRUB2 and linux are capable of getting a usable
  * text console, which uses the monitor's native resolution (even 1920x1080).
  * The graphical console (linux) does not work properly.
- * @TODO
+ * @todo
  * 1. Figure out what sequence we need to do to get the VGA BIOS running
  *    properly. Use the code provided by VIA and compare their sequence to ours,
  *    fill in any missing steps, etc.
diff --git a/src/northbridge/via/vx900/early_smbus.c b/src/northbridge/via/vx900/early_smbus.c
index f006ce4..0398562 100644
--- a/src/northbridge/via/vx900/early_smbus.c
+++ b/src/northbridge/via/vx900/early_smbus.c
@@ -48,7 +48,8 @@ static void smbus_delays(int delays)
 /**
  * Read a byte from the SMBus.
  *
- * @param dimm The address location of the DIMM on the SMBus.
+ * @param smbus_dev The PCI address of the SMBus device .
+ * @param addr The address location of the DIMM on the SMBus.
  * @param offset The offset the data is located at.
  */
 u8 smbus_read_byte(u32 smbus_dev, u8 addr, u8 offset)
diff --git a/src/northbridge/via/vx900/lpc.c b/src/northbridge/via/vx900/lpc.c
index ac5e4c8..76280d8 100644
--- a/src/northbridge/via/vx900/lpc.c
+++ b/src/northbridge/via/vx900/lpc.c
@@ -30,7 +30,7 @@
 #include "chip.h"
 
 /**
- * @file lpc.c
+ * @file vx900/lpc.c
  *
  * STATUS:
  * We do a fair bit of setup, and most of it seems to work fairly well. There
diff --git a/src/northbridge/via/vx900/northbridge.c b/src/northbridge/via/vx900/northbridge.c
index f9c225d..daff180 100644
--- a/src/northbridge/via/vx900/northbridge.c
+++ b/src/northbridge/via/vx900/northbridge.c
@@ -36,7 +36,7 @@ static uint64_t uma_memory_base = 0;
 static uint64_t uma_memory_size = 0;
 
 /**
- * @file northbridge.c
+ * @file vx900/northbridge.c
  *
  * STATUS: Pretty good
  * One thing that needs to be thoroughly tested is the remap above 4G logic.
diff --git a/src/northbridge/via/vx900/pcie.c b/src/northbridge/via/vx900/pcie.c
index 109e5c9..96664e0 100644
--- a/src/northbridge/via/vx900/pcie.c
+++ b/src/northbridge/via/vx900/pcie.c
@@ -26,7 +26,7 @@
 #include "vx900.h"
 
 /**
- * @file pcie.c
+ * @file vx900/pcie.c
  *
  * STATUS:
  * We do part of the sequence to initialize the PCIE link. The problem is that
diff --git a/src/northbridge/via/vx900/sata.c b/src/northbridge/via/vx900/sata.c
index 63295e5..4b831be 100644
--- a/src/northbridge/via/vx900/sata.c
+++ b/src/northbridge/via/vx900/sata.c
@@ -24,7 +24,7 @@
 #include "vx900.h"
 
 /**
- * @file sata.c
+ * @file vx900/sata.c
  *
  * STATUS: Pretty good
  * The only issue is the SATA EPHY configuration. We do not know if it is board
diff --git a/src/northbridge/via/vx900/traf_ctrl.c b/src/northbridge/via/vx900/traf_ctrl.c
index 2e73ea4..5183391 100644
--- a/src/northbridge/via/vx900/traf_ctrl.c
+++ b/src/northbridge/via/vx900/traf_ctrl.c
@@ -28,7 +28,7 @@
 #include "chip.h"
 
 /**
- * @file traf_ctrl.c
+ * @file vx900/traf_ctrl.c
  *
  * STATUS:
  * The same issues with the IOAPIC pointe in lpc.c also apply here.



More information about the coreboot-gerrit mailing list