efs: Simplify EfhBhdsIterator.

Author: daym

src/efs.rs

@@ -916,16 +916,11 @@ impl<
 			if position != 0xffff_ffff && position != 0
 			/* sigh.  Some images have 0 as "invalid" mark */
 			{
-				return match BhdDirectory::load(
+				return BhdDirectory::load(
 					self.storage,
 					position,
 					self.amd_physical_mode_mmio_size,
-				) {
-					Ok(e) => Some(e),
-					Err(e) => {
-						None // FIXME: error check
-					}
-				};
+				).ok() // FIXME: error check
 			}
 		}
 		None

src/ondisk.rs

@@ -449,21 +449,20 @@ impl ValueOrLocation {
 					directory_address_mode,
 					AddressMode::PhysicalAddress
 				) {
-					/* AMD retrofitted (introduced) two
-					   flag bits at the top bits in Milan.
-
-					   In Rome, you actually COULD use
-					   all the bits.
-
-					   Newer platform do not regularily use
-					   AddressMode::PhysicalAddress anyway.
-
-					   But if someone uses
-					   AddressMode::PhysicalAddress,
-					   they might do it on Rome and use
-					   those two top bits as part of the
-					   address.
-					*/
+					// AMD retrofitted (introduced) two
+					// flag bits at the top bits in Milan.
+					//
+					// In Rome, you actually COULD use
+					// all the bits.
+					//
+					// Newer platform do not regularily use
+					// AddressMode::PhysicalAddress anyway.
+					//
+					// But if someone uses
+					// AddressMode::PhysicalAddress,
+					// they might do it on Rome and use
+					// those two top bits as part of the
+					// address.
 					let v = u64::from(*x);
 					Ok(v)
 				} else {