[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v4 4/4] hw/arm: versal: Add a virtual Xilinx Versa
From: |
Edgar E. Iglesias |
Subject: |
Re: [Qemu-arm] [PATCH v4 4/4] hw/arm: versal: Add a virtual Xilinx Versal board |
Date: |
Fri, 2 Nov 2018 12:33:56 +0100 |
User-agent: |
Mutt/1.9.4 (2018-02-28) |
On Tue, Oct 30, 2018 at 01:31:44PM +0000, Peter Maydell wrote:
> On 22 October 2018 at 18:35, Edgar E. Iglesias <address@hidden> wrote:
> > From: "Edgar E. Iglesias" <address@hidden>
> >
> > Add a virtual Xilinx Versal board.
> >
> > This board is based on the Xilinx Versal SoC. The exact
> > details of what peripherals are attached to this board
> > will remain in control of QEMU. QEMU will generate an
> > FDT on the fly for Linux and other software to auto-discover
> > peripherals.
> >
> > Signed-off-by: Edgar E. Iglesias <address@hidden>
>
> > +static void fdt_add_fixed_link_nodes(VersalVirt *s, char *gemname,
> > + uint32_t phandle)
> > +{
> > + char *name = g_strdup_printf("%s/fixed-link", gemname);
> > +
> > + qemu_fdt_add_subnode(s->fdt, name);
> > + qemu_fdt_setprop_cell(s->fdt, name, "phandle", phandle);
> > + qemu_fdt_setprop_cells(s->fdt, name, "full-duplex");
>
> Hi. This fails to compile in a non-debug build:
>
> In file included from /home/peter.maydell/qemu/hw/arm/xlnx-versal-virt.c:16:0:
> /home/peter.maydell/qemu/hw/arm/xlnx-versal-virt.c: In function
> 'fdt_add_fixed_link_nodes':
> /home/peter.maydell/qemu/include/sysemu/device_tree.h:110:23: error:
> comparison of unsigned expression < 0 is always false
> [-Werror=type-limits]
> for (i = 0; i < ARRAY_SIZE(qdt_tmp); i++) {
> \
> ^
> /home/peter.maydell/qemu/hw/arm/xlnx-versal-virt.c:191:5: note: in
> expansion of macro 'qemu_fdt_setprop_cells'
> qemu_fdt_setprop_cells(s->fdt, name, "full-duplex");
> ^
>
> because qemu_fdt_setprop_cells() requires you to provide
> at least one cell value for the property being set.
> What was the intention here ?
Hi Peter,
The intent was to set a boolean property without a value.
I'll fix this with the following and send a new version:
qemu_fdt_setprop(s->fdt, name, "full-duplex", NULL, 0);
Cheers,
Edgar
>
>
> > + qemu_fdt_setprop_cell(s->fdt, name, "speed", 1000);
> > + g_free(name);
> > +}
>
> In the meantime, I'm dropping the versal patches from
> target-arm.next.
>
> thanks
> -- PMM
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-arm] [PATCH v4 4/4] hw/arm: versal: Add a virtual Xilinx Versal board,
Edgar E. Iglesias <=