On Wed, Oct 26, 2016 at 06:57:12PM +0100, Peter Maydell wrote:
> On 26 October 2016 at 17:57, Eduardo Habkost <ehabk...@redhat.com> wrote:
> > On Wed, Oct 26, 2016 at 05:48:03PM +0100, Peter Maydell wrote:
> >> On 26 October 2016 at 17:30, Eduardo Habkost <ehabk...@redhat.com> wrote:
> >> > The only remaining user of object_class_property_add() is
> >> > arm_cpu_post_init(), but removing it may take some work. While we
> >> > don't change it, warn people to not use the function in new code.
> >>
> >> This is about the third time I've seen this commit
> >> message go past, but:
> >>
> >> git grep object_class_property_add target-arm => no hits
> >
> > Erm. I meant qdev_property_add_static(). Sorry.
> > Commit message needs to be rewritten as:
> >
> > qdev: Warning about using qdev_property_add_static() in new code
> >
> > The only remaining user of qdev_property_add_static() is
> > arm_cpu_post_init(), but removing it may take some work. While we
> > don't change it, warn people to not use the function in new code.
>
> Can we say what the recommended replacement function is?
It's mentioned in the comment I added[1]. I assume I don't need to duplicate
the comment contents in the commit message.
[1] "Do not use this in new code. Either use qdev_class_set_props(), or
register regular QOM properties using object_property_add() or
object_class_property_add()."
--
Eduardo
