summaryrefslogtreecommitdiffstats
path: root/arch
diff options
context:
space:
mode:
authorAron Griffis <aron@hp.com>2008-01-08 22:29:38 -0500
committerTony Luck <tony.luck@intel.com>2008-02-04 15:35:06 -0800
commit410ab512e5c5716287a399145df0905c1dcddb04 (patch)
tree7e56cec14488041fab537b118bb237fc2b59f7eb /arch
parent965e7c8affeca27f7e5de75c97954e74d3b8052d (diff)
downloadop-kernel-dev-410ab512e5c5716287a399145df0905c1dcddb04.zip
op-kernel-dev-410ab512e5c5716287a399145df0905c1dcddb04.tar.gz
[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')
-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 269f4f4..d59134d 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);
OpenPOWER on IntegriCloud