Hi Alistair, On Thu, May 14, 2020 at 11:38 PM Alistair Francis <alistai...@gmail.com> wrote: > > On Thu, May 14, 2020 at 8:34 AM Bin Meng <bmeng...@gmail.com> wrote: > > > > On Fri, May 8, 2020 at 3:24 AM Alistair Francis > > <alistair.fran...@wdc.com> wrote: > > > > > > Signed-off-by: Alistair Francis <alistair.fran...@wdc.com> > > > --- > > > include/hw/riscv/boot.h | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/include/hw/riscv/boot.h b/include/hw/riscv/boot.h > > > index 474a940ad5..9daa98da08 100644 > > > --- a/include/hw/riscv/boot.h > > > +++ b/include/hw/riscv/boot.h > > > @@ -21,6 +21,7 @@ > > > #define RISCV_BOOT_H > > > > > > #include "exec/cpu-defs.h" > > > +#include "hw/loader.h" > > > > Why is this needed? Currently this does not break build. > > 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. >
Please include such in the commit message to help people understand. Reviewed-by: Bin Meng <bin.m...@windriver.com> Regards, Bin
