[coreboot-gerrit] [S] Change in coreboot[master]: ifdtool: Cleanup IFDv1 detection

[Patrick Rudolph (Code Review)]

Attention is currently required from: Stefan Reinauer.

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

Change subject: ifdtool: Cleanup IFDv1 detection
......................................................................

ifdtool: Cleanup IFDv1 detection

Change https://review.coreboot.org/c/coreboot/+/54305
"util/ifdtool: Use -p platform name to detect IFDv2 platform and chipset"

made "-p" agument mandatory for IFDv2 platforms.

Drop the IFDv2 platform CHIPSET_C620_SERIES_LEWISBURG from IFDv1 detection.

Change-Id: If29f8718b7aa696cdc07deef4c98be9a68c66f10
Signed-off-by: Patrick Rudolph 
---
M util/ifdtool/ifdtool.c
1 file changed, 17 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/68680/1

diff --git a/util/ifdtool/ifdtool.c b/util/ifdtool/ifdtool.c
index 6cc2c21..043e6ee 100644
--- a/util/ifdtool/ifdtool.c
+++ b/util/ifdtool/ifdtool.c
@@ -201,7 +201,6 @@
    uint32_t iccriba = (fdb->flmap2 >> 16) & 0xff;
    uint32_t msl = (fdb->flmap2 >> 8) & 0xff;
    uint32_t isl = (fdb->flmap1 >> 24);
-	uint32_t nm = (fdb->flmap1 >> 8) & 0x7;

/* Rest for IFD1 chipset type */
    if (iccriba == 0x00) {
@@ -224,8 +223,6 @@
    		return CHIPSET_8_SERIES_LYNX_POINT;
    	printf("Peculiar firmware descriptor, assuming Wildcat Point compatibility.\n");
    	return CHIPSET_9_SERIES_WILDCAT_POINT;
-	} else if (nm == 6) {
-		return CHIPSET_C620_SERIES_LEWISBURG;
    }
    return CHIPSET_PCH_UNKNOWN;
 }