[U-Boot] [PATCH 09/19] riscv: qemu: Probe cpus during boot

[Bin Meng]

Hi Lukas,

On Thu, Nov 15, 2018 at 6:22 AM Auer, Lukas
<lukas.auer at aisec.fraunhofer.de> wrote:
>
> Hi Bin,
>
> On Tue, 2018-11-13 at 00:21 -0800, Bin Meng wrote:
> > This calls cpu_probe_all() to probe all available cpus.
> >
> > Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> > ---
> >
> >  arch/riscv/cpu/qemu/Kconfig |  1 +
> >  arch/riscv/cpu/qemu/cpu.c   | 14 ++++++++++++++
> >  2 files changed, 15 insertions(+)
> >
>
> Reviewed-by: Lukas Auer <lukas.auer at aisec.fraunhofer.de>
>
> This could also go into the generic cpu/cpu.c, what do you think?
>

Yes, I think so. Let's do this in v2.

> > diff --git a/arch/riscv/cpu/qemu/Kconfig
> > b/arch/riscv/cpu/qemu/Kconfig
> > index ec5d934..e91cff5 100644
> > --- a/arch/riscv/cpu/qemu/Kconfig
> > +++ b/arch/riscv/cpu/qemu/Kconfig
> > @@ -4,6 +4,7 @@
> >
> >  config QEMU_RISCV
> >       bool
> > +     select ARCH_EARLY_INIT_R
> >       imply CPU
> >       imply CPU_RISCV
> >       imply RISCV_TIMER
> > diff --git a/arch/riscv/cpu/qemu/cpu.c b/arch/riscv/cpu/qemu/cpu.c
> > index 221f3a8..e98f624 100644
> > --- a/arch/riscv/cpu/qemu/cpu.c
> > +++ b/arch/riscv/cpu/qemu/cpu.c
> > @@ -4,7 +4,9 @@
> >   */
> >
> >  #include <common.h>
> > +#include <cpu.h>
> >  #include <dm.h>
> > +#include <log.h>
> >
> >  /*
> >   * cleanup_before_linux() is called just before we call linux
> > @@ -21,6 +23,18 @@ int cleanup_before_linux(void)
> >       return 0;
> >  }
> >
> > +int arch_early_init_r(void)
> > +{
> > +     int ret;
> > +
> > +     /* probe cpus so that risc-v timer can be bound */
> > +     ret = cpu_probe_all();
> > +     if (ret)
> > +             return log_msg_ret("risc-v cpus probe fails\n", ret);
>
> nit: RISC-V (here and in the comment above), failed instead of fails
>

Will fix in v2.

Regards,
Bin