[PATCH] riscv: Add do_irqinfo() for CONFIG_CMD_IRQ

Heinrich Schuchardt xypron.glpk at gmx.de
Mon Aug 24 15:31:07 CEST 2020


On 24.08.20 13:00, Pragnesh Patel wrote:
> Right now, do_irqinfo() done nothing and return 0 for

Nits:

%s/done/does/

> CONFIG_CMD_IRQ
>
> Signed-off-by: Pragnesh Patel <pragnesh.patel at sifive.com>
> ---
>  arch/riscv/lib/interrupts.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/arch/riscv/lib/interrupts.c b/arch/riscv/lib/interrupts.c
> index cd47e64487..d6273606b5 100644
> --- a/arch/riscv/lib/interrupts.c
> +++ b/arch/riscv/lib/interrupts.c
> @@ -145,3 +145,10 @@ __attribute__((weak)) void external_interrupt(struct pt_regs *regs)
>  __attribute__((weak)) void timer_interrupt(struct pt_regs *regs)
>  {
>  }
> +
> +#if defined(CONFIG_CMD_IRQ)
> +int do_irqinfo(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
> +{
> +	return 0;
> +}
> +#endif

Why do we allow CONFIG_CMD_IRQ for RISC-V if the command is not supported?

How about:

cmd/Kconfig:2224

 config CMD_IRQ
    bool "irq - Show information about interrupts"
-   depends on !ARM && !MIPS && !SH
+   depends on !ARM && !MIPS && !RISCV && !SH

Best regards

Heinrich

>



More information about the U-Boot mailing list