Re: [U-Boot] [PATCH v2 05/16] efi: Correct header order in efi_memory

2017-12-05 Thread Simon Glass
Hi Heinrich, On 4 December 2017 at 15:28, Heinrich Schuchardt wrote: > > > On 12/04/2017 10:28 PM, Simon Glass wrote: >> >> The headers are not in the correct order. Fix this. Also drop libfdt_env.h >> since it is not needed. > > > What do you consider as correct order? Do

Re: [U-Boot] [PATCH v2 05/16] efi: Correct header order in efi_memory

2017-12-04 Thread Heinrich Schuchardt
On 12/04/2017 10:28 PM, Simon Glass wrote: The headers are not in the correct order. Fix this. Also drop libfdt_env.h since it is not needed. What do you consider as correct order? Do you mean sorted alphabetically? Includes should be self sufficient and the correct execution be independent

[U-Boot] [PATCH v2 05/16] efi: Correct header order in efi_memory

2017-12-04 Thread Simon Glass
The headers are not in the correct order. Fix this. Also drop libfdt_env.h since it is not needed. Signed-off-by: Simon Glass --- Changes in v2: - Update commit message to dropping libfdt_env.h lib/efi_loader/efi_memory.c | 5 ++--- 1 file changed, 2 insertions(+), 3