Re: [PATCH 01/65] hw/core: Permit devices to define an array of link properties

[Peter Maydell <peter.maydell@linaro.org>]

On Sat, 21 Mar 2026 at 15:42, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> (+Kevin / Markus / Mark for previous discussions)
>
> On 23/2/26 18:01, Peter Maydell wrote:
> > Currently we allow devices to define "link properties" with
> > DEFINE_PROP_LINK(): these are a way to give a device a pointer to
> > another QOM object.  (Under the hood this is done by handing it the
> > canonical QOM path for the object.)
> >
> > We also allow devices to define "array properties" with
> > DEFINE_PROP_ARRAY(): these are a way to give a device a
> > variable-length array of properties.
> >
> > However, there is no way to define an array of link properties.  If
> > you try to do it by passing qdev_prop_link as the arrayprop argument
> > to DEFINE_PROP_ARRAY() you will get a crash because qdev_prop_link
> > does not provide the .set and .get methods in its PropertyInfo
> > struct.
> >
> > This patch implements a new DEFINE_PROP_LINK_ARRAY().


> Nitpicking, I'd add object_resolve_and_typecheck() in a preliminary
> patch. Otherwise patch LGTM!
>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Thanks. I've split the patch in two (it splits exactly into
"changes to object.[ch]" and "changes to qdev-properties.[ch]")
and added your R-by tag to both the resulting patches.

-- PMM