Re: [Qemu-devel] [PATCH] Use Aff1 with mpidr

[Pavel Fedin]

 Hi!

> I think we should take Igor's comment into account. The CPUS_PER_CLUSTER 
> should not be a
> constant, and maybe should be initialized in arm_cpu_initfn and
aarch64_{a53|a57}_initfn, as
> psci need to know it.

 Yes, of course. Just replace a #define with something like:

 int cpus_per_cluster = 8;

 and you're done with this. This provides good backwards compatibility as well 
as
possibility to change this on a per-machine basis.

> I understand that you suggest that I'll not use my 1 & 4 patches. I have no 
> problem with
that,
> but how do I commit? Should I write that the patch series is pending until 
> your patch
series is
> integrated?

 Just make it a part of your V3. I'm perfectly OK with that. Looks like 
maintainers don't
want to integrate it alone.

> I intend to release the V3 version during this week.

 Good.

Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia