[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH] hw/ppc: use 0 instead of fdt_path_offset(fdt, "/"
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [PATCH] hw/ppc: use 0 instead of fdt_path_offset(fdt, "/") |
Date: |
Wed, 4 Oct 2017 10:34:53 +1100 |
User-agent: |
Mutt/1.9.0 (2017-09-02) |
On Tue, Oct 03, 2017 at 04:13:11PM +0200, Greg Kurz wrote:
> The offset of the root node is guaranteed to be 0.
>
> This doesn't fix anything, it's just trivial cleanup of the two
> remaining places where this was done under hw/ppc.
>
> Signed-off-by: Greg Kurz <address@hidden>
Applied to ppc-for-2.11.
> ---
> hw/ppc/pnv.c | 3 +--
> hw/ppc/spapr.c | 3 +--
> 2 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
> index d46d91c76f5c..84b2389ea60b 100644
> --- a/hw/ppc/pnv.c
> +++ b/hw/ppc/pnv.c
> @@ -92,8 +92,7 @@ static int get_cpus_node(void *fdt)
> int cpus_offset = fdt_path_offset(fdt, "/cpus");
>
> if (cpus_offset < 0) {
> - cpus_offset = fdt_add_subnode(fdt, fdt_path_offset(fdt, "/"),
> - "cpus");
> + cpus_offset = fdt_add_subnode(fdt, 0, "cpus");
> if (cpus_offset) {
> _FDT((fdt_setprop_cell(fdt, cpus_offset, "#address-cells",
> 0x1)));
> _FDT((fdt_setprop_cell(fdt, cpus_offset, "#size-cells", 0x0)));
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index ff87f155d55e..352ff3d614e8 100644
> --- a/hw/ppc/spapr.c
> +++ b/hw/ppc/spapr.c
> @@ -353,8 +353,7 @@ static int spapr_fixup_cpu_dt(void *fdt,
> sPAPRMachineState *spapr)
>
> cpus_offset = fdt_path_offset(fdt, "/cpus");
> if (cpus_offset < 0) {
> - cpus_offset = fdt_add_subnode(fdt, fdt_path_offset(fdt, "/"),
> - "cpus");
> + cpus_offset = fdt_add_subnode(fdt, 0, "cpus");
> if (cpus_offset < 0) {
> return cpus_offset;
> }
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature