diff mbox series

hw/arm/npcm7xx: remove setting of mp-affinity

Message ID 20240504141733.14813-1-dorjoychy111@gmail.com (mailing list archive)
State New, archived
Headers show
Series hw/arm/npcm7xx: remove setting of mp-affinity | expand

Commit Message

Dorjoy Chowdhury May 4, 2024, 2:17 p.m. UTC
The value of the mp-affinity property being set in npcm7xx_realize is
always the same as the default value it would have when arm_cpu_realizefn
is called if the property is not set here. So there is no need to set
the property value in npcm7xx_realize function.

Signed-off-by: Dorjoy Chowdhury <dorjoychy111@gmail.com>
---
 hw/arm/npcm7xx.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Richard Henderson May 4, 2024, 2:32 p.m. UTC | #1
On 5/4/24 07:17, Dorjoy Chowdhury wrote:
> The value of the mp-affinity property being set in npcm7xx_realize is
> always the same as the default value it would have when arm_cpu_realizefn
> is called if the property is not set here. So there is no need to set
> the property value in npcm7xx_realize function.
> 
> Signed-off-by: Dorjoy Chowdhury<dorjoychy111@gmail.com>
> ---
>   hw/arm/npcm7xx.c | 3 ---
>   1 file changed, 3 deletions(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~
Philippe Mathieu-Daudé May 6, 2024, 7:36 a.m. UTC | #2
On 4/5/24 16:32, Richard Henderson wrote:
> On 5/4/24 07:17, Dorjoy Chowdhury wrote:
>> The value of the mp-affinity property being set in npcm7xx_realize is
>> always the same as the default value it would have when arm_cpu_realizefn
>> is called if the property is not set here. So there is no need to set
>> the property value in npcm7xx_realize function.
>>
>> Signed-off-by: Dorjoy Chowdhury<dorjoychy111@gmail.com>
>> ---
>>   hw/arm/npcm7xx.c | 3 ---
>>   1 file changed, 3 deletions(-)

Implicitly in
https://lore.kernel.org/qemu-devel/20231212162935.42910-31-philmd@linaro.org/

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Peter Maydell May 7, 2024, 9:43 a.m. UTC | #3
On Sat, 4 May 2024 at 15:17, Dorjoy Chowdhury <dorjoychy111@gmail.com> wrote:
>
> The value of the mp-affinity property being set in npcm7xx_realize is
> always the same as the default value it would have when arm_cpu_realizefn
> is called if the property is not set here. So there is no need to set
> the property value in npcm7xx_realize function.
>
> Signed-off-by: Dorjoy Chowdhury <dorjoychy111@gmail.com>
> ---



Applied to target-arm.next, thanks.

-- PMM
diff mbox series

Patch

diff --git a/hw/arm/npcm7xx.c b/hw/arm/npcm7xx.c
index 9f2d96c733..cb7791301b 100644
--- a/hw/arm/npcm7xx.c
+++ b/hw/arm/npcm7xx.c
@@ -487,9 +487,6 @@  static void npcm7xx_realize(DeviceState *dev, Error **errp)
 
     /* CPUs */
     for (i = 0; i < nc->num_cpus; i++) {
-        object_property_set_int(OBJECT(&s->cpu[i]), "mp-affinity",
-                                arm_build_mp_affinity(i, NPCM7XX_MAX_NUM_CPUS),
-                                &error_abort);
         object_property_set_int(OBJECT(&s->cpu[i]), "reset-cbar",
                                 NPCM7XX_GIC_CPU_IF_ADDR, &error_abort);
         object_property_set_bool(OBJECT(&s->cpu[i]), "reset-hivecs", true,