This patch removes a duplicated assignment of *md2.
It also replaces a switch statement with an if statement
which is much more compact in this instance.

Signed-Off-By: Simon Horman <[EMAIL PROTECTED]>

Index: kexec-tools-patch-boot-params/purgatory/arch/ia64/purgatory-ia64.c
===================================================================
--- kexec-tools-patch-boot-params.orig/purgatory/arch/ia64/purgatory-ia64.c     
2006-09-20 15:39:32.000000000 +0900
+++ kexec-tools-patch-boot-params/purgatory/arch/ia64/purgatory-ia64.c  
2006-09-20 15:44:32.000000000 +0900
@@ -167,14 +167,9 @@
                        continue;
                mstart = md1->phys_addr;
                mend = md1->phys_addr + (md1->num_pages << EFI_PAGE_SHIFT);
-               switch (md1->type) {
-                       case EFI_LOADER_DATA:
-                               *md2 = *md1;
-                               md2->type = EFI_CONVENTIONAL_MEMORY;
-                               break;
-                       default:
-                               *md2 = *md1;
-               }
+               *md2 = *md1;
+               if (md1->type == EFI_LOADER_DATA)
+                       md2->type = EFI_CONVENTIONAL_MEMORY;
                // segments are already sorted and aligned to 4K
                orig_type = md2->type;
                for (i = 0; i < params->loaded_segments_num; i++) {

--

-- 
Horms
  H: http://www.vergenet.net/~horms/
  W: http://www.valinux.co.jp/en/

_______________________________________________
fastboot mailing list
[email protected]
https://lists.osdl.org/mailman/listinfo/fastboot

Reply via email to