Hi Simon, > -----"Simon Glass" <s...@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 <s...@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.wall...@br-automation.com>
