[PATCH v2 2/3] lib: utils: check if CPU node is enabled

Xiang W wxjstz at 126.com
Mon May 9 23:08:48 PDT 2022


在 2022-05-09星期一的 15:56 +0200,Jan Remeš写道:
> Ignore CPU nodes in FDT that are not enabled.
> 
> Signed-off-by: Jan Remes <jan.remes at codasip.com>
Look good to me

Reviewed-by: Xiang W <wxjstz at 126.com>
> ---
>  lib/utils/fdt/fdt_domain.c            | 14 +++++++++++++-
>  lib/utils/fdt/fdt_fixup.c             |  3 +++
>  lib/utils/fdt/fdt_helper.c            |  6 ++++++
>  lib/utils/irqchip/fdt_irqchip_imsic.c |  3 +++
>  lib/utils/irqchip/fdt_irqchip_plic.c  |  3 +++
>  5 files changed, 28 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/utils/fdt/fdt_domain.c b/lib/utils/fdt/fdt_domain.c
> index 676c757..bd0eec3 100644
> --- a/lib/utils/fdt/fdt_domain.c
> +++ b/lib/utils/fdt/fdt_domain.c
> @@ -165,6 +165,9 @@ void fdt_domain_fixup(void *fdt)
>          if (err)
>              continue;
> 
> +        if (!fdt_node_is_enabled(fdt, doffset))
> +            continue;
> +
>          fdt_nop_property(fdt, doffset, "opensbi-domain");
>      }
> 
> @@ -308,6 +311,9 @@ static int __fdt_parse_domain(void *fdt, int
> domain_offset, void *opaque)
>              if (err)
>                  return err;
> 
> +            if (!fdt_node_is_enabled(fdt, cpu_offset))
> +                continue;
> +
>              sbi_hartmask_set_hart(val32, mask);
>          }
>      }
> @@ -347,7 +353,7 @@ static int __fdt_parse_domain(void *fdt, int
> domain_offset, void *opaque)
>      if (val && len >= 4) {
>          cpu_offset = fdt_node_offset_by_phandle(fdt,
>                               fdt32_to_cpu(*val));
> -        if (cpu_offset >= 0)
> +        if (cpu_offset >= 0 && fdt_node_is_enabled(fdt, cpu_offset))
>              fdt_parse_hart_id(fdt, cpu_offset, &val32);
>      } else {
>          if (domain_offset == *cold_domain_offset)
> @@ -414,6 +420,9 @@ static int __fdt_parse_domain(void *fdt, int
> domain_offset, void *opaque)
>          if (SBI_HARTMASK_MAX_BITS <= val32)
>              continue;
> 
> +        if (!fdt_node_is_enabled(fdt, cpu_offset))
> +            continue;
> +
>          val = fdt_getprop(fdt, cpu_offset, "opensbi-domain", &len);
>          if (!val || len < 4)
>              return SBI_EINVAL;
> @@ -460,6 +469,9 @@ int fdt_domains_populate(void *fdt)
>          if (hartid != cold_hartid)
>              continue;
> 
> +        if (!fdt_node_is_enabled(fdt, cpu_offset))
> +            continue;
> +
>          val = fdt_getprop(fdt, cpu_offset, "opensbi-domain", &len);
>          if (val && len >= 4)
>              cold_domain_offset = fdt_node_offset_by_phandle(fdt,
> diff --git a/lib/utils/fdt/fdt_fixup.c b/lib/utils/fdt/fdt_fixup.c
> index a80bd82..d1050bb 100644
> --- a/lib/utils/fdt/fdt_fixup.c
> +++ b/lib/utils/fdt/fdt_fixup.c
> @@ -38,6 +38,9 @@ void fdt_cpu_fixup(void *fdt)
>          if (err)
>              continue;
> 
> +        if (!fdt_node_is_enabled(fdt, cpu_offset))
> +            continue;
> +
>          /*
>           * Disable a HART DT node if one of the following is true:
>           * 1. The HART is not assigned to the current domain
> diff --git a/lib/utils/fdt/fdt_helper.c b/lib/utils/fdt/fdt_helper.c
> index 3b45ae8..a2736af 100644
> --- a/lib/utils/fdt/fdt_helper.c
> +++ b/lib/utils/fdt/fdt_helper.c
> @@ -283,6 +283,9 @@ int fdt_parse_max_hart_id(void *fdt, u32 *max_hartid)
>          if (err)
>              continue;
> 
> +        if (!fdt_node_is_enabled(fdt, cpu_offset))
> +            continue;
> +
>          if (hartid > *max_hartid)
>              *max_hartid = hartid;
>      }
> @@ -864,6 +867,9 @@ int fdt_parse_aclint_node(void *fdt, int
> nodeoffset, bool for_timer,
>          if (rc)
>              continue;
> 
> +        if (!fdt_node_is_enabled(fdt, cpu_offset))
> +            continue;
> +
>          if (SBI_HARTMASK_MAX_BITS <= hartid)
>              continue;
> 
> diff --git a/lib/utils/irqchip/fdt_irqchip_imsic.c
> b/lib/utils/irqchip/fdt_irqchip_imsic.c
> index b6962be..bbd8b8d 100644
> --- a/lib/utils/irqchip/fdt_irqchip_imsic.c
> +++ b/lib/utils/irqchip/fdt_irqchip_imsic.c
> @@ -51,6 +51,9 @@ static int irqchip_imsic_update_hartid_table(void
> *fdt, int nodeoff,
>          if (SBI_HARTMASK_MAX_BITS <= hartid)
>              return SBI_EINVAL;
> 
> +        if (!fdt_node_is_enabled(fdt, cpu_offset))
> +            continue;
> +
>          switch (hwirq) {
>          case IRQ_M_EXT:
>              err = imsic_map_hartid_to_data(hartid, id, i / 2);
> diff --git a/lib/utils/irqchip/fdt_irqchip_plic.c
> b/lib/utils/irqchip/fdt_irqchip_plic.c
> index b2d2a1b..94ddd08 100644
> --- a/lib/utils/irqchip/fdt_irqchip_plic.c
> +++ b/lib/utils/irqchip/fdt_irqchip_plic.c
> @@ -64,6 +64,9 @@ static int irqchip_plic_update_hartid_table(void
> *fdt, int nodeoff,
>          if (SBI_HARTMASK_MAX_BITS <= hartid)
>              continue;
> 
> +        if (!fdt_node_is_enabled(fdt, cpu_offset))
> +            continue;
> +
>          plic_hartid2data[hartid] = pd;
>          switch (hwirq) {
>          case IRQ_M_EXT:
> -- 
> 2.35.1
> 





More information about the opensbi mailing list