diff mbox series

[PATCHv3,2/8] board: lx2160a: Don't program the GIC RD tables if failed to reserve memory

Message ID 20200428021935.27659-3-Zhiqiang.Hou@nxp.com
State New
Headers show
Series fsl: layerscape: Initialize the GIC redistributor | expand

Commit Message

Zhiqiang Hou April 28, 2020, 2:19 a.m. UTC
From: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>

Program the GIC redistributor tables only when succeeded to reserve memory
for them, otherwise kernel will lose the chance to program them using
allocated memory.

Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
---
V3:
 - Change the return type to int of fdt_fixup_gic_lpi_memory().

 board/freescale/lx2160a/lx2160a.c | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

Comments

Wasim Khan April 28, 2020, 7:40 a.m. UTC | #1
> -----Original Message-----
> From: Z.q. Hou <zhiqiang.hou at nxp.com>
> Sent: Tuesday, April 28, 2020 7:49 AM
> To: u-boot at lists.denx.de; Priyanka Jain <priyanka.jain at nxp.com>; Wasim Khan
> <wasim.khan at nxp.com>; bmeng.cn at gmail.com
> Cc: Z.q. Hou <zhiqiang.hou at nxp.com>
> Subject: [PATCHv3 2/8] board: lx2160a: Don't program the GIC RD tables if failed
> to reserve memory
> 
> From: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> 
> Program the GIC redistributor tables only when succeeded to reserve memory
> for them, otherwise kernel will lose the chance to program them using allocated
> memory.
> 
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>

Reviewed-by: Wasim Khan <wasim.khan at nxp.com>

> ---
> V3:
>  - Change the return type to int of fdt_fixup_gic_lpi_memory().
> 
>  board/freescale/lx2160a/lx2160a.c | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
> 
> diff --git a/board/freescale/lx2160a/lx2160a.c
> b/board/freescale/lx2160a/lx2160a.c
> index 23ea1b6f16..eafd530763 100644
> --- a/board/freescale/lx2160a/lx2160a.c
> +++ b/board/freescale/lx2160a/lx2160a.c
> @@ -645,7 +645,7 @@ void board_quiesce_devices(void)  #endif
> 
>  #ifdef CONFIG_GIC_V3_ITS
> -void fdt_fixup_gic_lpi_memory(void *blob, u64 gic_lpi_base)
> +int fdt_fixup_gic_lpi_memory(void *blob, u64 gic_lpi_base)
>  {
>  	u32 phandle;
>  	int err;
> @@ -656,6 +656,8 @@ void fdt_fixup_gic_lpi_memory(void *blob, u64
> gic_lpi_base)
>  	err = fdtdec_add_reserved_memory(blob, "gic-lpi", &gic_lpi, &phandle);
>  	if (err < 0)
>  		debug("failed to add reserved memory: %d\n", err);
> +
> +	return err;
>  }
>  #endif
> 
> @@ -671,6 +673,7 @@ int ft_board_setup(void *blob, bd_t *bd)
>  	u64 mc_memory_size = 0;
>  	u16 total_memory_banks;
>  	u64 __maybe_unused gic_lpi_base;
> +	int ret;
> 
>  	ft_cpu_setup(blob, bd);
> 
> @@ -692,8 +695,9 @@ int ft_board_setup(void *blob, bd_t *bd)
> 
>  #ifdef CONFIG_GIC_V3_ITS
>  	gic_lpi_base = gd->arch.resv_ram - GIC_LPI_SIZE;
> -	gic_lpi_tables_init(gic_lpi_base, cpu_numcores());
> -	fdt_fixup_gic_lpi_memory(blob, gic_lpi_base);
> +	ret = fdt_fixup_gic_lpi_memory(blob, gic_lpi_base);
> +	if (!ret && gic_lpi_tables_init(gic_lpi_base, cpu_numcores()))
> +		debug("%s: failed to init gic-lpi-tables\n", __func__);
>  #endif
> 
>  #ifdef CONFIG_RESV_RAM
> --
> 2.17.1
diff mbox series

Patch

diff --git a/board/freescale/lx2160a/lx2160a.c b/board/freescale/lx2160a/lx2160a.c
index 23ea1b6f16..eafd530763 100644
--- a/board/freescale/lx2160a/lx2160a.c
+++ b/board/freescale/lx2160a/lx2160a.c
@@ -645,7 +645,7 @@  void board_quiesce_devices(void)
 #endif
 
 #ifdef CONFIG_GIC_V3_ITS
-void fdt_fixup_gic_lpi_memory(void *blob, u64 gic_lpi_base)
+int fdt_fixup_gic_lpi_memory(void *blob, u64 gic_lpi_base)
 {
 	u32 phandle;
 	int err;
@@ -656,6 +656,8 @@  void fdt_fixup_gic_lpi_memory(void *blob, u64 gic_lpi_base)
 	err = fdtdec_add_reserved_memory(blob, "gic-lpi", &gic_lpi, &phandle);
 	if (err < 0)
 		debug("failed to add reserved memory: %d\n", err);
+
+	return err;
 }
 #endif
 
@@ -671,6 +673,7 @@  int ft_board_setup(void *blob, bd_t *bd)
 	u64 mc_memory_size = 0;
 	u16 total_memory_banks;
 	u64 __maybe_unused gic_lpi_base;
+	int ret;
 
 	ft_cpu_setup(blob, bd);
 
@@ -692,8 +695,9 @@  int ft_board_setup(void *blob, bd_t *bd)
 
 #ifdef CONFIG_GIC_V3_ITS
 	gic_lpi_base = gd->arch.resv_ram - GIC_LPI_SIZE;
-	gic_lpi_tables_init(gic_lpi_base, cpu_numcores());
-	fdt_fixup_gic_lpi_memory(blob, gic_lpi_base);
+	ret = fdt_fixup_gic_lpi_memory(blob, gic_lpi_base);
+	if (!ret && gic_lpi_tables_init(gic_lpi_base, cpu_numcores()))
+		debug("%s: failed to init gic-lpi-tables\n", __func__);
 #endif
 
 #ifdef CONFIG_RESV_RAM