On Sun, 19 Apr 2020 at 17:27, Edgar E. Iglesias
<edgar.igles...@gmail.com> wrote:
>
> From: "Edgar E. Iglesias" <edgar.igles...@xilinx.com>
>
> Disable unsupported FDT firmware nodes if a user passes us
> a DTB with nodes enabled that the machine cannot support
> due to lack of EL3 or EL2 support.
>
> Signed-off-by: Edgar E. Iglesias <edgar.igles...@xilinx.com>
> ---
> hw/arm/xlnx-zcu102.c | 31 +++++++++++++++++++++++++++++++
> 1 file changed, 31 insertions(+)
> +static void zcu102_modify_dtb(const struct arm_boot_info *binfo, void *fdt)
> +{
> + XlnxZCU102 *s = container_of(binfo, XlnxZCU102, binfo);
> + bool method_is_hvc;
> + char **node_path;
> + const char *r;
> + int prop_len;
> + int i;
> +
> + /* If EL3 is enabled, we keep all firmware nodes active. */
> + if (!s->secure) {
> + node_path = qemu_fdt_node_path(fdt, NULL,
> + (char *)"xlnx,zynqmp-firmware",
> + &error_fatal);
Why do we need the 'char *' cast ?
thanks
-- PMM
