[flashrom-gerrit] Change in flashrom[master]: linux_mtd: display correct write-protect info

[David Hendricks (Code Review)]

David Hendricks has uploaded this change for review. ( https://review.coreboot.org/22507


Change subject: linux_mtd: display correct write-protect info
......................................................................

linux_mtd: display correct write-protect info

The wp_status call to linux_mtd currently display nothing if
write-protect is disabled.  Display correct message and match the output
with other driver.

BUG=chrome-os-partner:40208
TEST=`flashrom --wp-disable; flashrom --wp-status` should display:
     WP: write protect is disabled.
     WP: write protect range: start=0x00000000, len=0x00000000

Change-Id: Icc3727702d2813b875a7e3a7151b44a7853967bd
Reviewed-on: https://chromium-review.googlesource.com/340882
Commit-Ready: Wei-Ning Huang <wnhuang(a)chromium.org>
Tested-by: Wei-Ning Huang <wnhuang(a)chromium.org>
Reviewed-by: Ricky Liang <jcliang(a)chromium.org>
Reviewed-by: David Hendricks <dhendrix(a)chromium.org>
---
M linux_mtd.c
1 file changed, 6 insertions(+), 4 deletions(-)



  git pull ssh://review.coreboot.org:29418/flashrom refs/changes/07/22507/1

diff --git a/linux_mtd.c b/linux_mtd.c
index 6055aef..f09c572 100644
--- a/linux_mtd.c
+++ b/linux_mtd.c
@@ -505,14 +505,16 @@
 		}
 
 		if (start_found && end_found) {
-			msg_pinfo("WP: write protect range: start=0x%08x, "
-				"len=0x%08x\n", start, end - start);
-			/* TODO: Replace this break with "start = end = 0" if
-			 * we want to support non-contiguous locked regions */
+			/* TODO: changes required for supporting non-contiguous locked regions */
 			break;
 		}
 	}
 
+	msg_cinfo("WP: write protect is %s.\n",
+			(start_found && end_found)? "enabled": "disabled");
+	msg_pinfo("WP: write protect range: start=0x%08x, "
+			"len=0x%08x\n", start, end - start);
+
 	return 0;
 }
 

-- 
To view, visit https://review.coreboot.org/22507
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icc3727702d2813b875a7e3a7151b44a7853967bd
Gerrit-Change-Number: 22507
Gerrit-PatchSet: 1
Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com>