Hi Heinrich,
On Tue, 3 Dec 2024 at 07:38, Heinrich Schuchardt xypron.glpk@gmx.de wrote:
On 01.12.24 16:24, Simon Glass wrote:
Rather than an integer, it is better to use the enum provided, when referring to an EFI memory-type. Update existing uses.
Call the value 'mem_type' consistently. Fix up one instance of upper-case hex.
Fix up the calls in struct efi_boot_services so that they use the same enum, adding the missing parameter names and enum efi_allocate_type.
While we are here, rename the 'memory' parameter to 'memoryp' so that it is clear it is a return value.
Signed-off-by: Simon Glass sjg@chromium.org
Simon, I have no clue why you keep ignoring reviews. This is just annoying and won't lead to merging patches.
I got your feedback on [1] and incorporated it in v2 onwards. Did you notice?
NAK
Regards, Simon
[1] https://patchwork.ozlabs.org/project/uboot/patch/20241125204456.1525752-7-sj...