[U-Boot] [PATCH 2/9] riscv: Add a SYSCON driver for Andestech's PLIC
Rick Chen
rickchen36 at gmail.com
Thu Mar 21 08:39:59 UTC 2019
Bin Meng <bmeng.cn at gmail.com> 於 2019年3月21日 週四 下午3:32寫道：
>
> Hi Rick,
>
> On Thu, Mar 21, 2019 at 3:04 PM Rick Chen <rickchen36 at gmail.com> wrote:
> >
> > Hi Bin
> >
> > Bin Meng <bmeng.cn at gmail.com> 於 2019年3月20日 週三 下午3:22寫道：
> > >
> > > Hi Rick,
> > >
> > > On Tue, Mar 19, 2019 at 5:12 PM Andes <uboot at andestech.com> wrote:
> > > >
> > > > From: Rick Chen <rick at andestech.com>
> > > >
> > > > The Platform-Level Interrupt Controller(PLIC)
> > > > block holds memory-mapped claim and pending registers
> > > > associated with software interrupt.It is required
> > >
> > > nits: need one space after interrupt.
> >
> > OK
> > I will add space.
> >
> > >
> > > > for handling IPI.
> > > >
> > > > Signed-off-by: Rick Chen <rick at andestech.com>
> > > > Cc: Greentime Hu <greentime at andestech.com>
> > > > ---
> > > > arch/riscv/Kconfig | 9 ++++
> > > > arch/riscv/include/asm/global_data.h | 3 ++
> > > > arch/riscv/include/asm/syscon.h | 2 +-
> > > > arch/riscv/lib/Makefile | 1 +
> > > > arch/riscv/lib/nds_plic.c | 84 ++++++++++++++++++++++++++++++++++++
> > > > 5 files changed, 98 insertions(+), 1 deletion(-)
> > > > create mode 100644 arch/riscv/lib/nds_plic.c
> > > >
> > > > diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
> > > > index 3a4470d..fef11dd 100644
> > > > --- a/arch/riscv/Kconfig
> > > > +++ b/arch/riscv/Kconfig
> > >
> > > Probably it makes more sense to put this to arch/riscv/cpu/ax25/Kconfig?
> >
> > I just refer to SIFIVE_CLINT. It also not make sense to place here, right ?
>
> Maybe, but since the cpu directory is renamed to 'generic', I am not
> sure moving to that directory is a good idea.
Maybe I will still put it in /arch/riscv/Kconfig.
And we can move them to the place they belong individually together if
there have better place to accommodate to SIFIVE_CLINT.
How do you think ?
Rick
>
> >
> > >
> > > > @@ -109,6 +109,15 @@ config SIFIVE_CLINT
> > > > The SiFive CLINT block holds memory-mapped control and status registers
> > > > associated with software and timer interrupts.
> > > >
> > > > +config NDS_PLIC
> > >
> > > I am not sure if it is appropriate to call this "NDS_PLIC". Shouldn't
> > > it be "ANDES_PLIC", because ANDES_ and SIFIVE_ are vendor prefixes.
> >
> > OK
> > I will use ANDES_PLIC to replace NDS_PLIC.
> >
> > >
> > > > + bool
> > > > + depends on RISCV_MMODE
> > > > + select REGMAP
> > > > + select SYSCON
> > > > + help
> > > > + The Andes PLIC block holds memory-mapped claim and pending registers
> > > > + associated with software interrupt.
> > > > +
> > > > config RISCV_RDTIME
> > > > bool
> > > > default y if RISCV_SMODE
> > > > diff --git a/arch/riscv/include/asm/global_data.h b/arch/riscv/include/asm/global_data.h
> > > > index 80e3165..15867f5 100644
> > > > --- a/arch/riscv/include/asm/global_data.h
> > > > +++ b/arch/riscv/include/asm/global_data.h
> > > > @@ -18,6 +18,9 @@ struct arch_global_data {
> > > > #ifdef CONFIG_SIFIVE_CLINT
> > > > void __iomem *clint; /* clint base address */
> > > > #endif
> > > > +#ifdef CONFIG_NDS_PLIC
> > > > + void __iomem *plic; /* plic base address */
> > > > +#endif
> > > > #ifdef CONFIG_SMP
> > > > struct ipi_data ipi[CONFIG_NR_CPUS];
> > > > #endif
> > > > diff --git a/arch/riscv/include/asm/syscon.h b/arch/riscv/include/asm/syscon.h
> > > > index d311ee6..0229989 100644
> > > > --- a/arch/riscv/include/asm/syscon.h
> > > > +++ b/arch/riscv/include/asm/syscon.h
> > > > @@ -9,11 +9,11 @@
> > > > /*
> > > > * System controllers in a RISC-V system
> > > > *
> > > > - * So far only SiFive's Core Local Interruptor (CLINT) is defined.
> > > > */
> > > > enum {
> > > > RISCV_NONE,
> > > > RISCV_SYSCON_CLINT, /* Core Local Interruptor (CLINT) */
> > > > + RISCV_SYSCON_PLIC, /* Platform Level Interrup Controller (PLIC) */
> > >
> > > typo: Interrupt
> >
> > OK
> > I will correct it.
> >
> > >
> > > > };
> > > >
> > > > #endif /* _ASM_SYSCON_H */
> > > > diff --git a/arch/riscv/lib/Makefile b/arch/riscv/lib/Makefile
> > > > index 35dbf64..8187c2b 100644
> > > > --- a/arch/riscv/lib/Makefile
> > > > +++ b/arch/riscv/lib/Makefile
> > > > @@ -11,6 +11,7 @@ obj-$(CONFIG_CMD_GO) += boot.o
> > > > obj-y += cache.o
> > > > obj-$(CONFIG_RISCV_RDTIME) += rdtime.o
> > > > obj-$(CONFIG_SIFIVE_CLINT) += sifive_clint.o
> > > > +obj-$(CONFIG_NDS_PLIC) += nds_plic.o
> > > > obj-y += interrupts.o
> > > > obj-y += reset.o
> > > > obj-$(CONFIG_SBI_IPI) += sbi_ipi.o
> > > > diff --git a/arch/riscv/lib/nds_plic.c b/arch/riscv/lib/nds_plic.c
> > >
> > > And move this driver to arch/riscv/cpu/ax25 since it's only available
> > > in AX25 CPUs?
> >
> > Same as sifive_clint.c . Shall it also move away from /lib ?
> >
>
> I agree, but see comments above :)
>
> > >
> > > > new file mode 100644
> > > > index 0000000..563da7d
> > > > --- /dev/null
> > > > +++ b/arch/riscv/lib/nds_plic.c
> > > > @@ -0,0 +1,84 @@
> > > > +// SPDX-License-Identifier: GPL-2.0+
> > > > +/*
> > > > + * Copyright (C) 2019, Rick Chen <rick at andestech.com>
> > > > + *
> > > > + * U-Boot syscon driver for Andes's Platform Level Interrupt Controller (PLIC).
> > > > + * The PLIC block holds memory-mapped claim and pending registers
> > > > + * associated with software interrupt.
> > > > + */
> > > > +
> > > > +#include <common.h>
> > > > +#include <dm.h>
> > > > +#include <regmap.h>
> > > > +#include <syscon.h>
> > > > +#include <asm/io.h>
> > > > +#include <asm/syscon.h>
> > > > +
> > > > +/* pending register */
> > > > +#define PENDING_REG(base, hart) ((ulong)(base) + 0x1000 + (hart) * 8)
> > > > +/* enable register */
> > > > +#define ENABLE_REG(base, hart) ((ulong)(base) + 0x2000 + (hart) * 0x80)
> > > > +/* claim register */
> > > > +#define CLAIM_REG(base, hart) ((ulong)(base) + 0x200004 + (hart) * 0x1000)
> > > > +
> > > > +DECLARE_GLOBAL_DATA_PTR;
> > > > +
> > > > +#define PLIC_BASE_GET(void) \
> > > > + do { \
> > > > + long *ret; \
> > > > + \
> > > > + if (!gd->arch.plic) { \
> > > > + ret = syscon_get_first_range(RISCV_SYSCON_PLIC); \
> > > > + if (IS_ERR(ret)) \
> > > > + return PTR_ERR(ret); \
> > > > + gd->arch.plic = ret; \
> > > > + } \
> > > > + } while (0)
> > > > +
> > > > +int plic_init(int harts)
> > >
> > > Can we make this function be automatically called in PLIC_BASE_GET()?
> >
> > OK
> > I will move it in PLIC_BASE_GET()
> >
> > >
> > > > +{
> > > > + int i;
> > > > + int en = 0x80808080;
> > >
> > > Can we use some macros for this?
> >
> > OK
> > I will use macro to represent it.
> >
> > >
> > > > +
> > > > + PLIC_BASE_GET();
> > > > + for(i=0;i<harts;i++)
> > >
> > > nits: should have various spaces like i = 0;
> >
> > OK
> >
> > >
> > > > + {
> > > > + en = en >> i;
> > > > + writel(en, (void __iomem *)ENABLE_REG(gd->arch.plic, i));
> > > > + }
> > > > +
> > > > + return 0;
> > > > +}
> > > > +
> > > > +int riscv_send_ipi(int hart)
> > > > +{
> > > > + PLIC_BASE_GET();
> > > > +
> > > > + writel((0x80>>hart), (void __iomem *)PENDING_REG(gd->arch.plic, gd->arch.boot_hart));
> > >
> > > macro for 0x80?
> >
> > OK
> > I will use macro to represent it.
> >
> > Thanks for review.
> >
>
> Regards,
> Bin
More information about the U-Boot
mailing list