[U-Boot] [PATCH] mpx85xx/fdt: Add cpu-release-addr for all cores

[Kumar Gala]

On Aug 18, 2010, at 3:12 PM, Matthew McClintock wrote:

> We currently do not add a cpu-release-addr for core 0, this is needed
> when we want to reset core 0 and later restart it from Linux
> ---
> arch/powerpc/cpu/mpc85xx/fdt.c |   21 ++++++++-------------
> 1 files changed, 8 insertions(+), 13 deletions(-)
> 
> diff --git a/arch/powerpc/cpu/mpc85xx/fdt.c b/arch/powerpc/cpu/mpc85xx/fdt.c
> index 8e7b827..acc70bf 100644
> --- a/arch/powerpc/cpu/mpc85xx/fdt.c
> +++ b/arch/powerpc/cpu/mpc85xx/fdt.c
> @@ -47,25 +47,20 @@ void ft_fixup_cpu(void *blob, u64 memory_limit)
> 	int off;
> 	ulong spin_tbl_addr = get_spin_phys_addr();
> 	u32 bootpg = determine_mp_bootpg();
> -	u32 id = get_my_id();
> 
> 	off = fdt_node_offset_by_prop_value(blob, -1, "device_type", "cpu", 4);
> 	while (off != -FDT_ERR_NOTFOUND) {
> 		u32 *reg = (u32 *)fdt_getprop(blob, off, "reg", 0);
> 
> 		if (reg) {
> -			if (*reg == id) {
> -				fdt_setprop_string(blob, off, "status", "okay");

Don't we still want "status" = "okay" for boot cpu?

> -			} else {
> -				u64 val = *reg * SIZE_BOOT_ENTRY + spin_tbl_addr;
> -				val = cpu_to_fdt32(val);
> -				fdt_setprop_string(blob, off, "status",
> -								"disabled");
> -				fdt_setprop_string(blob, off, "enable-method",
> -								"spin-table");
> -				fdt_setprop(blob, off, "cpu-release-addr",
> -						&val, sizeof(val));
> -			}
> +			u64 val = *reg * SIZE_BOOT_ENTRY + spin_tbl_addr;
> +			val = cpu_to_fdt32(val);
> +			fdt_setprop_string(blob, off, "status",
> +							"disabled");
> +			fdt_setprop_string(blob, off, "enable-method",
> +							"spin-table");
> +			fdt_setprop(blob, off, "cpu-release-addr",
> +					&val, sizeof(val));
> 		} else {
> 			printf ("cpu NULL\n");
> 		}
> -- 
> 1.6.6.1
> 
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot