On Tue, 6 Jun 2023 at 19:24, Marcin Juszkiewicz <marcin.juszkiew...@linaro.org> wrote: > > We need GIC ITS information in DeviceTree so TF-A can pass it to EDK2. > > Bumping platform version to 0.2 as this is important hardware change. > > Signed-off-by: Marcin Juszkiewicz <marcin.juszkiew...@linaro.org> > > --- > hw/arm/sbsa-ref.c | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/hw/arm/sbsa-ref.c b/hw/arm/sbsa-ref.c > index 1520cd598c..2bd9e370a7 100644 > --- a/hw/arm/sbsa-ref.c > +++ b/hw/arm/sbsa-ref.c > @@ -183,8 +183,15 @@ static void sbsa_fdt_add_gic_node(SBSAMachineState *sms) > 2, sbsa_ref_memmap[SBSA_GIC_REDIST].base, > 2, sbsa_ref_memmap[SBSA_GIC_REDIST].size); > > + nodename = g_strdup_printf("/intc/its"); > + qemu_fdt_add_subnode(sms->fdt, nodename); > + qemu_fdt_setprop_sized_cells(sms->fdt, nodename, "reg", > + 2, sbsa_ref_memmap[SBSA_GIC_ITS].base, > + 2, sbsa_ref_memmap[SBSA_GIC_ITS].size); > + > g_free(nodename); > } > + > /* > * Firmware on this machine only uses ACPI table to load OS, these limited > * device tree nodes are just to let firmware know the info which varies from > @@ -221,7 +228,7 @@ static void create_fdt(SBSAMachineState *sms) > * fw compatibility. > */ > qemu_fdt_setprop_cell(fdt, "/", "machine-version-major", 0); > - qemu_fdt_setprop_cell(fdt, "/", "machine-version-minor", 1); > + qemu_fdt_setprop_cell(fdt, "/", "machine-version-minor", 2); > > if (ms->numa_state->have_numa_distance) { > int size = nb_numa_nodes * nb_numa_nodes * 3 * sizeof(uint32_t);
We should fold this patch into the previous one. If we are bumping the version-minor, we should add something to the documentation that says what the difference between 0.1 and 0.2 is. (And if we can remember what 0.0 was that would be worth noting.) thanks -- PMM