[coreboot-gerrit] New patch to review for coreboot: 87fdd6d southbridge/nvidia: Spelling/comment fix

Martin Roth (gaumless@gmail.com) gerrit at coreboot.org
Tue Dec 9 21:52:44 CET 2014


Martin Roth (gaumless at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7731

-gerrit

commit 87fdd6ddf83af734aa93e08c0bd0261b31e6d828
Author: Martin Roth <martin.roth at se-eng.com>
Date:   Tue Dec 9 13:49:34 2014 -0700

    southbridge/nvidia: Spelling/comment fix
    
    Change-Id: I4f9b2b8375abe4691f279df649eaf822b87509e5
    Signed-off-by: Martin Roth <martin.roth at se-eng.com>
---
 src/southbridge/nvidia/mcp55/mcp55.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/southbridge/nvidia/mcp55/mcp55.c b/src/southbridge/nvidia/mcp55/mcp55.c
index 8350579..0a43048 100644
--- a/src/southbridge/nvidia/mcp55/mcp55.c
+++ b/src/southbridge/nvidia/mcp55/mcp55.c
@@ -218,8 +218,10 @@ void mcp55_enable(device_t dev)
 	if (!dev->enabled) {
 		final_reg |= (1 << index); /* Disable it. */
 		/*
-		 * The reason for using final_reg, if diable func 1,
-		 * the func 2 will be func 1, so we need disable them one time.
+		 * The reason for using final_reg is that if func 1 is disabled,
+		 * then func 2 will become func 1.
+		 * Because of this, we need loop through disabling them all at
+		 * the same time.
 		 */
 	}
 



More information about the coreboot-gerrit mailing list