ich_descriptors: Remove `Dual Output Fast Read' for newer gens
This is queried via SFDP since the Cannon Point PCH / Gemini Lake.
Avoid using a relational comparison so we don't have to worry when
changing the enum order.
Tested `flashprog' on ADL-P and `ich_descriptors_tool' with various
descriptors.
Change-Id: I4c60e8c01a019303a830d46561c80e6ad462bb7c
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.sourcearcade.org/c/flashprog/+/172
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
diff --git a/ich_descriptors.c b/ich_descriptors.c
index f939389..cd92c51 100644
--- a/ich_descriptors.c
+++ b/ich_descriptors.c
@@ -374,9 +374,23 @@
if (desc->component.modes.fastread)
msg_pdbg2("Fast Read Clock Frequency: %s\n",
pprint_freq(cs, desc->component.modes.freq_fastread));
- if (cs > CHIPSET_6_SERIES_COUGAR_POINT)
+ switch (cs) {
+ case CHIPSET_7_SERIES_PANTHER_POINT:
+ case CHIPSET_8_SERIES_LYNX_POINT:
+ case CHIPSET_BAYTRAIL:
+ case CHIPSET_8_SERIES_LYNX_POINT_LP:
+ case CHIPSET_8_SERIES_WELLSBURG:
+ case CHIPSET_9_SERIES_WILDCAT_POINT:
+ case CHIPSET_9_SERIES_WILDCAT_POINT_LP:
+ case CHIPSET_100_SERIES_SUNRISE_POINT:
+ case CHIPSET_APOLLO_LAKE:
+ case CHIPSET_C620_SERIES_LEWISBURG:
msg_pdbg2("Dual Output Fast Read Support: %sabled\n",
desc->component.modes.dual_output ? "en" : "dis");
+ break;
+ default:
+ break;
+ }
bool has_forbidden_opcode = false;
if (desc->component.FLILL != 0) {