[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: |
Bin Meng |
Subject: |
Re: [Qemu-devel] [PATCH v2 11/28] riscv: sifive: Rename sifive_prci.{c, h} to sifive_e_prci.{c, h} |
Date: |
Wed, 7 Aug 2019 18:10:49 +0800 |
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
- [Qemu-devel] [PATCH v2 08/28] riscv: sifive_u: Update PLIC hart topology configuration string, (continued)
- [Qemu-devel] [PATCH v2 08/28] riscv: sifive_u: Update PLIC hart topology configuration string, Bin Meng, 2019/08/07
- [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