Re: [PATCH v3 1/9] riscv/boot: Add a missing header include

2020-05-20 Thread Alistair Francis
On Tue, May 19, 2020 at 11:01 PM Philippe Mathieu-Daudé wrote: > > On 5/19/20 11:31 PM, Alistair Francis wrote: > > Currently every c file that includes boot.h also includes loader.h > > before it. Which is why the build works fine. We should be able to > > include just boot.h though so this is a

Re: [PATCH v3 1/9] riscv/boot: Add a missing header include

2020-05-19 Thread Philippe Mathieu-Daudé
On 5/19/20 11:31 PM, Alistair Francis wrote: Currently every c file that includes boot.h also includes loader.h before it. Which is why the build works fine. We should be able to include just boot.h though so this is a small fixup to allow that. What about: 'As the functions declared in this h

[PATCH v3 1/9] riscv/boot: Add a missing header include

2020-05-19 Thread Alistair Francis
Currently every c file that includes boot.h also includes loader.h before it. Which is why the build works fine. We should be able to include just boot.h though so this is a small fixup to allow that. Signed-off-by: Alistair Francis Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Bin Meng ---