[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-riscv] [Qemu-devel] [PATCH v4 27/28] riscv: sifive_u: Remove h
From: |
Alistair Francis |
Subject: |
Re: [Qemu-riscv] [Qemu-devel] [PATCH v4 27/28] riscv: sifive_u: Remove handcrafted clock nodes for UART and ethernet |
Date: |
Thu, 22 Aug 2019 15:39:41 -0700 |
On Sun, Aug 18, 2019 at 10:31 PM Bin Meng <address@hidden> wrote:
>
> In the past we did not have a model for PRCI, hence two handcrafted
> clock nodes ("/soc/ethclk" and "/soc/uartclk") were created for the
> purpose of supplying hard-coded clock frequencies. But now since we
> have added the PRCI support in QEMU, we don't need them any more.
>
> Signed-off-by: Bin Meng <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Alistair
>
> ---
>
> Changes in v4:
> - new patch to remove handcrafted clock nodes for UART and ethernet
>
> Changes in v3: None
> Changes in v2: None
>
> hw/riscv/sifive_u.c | 24 +-----------------------
> include/hw/riscv/sifive_u.h | 3 +--
> 2 files changed, 2 insertions(+), 25 deletions(-)
>
> diff --git a/hw/riscv/sifive_u.c b/hw/riscv/sifive_u.c
> index 7a370e9..7d9fb3a 100644
> --- a/hw/riscv/sifive_u.c
> +++ b/hw/riscv/sifive_u.c
> @@ -89,8 +89,7 @@ static void create_fdt(SiFiveUState *s, const struct
> MemmapEntry *memmap,
> uint32_t *cells;
> char *nodename;
> char ethclk_names[] = "pclk\0hclk";
> - uint32_t plic_phandle, prci_phandle, ethclk_phandle, phandle = 1;
> - uint32_t uartclk_phandle;
> + uint32_t plic_phandle, prci_phandle, phandle = 1;
> uint32_t hfclk_phandle, rtcclk_phandle, phy_phandle;
>
> fdt = s->fdt = create_device_tree(&s->fdt_size);
> @@ -250,17 +249,6 @@ static void create_fdt(SiFiveUState *s, const struct
> MemmapEntry *memmap,
> g_free(cells);
> g_free(nodename);
>
> - ethclk_phandle = phandle++;
> - nodename = g_strdup_printf("/soc/ethclk");
> - qemu_fdt_add_subnode(fdt, nodename);
> - qemu_fdt_setprop_string(fdt, nodename, "compatible", "fixed-clock");
> - qemu_fdt_setprop_cell(fdt, nodename, "#clock-cells", 0x0);
> - qemu_fdt_setprop_cell(fdt, nodename, "clock-frequency",
> - SIFIVE_U_GEM_CLOCK_FREQ);
> - qemu_fdt_setprop_cell(fdt, nodename, "phandle", ethclk_phandle);
> - ethclk_phandle = qemu_fdt_get_phandle(fdt, nodename);
> - g_free(nodename);
> -
> phy_phandle = phandle++;
> nodename = g_strdup_printf("/soc/ethernet@%lx",
> (long)memmap[SIFIVE_U_GEM].base);
> @@ -292,16 +280,6 @@ static void create_fdt(SiFiveUState *s, const struct
> MemmapEntry *memmap,
> qemu_fdt_setprop_cell(fdt, nodename, "reg", 0x0);
> g_free(nodename);
>
> - uartclk_phandle = phandle++;
> - nodename = g_strdup_printf("/soc/uartclk");
> - qemu_fdt_add_subnode(fdt, nodename);
> - qemu_fdt_setprop_string(fdt, nodename, "compatible", "fixed-clock");
> - qemu_fdt_setprop_cell(fdt, nodename, "#clock-cells", 0x0);
> - qemu_fdt_setprop_cell(fdt, nodename, "clock-frequency", 3686400);
> - qemu_fdt_setprop_cell(fdt, nodename, "phandle", uartclk_phandle);
> - uartclk_phandle = qemu_fdt_get_phandle(fdt, nodename);
> - g_free(nodename);
> -
> nodename = g_strdup_printf("/soc/serial@%lx",
> (long)memmap[SIFIVE_U_UART0].base);
> qemu_fdt_add_subnode(fdt, nodename);
> diff --git a/include/hw/riscv/sifive_u.h b/include/hw/riscv/sifive_u.h
> index cba29e1..8880f9c 100644
> --- a/include/hw/riscv/sifive_u.h
> +++ b/include/hw/riscv/sifive_u.h
> @@ -72,8 +72,7 @@ enum {
> enum {
> SIFIVE_U_CLOCK_FREQ = 1000000000,
> SIFIVE_U_HFCLK_FREQ = 33333333,
> - SIFIVE_U_RTCCLK_FREQ = 1000000,
> - SIFIVE_U_GEM_CLOCK_FREQ = 125000000
> + SIFIVE_U_RTCCLK_FREQ = 1000000
> };
>
> #define SIFIVE_U_MANAGEMENT_CPU_COUNT 1
> --
> 2.7.4
>
>
- Re: [Qemu-riscv] [Qemu-devel] [PATCH v4 13/28] riscv: hart: Add a "hartid-base" property to RISC-V hart array, (continued)
- [Qemu-riscv] [PATCH v4 16/28] riscv: sifive_u: Update PLIC hart topology configuration string, Bin Meng, 2019/08/19
- [Qemu-riscv] [PATCH v4 20/28] riscv: sifive_u: Reference PRCI clocks in UART and ethernet nodes, Bin Meng, 2019/08/19
- [Qemu-riscv] [PATCH v4 15/28] riscv: sifive_u: Set the minimum number of cpus to 2, Bin Meng, 2019/08/19
- [Qemu-riscv] [PATCH v4 25/28] riscv: sifive_u: Instantiate OTP memory with a serial number, Bin Meng, 2019/08/19
- [Qemu-riscv] [PATCH v4 19/28] riscv: sifive_u: Add PRCI block to the SoC, Bin Meng, 2019/08/19
- [Qemu-riscv] [PATCH v4 26/28] riscv: sifive_u: Fix broken GEM support, Bin Meng, 2019/08/19
- [Qemu-riscv] [PATCH v4 27/28] riscv: sifive_u: Remove handcrafted clock nodes for UART and ethernet, Bin Meng, 2019/08/19
- Re: [Qemu-riscv] [Qemu-devel] [PATCH v4 27/28] riscv: sifive_u: Remove handcrafted clock nodes for UART and ethernet,
Alistair Francis <=
- [Qemu-riscv] [PATCH v4 17/28] riscv: sifive: Implement PRCI model for FU540, Bin Meng, 2019/08/19
- [Qemu-riscv] [PATCH v4 22/28] riscv: sifive_u: Change UART node name in device tree, Bin Meng, 2019/08/19
- [Qemu-riscv] [PATCH v4 28/28] riscv: sifive_u: Update model and compatible strings in device tree, Bin Meng, 2019/08/19
- [Qemu-riscv] [PATCH v4 21/28] riscv: sifive_u: Update UART base addresses and IRQs, Bin Meng, 2019/08/19
- [Qemu-riscv] [PATCH v4 23/28] riscv: roms: Update default bios for sifive_u machine, Bin Meng, 2019/08/19
- [Qemu-riscv] [PATCH v4 24/28] riscv: sifive: Implement a model for SiFive FU540 OTP, Bin Meng, 2019/08/19