Commit 410ab512 authored by Aron Griffis's avatar Aron Griffis Committed by Tony Luck

[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: default avatarAron Griffis <aron@hp.com>
Signed-off-by: default avatarTony Luck <tony.luck@intel.com>
parent 965e7c8a
...@@ -762,7 +762,7 @@ efi_mem_attribute (unsigned long phys_addr, unsigned long size) ...@@ -762,7 +762,7 @@ efi_mem_attribute (unsigned long phys_addr, unsigned long size)
if (!md || (md->attribute & ~EFI_MEMORY_RUNTIME) != attr) if (!md || (md->attribute & ~EFI_MEMORY_RUNTIME) != attr)
return 0; return 0;
} while (md); } while (md);
return 0; return 0; /* never reached */
} }
u64 u64
...@@ -798,7 +798,7 @@ kern_mem_attribute (unsigned long phys_addr, unsigned long size) ...@@ -798,7 +798,7 @@ kern_mem_attribute (unsigned long phys_addr, unsigned long size)
if (!md || md->attribute != attr) if (!md || md->attribute != attr)
return 0; return 0;
} while (md); } while (md);
return 0; return 0; /* never reached */
} }
EXPORT_SYMBOL(kern_mem_attribute); EXPORT_SYMBOL(kern_mem_attribute);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment