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.

target-arm/cpu.c:        qdev_property_add_static(DEVICE(obj), 
&arm_cpu_reset_cbar_property,
target-arm/cpu.c:        qdev_property_add_static(DEVICE(obj), 
&arm_cpu_reset_hivecs_property,
target-arm/cpu.c:        qdev_property_add_static(DEVICE(obj), 
&arm_cpu_rvbar_property,
target-arm/cpu.c:        qdev_property_add_static(DEVICE(obj), 
&arm_cpu_has_el3_property,
target-arm/cpu.c:        qdev_property_add_static(DEVICE(obj), 
&arm_cpu_has_mpu_property,
target-arm/cpu.c:            qdev_property_add_static(DEVICE(obj),

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.

-- 
Eduardo

Reply via email to