[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 11/28] riscv: sifive: Rename sifive_prci.{c,
From: |
Chih-Min Chao |
Subject: |
Re: [Qemu-devel] [PATCH v2 11/28] riscv: sifive: Rename sifive_prci.{c, h} to sifive_e_prci.{c, h} |
Date: |
Thu, 8 Aug 2019 22:00:04 +0800 |
On Wed, Aug 7, 2019 at 6:11 PM Bin Meng <address@hidden> wrote:
> On Wed, Aug 7, 2019 at 4:54 PM Chih-Min Chao <address@hidden>
> wrote:
> >
> >
> >
> > On Wed, Aug 7, 2019 at 3:49 PM Bin Meng <address@hidden> wrote:
> >>
> >> Current SiFive PRCI model only works with sifive_e machine, as it
> >> only emulates registers or PRCI block in the FE310 SoC.
> >>
> >> Rename the file name to make it clear that it is for sifive_e.
> >>
> >> Signed-off-by: Bin Meng <address@hidden>
> >> ---
> >>
> >> Changes in v2: None
> >>
> >> hw/riscv/Makefile.objs | 2 +-
> >> hw/riscv/sifive_e.c | 4 ++--
> >> hw/riscv/{sifive_prci.c => sifive_e_prci.c} | 14 +++++++-------
> >> include/hw/riscv/{sifive_prci.h => sifive_e_prci.h} | 14 +++++++-------
> >> 4 files changed, 17 insertions(+), 17 deletions(-)
> >> rename hw/riscv/{sifive_prci.c => sifive_e_prci.c} (90%)
> >> rename include/hw/riscv/{sifive_prci.h => sifive_e_prci.h} (82%)
> >>
> >> diff --git a/hw/riscv/Makefile.objs b/hw/riscv/Makefile.objs
> >> index eb9d4f9..c859697 100644
> >> --- a/hw/riscv/Makefile.objs
> >> +++ b/hw/riscv/Makefile.objs
> >> @@ -2,9 +2,9 @@ obj-y += boot.o
> >> obj-$(CONFIG_SPIKE) += riscv_htif.o
> >> obj-$(CONFIG_HART) += riscv_hart.o
> >> obj-$(CONFIG_SIFIVE_E) += sifive_e.o
> >> +obj-$(CONFIG_SIFIVE_E) += sifive_e_prci.o
> >> obj-$(CONFIG_SIFIVE) += sifive_clint.o
> >> obj-$(CONFIG_SIFIVE) += sifive_gpio.o
> >> -obj-$(CONFIG_SIFIVE) += sifive_prci.o
> >> obj-$(CONFIG_SIFIVE) += sifive_plic.o
> >> obj-$(CONFIG_SIFIVE) += sifive_test.o
> >> obj-$(CONFIG_SIFIVE_U) += sifive_u.o
> >> diff --git a/hw/riscv/sifive_e.c b/hw/riscv/sifive_e.c
> >> index 2a499d8..2d67670 100644
> >> --- a/hw/riscv/sifive_e.c
> >> +++ b/hw/riscv/sifive_e.c
> >> @@ -41,9 +41,9 @@
> >> #include "hw/riscv/riscv_hart.h"
> >> #include "hw/riscv/sifive_plic.h"
> >> #include "hw/riscv/sifive_clint.h"
> >> -#include "hw/riscv/sifive_prci.h"
> >> #include "hw/riscv/sifive_uart.h"
> >> #include "hw/riscv/sifive_e.h"
> >> +#include "hw/riscv/sifive_e_prci.h"
> >> #include "hw/riscv/boot.h"
> >> #include "chardev/char.h"
> >> #include "sysemu/arch_init.h"
> >> @@ -174,7 +174,7 @@ static void riscv_sifive_e_soc_realize(DeviceState
> *dev, Error **errp)
> >> SIFIVE_SIP_BASE, SIFIVE_TIMECMP_BASE, SIFIVE_TIME_BASE);
> >> sifive_mmio_emulate(sys_mem, "riscv.sifive.e.aon",
> >> memmap[SIFIVE_E_AON].base, memmap[SIFIVE_E_AON].size);
> >> - sifive_prci_create(memmap[SIFIVE_E_PRCI].base);
> >> + sifive_e_prci_create(memmap[SIFIVE_E_PRCI].base);
> >>
> >> /* GPIO */
> >>
> >
> > I think adding infix to function name is sufficient and keeping the
> filename the same may be better.
> > The U board PRCI or variant implementation in future could be included
> in the same files with different create function
> >
>
> I considered such approach when working on this one, but later I chose
> to implement a new file for SiFive U machine.
>
> The SiFive U and E PRCI blocks have different register blocks so if we
> put two variants into one file, their functions don't have much in
> common and we end up just merely physically put them into one file
> which does not bring too much benefit IMHO.
>
> Regards,
> Bin
>
agree that the difference between u and e prci are a lot and it make sense
to separate it
Reviewed-by: Chih-Min Chao <address@hidden>
- [Qemu-devel] [PATCH v2 05/28] riscv: hart: Support heterogeneous harts population, (continued)
- [Qemu-devel] [PATCH v2 05/28] riscv: hart: Support heterogeneous harts population, Bin Meng, 2019/08/07
- [Qemu-devel] [PATCH v2 09/28] riscv: sifive_u: Update UART base addresses, Bin Meng, 2019/08/07
- [Qemu-devel] [PATCH v2 07/28] riscv: sifive_u: Set the minimum number of cpus to 2, Bin Meng, 2019/08/07
- [Qemu-devel] [PATCH v2 11/28] riscv: sifive: Rename sifive_prci.{c, h} to sifive_e_prci.{c, h}, Bin Meng, 2019/08/07
- Re: [Qemu-devel] [PATCH v2 11/28] riscv: sifive: Rename sifive_prci.{c, h} to sifive_e_prci.{c, h}, Alistair Francis, 2019/08/09
[Qemu-devel] [PATCH v2 13/28] riscv: sifive_e: prci: Update the PRCI register block size, Bin Meng, 2019/08/07
[Qemu-devel] [PATCH v2 14/28] riscv: sifive: Implement PRCI model for FU540, Bin Meng, 2019/08/07
[Qemu-devel] [PATCH v2 16/28] riscv: sifive_u: Add PRCI block to the SoC, Bin Meng, 2019/08/07
[Qemu-devel] [PATCH v2 18/28] riscv: hw: Implement a model for SiFive FU540 OTP, Bin Meng, 2019/08/07
[Qemu-devel] [PATCH v2 15/28] riscv: sifive_u: Generate hfclk and rtcclk nodes, Bin Meng, 2019/08/07
[Qemu-devel] [PATCH v2 17/28] riscv: sifive_u: Change UART node name in device tree, Bin Meng, 2019/08/07
[Qemu-devel] [PATCH v2 19/28] riscv: sifive_u: Instantiate OTP memory with a serial number, Bin Meng, 2019/08/07