Nico Huber submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Nico Huber: Looks good to me, approved
sfdp: drop redundant check of the mandatory table size

Change-Id: I464856612a6d21c682f1d9ad5110fa11a0a276c2
Signed-off-by: Michael Niewöhner <foss@mniewoehner.de>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/61379
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
---
M sfdp.c
1 file changed, 0 insertions(+), 4 deletions(-)

diff --git a/sfdp.c b/sfdp.c
index 530588f..85e698d 100644
--- a/sfdp.c
+++ b/sfdp.c
@@ -131,10 +131,6 @@
int j;

msg_cdbg("Parsing JEDEC flash parameter table... ");
- if (len != 9 * 4 && len != 4 * 4) {
- msg_cdbg("%s: len out of spec\n", __func__);
- return 1;
- }
msg_cdbg2("\n");

/* 1. double word */

To view, visit change 61379. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I464856612a6d21c682f1d9ad5110fa11a0a276c2
Gerrit-Change-Number: 61379
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Niewöhner <foss@mniewoehner.de>
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Felix Singer <felixsinger@posteo.net>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged