On 09/09/2017 21:12, Laszlo Ersek wrote:
Repo: https://github.com/lersek/edk2.git Branch: udf_fixes_cleanups Patches #2, #3 and #4 are needed (and enough) for me to build OVMF for IA32 and X64 with clang-3.8, after the UDF introduction. Patches #1 and #5 are cleanups that I felt fit before patch #2 and after patch #4, respectively. Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> Cc: Eric Dong <eric.d...@intel.com> Cc: Paulo Alcantara <pca...@zytor.com> Cc: Ruiyu Ni <ruiyu...@intel.com> Cc: Star Zeng <star.z...@intel.com> Thanks Laszlo Laszlo Ersek (5): MdeModulePkg/UdfDxe: ASSERT() valid ReadFileInfo Flags for INLINE_DATA req MdeModulePkg/UdfDxe: don't return unset Status if INLINE_DATA req succeeds MdeModulePkg/UdfDxe: replace zero-init of local variables with ZeroMem() MdeModulePkg/PartitionDxe: don't divide 64-bit values with C operators MdeModulePkg/PartitionDxe: remove always false comparison MdeModulePkg/Universal/Disk/PartitionDxe/Udf.c | 9 +++++++-- MdeModulePkg/Universal/Disk/UdfDxe/File.c | 6 ++++-- MdeModulePkg/Universal/Disk/UdfDxe/FileSystemOperations.c | 5 +++++ 3 files changed, 16 insertions(+), 4 deletions(-)
Looks good to me. Tested it with OVMF X64 and AARCH64. Reviewed-by: Paulo Alcantara <pca...@zytor.com> Thanks! Paulo _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel