Change in coreboot[master]: AGESA: Introduce AGESA_LEGACY and its counterpart

[Martin Roth (Code Review)]

Martin Roth has posted comments on this change. ( https://review.coreboot.org/18619 )

Change subject: AGESA: Introduce AGESA_LEGACY and its counterpart
......................................................................

Patch Set 4:

(4 comments)

https://review.coreboot.org/#/c/18619/4/src/cpu/amd/agesa/family12/romstage....
File src/cpu/amd/agesa/family12/romstage.c:

PS4, Line 16: #include <stdint.h>
            : #include <string.h>
            : #include <device/pci_def.h>
            : #include <device/pci_ids.h>
            : #include <arch/io.h>
            : #include <arch/stages.h>
            : #include <device/pnp_def.h>
            : #include <cpu/x86/lapic.h>
            : #include <console/console.h>
            : #include <commonlib/loglevel.h>
            : #include <cpu/amd/car.h>
            : #include <northbridge/amd/agesa/agesawrapper.h>
            : #include <northbridge/amd/agesa/agesa_helper.h>
            : #include <northbridge/amd/agesa/state_machine.h>
            : #include <cpu/x86/bist.h>
            : #include <superio/smsc/kbc1100/kbc1100.h>
            : #include <cpu/x86/lapic.h>
            : #include "sb_cimx.h"
            : #include "SbPlatform.h"
            : #include <arch/cpu.h>
            : #include "platform_cfg.h"
These are the includes that are actually needed.  I suspect the other platforms can be cleaned up similarly in a follow-on patch.

#include <arch/stages.h>
#include <cpu/amd/car.h>
#include <cpu/x86/bist.h>
#include <cpu/x86/lapic.h>
#include <northbridge/amd/agesa/agesawrapper.h>
#include <northbridge/amd/agesa/agesa_helper.h>
#include <northbridge/amd/agesa/state_machine.h>
#include "sb_cimx.h"
#include "SbPlatform.h"

https://review.coreboot.org/#/c/18619/4/src/cpu/amd/agesa/family14/romstage....
File src/cpu/amd/agesa/family14/romstage.c:

Line 104: 
It might be nice to add a couple of additional places that the mainboard can hook.  Something like:

board_after_agesa(cb);
board_end_romstage(cb);

https://review.coreboot.org/#/c/18619/4/src/northbridge/amd/agesa/state_mach...
File src/northbridge/amd/agesa/state_machine.h:

PS4, Line 16: _STATE_MACHINE_H_
I'm not crazy about the name - I think something more generic like agesa_sysinfo.h might be more appropriate.

PS4, Line 26: board_BeforeAgesa
why is this camelcase?

board_before_agesa();