Change in coreboot[master]: cpu/intel/speedstep: Add comment to clarify define

[Patrick Rudolph (Code Review)]

Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/34360 )

Change subject: cpu/intel/speedstep: Add comment to clarify define
......................................................................

cpu/intel/speedstep: Add comment to clarify define

Add a comment to clarify that the define must match the define in
southbridge headers.

Change-Id: Ie0e97b170c81e3bca38975d6a3fe9a368c70e622
Signed-off-by: Patrick Rudolph 
---
M src/include/cpu/intel/speedstep.h
1 file changed, 1 insertion(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/60/34360/1

diff --git a/src/include/cpu/intel/speedstep.h b/src/include/cpu/intel/speedstep.h
index 05d83ed..660e5ae 100644
--- a/src/include/cpu/intel/speedstep.h
+++ b/src/include/cpu/intel/speedstep.h
@@ -25,6 +25,7 @@

/* MWAIT coordination I/O base address. This must match
  * the _PR_.CP00 PM base address.
+ * PMB0_BASE - 0x10 should match PMBASE for legacy C-state emulation.
  */
 #define PMB0_BASE 0x510

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34360 )

Change subject: cpu/intel/speedstep: Add comment to clarify define
......................................................................

Patch Set 1: Code-Review+1

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34360 )

Change subject: cpu/intel/speedstep: Add comment to clarify define
......................................................................

Patch Set 1: Code-Review+1

[Christian Walter (Code Review)]

Christian Walter has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34360 )

Change subject: cpu/intel/speedstep: Add comment to clarify define
......................................................................

Patch Set 1: Code-Review+1

[XS] Change in coreboot[master]: cpu/intel/speedstep: Add comment to clarify define

[Martin L Roth (Code Review)]

Martin L Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/34360?usp=email )

Change subject: cpu/intel/speedstep: Add comment to clarify define
......................................................................

Abandoned

This patch has not been touched in over 12 months. Anyone who wants to take over work on this patch, please feel free to restore it and do any work needed to get it merged. If you create a new patch based on this work, please credit the original author.