16 Jun
2020
16 Jun
'20
1:31 p.m.
Hi Simon,
-----"Simon Glass" sjg@chromium.org schrieb: ----- Betreff: [PATCH v2 24/25] x86: mp: Add more comments to the module
Add a description of how this module works and also some missing function comments.
Great to see more documenation :)
Drop struct cpu_map since it is not used.
Signed-off-by: Simon Glass sjg@chromium.org
Changes in v2:
- Add a new patch with more comments
arch/x86/cpu/mp_init.c | 91 ++++++++++++++++++++++++++++++++++++++- arch/x86/include/asm/mp.h | 14 +++++- 2 files changed, 102 insertions(+), 3 deletions(-)
[snip]
- After being started, each AP runs the code in ap_start16, switches to 32-bit
- mode, runs the code at ap_start, then jumps to c_handler which is ap_init().
- This runs a very simple 'flight plan' described in* mp_steps. This sets up
Should this be "in *mp_steps"? Or "in mp_steps"?
[snip]
Reviewed-by: Wolfgang Wallner wolfgang.wallner@br-automation.com