mbox series

[0/2] x86: apl: Add support for different number of cores

Message ID 20200225121949.35950-1-wolfgang.wallner@br-automation.com
Headers show
Series x86: apl: Add support for different number of cores | expand

Message

Wolfgang Wallner Feb. 25, 2020, 12:19 p.m. UTC 
Currently the Apollo Lake implementation of cpu_ops.get_count returns
a hardcoded value of 4.

The function cpu_x86_get_count() provides a more generic implementation
that reads the number of cores from device tree. But this function is
currently static.

This series:
 - makes cpu_x86_get_count() non-static
 - drops the Apollo Lake specific get_count implementation and
   uses cpu_x86_get_count() instead

With the changes in this series also Apollo Lake variants with
a different number of cores boot correctly.


Wolfgang Wallner (2):
  x86: cpu_x86: Make cpu_x86_get_count() non-static
  x86: apl: Use cpu_x86_get_count() for cpu_ops.get_count

 arch/x86/cpu/apollolake/cpu.c  |  7 +------
 arch/x86/cpu/cpu_x86.c         |  2 +-
 arch/x86/include/asm/cpu_x86.h | 12 ++++++++++++
 3 files changed, 14 insertions(+), 7 deletions(-)

Comments

Bin Meng Feb. 28, 2020, 8:05 a.m. UTC | #1 
On Tue, Feb 25, 2020 at 8:19 PM Wolfgang Wallner
<wolfgang.wallner@br-automation.com> wrote:
>
>
> Currently the Apollo Lake implementation of cpu_ops.get_count returns
> a hardcoded value of 4.
>
> The function cpu_x86_get_count() provides a more generic implementation
> that reads the number of cores from device tree. But this function is
> currently static.
>
> This series:
>  - makes cpu_x86_get_count() non-static
>  - drops the Apollo Lake specific get_count implementation and
>    uses cpu_x86_get_count() instead
>
> With the changes in this series also Apollo Lake variants with
> a different number of cores boot correctly.
>
>
> Wolfgang Wallner (2):
>   x86: cpu_x86: Make cpu_x86_get_count() non-static
>   x86: apl: Use cpu_x86_get_count() for cpu_ops.get_count

series applied to u-boot-x86, thanks!