[coreboot-gerrit] Change in coreboot[master]: Documentation/nb/intel/sandybridge/nri_registers.md: Fix mistake

Martin Roth (Code Review) gerrit at coreboot.org
Thu Aug 30 16:47:44 CEST 2018


Martin Roth has submitted this change and it was merged. ( https://review.coreboot.org/28377 )

Change subject: Documentation/nb/intel/sandybridge/nri_registers.md: Fix mistake
......................................................................

Documentation/nb/intel/sandybridge/nri_registers.md: Fix mistake

According to a comment on fa1a07b, the 100MHz clock is the Ivy Bridge
only clock, not the 133MHz one.

Change-Id: I28fed4a9264b96f93b9e88325f547a5db512514c
Signed-off-by: Angel Pons <th3fanbus at gmail.com>
Reviewed-on: https://review.coreboot.org/28377
Reviewed-by: Felix Held <felix-coreboot at felixheld.de>
Tested-by: build bot (Jenkins) <no-reply at coreboot.org>
---
M Documentation/northbridge/intel/sandybridge/nri_registers.md
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Felix Held: Looks good to me, approved



diff --git a/Documentation/northbridge/intel/sandybridge/nri_registers.md b/Documentation/northbridge/intel/sandybridge/nri_registers.md
index d5857ec..aa16644 100644
--- a/Documentation/northbridge/intel/sandybridge/nri_registers.md
+++ b/Documentation/northbridge/intel/sandybridge/nri_registers.md
@@ -2137,8 +2137,8 @@
 +===========+==================================================================+
 |        0:7|                Selected multiplier: 100Mhz [7,12], 133Mhz [3,19] |
 +-----------+------------------------------------------------------------------+
-|         8 | - 1: 100Mhz reference clock                                      |
-|           | - 0: 133Mhz reference clock (Ivy Bridge only)                    |
+|         8 | - 1: 100Mhz reference clock (Ivy Bridge only)                    |
+|           | - 0: 133Mhz reference clock                                      |
 +-----------+------------------------------------------------------------------+
 |        31 |                                                         PLL busy |
 +-----------+------------------------------------------------------------------+

-- 
To view, visit https://review.coreboot.org/28377
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I28fed4a9264b96f93b9e88325f547a5db512514c
Gerrit-Change-Number: 28377
Gerrit-PatchSet: 2
Gerrit-Owner: Angel Pons <th3fanbus at gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus at gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot at felixheld.de>
Gerrit-Reviewer: Martin Roth <martinroth at google.com>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180830/eb0a6137/attachment.html>


More information about the coreboot-gerrit mailing list