summaryrefslogtreecommitdiffstats
path: root/arch/ia64
diff options
context:
space:
mode:
authorAron Griffis <aron@hp.com>2008-01-09 04:29:38 +0100
committerTony Luck <tony.luck@intel.com>2008-02-05 00:35:06 +0100
commit410ab512e5c5716287a399145df0905c1dcddb04 (patch)
tree7e56cec14488041fab537b118bb237fc2b59f7eb /arch/ia64
parent[IA64] efi.c Spelling/punctuation fixes (diff)
downloadlinux-410ab512e5c5716287a399145df0905c1dcddb04.tar.xz
linux-410ab512e5c5716287a399145df0905c1dcddb04.zip
[IA64] efi.c Add /* never reached */ annotation
As written, this loop could be for (;;) instead of do while (md). The tests inside the loop always result in a return so the loop never terminates normally. Signed-off-by: Aron Griffis <aron@hp.com> Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'arch/ia64')
-rw-r--r--arch/ia64/kernel/efi.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/ia64/kernel/efi.c b/arch/ia64/kernel/efi.c
index 269f4f4adfed..d59134d7e73c 100644
--- a/arch/ia64/kernel/efi.c
+++ b/arch/ia64/kernel/efi.c
@@ -762,7 +762,7 @@ efi_mem_attribute (unsigned long phys_addr, unsigned long size)
if (!md || (md->attribute & ~EFI_MEMORY_RUNTIME) != attr)
return 0;
} while (md);
- return 0;
+ return 0; /* never reached */
}
u64
@@ -798,7 +798,7 @@ kern_mem_attribute (unsigned long phys_addr, unsigned long size)
if (!md || md->attribute != attr)
return 0;
} while (md);
- return 0;
+ return 0; /* never reached */
}
EXPORT_SYMBOL(kern_mem_attribute);