diff options
author | Aron Griffis <aron@hp.com> | 2008-01-08 22:29:38 -0500 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2008-02-04 18:35:06 -0500 |
commit | 410ab512e5c5716287a399145df0905c1dcddb04 (patch) | |
tree | 7e56cec14488041fab537b118bb237fc2b59f7eb /arch/ia64 | |
parent | 965e7c8affeca27f7e5de75c97954e74d3b8052d (diff) |
[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.c | 4 |
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) | |||
762 | if (!md || (md->attribute & ~EFI_MEMORY_RUNTIME) != attr) | 762 | if (!md || (md->attribute & ~EFI_MEMORY_RUNTIME) != attr) |
763 | return 0; | 763 | return 0; |
764 | } while (md); | 764 | } while (md); |
765 | return 0; | 765 | return 0; /* never reached */ |
766 | } | 766 | } |
767 | 767 | ||
768 | u64 | 768 | u64 |
@@ -798,7 +798,7 @@ kern_mem_attribute (unsigned long phys_addr, unsigned long size) | |||
798 | if (!md || md->attribute != attr) | 798 | if (!md || md->attribute != attr) |
799 | return 0; | 799 | return 0; |
800 | } while (md); | 800 | } while (md); |
801 | return 0; | 801 | return 0; /* never reached */ |
802 | } | 802 | } |
803 | EXPORT_SYMBOL(kern_mem_attribute); | 803 | EXPORT_SYMBOL(kern_mem_attribute); |
804 | 804 | ||