Re: [PATCH v1 13/14] xen/riscv: initialize interrupt controller

[Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>]

On 4/15/25 5:59 PM, Jan Beulich wrote:

> On 08.04.2025 17:57, Oleksii Kurochko wrote:
> > Call intc_init() to do basic initialization steps for APLIC and IMISC.
> >
> > Signed-off-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
> >       
> Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
> yet ...
>
> > --- a/xen/arch/riscv/setup.c
> > +++ b/xen/arch/riscv/setup.c
> > @@ -136,6 +136,8 @@ void __init noreturn start_xen(unsigned long bootcpu_id,
> >  
> >      intc_preinit();
> >  
> > +    intc_init();
> > +
> >      printk("All set up\n");
> >  
> >      machine_halt();
> >       
> ... this being everything here I wonder if this can't be folded with the
> patch where the function is introduced.
>     
Sure, it can be folded. I will do that to reduce patch series.

~ Oleksii