David Hendricks merged this change.

View Change

Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Nico Huber: Looks good to me, approved
ich_descriptors: Modify limits for C620/Lewisburg PCH

Change-Id: Ic8adc4b87993e65096166fa6d665432697070b4c
Signed-off-by: David Hendricks <dhendricks@fb.com>
Reviewed-on: https://review.coreboot.org/20936
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Nico Huber <nico.h@gmx.de>
---
M ich_descriptors.h
1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/ich_descriptors.h b/ich_descriptors.h
index 8c313d1..574bc5f 100644
--- a/ich_descriptors.h
+++ b/ich_descriptors.h
@@ -165,7 +165,7 @@
};
};

-#define MAX_NUM_FLREGS 10 /* 16 on unsupported Lewisburg PCH */
+#define MAX_NUM_FLREGS 16
struct ich_desc_region {
/*
* Number of entries and width differ on various generations:
@@ -174,6 +174,7 @@
* ICH8 .. Panther Point/7 5 13
* Lynx Point/8 .. Wildcat Point/9 7 15
* Sunrise Point/100 .. 10 15
+ * Lewisburg/100 .. 16 15
*/
union {
uint32_t FLREGs[MAX_NUM_FLREGS]; /* Flash Descriptor Regions */
@@ -188,7 +189,7 @@
};
};

-#define MAX_NUM_MASTERS 5 /* 6 on unsupported Lewisburg PCH */
+#define MAX_NUM_MASTERS 6 /* 5 prior to C620/Lewisburg PCH */
struct ich_desc_master {
union {
uint32_t FLMSTRs[MAX_NUM_MASTERS]; /* Flash Masters */

To view, visit change 20936. To unsubscribe, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: staging
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic8adc4b87993e65096166fa6d665432697070b4c
Gerrit-Change-Number: 20936
Gerrit-PatchSet: 2
Gerrit-Owner: David Hendricks <david.hendricks@gmail.com>
Gerrit-Reviewer: David Hendricks <david.hendricks@gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>