summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPaul Menzel <paulepanter@users.sourceforge.net>2017-05-07 08:57:53 +0200
committerPatrick Georgi <pgeorgi@google.com>2017-05-11 16:41:56 +0200
commit90d41779d1c4cd64a83ec9a80256988b93ec8d08 (patch)
tree3be35726da1ae93fc2f5de11b5ca64d0149afd4e
parentd84a1cae093ffc6d4a3d1f4ec7599f2ba5da81f2 (diff)
downloadcoreboot-90d41779d1c4cd64a83ec9a80256988b93ec8d08.tar.xz
intelmetool: Comment out unused variable
Only commented out code uses the variable `csr`, and GCC complains about it, when enabling the warning *unused-but-set-variable*. ``` Checking for pciutils and zlib... me.c: In function ‘mei_dump’: me.c:50:18: warning: variable ‘csr’ set but not used [-Wunused-but-set-variable] struct mei_csr *csr; ^~~ ``` As the code is commented, also comment out the declaration of the variable. Change-Id: I4ecb2b5e9f32906ccfc8a0628d2e0f2d3ad39a02 Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-on: https://review.coreboot.org/19612 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Patrick Georgi <pgeorgi@google.com>
-rw-r--r--util/intelmetool/me.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/util/intelmetool/me.c b/util/intelmetool/me.c
index 8c703a3465..e9fea6bc1c 100644
--- a/util/intelmetool/me.c
+++ b/util/intelmetool/me.c
@@ -47,14 +47,14 @@ static uint8_t* mei_mmap;
static void mei_dump(void *ptr, int dword, int offset, const char *type)
{
- struct mei_csr *csr;
+ /* struct mei_csr *csr; */
switch (offset) {
case MEI_H_CSR:
case MEI_ME_CSR_HA:
- csr = ptr;
-/* if (!csr) {
+/* csr = ptr;
+ if (!csr) {
printf("%-9s[%02x] : ", type, offset);
printf("ERROR: 0x%08x\n", dword);
break;