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
